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