From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH]KVM: kvm/ia64: select ANON_INODES PREEMPT_NOTIFIERS, once Date: Mon, 25 Aug 2008 12:03:00 +0300 Message-ID: <48B27544.7090004@qumranet.com> References: <42DFA526FC41B1429CE7279EF83C6BDC01845A71@pdsmsx415.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: kvm-ia64@vger.kernel.org, kvm@vger.kernel.org To: "Zhang, Xiantao" Return-path: Received: from il.qumranet.com ([212.179.150.194]:56181 "EHLO il.qumranet.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753828AbYHYJDD (ORCPT ); Mon, 25 Aug 2008 05:03:03 -0400 In-Reply-To: <42DFA526FC41B1429CE7279EF83C6BDC01845A71@pdsmsx415.ccr.corp.intel.com> Sender: kvm-owner@vger.kernel.org List-ID: Zhang, Xiantao wrote: > From 2c14c54eed790553fddbcb3970f0967a4aa79dca Mon Sep 17 00:00:00 2001 > From: Xiantao Zhang > Date: Sat, 23 Aug 2008 10:21:06 +0800 > Subject: [PATCH] KVM: kvm/ia64: select ANON_INODES PREEMPT_NOTIFIERS, > once > virtualization configured. > > Once virtulization configured, make PREEMPT_NOTIFIERS, ANON_INODES > selected by default. > What is the motivation for this? Don't things work as is? > Signed-off-by: Xiantao Zhang > --- > arch/ia64/kvm/Kconfig | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/ia64/kvm/Kconfig b/arch/ia64/kvm/Kconfig > index 7914e48..07f6b69 100644 > --- a/arch/ia64/kvm/Kconfig > +++ b/arch/ia64/kvm/Kconfig > @@ -7,6 +7,8 @@ config HAVE_KVM > menuconfig VIRTUALIZATION > bool "Virtualization" > depends on HAVE_KVM || IA64 > + select PREEMPT_NOTIFIERS > + select ANON_INODES > default y > Suppose VIRTUALIZATION is selected but KVM is not. Why select ANON_INODES and PREEMPT_NOTIFIERS? > ---help--- > Say Y here to get to see options for using your Linux host to > run other > @@ -20,8 +22,6 @@ if VIRTUALIZATION > config KVM > tristate "Kernel-based Virtual Machine (KVM) support" > depends on HAVE_KVM && EXPERIMENTAL > - select PREEMPT_NOTIFIERS > - select ANON_INODES > ---help--- > Support hosting fully virtualized guest machines using > hardware > virtualization extensions. You will need a fairly recent > -- error compiling committee.c: too many arguments to function