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

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

 



Quoting Paulo Zanoni (2017-11-16 20:45:26)
> Em Qui, 2017-11-16 às 20:24 +0200, David Weinehall escreveu:
> > 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.
> 
> What happens if you keep the autosuspend delay to 10000?

The problem with the extra hpd triggering a render and flip still
remains, but less frequent.

Remove compiz + desktop manager from the picture and see if that cures
it. Then obviously you have to try each of the main desktop managers /
display servers to see which have good/bad behaviour (if my hunch is
correct). And if it does, we can't proceed with an aggressive default,
but will have to recommend that the display managers set it according to
their needs.
-Chris
_______________________________________________
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