Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1357119

Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines.

Show all headers | View raw

On Mon, Mar 14, 2016 at 07:47:39PM +1000, Dave Airlie wrote:
> >
> >> -     if (pcie_port_runtime_suspend_allowed(dev))
> >> +     if (pcie_port_runtime_suspend_allowed(dev)) {
> >> +             pm_runtime_allow(&dev->dev);
> >
> > PCI drivers typically have left this decision up to the userspace. I'm
> > wondering whether it is good idea to deviate from that here? Of course
> > this allows immediate power savings but could potentially cause problems
> > as well.
> >
> 
> No distro has ever shipped userspace to do this, I really think this
> is a bad design.
> We have wasted countless watts of power on this stupid idea that people will
> run powertop, only a few people in the world run powertop, lots of
> people use Linux.
> 
> The kernel should power stuff down not wait for the user to run powertop,
> At least for the GPU it's in the area of 8W of power, and I've got the
> GPU drivers doing this themselves,
> 
> I could have the GPU driver call runtime allow for it's host bridge I suppose,
> if we insist on the userspace cares, but I'd prefer not doing so.

Yes, fully agreed. Runtime PM that's not enabled by default is useless,
since no one runs powertop, which also means it's buggy because no one
tests it. Not enabling power saving features by default is imo just a lame
excuse. Shit better just work, and tuneables should only exposed if
there's a real reason why autotuning is infeasible. E.g. we auto-ramp the
gpu clock down/up in i915 in software to compensate for some of the
silliness in how the hw does it alone. And the sysfs tunables are mostly
just to make benchmarking (where slower, but without any thermal
throttling) is sometimes preferred.

And yes i915 hasn't enabled rpm yet by default because there's bugs left
:(
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. Dave Airlie <airlied@gmail.com> - 2016-03-09 07:20 +0100
  Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. "Rafael J. Wysocki" <rafael@kernel.org> - 2016-03-09 14:20 +0100
    Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. Dave Airlie <airlied@gmail.com> - 2016-03-09 23:00 +0100
      Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-03-10 22:00 +0100
        Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10  machines. Mika Westerberg <mika.westerberg@linux.intel.com> - 2016-03-11 12:00 +0100
          Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-03-11 14:50 +0100
            Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. Dave Airlie <airlied@gmail.com> - 2016-03-14 03:20 +0100
              Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. Dave Airlie <airlied@gmail.com> - 2016-03-14 10:50 +0100
                Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10  machines. Daniel Vetter <daniel@ffwll.ch> - 2016-03-14 11:10 +0100
                Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10  machines. Mika Westerberg <mika.westerberg@linux.intel.com> - 2016-03-14 11:30 +0100
                Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. "Rafael J. Wysocki" <rafael@kernel.org> - 2016-03-14 14:00 +0100
                Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10  machines. Mika Westerberg <mika.westerberg@linux.intel.com> - 2016-03-14 15:40 +0100
              Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10  machines. Mika Westerberg <mika.westerberg@linux.intel.com> - 2016-03-14 10:50 +0100
                Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10  machines. Lukas Wunner <lukas@wunner.de> - 2016-03-15 14:40 +0100
                Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10  machines. Mika Westerberg <mika.westerberg@linux.intel.com> - 2016-03-15 15:00 +0100
              Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. Alex Deucher <alexdeucher@gmail.com> - 2016-03-14 15:40 +0100
  Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10  machines. Lukas Wunner <lukas@wunner.de> - 2016-03-09 15:40 +0100
    Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. Alex Deucher <alexdeucher@gmail.com> - 2016-03-09 18:00 +0100
      Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10  machines. Lukas Wunner <lukas@wunner.de> - 2016-03-09 21:20 +0100
        Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. Alex Deucher <alexdeucher@gmail.com> - 2016-03-09 21:30 +0100
        Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. Dave Airlie <airlied@gmail.com> - 2016-03-09 23:10 +0100
    Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. Dave Airlie <airlied@gmail.com> - 2016-03-09 23:10 +0100

csiph-web