Re: [patch] config: fix make kvmconfig

From: David Rientjes <rientjes@google.com>
To: Ingo Molnar <mingo@kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Pekka Enberg <penberg@kernel.org>,
	Sasha Levin <levinsasha928@gmail.com>,
	Randy Dunlap <rdunlap@infradead.org>,
	David Woodhouse <dwmw2@infradead.org>,
	Michal Marek <mmarek@suse.cz>, "H. Peter Anvin" <hpa@zytor.com>,
	tglx@linutronix.de, "H. Peter Anvin" <hpa@linux.intel.com>,
	linux-kernel@vger.kernel.org
Subject: Re: [patch] config: fix make kvmconfig
Date: Mon, 4 Feb 2013 10:57:35 -0800 (PST)	[thread overview]
Message-ID: <alpine.DEB.2.02.1302041053330.6063@chino.kir.corp.google.com> (raw)
In-Reply-To: <20130204184436.GA13256@gmail.com>

On Mon, 4 Feb 2013, Ingo Molnar wrote:

> >  arch/x86/Kconfig | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
> > index 6979498..f9459b3 100644
> > --- a/arch/x86/Kconfig
> > +++ b/arch/x86/Kconfig
> > @@ -606,6 +606,7 @@ config KVMTOOL_TEST_ENABLE
> >  	select NETWORK_FILESYSTEMS
> >  	select INET
> >  	select EXPERIMENTAL
> > +	select TTY
> >  	select SERIAL_8250
> >  	select SERIAL_8250_CONSOLE
> >  	select IP_PNP
> 
> There's no 'config TTY' in the kernel AFAICS.
> 

It's added in linux-next by 4f73bc4dd3e8 ("tty: Added a CONFIG_TTY option 
to allow removal of TTY").

Adding Greg to the cc to figure out which tree this should be merged 
through.  Greg, please find the patch inline.

The better alternative would be to remove KVMTOOL_TEST_ENABLE entirely 
since it has been sitting in linux-next for over a year, Linus has said he 
won't pull it, and it consistently needs updates both inside and outside 
of Pekka's tree.

-----o<-----

config: fix make kvmconfig

make kvmconfig selects CONFIG_SERIAL_8250 without selecting its 
dependencies causing some configs to fail:

drivers/tty/serial/8250/8250.c:57:32: error: 'CONFIG_SERIAL_8250_RUNTIME_UARTS' undeclared here (not in a function)
drivers/tty/serial/8250/8250.c:2712:47: error: 'CONFIG_SERIAL_8250_NR_UARTS' undeclared here (not in a function)
drivers/tty/serial/8250/8250.c: In function 'serial8250_console_write':
drivers/tty/serial/8250/8250.c:2843:10: error: 'struct uart_port' has no member named 'sysrq'
drivers/tty/serial/8250/8250.c: In function 'early_serial_setup':
drivers/tty/serial/8250/8250.c:2971:20: error: negative width in bit-field '<anonymous>'
drivers/tty/serial/8250/8250.c: At top level:
drivers/tty/serial/8250/8250.c:2712:30: warning: 'serial8250_ports' defined but not used [-Wunused-variable]

Fix this by selecting CONFIG_TTY as well, which CONFIG_SERIAL_8250 
requires.

Signed-off-by: David Rientjes <rientjes@google.com>
---
 arch/x86/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index 6979498..f9459b3 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -606,6 +606,7 @@ config KVMTOOL_TEST_ENABLE
 	select NETWORK_FILESYSTEMS
 	select INET
 	select EXPERIMENTAL
+	select TTY
 	select SERIAL_8250
 	select SERIAL_8250_CONSOLE
 	select IP_PNP
