RE: [PATCH] drm/amdgpu: Report firmware versions with sysfs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The debugfs won't have anything in it that this interface won't provide. It does FW+VBIOS, and there will be separate files for each of those components.

>From a housekeeping standpoint, should we make a subfolder called fw_version to dump the files into, or are they fine in the base sysfs tree?

 Kent

-----Original Message-----
From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Christian König
Sent: Tuesday, May 7, 2019 1:35 PM
To: Messinger, Ori <Ori.Messinger@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx
Subject: Re: [PATCH] drm/amdgpu: Report firmware versions with sysfs

[CAUTION: External Email]

Am 07.05.19 um 19:30 schrieb Messinger, Ori:
> Firmware versions can be found as separate sysfs files at:
> /sys/class/drm/cardX/device/ (where X is the card number) The firmware 
> versions are displayed in hexadecimal.
>
> Change-Id: I10cae4c0ca6f1b6a9ced07da143426e1d011e203
> Signed-off-by: Ori Messinger <ori.messinger@xxxxxxx>

Well that looks like a really nice one, patch is Reviewed-by: Christian König <christian.koenig@xxxxxxx>

Could we remove the debugfs interface now or should we keep it?

Christian.

> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_device.c |  5 ++
>   drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c  | 71 ++++++++++++++++++++++
>   drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h  |  2 +
>   3 files changed, 78 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> index 3f1c6b2d3d87..6bfee8d1f1c3 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> @@ -2701,6 +2701,10 @@ int amdgpu_device_init(struct amdgpu_device *adev,
>       if (r)
>               DRM_ERROR("registering pm debugfs failed (%d).\n", r);
>
> +     r = amdgpu_ucode_sysfs_init(adev);
> +     if (r)
> +             DRM_ERROR("Creating firmware sysfs failed (%d).\n", r);
> +
>       r = amdgpu_debugfs_gem_init(adev);
>       if (r)
>               DRM_ERROR("registering gem debugfs failed (%d).\n", r); 
> @@ -2813,6 +2817,7 @@ void amdgpu_device_fini(struct amdgpu_device *adev)
>       amdgpu_device_doorbell_fini(adev);
>       amdgpu_debugfs_regs_cleanup(adev);
>       device_remove_file(adev->dev, &dev_attr_pcie_replay_count);
> +     amdgpu_ucode_sysfs_fini(adev);
>   }
>
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
> index 7b33867036e7..3aa750e6bbf6 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c
> @@ -313,6 +313,77 @@ amdgpu_ucode_get_load_type(struct amdgpu_device *adev, int load_type)
>       return AMDGPU_FW_LOAD_DIRECT;
>   }
>
> +#define FW_VERSION_ATTR(name, mode, field)                           \
> +static ssize_t show_##name(struct device *dev,                               \
> +                       struct device_attribute *attr,                \
> +                       char *buf)                                    \
> +{                                                                    \
> +     struct drm_device *ddev = dev_get_drvdata(dev);                 \
> +     struct amdgpu_device *adev = ddev->dev_private;                 \
> +                                                                     \
> +     return snprintf(buf, PAGE_SIZE, "0x%08x\n", adev->field);       \
> +}                                                                    \
> +static DEVICE_ATTR(name, mode, show_##name, NULL)
> +
> +FW_VERSION_ATTR(vce_fw_version, 0444, vce.fw_version); 
> +FW_VERSION_ATTR(uvd_fw_version, 0444, uvd.fw_version); 
> +FW_VERSION_ATTR(mc_fw_version, 0444, gmc.fw_version); 
> +FW_VERSION_ATTR(me_fw_version, 0444, gfx.me_fw_version); 
> +FW_VERSION_ATTR(pfp_fw_version, 0444, gfx.pfp_fw_version); 
> +FW_VERSION_ATTR(ce_fw_version, 0444, gfx.ce_fw_version); 
> +FW_VERSION_ATTR(rlc_fw_version, 0444, gfx.rlc_fw_version); 
> +FW_VERSION_ATTR(rlc_srlc_fw_version, 0444, gfx.rlc_srlc_fw_version); 
> +FW_VERSION_ATTR(rlc_srlg_fw_version, 0444, gfx.rlc_srlg_fw_version); 
> +FW_VERSION_ATTR(rlc_srls_fw_version, 0444, gfx.rlc_srls_fw_version); 
> +FW_VERSION_ATTR(mec_fw_version, 0444, gfx.mec_fw_version); 
> +FW_VERSION_ATTR(mec2_fw_version, 0444, gfx.mec2_fw_version); 
> +FW_VERSION_ATTR(sos_fw_version, 0444, psp.sos_fw_version); 
> +FW_VERSION_ATTR(asd_fw_version, 0444, psp.asd_fw_version); 
> +FW_VERSION_ATTR(ta_ras_fw_version, 0444, psp.ta_fw_version); 
> +FW_VERSION_ATTR(ta_xgmi_fw_version, 0444, psp.ta_fw_version); 
> +FW_VERSION_ATTR(smc_fw_version, 0444, pm.fw_version); 
> +FW_VERSION_ATTR(sdma_fw_version, 0444, sdma.instance[0].fw_version); 
> +FW_VERSION_ATTR(sdma2_fw_version, 0444, sdma.instance[1].fw_version); 
> +FW_VERSION_ATTR(vcn_fw_version, 0444, vcn.fw_version); 
> +FW_VERSION_ATTR(dmcu_fw_version, 0444, dm.dmcu_fw_version);
> +
> +static struct device_attribute *dev_fw_attr[] = {
> +     &dev_attr_vce_fw_version, &dev_attr_uvd_fw_version,
> +     &dev_attr_mc_fw_version, &dev_attr_me_fw_version,
> +     &dev_attr_pfp_fw_version, &dev_attr_ce_fw_version,
> +     &dev_attr_rlc_fw_version, &dev_attr_rlc_srlc_fw_version,
> +     &dev_attr_rlc_srlg_fw_version, &dev_attr_rlc_srls_fw_version,
> +     &dev_attr_mec_fw_version, &dev_attr_mec2_fw_version,
> +     &dev_attr_sos_fw_version, &dev_attr_asd_fw_version,
> +     &dev_attr_ta_ras_fw_version, &dev_attr_ta_xgmi_fw_version,
> +     &dev_attr_smc_fw_version, &dev_attr_sdma_fw_version,
> +     &dev_attr_sdma2_fw_version, &dev_attr_vcn_fw_version,
> +     &dev_attr_dmcu_fw_version
> +};
> +
> +void amdgpu_ucode_sysfs_fini(struct amdgpu_device *adev) {
> +     int i;
> +
> +     for (i = 0; i < ARRAY_SIZE(dev_fw_attr); i++)
> +             device_remove_file(adev->dev, dev_fw_attr[i]); }
> +
> +int amdgpu_ucode_sysfs_init(struct amdgpu_device *adev) {
> +     int i, ret;
> +
> +     for (i = 0; i < ARRAY_SIZE(dev_fw_attr); i++) {
> +             ret = device_create_file(adev->dev, dev_fw_attr[i]);
> +             if (ret) {
> +                     DRM_ERROR("Failed to create %s\n",
> +                               dev_fw_attr[i]->attr.name);
> +                     return ret;
> +             }
> +     }
> +     return 0;
> +}
> +
>   static int amdgpu_ucode_init_single_fw(struct amdgpu_device *adev,
>                                      struct amdgpu_firmware_info *ucode,
>                                      uint64_t mc_addr, void *kptr) 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h
> index 7ac25a1c7853..ec4c2ea1f05a 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h
> @@ -291,7 +291,9 @@ bool amdgpu_ucode_hdr_version(union 
> amdgpu_firmware_header *hdr,
>
>   int amdgpu_ucode_init_bo(struct amdgpu_device *adev);
>   int amdgpu_ucode_create_bo(struct amdgpu_device *adev);
> +int amdgpu_ucode_sysfs_init(struct amdgpu_device *adev);
>   void amdgpu_ucode_free_bo(struct amdgpu_device *adev);
> +void amdgpu_ucode_sysfs_fini(struct amdgpu_device *adev);
>
>   enum amdgpu_firmware_load_type
>   amdgpu_ucode_get_load_type(struct amdgpu_device *adev, int 
> load_type);

_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux