drm/i915: rename debugfs_gt files
authorLucas De Marchi <lucas.demarchi@intel.com>
Sat, 18 Sep 2021 02:57:51 +0000 (19:57 -0700)
committerLucas De Marchi <lucas.demarchi@intel.com>
Sun, 19 Sep 2021 06:33:19 +0000 (23:33 -0700)
We shouldn't be using debugfs_ namespace for this functionality. Rename
debugfs_gt.[ch] to intel_gt_debugfs.[ch] and then make functions,
defines and structs follow suit.

While at it and since we are renaming the header, sort the includes
alphabetically.

Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210918025754.1254705-1-lucas.demarchi@intel.com
13 files changed:
drivers/gpu/drm/i915/Makefile
drivers/gpu/drm/i915/gt/debugfs_engines.c
drivers/gpu/drm/i915/gt/debugfs_gt.c [deleted file]
drivers/gpu/drm/i915/gt/debugfs_gt.h [deleted file]
drivers/gpu/drm/i915/gt/debugfs_gt_pm.c
drivers/gpu/drm/i915/gt/intel_gt.c
drivers/gpu/drm/i915/gt/intel_gt_debugfs.c [new file with mode: 0644]
drivers/gpu/drm/i915/gt/intel_gt_debugfs.h [new file with mode: 0644]
drivers/gpu/drm/i915/gt/intel_sseu_debugfs.c
drivers/gpu/drm/i915/gt/uc/intel_guc_debugfs.c
drivers/gpu/drm/i915/gt/uc/intel_guc_log_debugfs.c
drivers/gpu/drm/i915/gt/uc/intel_huc_debugfs.c
drivers/gpu/drm/i915/gt/uc/intel_uc_debugfs.c

index 335ba9f..7eab4ed 100644 (file)
@@ -79,7 +79,6 @@ i915-$(CONFIG_PERF_EVENTS) += i915_pmu.o
 # "Graphics Technology" (aka we talk to the gpu)
 gt-y += \
        gt/debugfs_engines.o \
-       gt/debugfs_gt.o \
        gt/debugfs_gt_pm.o \
        gt/gen2_engine_cs.o \
        gt/gen6_engine_cs.o \
@@ -100,6 +99,7 @@ gt-y += \
        gt/intel_gt.o \
        gt/intel_gt_buffer_pool.o \
        gt/intel_gt_clock_utils.o \
+       gt/intel_gt_debugfs.o \
        gt/intel_gt_irq.o \
        gt/intel_gt_pm.o \
        gt/intel_gt_pm_irq.o \
index 5e3725e..2980dac 100644 (file)
@@ -7,9 +7,9 @@
 #include <drm/drm_print.h>
 
 #include "debugfs_engines.h"
-#include "debugfs_gt.h"
 #include "i915_drv.h" /* for_each_engine! */
 #include "intel_engine.h"
+#include "intel_gt_debugfs.h"
 
 static int engines_show(struct seq_file *m, void *data)
 {
@@ -24,11 +24,11 @@ static int engines_show(struct seq_file *m, void *data)
 
        return 0;
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(engines);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(engines);
 
 void debugfs_engines_register(struct intel_gt *gt, struct dentry *root)
 {
-       static const struct debugfs_gt_file files[] = {
+       static const struct intel_gt_debugfs_file files[] = {
                { "engines", &engines_fops },
        };
 
diff --git a/drivers/gpu/drm/i915/gt/debugfs_gt.c b/drivers/gpu/drm/i915/gt/debugfs_gt.c
deleted file mode 100644 (file)
index 591eb60..0000000
+++ /dev/null
@@ -1,47 +0,0 @@
-// SPDX-License-Identifier: MIT
-/*
- * Copyright © 2019 Intel Corporation
- */
-
-#include <linux/debugfs.h>
-
-#include "debugfs_engines.h"
-#include "debugfs_gt.h"
-#include "debugfs_gt_pm.h"
-#include "intel_sseu_debugfs.h"
-#include "uc/intel_uc_debugfs.h"
-#include "i915_drv.h"
-
-void debugfs_gt_register(struct intel_gt *gt)
-{
-       struct dentry *root;
-
-       if (!gt->i915->drm.primary->debugfs_root)
-               return;
-
-       root = debugfs_create_dir("gt", gt->i915->drm.primary->debugfs_root);
-       if (IS_ERR(root))
-               return;
-
-       debugfs_engines_register(gt, root);
-       debugfs_gt_pm_register(gt, root);
-       intel_sseu_debugfs_register(gt, root);
-
-       intel_uc_debugfs_register(&gt->uc, root);
-}
-
-void intel_gt_debugfs_register_files(struct dentry *root,
-                                    const struct debugfs_gt_file *files,
-                                    unsigned long count, void *data)
-{
-       while (count--) {
-               umode_t mode = files->fops->write ? 0644 : 0444;
-
-               if (!files->eval || files->eval(data))
-                       debugfs_create_file(files->name,
-                                           mode, root, data,
-                                           files->fops);
-
-               files++;
-       }
-}
diff --git a/drivers/gpu/drm/i915/gt/debugfs_gt.h b/drivers/gpu/drm/i915/gt/debugfs_gt.h
deleted file mode 100644 (file)
index f77540f..0000000
+++ /dev/null
@@ -1,38 +0,0 @@
-/* SPDX-License-Identifier: MIT */
-/*
- * Copyright © 2019 Intel Corporation
- */
-
-#ifndef DEBUGFS_GT_H
-#define DEBUGFS_GT_H
-
-#include <linux/file.h>
-
-struct intel_gt;
-
-#define DEFINE_GT_DEBUGFS_ATTRIBUTE(__name)                            \
-       static int __name ## _open(struct inode *inode, struct file *file) \
-{                                                                      \
-       return single_open(file, __name ## _show, inode->i_private);    \
-}                                                                      \
-static const struct file_operations __name ## _fops = {                        \
-       .owner = THIS_MODULE,                                           \
-       .open = __name ## _open,                                        \
-       .read = seq_read,                                               \
-       .llseek = seq_lseek,                                            \
-       .release = single_release,                                      \
-}
-
-void debugfs_gt_register(struct intel_gt *gt);
-
-struct debugfs_gt_file {
-       const char *name;
-       const struct file_operations *fops;
-       bool (*eval)(void *data);
-};
-
-void intel_gt_debugfs_register_files(struct dentry *root,
-                                    const struct debugfs_gt_file *files,
-                                    unsigned long count, void *data);
-
-#endif /* DEBUGFS_GT_H */
index f6733f2..9222cf6 100644 (file)
@@ -6,11 +6,11 @@
 
 #include <linux/seq_file.h>
 
-#include "debugfs_gt.h"
 #include "debugfs_gt_pm.h"
 #include "i915_drv.h"
 #include "intel_gt.h"
 #include "intel_gt_clock_utils.h"
+#include "intel_gt_debugfs.h"
 #include "intel_gt_pm.h"
 #include "intel_llc.h"
 #include "intel_rc6.h"
@@ -36,7 +36,7 @@ static int fw_domains_show(struct seq_file *m, void *data)
 
        return 0;
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(fw_domains);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(fw_domains);
 
 static void print_rc6_res(struct seq_file *m,
                          const char *title,
@@ -238,7 +238,7 @@ static int drpc_show(struct seq_file *m, void *unused)
 
        return err;
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(drpc);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(drpc);
 
 static int frequency_show(struct seq_file *m, void *unused)
 {
@@ -480,7 +480,7 @@ static int frequency_show(struct seq_file *m, void *unused)
 
        return 0;
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(frequency);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(frequency);
 
 static int llc_show(struct seq_file *m, void *data)
 {
@@ -533,7 +533,7 @@ static bool llc_eval(void *data)
        return HAS_LLC(gt->i915);
 }
 
-DEFINE_GT_DEBUGFS_ATTRIBUTE(llc);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(llc);
 
 static const char *rps_power_to_str(unsigned int power)
 {
@@ -612,11 +612,11 @@ static bool rps_eval(void *data)
        return HAS_RPS(gt->i915);
 }
 
-DEFINE_GT_DEBUGFS_ATTRIBUTE(rps_boost);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(rps_boost);
 
 void debugfs_gt_pm_register(struct intel_gt *gt, struct dentry *root)
 {
-       static const struct debugfs_gt_file files[] = {
+       static const struct intel_gt_debugfs_file files[] = {
                { "drpc", &drpc_fops, NULL },
                { "frequency", &frequency_fops, NULL },
                { "forcewake", &fw_domains_fops, NULL },
index 55e87af..985922c 100644 (file)
@@ -3,7 +3,7 @@
  * Copyright © 2019 Intel Corporation
  */
 
-#include "debugfs_gt.h"
+#include "intel_gt_debugfs.h"
 
 #include "gem/i915_gem_lmem.h"
 #include "i915_drv.h"
 #include "intel_gt_requests.h"
 #include "intel_migrate.h"
 #include "intel_mocs.h"
+#include "intel_pm.h"
 #include "intel_rc6.h"
 #include "intel_renderstate.h"
 #include "intel_rps.h"
 #include "intel_uncore.h"
-#include "intel_pm.h"
 #include "shmem_utils.h"
 
 void intel_gt_init_early(struct intel_gt *gt, struct drm_i915_private *i915)
@@ -434,7 +434,7 @@ void intel_gt_driver_register(struct intel_gt *gt)
 {
        intel_rps_driver_register(&gt->rps);
 
-       debugfs_gt_register(gt);
+       intel_gt_debugfs_register(gt);
 }
 
 static int intel_gt_init_scratch(struct intel_gt *gt, unsigned int size)
diff --git a/drivers/gpu/drm/i915/gt/intel_gt_debugfs.c b/drivers/gpu/drm/i915/gt/intel_gt_debugfs.c
new file mode 100644 (file)
index 0000000..b196480
--- /dev/null
@@ -0,0 +1,47 @@
+// SPDX-License-Identifier: MIT
+/*
+ * Copyright © 2019 Intel Corporation
+ */
+
+#include <linux/debugfs.h>
+
+#include "debugfs_engines.h"
+#include "debugfs_gt_pm.h"
+#include "i915_drv.h"
+#include "intel_gt_debugfs.h"
+#include "intel_sseu_debugfs.h"
+#include "uc/intel_uc_debugfs.h"
+
+void intel_gt_debugfs_register(struct intel_gt *gt)
+{
+       struct dentry *root;
+
+       if (!gt->i915->drm.primary->debugfs_root)
+               return;
+
+       root = debugfs_create_dir("gt", gt->i915->drm.primary->debugfs_root);
+       if (IS_ERR(root))
+               return;
+
+       debugfs_engines_register(gt, root);
+       debugfs_gt_pm_register(gt, root);
+       intel_sseu_debugfs_register(gt, root);
+
+       intel_uc_debugfs_register(&gt->uc, root);
+}
+
+void intel_gt_debugfs_register_files(struct dentry *root,
+                                    const struct intel_gt_debugfs_file *files,
+                                    unsigned long count, void *data)
+{
+       while (count--) {
+               umode_t mode = files->fops->write ? 0644 : 0444;
+
+               if (!files->eval || files->eval(data))
+                       debugfs_create_file(files->name,
+                                           mode, root, data,
+                                           files->fops);
+
+               files++;
+       }
+}
diff --git a/drivers/gpu/drm/i915/gt/intel_gt_debugfs.h b/drivers/gpu/drm/i915/gt/intel_gt_debugfs.h
new file mode 100644 (file)
index 0000000..8b6fca0
--- /dev/null
@@ -0,0 +1,38 @@
+/* SPDX-License-Identifier: MIT */
+/*
+ * Copyright © 2019 Intel Corporation
+ */
+
+#ifndef INTEL_GT_DEBUGFS_H
+#define INTEL_GT_DEBUGFS_H
+
+#include <linux/file.h>
+
+struct intel_gt;
+
+#define DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(__name)                              \
+       static int __name ## _open(struct inode *inode, struct file *file) \
+{                                                                      \
+       return single_open(file, __name ## _show, inode->i_private);    \
+}                                                                      \
+static const struct file_operations __name ## _fops = {                        \
+       .owner = THIS_MODULE,                                           \
+       .open = __name ## _open,                                        \
+       .read = seq_read,                                               \
+       .llseek = seq_lseek,                                            \
+       .release = single_release,                                      \
+}
+
+void intel_gt_debugfs_register(struct intel_gt *gt);
+
+struct intel_gt_debugfs_file {
+       const char *name;
+       const struct file_operations *fops;
+       bool (*eval)(void *data);
+};
+
+void intel_gt_debugfs_register_files(struct dentry *root,
+                                    const struct intel_gt_debugfs_file *files,
+                                    unsigned long count, void *data);
+
+#endif /* INTEL_GT_DEBUGFS_H */
index 1ba8b7d..8bb3a91 100644 (file)
@@ -4,9 +4,9 @@
  * Copyright © 2020 Intel Corporation
  */
 
-#include "debugfs_gt.h"
-#include "intel_sseu_debugfs.h"
 #include "i915_drv.h"
+#include "intel_gt_debugfs.h"
+#include "intel_sseu_debugfs.h"
 
 static void sseu_copy_subslices(const struct sseu_dev_info *sseu,
                                int slice, u8 *to_mask)
@@ -282,7 +282,7 @@ static int sseu_status_show(struct seq_file *m, void *unused)
 
        return intel_sseu_status(m, gt);
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(sseu_status);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(sseu_status);
 
 static int rcs_topology_show(struct seq_file *m, void *unused)
 {
@@ -293,11 +293,11 @@ static int rcs_topology_show(struct seq_file *m, void *unused)
 
        return 0;
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(rcs_topology);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(rcs_topology);
 
 void intel_sseu_debugfs_register(struct intel_gt *gt, struct dentry *root)
 {
-       static const struct debugfs_gt_file files[] = {
+       static const struct intel_gt_debugfs_file files[] = {
                { "sseu_status", &sseu_status_fops, NULL },
                { "rcs_topology", &rcs_topology_fops, NULL },
        };
index 887c8c8..25f09a4 100644 (file)
@@ -5,14 +5,14 @@
 
 #include <drm/drm_print.h>
 
-#include "gt/debugfs_gt.h"
+#include "gt/intel_gt_debugfs.h"
+#include "gt/uc/intel_guc_ads.h"
+#include "gt/uc/intel_guc_ct.h"
+#include "gt/uc/intel_guc_slpc.h"
+#include "gt/uc/intel_guc_submission.h"
 #include "intel_guc.h"
 #include "intel_guc_debugfs.h"
 #include "intel_guc_log_debugfs.h"
-#include "gt/uc/intel_guc_ct.h"
-#include "gt/uc/intel_guc_ads.h"
-#include "gt/uc/intel_guc_submission.h"
-#include "gt/uc/intel_guc_slpc.h"
 
 static int guc_info_show(struct seq_file *m, void *data)
 {
@@ -35,7 +35,7 @@ static int guc_info_show(struct seq_file *m, void *data)
 
        return 0;
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(guc_info);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(guc_info);
 
 static int guc_registered_contexts_show(struct seq_file *m, void *data)
 {
@@ -49,7 +49,7 @@ static int guc_registered_contexts_show(struct seq_file *m, void *data)
 
        return 0;
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(guc_registered_contexts);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(guc_registered_contexts);
 
 static int guc_slpc_info_show(struct seq_file *m, void *unused)
 {
@@ -62,7 +62,7 @@ static int guc_slpc_info_show(struct seq_file *m, void *unused)
 
        return intel_guc_slpc_print_info(slpc, &p);
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(guc_slpc_info);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(guc_slpc_info);
 
 static bool intel_eval_slpc_support(void *data)
 {
@@ -73,7 +73,7 @@ static bool intel_eval_slpc_support(void *data)
 
 void intel_guc_debugfs_register(struct intel_guc *guc, struct dentry *root)
 {
-       static const struct debugfs_gt_file files[] = {
+       static const struct intel_gt_debugfs_file files[] = {
                { "guc_info", &guc_info_fops, NULL },
                { "guc_registered_contexts", &guc_registered_contexts_fops, NULL },
                { "guc_slpc_info", &guc_slpc_info_fops, &intel_eval_slpc_support},
index 64e0b86..46026c2 100644 (file)
@@ -6,7 +6,7 @@
 #include <linux/fs.h>
 #include <drm/drm_print.h>
 
-#include "gt/debugfs_gt.h"
+#include "gt/intel_gt_debugfs.h"
 #include "intel_guc.h"
 #include "intel_guc_log.h"
 #include "intel_guc_log_debugfs.h"
@@ -17,7 +17,7 @@ static int guc_log_dump_show(struct seq_file *m, void *data)
 
        return intel_guc_log_dump(m->private, &p, false);
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(guc_log_dump);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(guc_log_dump);
 
 static int guc_load_err_log_dump_show(struct seq_file *m, void *data)
 {
@@ -25,7 +25,7 @@ static int guc_load_err_log_dump_show(struct seq_file *m, void *data)
 
        return intel_guc_log_dump(m->private, &p, true);
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(guc_load_err_log_dump);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(guc_load_err_log_dump);
 
 static int guc_log_level_get(void *data, u64 *val)
 {
@@ -109,7 +109,7 @@ static const struct file_operations guc_log_relay_fops = {
 void intel_guc_log_debugfs_register(struct intel_guc_log *log,
                                    struct dentry *root)
 {
-       static const struct debugfs_gt_file files[] = {
+       static const struct intel_gt_debugfs_file files[] = {
                { "guc_log_dump", &guc_log_dump_fops, NULL },
                { "guc_load_err_log_dump", &guc_load_err_log_dump_fops, NULL },
                { "guc_log_level", &guc_log_level_fops, NULL },
index 5733c15..1599896 100644 (file)
@@ -5,7 +5,7 @@
 
 #include <drm/drm_print.h>
 
-#include "gt/debugfs_gt.h"
+#include "gt/intel_gt_debugfs.h"
 #include "intel_huc.h"
 #include "intel_huc_debugfs.h"
 
@@ -21,11 +21,11 @@ static int huc_info_show(struct seq_file *m, void *data)
 
        return 0;
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(huc_info);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(huc_info);
 
 void intel_huc_debugfs_register(struct intel_huc *huc, struct dentry *root)
 {
-       static const struct debugfs_gt_file files[] = {
+       static const struct intel_gt_debugfs_file files[] = {
                { "huc_info", &huc_info_fops, NULL },
        };
 
index 089d986..c2f7924 100644 (file)
@@ -6,7 +6,7 @@
 #include <linux/debugfs.h>
 #include <drm/drm_print.h>
 
-#include "gt/debugfs_gt.h"
+#include "gt/intel_gt_debugfs.h"
 #include "intel_guc_debugfs.h"
 #include "intel_huc_debugfs.h"
 #include "intel_uc.h"
@@ -32,11 +32,11 @@ static int uc_usage_show(struct seq_file *m, void *data)
 
        return 0;
 }
-DEFINE_GT_DEBUGFS_ATTRIBUTE(uc_usage);
+DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(uc_usage);
 
 void intel_uc_debugfs_register(struct intel_uc *uc, struct dentry *gt_root)
 {
-       static const struct debugfs_gt_file files[] = {
+       static const struct intel_gt_debugfs_file files[] = {
                { "usage", &uc_usage_fops, NULL },
        };
        struct dentry *root;