On 2022-09-27 18:59, Sharma, Shashank wrote: > 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.
In general, it's not possible to review UAPI without the corresponding user-space code. I don't think this is an exception. -- Earthling Michel Dänzer | https://redhat.com Libre software enthusiast | Mesa and Xwayland developer