Re: [PATCH] config CONSOLE

[Greg Ungerer <gerg@snapgear.com>]

Hi Christoph,

Christoph Egger wrote:
> 	As part of the VAMOS[0] research project at the University of
> Erlangen we are looking at multiple integrity errors in linux'
> configuration system.
> 
> 	CONSOLE is such a generic term that maybe someone just
> expected this option exists. Or maybe it existed ages ago but I can't
> find any trace of it. Anyway that code has been dead for quite some
> time withoutanybody noticing so it's probably just unneeded.

As far as I can tell it comes from the old 2.0.x out-of-main-line
uClinux patches. But yes, dead and gone for quite some time.


> 	Please keep me informed of this patch getting confirmed /
> merged so we can keep track of it.

I am happy to apply to the m68knommu git tree for eventual pushing
to main-line from there.

Regards
Greg



> [0] http://vamos1.informatik.uni-erlangen.de/
> 
> ----
>>From 4a3b6fda1bb3a229c7ea8ca8992ed2573f07e5ba Mon Sep 17 00:00:00 2001
> From: Christoph Egger <siccegge@cs.fau.de>
> Date: Wed, 26 May 2010 15:18:26 +0200
> Subject: [PATCH] There's no config CONSOLE
> 
> as there's no config CONSOLE (never has been as far as I can tell) and
> noone has ever missed that piece of code, it should be safe to remove
> it making the kernel a tiny bit less complex.
> 
> Signed-off-by: Christoph Egger <siccegge@cs.fau.de>
> ---
>  drivers/serial/68328serial.c |    7 -------
>  1 files changed, 0 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/serial/68328serial.c b/drivers/serial/68328serial.c
> index 3046386..193a242 100644
> --- a/drivers/serial/68328serial.c
> +++ b/drivers/serial/68328serial.c
> @@ -78,10 +78,6 @@ struct m68k_serial *m68k_consinfo = 0;
>  
>  #define M68K_CLOCK (16667000) /* FIXME: 16MHz is likely wrong */
>  
> -#ifdef CONFIG_CONSOLE
> -extern wait_queue_head_t keypress_wait; 
> -#endif
> -
>  struct tty_driver *serial_driver;
>  
>  /* number of characters left in xmit buffer before we ask for more */
> @@ -301,9 +297,5 @@ static void receive_chars(struct m68k_serial *info, unsigned short rx)
>  #endif /* CONFIG_MAGIC_SYSRQ */
>  			}
> - 			/* It is a 'keyboard interrupt' ;-) */
> -#ifdef CONFIG_CONSOLE
> -			wake_up(&keypress_wait);
> -#endif			
>  		}
>  
>  		if(!tty)


-- 
------------------------------------------------------------------------
Greg Ungerer  --  Principal Engineer        EMAIL:     gerg@snapgear.com
SnapGear Group, McAfee                      PHONE:       +61 7 3435 2888
8 Gardner Close                             FAX:         +61 7 3217 5323
Milton, QLD, 4064, Australia                WEB: http://www.SnapGear.com