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

[Randy Dunlap]

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

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


-- 
~Randy