Re: [PATCH 1/2] MIPS: add CONFIG_VIRTUALIZATION for virtio support

[Aurelien Jarno <aurelien@aurel32.net>]

On Wed, Jan 19, 2011 at 03:30:49PM +0100, Christoph Hellwig wrote:
> On Tue, Jan 18, 2011 at 12:20:44PM +0100, Aurelien Jarno wrote:
> > Add CONFIG_VIRTUALIZATION to the MIPS architecture and include the
> > the virtio code there. Used to enable the virtio drivers under QEMU.
> > 
> > Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
> > ---
> >  arch/mips/Kconfig |   16 ++++++++++++++++
> 
> > +menuconfig VIRTUALIZATION
> > +	bool "Virtualization"
> > +	default n
> > +	---help---
> > +	  Say Y here to get to see options for using your Linux host to run other
> > +	  operating systems inside virtual machines (guests).
> > +	  This option alone does not add any kernel code.
> > +
> > +	  If you say N, all options in this submenu will be skipped and disabled.
> 
> This item seems rather misleading as you're using virtio drivers as a
> guest.  I think the right fix is to just remove the VIRTUALIZATION
> dependency for the qemu drivers and just include them from
> drivers/Kconfig for all architectures.  There aren't a whole lot Linux
> architectures that don't have a qemu emulation these days.
> 

Pulling drivers/Kconfig from drivers may not be the best option either,
as it only pulls "PCI driver for virtio devices" and "Virtio balloon
driver", which actually doesn't really belong to drivers, but maybe to 
"Bus options (PCI etc.)" and "Processor type and features".

-- 
Aurelien Jarno	                        GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net