All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Airlie <airlied@gmail.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 2/5] drm: Add initial dnyamic power off feature
Date: Mon, 10 Sep 2012 21:16:34 +1000	[thread overview]
Message-ID: <CAPM=9tzFQQ7voLCaTs-Pnh0dx7cvpWmZsS7FXe10Kv1_28NiTw@mail.gmail.com> (raw)
In-Reply-To: <20120910120758.3bd752bc@pyramind.ukuu.org.uk>

On Mon, Sep 10, 2012 at 9:07 PM, Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:
> On Mon, 10 Sep 2012 14:31:52 +1000
> Dave Airlie <airlied@gmail.com> wrote:
>
>> From: Dave Airlie <airlied@redhat.com>
>>
>> For secondary GPUs in laptops, i.e. optimus or powerxpress, we have
>> methods for powering down the GPU completely. This adds support
>> to the drm core for powering back up the GPU on any access from
>> ioctls or sysfs interfaces, and fires a 5s timer to test if
>> we can power the GPU off.
>
>
> Is there a reason for basically re-inventing the existing Linux dynamic
> power management layer ?

Not really, wanted to have something simple to demo first. This

"bus type drivers of the buses the devices are on are
 responsible for the actual handling of the autosuspend requests and
 wake-up events."

seemed to imply there was some magic in the PCI bus layer that is
generic, and in this case that doesn't exist.

We have a 3 specific ACPI calls to power off the GPU slot for nvidia,
radeon and apple.

Dave.

  reply	other threads:[~2012-09-10 11:16 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-10  4:31 [RFC] drm dynamic power off support Dave Airlie
2012-09-10  4:31 ` [PATCH 1/5] gpu/vga_switcheroo: add driver control power feature Dave Airlie
2012-09-10  4:31 ` [PATCH 2/5] drm: Add initial dnyamic power off feature Dave Airlie
2012-09-10  7:18   ` Daniel Vetter
2012-09-10  8:23     ` Dave Airlie
2012-09-10  8:36       ` Chris Wilson
2012-09-10 10:55         ` Alan Cox
2012-09-10  9:00       ` Daniel Vetter
2012-09-10 11:07   ` Alan Cox
2012-09-10 11:16     ` Dave Airlie [this message]
2012-09-10  4:31 ` [PATCH 3/5] nouveau: Add interface to detect optimus support Dave Airlie
2012-09-10 16:25   ` Lekensteyn
2012-09-10 20:24     ` Dave Airlie
2012-09-10  4:31 ` [PATCH 4/5] nouveau: add dynamic gpu power off support Dave Airlie
2012-09-10 16:30   ` Peter Wu
2012-09-10  4:31 ` [PATCH 5/5] radeon: add dynamic " Dave Airlie
2012-09-10  5:04 ` [RFC] drm " Dave Airlie
2012-09-10  8:47   ` Takashi Iwai
2012-09-10  8:50     ` Dave Airlie
2012-09-11 13:32       ` Takashi Iwai

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='CAPM=9tzFQQ7voLCaTs-Pnh0dx7cvpWmZsS7FXe10Kv1_28NiTw@mail.gmail.com' \
    --to=airlied@gmail.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=dri-devel@lists.freedesktop.org \
    /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.