LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] x86: nmi - nmi_watchdog boot param docs cleanup
@ 2008-11-01 15:06 Cyrill Gorcunov
  2008-11-03  8:18 ` Ingo Molnar
  0 siblings, 1 reply; 2+ messages in thread
From: Cyrill Gorcunov @ 2008-11-01 15:06 UTC (permalink / raw)
  To: Ingo Molnar, LKML
  Cc: H. Peter Anvin, Maciej W. Rozycki, Thomas Gleixner, Yinghai Lu,
	Aristeu Rozanski

1) nmi_watchdog boot parameter is common to 32/64 bit modes. So
move it from Documentation/x86/x86_64/boot-options.txt to
Documentation/kernel-parameters.txt and integrate with.

2) Also fix [panic] keyword placement -- it ought to be at first
position otherwise it will not be recognized.

3) Document lapoc and ioapic keywords.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>
---

It's a bit fun but 'panic' was placed on a second
position which is just a plain wrong :-)

On top of -tip dda51106620c328b50007c259b9ef8cea2b7c3d2

Index: linux-2.6.git/Documentation/kernel-parameters.txt
===================================================================
--- linux-2.6.git.orig/Documentation/kernel-parameters.txt	2008-10-27 21:35:08.000000000 +0300
+++ linux-2.6.git/Documentation/kernel-parameters.txt	2008-11-01 17:53:08.000000000 +0300
@@ -1403,7 +1403,20 @@ and is between 256 and 4096 characters. 
 			when a NMI is triggered.
 			Format: [state][,regs][,debounce][,die]
 
-	nmi_watchdog=	[KNL,BUGS=X86-32] Debugging features for SMP kernels
+	nmi_watchdog=	[KNL,BUGS=X86-32,X86-64] Debugging features for SMP kernels
+			Format: [panic,][num]
+			Valid num: 0,1,2
+			0 - turn nmi_watchdog off
+			1 - use the IO-APIC timer for the NMI watchdog
+			2 - use the local APIC for the NMI watchdog using
+			a performance counter. Note: This will use one performance
+			counter and the local APIC's performance vector.
+			When panic is specified panic when an NMI watchdog timeout occurs.
+			This is useful when you use a panic=... timeout and need the box
+			quickly up again.
+			Instead of 1 and 2 it is possible to use the following
+			symbolic names: lapic and ioapic
+			Example: nmi_watchdog=2 or nmi_watchdog=panic,lapic
 
 	no387		[BUGS=X86-32] Tells the kernel to use the 387 maths
 			emulation library even if a 387 maths coprocessor
Index: linux-2.6.git/Documentation/x86/x86_64/boot-options.txt
===================================================================
--- linux-2.6.git.orig/Documentation/x86/x86_64/boot-options.txt	2008-10-27 21:35:08.000000000 +0300
+++ linux-2.6.git/Documentation/x86/x86_64/boot-options.txt	2008-11-01 17:52:22.000000000 +0300
@@ -79,17 +79,6 @@ Timing
   Report when timer interrupts are lost because some code turned off
   interrupts for too long.
 
-  nmi_watchdog=NUMBER[,panic]
-  NUMBER can be:
-  0 don't use an NMI watchdog
-  1 use the IO-APIC timer for the NMI watchdog
-  2 use the local APIC for the NMI watchdog using a performance counter. Note
-  This will use one performance counter and the local APIC's performance
-  vector.
-  When panic is specified panic when an NMI watchdog timeout occurs.
-  This is useful when you use a panic=... timeout and need the box
-  quickly up again.
-
   nohpet
   Don't use the HPET timer.
 

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] x86: nmi - nmi_watchdog boot param docs cleanup
  2008-11-01 15:06 [PATCH] x86: nmi - nmi_watchdog boot param docs cleanup Cyrill Gorcunov
@ 2008-11-03  8:18 ` Ingo Molnar
  0 siblings, 0 replies; 2+ messages in thread
From: Ingo Molnar @ 2008-11-03  8:18 UTC (permalink / raw)
  To: Cyrill Gorcunov
  Cc: LKML, H. Peter Anvin, Maciej W. Rozycki, Thomas Gleixner,
	Yinghai Lu, Aristeu Rozanski


* Cyrill Gorcunov <gorcunov@gmail.com> wrote:

> 1) nmi_watchdog boot parameter is common to 32/64 bit modes. So
> move it from Documentation/x86/x86_64/boot-options.txt to
> Documentation/kernel-parameters.txt and integrate with.
> 
> 2) Also fix [panic] keyword placement -- it ought to be at first
> position otherwise it will not be recognized.
> 
> 3) Document lapoc and ioapic keywords.
> 
> Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

applied to tip/x86/nmi-watchdog, thanks Cyrill!

	Ingo

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-11-03  8:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-11-01 15:06 [PATCH] x86: nmi - nmi_watchdog boot param docs cleanup Cyrill Gorcunov
2008-11-03  8:18 ` Ingo Molnar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).