WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] Re: [PATCH] xen: fix XEN_SAVE_RESTORE Kconfig dependenci

To: "Jan Beulich" <JBeulich@xxxxxxxxxx>
Subject: Re: [Xen-devel] Re: [PATCH] xen: fix XEN_SAVE_RESTORE Kconfig dependencies
From: "Rafael J. Wysocki" <rjw@xxxxxxx>
Date: Fri, 25 Feb 2011 19:19:26 +0100
Cc: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>, Shriram Rajagopalan <rshriram@xxxxxxxxx>, "linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx" <linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Pavel Machek <pavel@xxxxxx>
Delivery-date: Fri, 25 Feb 2011 10:20:35 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4D677B260200007800033A60@xxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <1298446066-11754-1-git-send-email-rshriram@xxxxxxxxx> <1298459402.16356.10779.camel@xxxxxxxxxxxxxxxxxxxxxx> <4D677B260200007800033A60@xxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: KMail/1.13.5 (Linux/2.6.38-rc6+; KDE/4.4.4; x86_64; ; )
On Friday, February 25, 2011, Jan Beulich wrote:
> >>> On 23.02.11 at 12:10, Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx> wrote:
> > On Wed, 2011-02-23 at 10:58 +0000, Jan Beulich wrote:
> >> >>> On 23.02.11 at 10:08, Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx> wrote:
> >> > On Wed, 2011-02-23 at 07:27 +0000, Shriram Rajagopalan wrote:
> >> >> Make XEN_SAVE_RESTORE user visible and enable HIBERNATION and SWAP,
> >> >> if this option is enabled.
> >> > 
> >> > I was under the (perhaps mistaken?) impression that selecting user
> >> > visible Kconfig symbols was verboten since it has side-effects which
> >> > make it hard for users to figure out why something cannot be switched
> >> > off and/or how to turn things on etc.
> >> > 
> >> > In any case I think I preferred the "depends HIBERNATION" idea -- I
> >> > thought that was the consensus of the thread too?
> >> 
> >> Why HIBERNATION (and SWAP) instead of SUSPEND? Xen doesn't
> >> support the former (other than the latter)...
> > 
> > HIBERNATION provides the FREEZE, THAW and RESUME methods which we would
> > like to use to integrate save/restore/checkpoint with the PM core i.e.
> > choose THAW vs. RESUME based on whether the suspend was cancelled (aka a
> > checkpoint) or not. See Shriram's other recent patch for details.
> 
> But imo it's nevertheless wrong to select HIBERNATION (and bogus
> to select SWAP), as that'll create to user space the impression that
> the kernel is capable of doing hibernation, which is wrong. If you
> need the particular PM operations but no hibernation, then an
> intermediary option will need to be introduced.

I think we can introduce CONFIG_HIBERNATE_INTERFACE that will be user-visible
option instead of CONFIG_HIBERNATION and will select the latter.  Then,
CONFIG_XEN_SAVE_RESTORE will also be able to select CONFIG_HIBERNATION without
building the hibernate interface in, which will prevent user space from being
confused, but that will cause too much code to be built anyway.

Thanks,
Rafael

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel