linux-kernel - Re: [PATCH] panic: keep blinking in spite of long spin timer mode

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

Message-Id: <20100528152241.4f5db28f.randy.dunlap@oracle.com>
Date:	Fri, 28 May 2010 15:22:41 -0700
From:	Randy Dunlap <randy.dunlap@...cle.com>
To:	TAMUKI Shoichi <tamuki@...et.gr.jp>
Cc:	Ingo Molnar <mingo@...e.hu>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Anton Blanchard <anton@...ba.org>,
	Andi Kleen <andi@...stfloor.org>,
	Andy Green <andy@...mcat.com>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] panic: keep blinking in spite of long spin timer mode

On Fri, 28 May 2010 06:56:45 +0900 TAMUKI Shoichi wrote:

> To keep panic_timeout accuracy when running under a hypervisor, the
> current implementation only spins on long time (1 second) calls to
> mdelay.  That brings a good effect, but we must give up blinking even
> if we have a panic_blink.
> 
> This patch keeps blinking in spite of long spin timer mode.
> 
> We now have new kernel parameter (panic_longspin) to enable long spin
> timer mode when kernel panics.  This is to be used when running under
> a hypervisor (default is disable).
> 
> Signed-off-by: TAMUKI Shoichi <tamuki@...et.gr.jp>
> ---
>  Documentation/kernel-parameters.txt |    7 +-
>  arch/arm/mach-s3c2440/mach-gta02.c  |   17 +----
>  drivers/input/serio/i8042.c         |   25 +-------
>  include/linux/kernel.h              |    2 
>  kernel/panic.c                      |   81 +++++++++++++++++---------
>  5 files changed, 70 insertions(+), 62 deletions(-)
> 
> diff --git a/Documentation/kernel-parameters.txt b/Documentation/kernel-parameters.txt
> index fc89cac..7709450 100644
> --- a/Documentation/kernel-parameters.txt
> +++ b/Documentation/kernel-parameters.txt
> @@ -918,9 +918,6 @@ and is between 256 and 4096 characters. It is defined in the file
>  			     controller
>  	i8042.nopnp	[HW] Don't use ACPIPnP / PnPBIOS to discover KBD/AUX
>  			     controllers
> -	i8042.panicblink=
> -			[HW] Frequency with which keyboard LEDs should blink
> -			     when kernel panics (default is 0.5 sec)
>  	i8042.reset	[HW] Reset the controller during init and cleanup
>  	i8042.unlock	[HW] Unlock (ignore) the keylock
>  
> @@ -1870,6 +1867,10 @@ and is between 256 and 4096 characters. It is defined in the file
>  
>  	panic=		[KNL] Kernel behaviour on panic
>  			Format: <timeout>
> +	panic_longspin	[KNL] Enable long spin timer mode when kernel panics.
> +			      This is to be used when running under a hyper-
> +			      visor (default is disable)
> +	panicblink=	[KNL] The speed of panic blink (default is 12 wpm)

What is "wpm"?  In typing, it's words per minute.
Is it "winks per minute" here?  If not, what?


>  	parkbd.port=	[HW] Parallel port number the keyboard adapter is
>  			connected to, default is 0.


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/