Re: [PATCH] drm/i915: Enable runtime pm

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Nov 16, 2017 at 08:24:02PM +0200, David Weinehall wrote:
> On Wed, Nov 08, 2017 at 04:25:42PM +0200, David Weinehall wrote:
> > On Tue, Nov 07, 2017 at 05:18:21PM +0100, Daniel Vetter wrote:
> > > Now that we have CI, and that pm_rpm fully passes (I guess the audio
> > > folks have implemented proper runtime pm for snd-hda, hooray, pls
> > > confirm) it's time to enable this again by default.
> > > 
> > > Real goal here is to have 1 configuration only that we fully support,
> > > instead of tons of different codes with every user/customer tuning it
> > > differently. And really, power stuff should work by default, and
> > > should be enabled by everywhere where it is save to do so.
> > > 
> > > v2: Completely new commit message, a few years passed since v1 ...
> > > 
> > > Cc: Takashi Iwai <tiwai@xxxxxxx>
> > > Cc: Liam Girdwood <liam.r.girdwood@xxxxxxxxx>
> > > Cc: "Yang, Libin" <libin.yang@xxxxxxxxx>
> > > Cc: "Lin, Mengdong" <mengdong.lin@xxxxxxxxx>
> > > Cc: "Li, Jocelyn" <jocelyn.li@xxxxxxxxx>
> > > Cc: "Kaskinen, Tanu" <tanu.kaskinen@xxxxxxxxx>
> > > Cc: "Zanoni, Paulo R" <paulo.r.zanoni@xxxxxxxxx>
> > > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx>
> > 
> > lgtm,
> > 
> > Reviewed-by: David Weinehall <david.weinehall@xxxxxxxxxxxxxxx>
> 
> Been testing a bit; test results are a bit discouraging when it comes to
> idle power consumption.
> 
> Our nightly test suite logs idle power consumption before running any
> tests, after running all tests, and then finally after running
> powertop --auto-tune.
> 
> The before and after cases are similar with and without patch,
> but not so for the auto-tune case.
> 
> Without this patch, powertop --auto-tune yields a massive improvement
> (on NUC6i5 the power consumption drops with more than a factor of 3),
> while with this patch the power consumption just drops with 10% or so.

With the 10s => 0.1s bit of the patch reverted there's no longer any
--auto-tune regression.


Kind regards, David
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux