[PATCH] drm/amdgpu: Add sysfs file for VBIOS

[Russell, Kent]

Right. And according to Christian, it doesn't fit the format anyways, so power usage can stay. I'm working on the patch to move the vbios version to sysfs from debugfs. Should be up shortly. Maybe I can just make a new sysfs file for Average GPU usage by itself. Another task for another day. But I won't be moving the sensors over. Just vbios version for now, maybe average GPU power usage later.

 Kent

-----Original Message-----
From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf Of Tom St Denis
Sent: Friday, August 25, 2017 9:34 AM
To: amd-gfx at lists.freedesktop.org
Subject: Re: [PATCH] drm/amdgpu: Add sysfs file for VBIOS

On 25/08/17 09:05 AM, Tom St Denis wrote:
> On 25/08/17 08:59 AM, Russell, Kent wrote:
>> There is GPU Power usage reported through amdgpu_pm_info, which also 
>> has some other information as well. I'd like that in sysfs, but I am 
>> unsure if we are  allowed to due to the other information reported there.
> 
> I thought the sensors were added to the DRM ioctl but apparently not.

Nope, it was actually (see amdgpu_kms.c).  Under an AMDGPU_INFO command you ask for AMDGPU_INFO_SENSOR and then fill out sensor_info.type.

Tom
_______________________________________________
amd-gfx mailing list
amd-gfx at lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx