drm/i915/gt: Create per-tile RC6 sysfs interface
authorAndi Shyti <andi.shyti@linux.intel.com>
Fri, 18 Mar 2022 23:39:35 +0000 (01:39 +0200)
committerMatthew Auld <matthew.auld@intel.com>
Mon, 21 Mar 2022 08:37:42 +0000 (08:37 +0000)
Now tiles have their own sysfs interfaces under the gt/
directory. Because RC6 is a property that can be configured on a
tile basis, then each tile should have its own interface

The new sysfs structure will have a similar layout for the 4 tile
case:

/sys/.../card0
         ├── gt
         │   ├── gt0
         │   │   ├── id
         │   │   ├── rc6_enable
         │   │   ├── rc6_residency_ms
         .   .   .
         .   .   .
         .   .
         │   └── gtN
         │       ├── id
         │       ├── rc6_enable
         │       ├── rc6_residency_ms
         │       .
         │       .
         │
         └── power/                -+
              ├── rc6_enable        |    Original interface
              ├── rc6_residency_ms  +->  kept as existing ABI;
              .                     |    it multiplexes over
              .                     |    the GTs
                                   -+

The existing interfaces have been kept in their original location
to preserve the existing ABI. They act on all the GTs: when
reading they provide the average value from all the GTs.

This patch is not really adding exposing new interfaces (new
ABI) other than adapting the existing one to more tiles. In any
case this new set of interfaces will be a basic tool for system
managers and administrators when using i915.

Signed-off-by: Andi Shyti <andi.shyti@linux.intel.com>
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Matt Roper <matthew.d.roper@intel.com>
Cc: Sujaritha Sundaresan <sujaritha.sundaresan@intel.com>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Reviewed-by: Andrzej Hajda <andrzej.hajda@intel.com>
Signed-off-by: Matthew Auld <matthew.auld@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220318233938.149744-6-andi.shyti@linux.intel.com
drivers/gpu/drm/i915/Makefile
drivers/gpu/drm/i915/gt/intel_gt_sysfs.c
drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.c [new file with mode: 0644]
drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.h [new file with mode: 0644]
drivers/gpu/drm/i915/i915_sysfs.c

index f68d62d2bbea78934eb44c9c597ad1521c58f8db..9801aaa9885d42ff721d17bb79a3a34dcb662242 100644 (file)
@@ -106,6 +106,7 @@ gt-y += \
        gt/intel_gt_pm_irq.o \
        gt/intel_gt_requests.o \
        gt/intel_gt_sysfs.o \
+       gt/intel_gt_sysfs_pm.o \
        gt/intel_gtt.o \
        gt/intel_llc.o \
        gt/intel_lrc.o \
index d5083196129445d79a806db67f877a168751e0e1..8ec8bc660c8c2bbf88c551bcfac3294753b0123b 100644 (file)
@@ -13,6 +13,7 @@
 #include "i915_sysfs.h"
 #include "intel_gt.h"
 #include "intel_gt_sysfs.h"
+#include "intel_gt_sysfs_pm.h"
 #include "intel_gt_types.h"
 #include "intel_rc6.h"
 
@@ -50,6 +51,11 @@ struct intel_gt *intel_gt_sysfs_get_drvdata(struct device *dev,
        return kobj_to_gt(kobj);
 }
 
+static struct kobject *gt_get_parent_obj(struct intel_gt *gt)
+{
+       return &gt->i915->drm.primary->kdev->kobj;
+}
+
 static ssize_t id_show(struct device *dev,
                       struct device_attribute *attr,
                       char *buf)
@@ -81,6 +87,17 @@ void intel_gt_sysfs_register(struct intel_gt *gt)
 {
        struct kobj_gt *kg;
 
+       /*
+        * We need to make things right with the
+        * ABI compatibility. The files were originally
+        * generated under the parent directory.
+        *
+        * We generate the files only for gt 0
+        * to avoid duplicates.
+        */
+       if (gt_is_root(gt))
+               intel_gt_sysfs_pm_init(gt, gt_get_parent_obj(gt));
+
        kg = kzalloc(sizeof(*kg), GFP_KERNEL);
        if (!kg)
                goto exit_fail;
@@ -92,6 +109,8 @@ void intel_gt_sysfs_register(struct intel_gt *gt)
        if (kobject_add(&kg->base, gt->i915->sysfs_gt, "gt%d", gt->info.id))
                goto exit_kobj_put;
 
+       intel_gt_sysfs_pm_init(gt, &kg->base);
+
        return;
 
 exit_kobj_put:
diff --git a/drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.c b/drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.c
new file mode 100644 (file)
index 0000000..144b004
--- /dev/null
@@ -0,0 +1,244 @@
+// SPDX-License-Identifier: MIT
+/*
+ * Copyright © 2022 Intel Corporation
+ */
+
+#include <drm/drm_device.h>
+#include <linux/sysfs.h>
+#include <linux/printk.h>
+
+#include "i915_drv.h"
+#include "i915_sysfs.h"
+#include "intel_gt.h"
+#include "intel_gt_regs.h"
+#include "intel_gt_sysfs.h"
+#include "intel_gt_sysfs_pm.h"
+#include "intel_rc6.h"
+
+#ifdef CONFIG_PM
+enum intel_gt_sysfs_op {
+       INTEL_GT_SYSFS_MIN = 0,
+       INTEL_GT_SYSFS_MAX,
+};
+
+static u32
+sysfs_gt_attribute_r_func(struct device *dev, struct device_attribute *attr,
+                         u32 (func)(struct intel_gt *gt),
+                         enum intel_gt_sysfs_op op)
+{
+       struct intel_gt *gt;
+       u32 ret;
+
+       ret = (op == INTEL_GT_SYSFS_MAX) ? 0 : (u32) -1;
+
+       if (!is_object_gt(&dev->kobj)) {
+               int i;
+               struct drm_i915_private *i915 = kdev_minor_to_i915(dev);
+
+               for_each_gt(gt, i915, i) {
+                       u32 val = func(gt);
+
+                       switch (op) {
+                       case INTEL_GT_SYSFS_MIN:
+                               if (val < ret)
+                                       ret = val;
+                               break;
+
+                       case INTEL_GT_SYSFS_MAX:
+                               if (val > ret)
+                                       ret = val;
+                               break;
+                       }
+               }
+       } else {
+               gt = intel_gt_sysfs_get_drvdata(dev, attr->attr.name);
+               ret = func(gt);
+       }
+
+       return ret;
+}
+
+/* RC6 interfaces will show the minimum RC6 residency value */
+#define sysfs_gt_attribute_r_min_func(d, a, f) \
+               sysfs_gt_attribute_r_func(d, a, f, INTEL_GT_SYSFS_MIN)
+
+#define sysfs_gt_attribute_r_max_func(d, a, f) \
+               sysfs_gt_attribute_r_func(d, a, f, INTEL_GT_SYSFS_MAX)
+
+static u32 get_residency(struct intel_gt *gt, i915_reg_t reg)
+{
+       intel_wakeref_t wakeref;
+       u64 res = 0;
+
+       with_intel_runtime_pm(gt->uncore->rpm, wakeref)
+               res = intel_rc6_residency_us(&gt->rc6, reg);
+
+       return DIV_ROUND_CLOSEST_ULL(res, 1000);
+}
+
+static ssize_t rc6_enable_show(struct device *dev,
+                              struct device_attribute *attr,
+                              char *buff)
+{
+       struct intel_gt *gt = intel_gt_sysfs_get_drvdata(dev, attr->attr.name);
+       u8 mask = 0;
+
+       if (HAS_RC6(gt->i915))
+               mask |= BIT(0);
+       if (HAS_RC6p(gt->i915))
+               mask |= BIT(1);
+       if (HAS_RC6pp(gt->i915))
+               mask |= BIT(2);
+
+       return sysfs_emit(buff, "%x\n", mask);
+}
+
+static u32 __rc6_residency_ms_show(struct intel_gt *gt)
+{
+       return get_residency(gt, GEN6_GT_GFX_RC6);
+}
+
+static ssize_t rc6_residency_ms_show(struct device *dev,
+                                    struct device_attribute *attr,
+                                    char *buff)
+{
+       u32 rc6_residency = sysfs_gt_attribute_r_min_func(dev, attr,
+                                                     __rc6_residency_ms_show);
+
+       return sysfs_emit(buff, "%u\n", rc6_residency);
+}
+
+static u32 __rc6p_residency_ms_show(struct intel_gt *gt)
+{
+       return get_residency(gt, GEN6_GT_GFX_RC6p);
+}
+
+static ssize_t rc6p_residency_ms_show(struct device *dev,
+                                     struct device_attribute *attr,
+                                     char *buff)
+{
+       u32 rc6p_residency = sysfs_gt_attribute_r_min_func(dev, attr,
+                                               __rc6p_residency_ms_show);
+
+       return sysfs_emit(buff, "%u\n", rc6p_residency);
+}
+
+static u32 __rc6pp_residency_ms_show(struct intel_gt *gt)
+{
+       return get_residency(gt, GEN6_GT_GFX_RC6pp);
+}
+
+static ssize_t rc6pp_residency_ms_show(struct device *dev,
+                                      struct device_attribute *attr,
+                                      char *buff)
+{
+       u32 rc6pp_residency = sysfs_gt_attribute_r_min_func(dev, attr,
+                                               __rc6pp_residency_ms_show);
+
+       return sysfs_emit(buff, "%u\n", rc6pp_residency);
+}
+
+static u32 __media_rc6_residency_ms_show(struct intel_gt *gt)
+{
+       return get_residency(gt, VLV_GT_MEDIA_RC6);
+}
+
+static ssize_t media_rc6_residency_ms_show(struct device *dev,
+                                          struct device_attribute *attr,
+                                          char *buff)
+{
+       u32 rc6_residency = sysfs_gt_attribute_r_min_func(dev, attr,
+                                               __media_rc6_residency_ms_show);
+
+       return sysfs_emit(buff, "%u\n", rc6_residency);
+}
+
+static DEVICE_ATTR_RO(rc6_enable);
+static DEVICE_ATTR_RO(rc6_residency_ms);
+static DEVICE_ATTR_RO(rc6p_residency_ms);
+static DEVICE_ATTR_RO(rc6pp_residency_ms);
+static DEVICE_ATTR_RO(media_rc6_residency_ms);
+
+static struct attribute *rc6_attrs[] = {
+       &dev_attr_rc6_enable.attr,
+       &dev_attr_rc6_residency_ms.attr,
+       NULL
+};
+
+static struct attribute *rc6p_attrs[] = {
+       &dev_attr_rc6p_residency_ms.attr,
+       &dev_attr_rc6pp_residency_ms.attr,
+       NULL
+};
+
+static struct attribute *media_rc6_attrs[] = {
+       &dev_attr_media_rc6_residency_ms.attr,
+       NULL
+};
+
+static const struct attribute_group rc6_attr_group[] = {
+       { .attrs = rc6_attrs, },
+       { .name = power_group_name, .attrs = rc6_attrs, },
+};
+
+static const struct attribute_group rc6p_attr_group[] = {
+       { .attrs = rc6p_attrs, },
+       { .name = power_group_name, .attrs = rc6p_attrs, },
+};
+
+static const struct attribute_group media_rc6_attr_group[] = {
+       { .attrs = media_rc6_attrs, },
+       { .name = power_group_name, .attrs = media_rc6_attrs, },
+};
+
+static int __intel_gt_sysfs_create_group(struct kobject *kobj,
+                                        const struct attribute_group *grp)
+{
+       return is_object_gt(kobj) ?
+              sysfs_create_group(kobj, &grp[0]) :
+              sysfs_merge_group(kobj, &grp[1]);
+}
+
+static void intel_sysfs_rc6_init(struct intel_gt *gt, struct kobject *kobj)
+{
+       int ret;
+
+       if (!HAS_RC6(gt->i915))
+               return;
+
+       ret = __intel_gt_sysfs_create_group(kobj, rc6_attr_group);
+       if (ret)
+               drm_warn(&gt->i915->drm,
+                        "failed to create gt%u RC6 sysfs files (%pe)\n",
+                        gt->info.id, ERR_PTR(ret));
+
+       /*
+        * cannot use the is_visible() attribute because
+        * the upper object inherits from the parent group.
+        */
+       if (HAS_RC6p(gt->i915)) {
+               ret = __intel_gt_sysfs_create_group(kobj, rc6p_attr_group);
+               if (ret)
+                       drm_warn(&gt->i915->drm,
+                                "failed to create gt%u RC6p sysfs files (%pe)\n",
+                                gt->info.id, ERR_PTR(ret));
+       }
+
+       if (IS_VALLEYVIEW(gt->i915) || IS_CHERRYVIEW(gt->i915)) {
+               ret = __intel_gt_sysfs_create_group(kobj, media_rc6_attr_group);
+               if (ret)
+                       drm_warn(&gt->i915->drm,
+                                "failed to create media %u RC6 sysfs files (%pe)\n",
+                                gt->info.id, ERR_PTR(ret));
+       }
+}
+#else
+static void intel_sysfs_rc6_init(struct intel_gt *gt, struct kobject *kobj)
+{
+}
+#endif /* CONFIG_PM */
+
+void intel_gt_sysfs_pm_init(struct intel_gt *gt, struct kobject *kobj)
+{
+       intel_sysfs_rc6_init(gt, kobj);
+}
diff --git a/drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.h b/drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.h
new file mode 100644 (file)
index 0000000..f567105
--- /dev/null
@@ -0,0 +1,15 @@
+/* SPDX-License-Identifier: MIT */
+/*
+ * Copyright © 2022 Intel Corporation
+ */
+
+#ifndef __SYSFS_GT_PM_H__
+#define __SYSFS_GT_PM_H__
+
+#include <linux/kobject.h>
+
+#include "intel_gt_types.h"
+
+void intel_gt_sysfs_pm_init(struct intel_gt *gt, struct kobject *kobj);
+
+#endif /* SYSFS_RC6_H */
index 5ff915b93c21abdce86c6973c7e8fe2f7dcd7d70..2c5d62a521f64e462adfe5907fbd20b80fe7dabb 100644 (file)
@@ -45,107 +45,6 @@ struct drm_i915_private *kdev_minor_to_i915(struct device *kdev)
        return to_i915(minor->dev);
 }
 
-#ifdef CONFIG_PM
-static u32 calc_residency(struct drm_i915_private *dev_priv,
-                         i915_reg_t reg)
-{
-       intel_wakeref_t wakeref;
-       u64 res = 0;
-
-       with_intel_runtime_pm(&dev_priv->runtime_pm, wakeref)
-               res = intel_rc6_residency_us(&to_gt(dev_priv)->rc6, reg);
-
-       return DIV_ROUND_CLOSEST_ULL(res, 1000);
-}
-
-static ssize_t rc6_enable_show(struct device *kdev,
-                              struct device_attribute *attr, char *buf)
-{
-       struct drm_i915_private *dev_priv = kdev_minor_to_i915(kdev);
-       unsigned int mask;
-
-       mask = 0;
-       if (HAS_RC6(dev_priv))
-               mask |= BIT(0);
-       if (HAS_RC6p(dev_priv))
-               mask |= BIT(1);
-       if (HAS_RC6pp(dev_priv))
-               mask |= BIT(2);
-
-       return sysfs_emit(buf, "%x\n", mask);
-}
-
-static ssize_t rc6_residency_ms_show(struct device *kdev,
-                                    struct device_attribute *attr, char *buf)
-{
-       struct drm_i915_private *dev_priv = kdev_minor_to_i915(kdev);
-       u32 rc6_residency = calc_residency(dev_priv, GEN6_GT_GFX_RC6);
-       return sysfs_emit(buf, "%u\n", rc6_residency);
-}
-
-static ssize_t rc6p_residency_ms_show(struct device *kdev,
-                                     struct device_attribute *attr, char *buf)
-{
-       struct drm_i915_private *dev_priv = kdev_minor_to_i915(kdev);
-       u32 rc6p_residency = calc_residency(dev_priv, GEN6_GT_GFX_RC6p);
-       return sysfs_emit(buf, "%u\n", rc6p_residency);
-}
-
-static ssize_t rc6pp_residency_ms_show(struct device *kdev,
-                                      struct device_attribute *attr, char *buf)
-{
-       struct drm_i915_private *dev_priv = kdev_minor_to_i915(kdev);
-       u32 rc6pp_residency = calc_residency(dev_priv, GEN6_GT_GFX_RC6pp);
-       return sysfs_emit(buf, "%u\n", rc6pp_residency);
-}
-
-static ssize_t media_rc6_residency_ms_show(struct device *kdev,
-                                          struct device_attribute *attr, char *buf)
-{
-       struct drm_i915_private *dev_priv = kdev_minor_to_i915(kdev);
-       u32 rc6_residency = calc_residency(dev_priv, VLV_GT_MEDIA_RC6);
-       return sysfs_emit(buf, "%u\n", rc6_residency);
-}
-
-static DEVICE_ATTR_RO(rc6_enable);
-static DEVICE_ATTR_RO(rc6_residency_ms);
-static DEVICE_ATTR_RO(rc6p_residency_ms);
-static DEVICE_ATTR_RO(rc6pp_residency_ms);
-static DEVICE_ATTR_RO(media_rc6_residency_ms);
-
-static struct attribute *rc6_attrs[] = {
-       &dev_attr_rc6_enable.attr,
-       &dev_attr_rc6_residency_ms.attr,
-       NULL
-};
-
-static const struct attribute_group rc6_attr_group = {
-       .name = power_group_name,
-       .attrs =  rc6_attrs
-};
-
-static struct attribute *rc6p_attrs[] = {
-       &dev_attr_rc6p_residency_ms.attr,
-       &dev_attr_rc6pp_residency_ms.attr,
-       NULL
-};
-
-static const struct attribute_group rc6p_attr_group = {
-       .name = power_group_name,
-       .attrs =  rc6p_attrs
-};
-
-static struct attribute *media_rc6_attrs[] = {
-       &dev_attr_media_rc6_residency_ms.attr,
-       NULL
-};
-
-static const struct attribute_group media_rc6_attr_group = {
-       .name = power_group_name,
-       .attrs =  media_rc6_attrs
-};
-#endif
-
 static int l3_access_valid(struct drm_i915_private *i915, loff_t offset)
 {
        if (!HAS_L3_DPF(i915))
@@ -492,29 +391,6 @@ void i915_setup_sysfs(struct drm_i915_private *dev_priv)
        struct device *kdev = dev_priv->drm.primary->kdev;
        int ret;
 
-#ifdef CONFIG_PM
-       if (HAS_RC6(dev_priv)) {
-               ret = sysfs_merge_group(&kdev->kobj,
-                                       &rc6_attr_group);
-               if (ret)
-                       drm_err(&dev_priv->drm,
-                               "RC6 residency sysfs setup failed\n");
-       }
-       if (HAS_RC6p(dev_priv)) {
-               ret = sysfs_merge_group(&kdev->kobj,
-                                       &rc6p_attr_group);
-               if (ret)
-                       drm_err(&dev_priv->drm,
-                               "RC6p residency sysfs setup failed\n");
-       }
-       if (IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv)) {
-               ret = sysfs_merge_group(&kdev->kobj,
-                                       &media_rc6_attr_group);
-               if (ret)
-                       drm_err(&dev_priv->drm,
-                               "Media RC6 residency sysfs setup failed\n");
-       }
-#endif
        if (HAS_L3_DPF(dev_priv)) {
                ret = device_create_bin_file(kdev, &dpf_attrs);
                if (ret)
@@ -560,8 +436,4 @@ void i915_teardown_sysfs(struct drm_i915_private *dev_priv)
                sysfs_remove_files(&kdev->kobj, gen6_attrs);
        device_remove_bin_file(kdev,  &dpf_attrs_1);
        device_remove_bin_file(kdev,  &dpf_attrs);
-#ifdef CONFIG_PM
-       sysfs_unmerge_group(&kdev->kobj, &rc6_attr_group);
-       sysfs_unmerge_group(&kdev->kobj, &rc6p_attr_group);
-#endif
 }