linux-kernel - Re: [Patch -v3 4/4] Make reboot

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]

Message-ID: <20130416090109.GB8438@gmail.com>
Date:	Tue, 16 Apr 2013 11:01:09 +0200
From:	Ingo Molnar <mingo@...nel.org>
To:	Robin Holt <holt@....com>
Cc:	"H. Peter Anvin" <hpa@...or.com>, Ingo Molnar <mingo@...hat.com>,
	Russ Anderson <rja@....com>, Shawn Guo <shawn.guo@...aro.org>,
	Oleg Nesterov <oleg@...hat.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Lai Jiangshan <laijs@...fujitsu.com>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Michel Lespinasse <walken@...gle.com>,
	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>,
	Paul Mackerras <paulus@...ba.org>,
	Peter Zijlstra <peterz@...radead.org>,
	"rusty@...tcorp.com.au" <rusty@...tcorp.com.au>,
	Tejun Heo <tj@...nel.org>,
	the arch/x86 maintainers <x86@...nel.org>,
	Thomas Gleixner <tglx@...utronix.de>
Subject: Re: [Patch -v3 4/4] Make reboot_cpuid a kernel parameter.


* Robin Holt <holt@....com> wrote:

> On Mon, Apr 15, 2013 at 10:47:49AM -0700, H. Peter Anvin wrote:
> > We usually don't break working user setups as a matter if policy.  Only if it can't be avoided.
> > 
> > Robin Holt <holt@....com> wrote:
> > 
> > >On Mon, Apr 15, 2013 at 10:18:31AM -0700, H. Peter Anvin wrote:
> > >> Um... doesn't this break anyone who currently depends on this?
> > >
> > >Yes.  That is also why I kept this separate and not tagged for
> > >stable. They would need to start using the more general kernel
> > >parameter
> > >to get the same functionality.  Should I have done it differently?
> 
> How about:
> Author: Robin Holt <holt@....com>
> Date:   Mon Apr 15 11:39:33 2013 -0500
> 
>     Make reboot_cpuid a kernel parameter.
>     
>     Moving the reboot=s<##> parameter for x86 to a kernel parameter
>     proper.  I did not find any other arch that was specifying the
>     reboot cpu.
>     
>     I left a compatibility mode in there.  The new parameter always
>     takes precedence.  I also fixed up the current code to support
>     up to cpuid's up to the current max of 4096.

Looks pretty nice and clean to me, as the old boot parameter still keeps working.

Mind sending an updated series that includes all the fixes and the Tested-by from 
Shawn Guo?

Thanks,

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/