[PATCH 4/4] drm/amd/powerplay: add profiling mode in dpm level

[Zhu, Rex]

Thanks for quick response.

>>>The patch doesn't apply directly to drm-next-4.10-wip, here's what I tested -

Sorry, it is because code base has been changed in my local.

Please review the attached patch.

Best Regards
Rex


-----Original Message-----
From: Andy Furniss [mailto:adf.lists at gmail.com] 
Sent: Monday, January 09, 2017 6:29 PM
To: Zhu, Rex; Alex Deucher
Cc: amd-gfx list
Subject: Re: [PATCH 4/4] drm/amd/powerplay: add profiling mode in dpm level

Hi, that change does fix it for me.

The patch doesn't apply directly to drm-next-4.10-wip, here's what I tested -

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
index 0345fbb..ac2e5f4 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c
@@ -162,7 +162,7 @@ static ssize_t
amdgpu_set_dpm_forced_performance_level(struct device *dev,
         }

         if (current_level == level)
-               return 0;
+               return count;

         if (level == AMD_DPM_FORCED_LEVEL_PROFILING)
                 amdgpu_set_clockgating_state(adev, AMD_IP_BLOCK_TYPE_GFX,



Zhu, Rex wrote:
> Thanks Andy,
>
> The attached patch can fix this bug.
> Please review.
>
> Best Regards
> Rex
>
> -----Original Message-----
> From: Andy Furniss [mailto:adf.lists at gmail.com]
> Sent: Monday, January 09, 2017 4:18 AM
> To: Alex Deucher; Zhu, Rex
> Cc: amd-gfx list
> Subject: Re: [PATCH 4/4] drm/amd/powerplay: add profiling mode in dpm 
> level
>
> Alex Deucher wrote:
>> On Fri, Dec 23, 2016 at 3:45 AM, Rex Zhu <Rex.Zhu at amd.com> wrote:
>>> Change-Id: I4a46440882cd94fe5e77e3f351aaccc218a2ece5
>>
>> Patches 1-3:
>> Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
>>
>> patch 4:
>> Please add a better patch description and explain what profiling mode 
>> is good for, etc.
>
> This on drm-next-4.10-wip regresses setting profile a bit on R9285.
>
> It works eg. if in auto then doing
>
> echo high > 
> /sys/class/drm/card0/device/power_dpm_force_performance_level
>
> is OK and echo auto  > ... is also OK.
>
> The issue if I am in auto and echo auto > .... then I don't get my prompt back.
>
> echo high from elsewhere will make it return - but it doesn't set high.
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-drm-amdgpu-fix-bug-return-invaild-value-to-sysfs.patch
Type: application/octet-stream
Size: 901 bytes
Desc: 0001-drm-amdgpu-fix-bug-return-invaild-value-to-sysfs.patch
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20170109/5d8dd0bb/attachment.obj>