[PATCH 4/4] drm/amdgpu: extend profiling mode.

[Alex Deucher]

On Tue, Feb 7, 2017 at 4:48 AM, Zhu, Rex <Rex.Zhu at amd.com> wrote:
>
> Not set max fan speed in high performance level because of the fan noise.

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

>
> Best Regards
> Rex
> ________________________________
> From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> on behalf of Zhu, Rex
> <Rex.Zhu at amd.com>
> Sent: Monday, February 6, 2017 10:09 PM
> To: Andy Furniss; Alex Deucher
> Cc: amd-gfx list
> Subject: RE: [PATCH 4/4] drm/amdgpu: extend profiling mode.
>
> Please see in line.
>
> Best Regards
> Rex
>
> -----Original Message-----
> From: Andy Furniss [mailto:adf.lists at gmail.com]
> Sent: Monday, February 06, 2017 8:32 PM
> To: Zhu, Rex; Alex Deucher
> Cc: amd-gfx list
> Subject: Re: [PATCH 4/4] drm/amdgpu: extend profiling mode.
>
> Zhu, Rex wrote:
>> Sorry for the late response.
>>
>> Yes, I set the fan speed to max in this patch when user set high
>> performance.
>>
>> Considering that: 1. set fan speed to max is helpful to let GPU run under
>> highest clock as long as possible.
>>                   2. avoid GPU rapid temperature rise in some case.
>
>>>OK, I guess you know if it's needed or not.
>>>It is somewhat annoying noise wise, maybe one day there will be a way for
>>> users to set fan back on auto?
>
> Rex: sure, you can echo "2"> /sys/class/drm/card0/device/hwmon/hwmon3(maybe
> not 3)/pwm1_enable
>
>>>Accepting I don't know how the h/w works your point 2 would imply that
>>> with perf on auto doing something like running a benchmark that pegs all the
>>> clocks high should also
>
> Rex: based on the thermal,  the fan speed.will be dynamically adjusted.
>
>>>I don't think that would be popular WRT noise.
>
> Rex: maybe I worried unnecessarily. I will change the code not to set fan
> speed to max. thanks.
>
>
>
>>
>> Best Regards
>> Rex
>>
>>
>>
>> -----Original Message-----
>> From: Andy Furniss [mailto:adf.lists at gmail.com]
>> Sent: Wednesday, February 01, 2017 11:37 PM
>> To: Alex Deucher
>> Cc: Zhu, Rex; amd-gfx list
>> Subject: Re: [PATCH 4/4] drm/amdgpu: extend profiling mode.
>>
>> Alex Deucher wrote:
>>> On Tue, Jan 31, 2017 at 6:19 PM, Andy Furniss <adf.lists at gmail.com>
>>> wrote:
>>>> Andy Furniss wrote:
>>>>>
>>>>> Rex Zhu wrote:
>>>>>>
>>>>>> in profiling mode, powerplay will fix power state as stable as
>>>>>> possible.and disable gfx cg and LBPW feature.
>>>>>>
>>>>>> profile_standard: as a prerequisite, ensure power and thermal
>>>>>> sustainable, set clocks ratio as close to the highest clock ratio
>>>>>> as possible.
>>>>>> profile_min_sclk: fix mclk as profile_normal, set lowest sclk
>>>>>> profile_min_mclk: fix sclk as profile_normal, set lowest mclk
>>>>>> profile_peak: set highest sclk and mclk, power and thermal not
>>>>>> sustainable
>>>>>> profile_exit: exit profile mode. enable gfx cg/lbpw feature.
>>>>>
>>>>>
>>>>> Testing R9 285 Tonga on drm-next-4.11-wip
>>>>>
>>>>> This commit has the effect that doing
>>>>>
>>>>> echo high >
>>>>> /sys/class/drm/card0/device/power_dpm_force_performance_level
>>>>>
>>>>> instantly forces fan to (I guess) max, where normally it doesn't
>>>>> need anything like as fast with the clocks high when doing nothing
>>>>> else.
>>>>
>>>>
>>>> Ping - just in case this got missed, still the same on current
>>>> drm-next-4.11-wip
>>>
>>> Just a heads up, Rex was looking at this, but it's Chinese New Year this
>>> week.
>>
>> OK, thanks & Happy new year.
>>
>>
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx