All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Eric Rannaud <eric.rannaud@gmail.com>
Cc: Jani Nikula <jani.nikula@intel.com>,
	Daniel Vetter <daniel.vetter@ffwll.ch>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Jon Kristensen <info@jonkri.com>
Subject: Re: i915: Regression: +4W in idle power use on Macbook Pro 15 (late 2013)
Date: Wed, 27 Aug 2014 12:17:26 +0300	[thread overview]
Message-ID: <20140827091726.GP4193@intel.com> (raw)
In-Reply-To: <CA+zRj8VE4Qnsoi53+onX9pyUGkDNYQhOFFSGvGpe9YjsR5MdBQ@mail.gmail.com>

On Tue, Aug 26, 2014 at 04:00:51PM -0700, Eric Rannaud wrote:
> On Tue, Aug 26, 2014 at 1:59 PM, Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
> >> Forcing FBC with i915.enable_fbc=1 brings the idle power consumption
> >> back to under 7W, however.
> >> This is all on 3.15.4-ARCH-00041-gf4db98240ac2.
> >
> > Any significant changes in package C state as reported in powertop?
> > Indeed fairly impressive how much fbc saves here ...
> 
> Not that I can tell.
>     Powertop report with FBC: http://pastebin.com/5qfJKpTQ
>     Without FBC: http://pastebin.com/NaYkR4n0

Seems to have gotten messed up somehow. I can't see the package c-state
info there at all for some reason. Also core c-states go only up to c7s.
On hsw I would have expected to see deeper c-states, but maybe it's not
a ult machine or something. My hsw-ult goes up to c10.

> Some highly uneducated guesses on what could explain a +4W jump with no FBC:
> 
> #1- The higher DRAM and bus duty cycle during scanout is enough to
> prevent some DRAM subsystems from sleeping, by crossing some tight
> threshold (maybe Apple has FBC enabled, so parameters somewhere in
> firmware are tuned for the lower level of background activity they
> expect with FBC on?). Not much we can do about that, unless such
> parameters can be tweaked by us.
> 
> #2- Without FBC, the FB doesn't fit in L3 (i7-4750HQ has 6MB,
> 2880x1800 compressed at least 1:4 fits), keeping the DRAM awake more.
> 
> #3- Disabling FBC somehow affects the layout of the framebuffer in
> DRAM, keeping more of the DRAM active and awake during scanout.
> Different tiling, swizzling, etc. parameters? Is it worth looking at
> the code for that kind of thing? To be clear, I'm (blindly) suggesting
> that it might be possible to increase the locality of the framebuffer
> in physical DRAM, even without compression enabled.
> 
> Actually, with an image of white-noise displayed fullscreen while
> powertop takes a 20 second measurement, the idle power consumption
> shoots up to 11W, with FBC enabled. This experiment actually
> invalidates my guess #3. The white noise image will not compress much
> at all, while my typical test screen is a couple of static
> black-and-white terminal windows (filling up the screen), which will
> compress well.
> 
> So it would appear 4W is the actual cost of having a full-size 20MB
> framebuffer on this system.

I guess the monster resolution just really hurts w/o fbc. My hsw has
1920x1080 panel which, assuming the same refresh rate, means your
display refresh requires 2.5x the bandwidth mine does. Sadly my hsw
seems to have a a dead battery so i can't check the power consumption
figures. pc7 residency definitely drops quite a bit on that machine if
I disable fbc. I don't remember if I ever measured the effect of fbc
on that thing before the battery died, but on my ivb it's definitely
somewhere in the .3W ballpark, though that machine has an even lower
resolution than the hsw.

> 
> Anything an outsider can do to help getting FBC enabled by default again?

Either someone needs to review my patches or fix the fbc code in
some other way. Even if someone wants to do the frontbuffer tracking
some other way, there was plenty of stuff in my patches that can
still be applied (there were even simple bugfixes included iirc).

-- 
Ville Syrjälä
Intel OTC

  reply	other threads:[~2014-08-27  9:19 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-22 17:40 i915: Regression: +4W in idle power use on Macbook Pro 15 (late 2013) Eric Rannaud
2014-08-25 10:19 ` Jani Nikula
2014-08-25 10:22   ` Jani Nikula
2014-08-25 12:19     ` Eric Rannaud
2014-08-26 11:06       ` Ville Syrjälä
2014-08-26 13:38         ` Eric Rannaud
2014-08-26 14:57           ` Eric Rannaud
2014-08-26 20:59             ` Daniel Vetter
2014-08-26 23:00               ` Eric Rannaud
2014-08-27  9:17                 ` Ville Syrjälä [this message]
2014-08-27 17:50                   ` Eric Rannaud
2014-08-28 20:08                 ` Lu, Ran
2014-08-28 16:41     ` Sean V Kelley
2014-08-28 17:19       ` Eric Rannaud

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140827091726.GP4193@intel.com \
    --to=ville.syrjala@linux.intel.com \
    --cc=daniel.vetter@ffwll.ch \
    --cc=eric.rannaud@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=info@jonkri.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=jani.nikula@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.