To avoid conflicts and confusion, I think better remove two old sysfs.
Well that is not a matter of personal opinion. The rule is simply that
sysfs needs to be stable, otherwise we can't upstream a sysfs interface.
Especially since you already noted that current ROCM tool and KFD depend
on them.
What we can argue is that if this never worked on Vega10 we can remove
it for Vega10 but keep it for older ASICs.
Alternative we can have some sort of compatibility code which translates
the old interface into the new one, e.g. in this case I think a simple
symlink should do.
Regards,
Christian.
Am 26.02.2018 um 03:54 schrieb Zhu, Rex:
We have added new sysfs pp_power_profile_mode.
The old sysfs (pp_compute_power_profile and pp_gfx_power_profile)
were used internally currently by ROCM tool and KFD to switch mode between gfx
and compute only on VI/CI asics.
We can't extend more modes for other workloads, for example: video, power
saving mode,etc.
And the old sysfs can't support VEGA.
With new sysfs, User can switch mode based on workloads(video, power save, gfx,
compute, VR), or configure custom mode.
To avoid conflicts and confusion, I think better remove two old sysfs.
Best Regards
Rex
-----Original Message-----
From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com]
Sent: Sunday, February 25, 2018 9:19 PM
To: Zhu, Rex; amd-gfx@lists.freedesktop.org
Subject: Re: [PATCH 01/10] drm/amdgpu: bump version for new power profile sysfs
Am 24.02.2018 um 14:55 schrieb Rex Zhu:
Add new power profile sysfs: pp_power_profile_mode old sysfs:
pp_compute_power_profile/pp_gfx_power_profile
will be removed
Well sysfs is supposed to be stable, so removing the old interface won't be
allowed.
Change-Id: Ief35efb81b313115b33c430dfb328ad83ce7975d
Signed-off-by: Rex Zhu <rex....@amd.com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
index 69cf490..4c92239 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
@@ -75,6 +75,7 @@
* - 3.23.0 - Add query for VRAM lost counter
* - 3.24.0 - Add high priority compute support for gfx9
* - 3.25.0 - Add support for sensor query info (stable pstate sclk/mclk).
+ * - 3.26.0 - Delete old gfx/compute profile sysfs, add common profile sysfs.
*/
#define KMS_DRIVER_MAJOR 3
#define KMS_DRIVER_MINOR 25
Well you actually also need to bump the version, e.g. modify KMS_DRIVER_MINOR.
Christian.
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
