[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.DEB.2.02.1305011531320.8804@chino.kir.corp.google.com>
Date:	Wed, 1 May 2013 15:37:28 -0700 (PDT)
From:	David Rientjes <rientjes@...gle.com>
To:	Borislav Petkov <bp@...en8.de>
cc:	Ingo Molnar <mingo@...nel.org>, "H. Peter Anvin" <hpa@...or.com>,
	linux-kernel@...r.kernel.org, Pekka Enberg <penberg@...nel.org>,
	levinsasha928@...il.com, mtosatti@...hat.com, tglx@...utronix.de,
	Borislav Petkov <bp@...e.de>, fengguang.wu@...el.com,
	linux-tip-commits@...r.kernel.org
Subject: Re: [tip:x86/urgent] x86/kconfig:  Add a Kconfig shortcut for building
 working KVM guest kernels
On Thu, 2 May 2013, Borislav Petkov wrote:
> Ok, so let's step back a bit here: I've added this option with the idea
> to have it as a shortcut so that you don't go search for every commodity
> option when you want to boot the kernel as a kvm guest. Currently I have
> it in all.config when doing randconfig builds here. While I'm doing the
> randbuilds, why not boot it in kvm too while at it.
> 
If that shortcut is enabling options that fail to build, it may not be 
considered so useful.
> So I'm pretty sure we won't be able to plug all holes. However, let's
> try to plug only those which hinder *real* usage instead of contrived
> stuff.
> 
It's contrived to not have CONFIG_NET_CORE enabled?
And why are you enabling CONFIG_PCI_MSI for systems that don't have 
ARCH_SUPPORTS_MSI?  You need CONFIG_X86_LOCAL_APIC and CONFIG_X86_IO_APIC.
So let's take a step back from your step back: you want to have a shortcut 
to build a guest.  Great, so you're enabling options that such a kernel 
needs.  The problem is that you're neglecting options that those options 
need.  It means your patch is incomplete.
--
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/
Powered by blists - more mailing lists
 
