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

[Andy Furniss]

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?

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 instantly force fan high?

I don't think that would be popular WRT noise.

>
>
> 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.
>
>