linux-kernel - Re: [PATCH] leds/trigger/cpu: move from CPU

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 22 Jul 2016 17:24:29 +0200
From:	Linus Walleij <linus.walleij@...aro.org>
To:	Sebastian Andrzej Siewior <bigeasy@...utronix.de>
Cc:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Ingo Molnar <mingo@...nel.org>,
	Jacek Anaszewski <j.anaszewski@...sung.com>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Paul Gortmaker <paul.gortmaker@...driver.com>,
	Peter Zijlstra <peterz@...radead.org>,
	Richard Purdie <rpurdie@...ys.net>,
	Thomas Gleixner <tglx@...utronix.de>,
	"linux-leds@...r.kernel.org" <linux-leds@...r.kernel.org>,
	rt@...utronix.de
Subject: Re: [PATCH] leds/trigger/cpu: move from CPU_STARTING to ONLINE level

On Wed, Jul 20, 2016 at 5:24 PM, Sebastian Andrzej Siewior
<bigeasy@...utronix.de> wrote:

> There is no need the ledtriger to be called *that* early in the hotplug
> process (+ with disabled interrupts). As explained by Jacek Anaszewski [0]
> there is no need for it.
> Therefore this patch moves it to the ONLINE/PREPARE_DOWN level using the
> dynamic registration for the id.
>
> [0] https://lkml.kernel.org/r/578C92BC.2070603@samsung.com
>
> Cc: Jacek Anaszewski <j.anaszewski@...sung.com>
> Cc: Linus Torvalds <torvalds@...ux-foundation.org>
> Cc: Linus Walleij <linus.walleij@...aro.org>
> Cc: Paul Gortmaker <paul.gortmaker@...driver.com>
> Cc: Peter Zijlstra <peterz@...radead.org>
> Cc: Richard Purdie <rpurdie@...ys.net>
> Cc: Thomas Gleixner <tglx@...utronix.de>
> Cc: linux-leds@...r.kernel.org
> Cc: rt@...utronix.de
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@...utronix.de>

Read back on the discussion, this change makes all kind of sense.
Acked-by: Linus Walleij <linus.walleij@...aro.org>

Yours,
Linus Walleij