[PATCH v2] powerpc: kernel: remove useless code which related with 'max_cpus'
Benjamin Herrenschmidt
benh at kernel.crashing.org
Thu Jul 25 18:28:31 EST 2013
On Thu, 2013-07-25 at 16:22 +0800, Chen Gang wrote:
> On 07/25/2013 04:06 PM, Benjamin Herrenschmidt wrote:
> > On Thu, 2013-07-25 at 15:59 +0800, Chen Gang wrote:
> >>
> >> For my opinion: one fix may like below (assume have removed max_cpus)
> >> which is more reasonable for code readers.
> >
> > So instead of just failing to bring the secondary CPUs, but potentially
> > still having a working system, you crash during boot.... potentially
> > before a console is even visible. And this is good how ?
> >
>
> Hmm... how about the above DBG("...") within this function ?
>
> One implementation of BUG_ON() is use printk() and coredump, if it is a
> critical failure, I suggest to use it (if console is really invisible, I
> guess still can generate the coredump).
Whatever ... looks like you don't feel like listening so I'm not going
to waste my breath anymore, nor will I accept your patches.
Ben.
More information about the Linuxppc-dev
mailing list