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

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] [day] [month] [year] [list]

Date:   Tue, 21 Nov 2017 09:04:33 +0800
From:   Dou Liyang <douly.fnst@...fujitsu.com>
To:     Randy Dunlap <rdunlap@...radead.org>,
        <linux-kernel@...r.kernel.org>, <x86@...nel.org>
CC:     <tglx@...utronix.de>, <mingo@...hat.com>, <hpa@...or.com>,
        <corbet@....net>, <peterz@...radead.org>, <bp@...e.de>
Subject: Re: [PATCH 2/3] x86/apic: Update the 'apic=' description of setting
 APIC driver

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@...fujitsu.com>
>> ---
>>  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 }
>>
>
>