From: Pavel Machek <pavel@ucw.cz>
To: Guenter Roeck <linux@roeck-us.net>
Cc: linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org,
Alan Cox <gnomes@lxorguk.ukuu.org.uk>,
Alexander Graf <agraf@suse.de>,
Andrew Morton <akpm@linux-foundation.org>,
Geert Uytterhoeven <geert@linux-m68k.org>,
Heiko Stuebner <heiko@sntech.de>,
LeeJon@roeck-us.net
Subject: Re: [PATCH v6 00/48] kernel: Add support for power-off handler call chain
Date: Tue, 11 Nov 2014 19:50:20 +0100 [thread overview]
Message-ID: <20141111185019.GA29452@amd> (raw)
In-Reply-To: <20141111182023.GA21970@roeck-us.net>
On Tue 2014-11-11 10:20:23, Guenter Roeck wrote:
> On Mon, Nov 10, 2014 at 09:33:48AM +0100, Pavel Machek wrote:
> > Hi!
> >
> > > Introduce a system power-off handler call chain to solve the described
> > > problems. This call chain is expected to be executed from the architecture
> > > specific machine_power_off() function. Drivers providing system power-off
> > > functionality are expected to register with this call chain. By using the
> > > priority field in the notifier block, callers can control power-off
> > > handler
> >
> > Linus rather disliked the idea of notifier chains for this... And I
> > don't see how it got addressed.
> >
> Hi all,
>
> After more thought, I concluded that it is technically impossible to support
> multiple power-off handlers without some kind of list or call chain, no matter
> how it is called. Given the opposition from Linus and the power maintainers
> to the series, I decided to shelf it.
Well, you can still do preparations -- current code directly setting
pm_power_off is ugly -- so that if you want to switch to call chain
later, it will be easy.
And if that does not happen, we still get the cleanup :-).
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
next prev parent reply other threads:[~2014-11-11 18:50 UTC|newest]
Thread overview: 80+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-10 1:42 [PATCH v6 00/48] kernel: Add support for power-off handler call chain Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 01/48] " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 02/48] memory: emif: Use API function to determine power-off capability Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 03/48] hibernate: Call have_kernel_power_off instead of checking pm_power_off Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 04/48] m68k: Replace mach_power_off with pm_power_off Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 05/48] mfd: as3722: Drop reference to pm_power_off from devicetree bindings Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 06/48] gpio-poweroff: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 07/48] qnap-poweroff: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 08/48] kernel: Move pm_power_off to common code Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 09/48] mfd: palmas: Register with kernel power-off handler Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 10/48] mfd: axp20x: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 11/48] mfd: retu: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 12/48] mfd: ab8500-sysctrl: " Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 13/48] mfd: max8907: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 14/48] mfd: tps80031: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 15/48] mfd: dm355evm_msp: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 16/48] mfd: tps6586x: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 17/48] mfd: tps65910: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 18/48] mfd: twl4030-power: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 19/48] mfd: rk808: Register power-off handler " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 20/48] mfd: rn5t618: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 21/48] ipmi: Register " Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 22/48] power/reset: restart-poweroff: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 23/48] power/reset: gpio-poweroff: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 24/48] power/reset: as3722-poweroff: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 25/48] power/reset: qnap-poweroff: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 26/48] power/reset: msm-poweroff: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 27/48] power/reset: vexpress-poweroff: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 28/48] power/reset: at91-poweroff: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 29/48] power/reset: ltc2952-poweroff: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 30/48] x86: iris: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 31/48] x86: apm: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 32/48] x86: olpc: Register xo1 power-off handler " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 33/48] staging: nvec: Register " Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 34/48] acpi: Register power-off handler " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 35/48] arm: Register " Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 36/48] arm64: psci: " Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 37/48] avr32: atngw100: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 38/48] ia64: " Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 39/48] m68k: " Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 40/48] mips: " Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 41/48] powerpc: " Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-11 7:42 ` Michael Ellerman
2014-11-11 7:42 ` Michael Ellerman
2014-11-11 10:42 ` Guenter Roeck
2014-11-11 10:42 ` Guenter Roeck
2014-11-10 1:42 ` [PATCH v6 42/48] sh: " Guenter Roeck
2014-11-10 1:42 ` Guenter Roeck
2014-11-10 1:43 ` [PATCH v6 43/48] x86: lguest: " Guenter Roeck
2014-11-10 1:43 ` [PATCH v6 44/48] x86: ce4100: " Guenter Roeck
2014-11-10 1:43 ` [PATCH v6 45/48] x86: intel-mid: Drop registration of dummy power-off handlers Guenter Roeck
2014-11-10 1:43 ` [PATCH v6 46/48] x86: pmc_atom: Register power-off handler with kernel power-off handler Guenter Roeck
2014-11-10 1:43 ` [PATCH v6 47/48] efi: " Guenter Roeck
2014-11-10 1:43 ` [PATCH v6 48/48] kernel: Remove pm_power_off Guenter Roeck
2014-11-10 8:33 ` [PATCH v6 00/48] kernel: Add support for power-off handler call chain Pavel Machek
2014-11-10 14:12 ` Guenter Roeck
2014-11-11 18:20 ` Guenter Roeck
2014-11-11 18:50 ` Pavel Machek [this message]
2014-11-11 19:17 ` Guenter Roeck
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=20141111185019.GA29452@amd \
--to=pavel@ucw.cz \
--cc=LeeJon@roeck-us.net \
--cc=agraf@suse.de \
--cc=akpm@linux-foundation.org \
--cc=geert@linux-m68k.org \
--cc=gnomes@lxorguk.ukuu.org.uk \
--cc=heiko@sntech.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux@roeck-us.net \
/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.