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

From: Dou Liyang
Date: Mon Nov 20 2017 - 20:04:46 EST


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 }