Hey Michel,
Thanks for the review coments.
On 9/27/2022 6:24 PM, Michel Dänzer wrote:
On 2022-09-26 23:40, Shashank Sharma wrote:
AMDGPU SOCs supports dynamic workload based power profiles, which can
provide fine-tuned performance for a particular type of workload.
This patch series adds an interface to set/reset these power profiles
based on the workload type hints. A user can set a hint of workload
type being submistted to GPU, and the driver can dynamically switch
the power profiles which is best suited to this kind of workload.
Currently supported workload profiles are:
"None", "3D", "Video", "VR", "Compute"
V2: This version addresses the review comment from Christian about
chaning the design to set workload mode in a more dynamic method
than during the context creation.
V3: Addressed review comment from Christian, Removed the get_workload()
calls from UAPI, keeping only the set_workload() call.
Shashank Sharma (5):
drm/amdgpu: add UAPI for workload hints to ctx ioctl
drm/amdgpu: add new functions to set GPU power profile
drm/amdgpu: set GPU workload via ctx IOCTL
drm/amdgpu: switch GPU workload profile
drm/amdgpu: switch workload context to/from compute
Where are the corresponding Mesa changes?
This series here was to get the feedback on the kernel side design
first. As you can see from the patch history, we have already changed
the design once and this is V2. So I thought it would be a good idea to
get the feedback on kernel UAPI, before starting sending patches to
mesa. The mesa/libdrm changes are ready and I was using those mixed with
libdrm/test/amdgpu stuff to validate the series.
Now I will fine tune them to match the feedback here, and send the
updated series.
- Shashank
