Re: [PATCH 2/3] x86/apic: Update the 'apic=' description of setting APIC driver

[Dou Liyang]

Hi Randy,

At 11/21/2017 01:21 AM, Randy Dunlap wrote:
> On 11/20/2017 05:27 AM, Dou Liyang wrote:
> > There are two consumers of apic=: the APIC debug level and the low
> > level generic architecture code, but Linux just documented the first
> > one.
> >
> > Append the second description.
> >
> > Signed-off-by: Dou Liyang <douly.fnst@xxxxxxxxxxxxxx>
> > ---
> >  Documentation/admin-guide/kernel-parameters.txt | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
> > index 62436bd..fdd2382 100644
> > --- a/Documentation/admin-guide/kernel-parameters.txt
> > +++ b/Documentation/admin-guide/kernel-parameters.txt
> > @@ -328,11 +328,14 @@
> >  			not play well with APC CPU idle - disable it if you have
> >  			APC and your system crashes randomly.
> >
> > -	apic=		[APIC,X86-32] Advanced Programmable Interrupt Controller
> > +	apic=		[APIC,X86] Advanced Programmable Interrupt Controller
> >  			Change the output verbosity whilst booting
> >  			Format: { quiet (default) | verbose | debug }
> >  			Change the amount of debugging information output
> >  			when initialising the APIC and IO-APIC components.
> > +			If in X86-32, one more function is registering APIC
> > +			driver.
> > +			Examples: apic=bigsmp, bigsmp is an APIC driver name.
>
> How about this?
>
> 			For X86-32, this can also be used to specify an APIC
> 			driver name.
> 			Format: apic=drivername
> 			Example: apic=bigsmp

Pretty good! Will use it in the next version.

Thanks.
	dou.
> >  	apic_extnmi=	[APIC,X86] External NMI delivery setting
> >  			Format: { bsp (default) | all | none }