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@xxxxxxxxxxxxxxxxxxxxx] 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
