From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [RFC v1 0/8] xen: kconfig changes Date: Wed, 18 Feb 2015 15:20:54 -0500 Message-ID: <20150218202054.GA19571@l.oracle.com> References: <1423720996-3332-1-git-send-email-mcgrof@do-not-panic.com> <54DC8974.5060908@citrix.com> <54E2ED22.7080607@suse.com> <20150218181120.GL8152@l.oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta14.messagelabs.com ([193.109.254.103]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1YOB7I-0003Jw-2a for xen-devel@lists.xenproject.org; Wed, 18 Feb 2015 20:21:04 +0000 Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "Luis R. Rodriguez" Cc: Juergen Gross , xen-devel@lists.xenproject.org, Boris Ostrovsky , David Vrabel , "Luis R. Rodriguez" List-Id: xen-devel@lists.xenproject.org On Wed, Feb 18, 2015 at 12:01:43PM -0800, Luis R. Rodriguez wrote: > On Wed, Feb 18, 2015 at 10:11 AM, Konrad Rzeszutek Wilk > wrote: > > On Tue, Feb 17, 2015 at 11:31:08AM -0800, Luis R. Rodriguez wrote: > >> On Mon, Feb 16, 2015 at 11:26 PM, Juergen Gross wrote: > >> > On 02/17/2015 01:25 AM, Luis R. Rodriguez wrote: > >> >> > >> >> On Mon, Feb 16, 2015 at 4:20 PM, Luis R. Rodriguez > >> >> wrote: > >> >>> > >> >>> As it is per our agreed upon changes we can in theory enable a > >> >>> XEN_PVHVM system without XEN_PV or XEN_PVH. If this is indeed > >> >>> desirable this poses an issue at build time > >> >> > >> >> > >> >> And this also raises the question of whether or not we should make > >> >> XEN_PVHVM a user selectable option, right now it is a def_bool and is > >> >> therefore not human selectable. You can implicitly disable it by > >> >> disabling PCI for example though. If we want that to be exposed to the > >> >> user we can then enable some description of what that means, and the > >> >> user will then be able to read / select / enable XEN_PV., XEN_PVHVM, > >> >> XEN_PVH. Right now they'd only be able to select XEN_PV and/or > >> >> XEN_PVH, XEN_PVHVM is implicit. > >> > > >> > > >> > I think making XEN_PVHVM user selectable is okay. > >> > >> OK I'll enable this then. > > > > Please don't. We had bugs in the past because distros did not select > > it (they made it an module) and the PV drivers were not loaded. > > Oy vey. > > > There should be an history in the git tree behind the desire to make > > it non selectable. > > OK how about we enable the user selection only under CONFIG_EXPERT, > otherwise make it hidden. The CONFIG_EXPERT is gone from the kernel.. > > Luis