[PATCH] drm/amd/amdgpu: Add a GPU_LOAD entry to sysfs

Alex Deucher alexdeucher at gmail.com
Wed Jun 20 14:52:58 UTC 2018


On Wed, Jun 20, 2018 at 8:31 AM, Tom St Denis <tom.stdenis at amd.com> wrote:
> This adds what should be a stable interface to read GPU
> load from userspace.
>
> Signed-off-by: Tom St Denis <tom.stdenis at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c | 41 ++++++++++++++++++++++++++++++++++
>  1 file changed, 41 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
> index 113edffb5960..d57b414ac228 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
> @@ -918,6 +918,37 @@ static ssize_t amdgpu_set_pp_power_profile_mode(struct device *dev,
>         return -EINVAL;
>  }
>
> +static ssize_t amdgpu_get_busy_level(struct device *dev,
> +               struct device_attribute *attr,
> +               char *buf)
> +{
> +       struct drm_device *ddev = dev_get_drvdata(dev);
> +       struct amdgpu_device *adev = ddev->dev_private;
> +       int r, value, size = sizeof(value);
> +
> +       /* sanity check PP is enabled */
> +       if (!(adev->powerplay.pp_funcs &&
> +             adev->powerplay.pp_funcs->read_sensor))
> +               return -EINVAL;
> +
> +       /* get the temperature */
> +       r = amdgpu_dpm_read_sensor(adev, AMDGPU_PP_SENSOR_GPU_LOAD,
> +                                  (void *)&value, &size);
> +       if (r)
> +               return r;
> +
> +       return snprintf(buf, PAGE_SIZE, "%d\n", value);
> +}
> +
> +static ssize_t amdgpu_set_gpu_busy_level(struct device *dev,
> +               struct device_attribute *attr,
> +               const char *buf,
> +               size_t count)
> +{
> +       return -EINVAL;
> +}
> +
> +
>  static DEVICE_ATTR(power_dpm_state, S_IRUGO | S_IWUSR, amdgpu_get_dpm_state, amdgpu_set_dpm_state);
>  static DEVICE_ATTR(power_dpm_force_performance_level, S_IRUGO | S_IWUSR,
>                    amdgpu_get_dpm_forced_performance_level,
> @@ -951,6 +982,8 @@ static DEVICE_ATTR(pp_power_profile_mode, S_IRUGO | S_IWUSR,
>  static DEVICE_ATTR(pp_od_clk_voltage, S_IRUGO | S_IWUSR,
>                 amdgpu_get_pp_od_clk_voltage,
>                 amdgpu_set_pp_od_clk_voltage);
> +static DEVICE_ATTR(gpu_busy_level, S_IRUGO | S_IWUSR,
> +               amdgpu_get_busy_level, amdgpu_set_gpu_busy_level);


Maybe call the node gpu_busy_percent since it returns a percentage?

>
>  static ssize_t amdgpu_hwmon_show_temp(struct device *dev,
>                                       struct device_attribute *attr,
> @@ -1854,6 +1887,13 @@ int amdgpu_pm_sysfs_init(struct amdgpu_device *adev)
>                                 "pp_od_clk_voltage\n");
>                 return ret;
>         }
> +       ret = device_create_file(adev->dev,
> +                       &dev_attr_gpu_busy_level);
> +       if (ret) {
> +               DRM_ERROR("failed to create device file "
> +                               "gpu_busy_level\n");
> +               return ret;
> +       }
>         ret = amdgpu_debugfs_pm_init(adev);
>         if (ret) {
>                 DRM_ERROR("Failed to register debugfs file for dpm!\n");
> @@ -1889,6 +1929,7 @@ void amdgpu_pm_sysfs_fini(struct amdgpu_device *adev)
>                         &dev_attr_pp_power_profile_mode);
>         device_remove_file(adev->dev,
>                         &dev_attr_pp_od_clk_voltage);
> +       device_remove_file(adev->dev, &dev_attr_gpu_busy_level);
>  }
>
>  void amdgpu_pm_compute_clocks(struct amdgpu_device *adev)
> --
> 2.14.4
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


More information about the amd-gfx mailing list