Re: [Xen-devel] [PATCH 2/3] xen: use freeze/restore/thaw PM events for suspend/resume/chkpt

[Shriram Rajagopalan <rshriram@xxxxxxxxx>]

On Tue, Feb 22, 2011 at 2:36 PM, Konrad Rzeszutek Wilk
<konrad.wilk@xxxxxxxxxx> wrote:
>> > Could we just make CONFIG_XEN_SAVE_RESTORE depend on CONFIG_HIBERANTION?
>> > Like this:
>> >
>> >
>> > diff --git a/arch/x86/xen/Kconfig b/arch/x86/xen/Kconfig
>> > index 5b54892..838e20c 100644
>> > --- a/arch/x86/xen/Kconfig
>> > +++ b/arch/x86/xen/Kconfig
>> > @@ -38,7 +38,7 @@ config XEN_MAX_DOMAIN_MEMORY
>> >
>> >  config XEN_SAVE_RESTORE
>> >        bool
>> > -       depends on XEN && PM
>> > +       depends on XEN && PM && HIBERNATION
>> >        default y
>> >
>> >  config XEN_DEBUG_FS
>> >
>> >
>> On that aspect, just noticed
>> config PM_SLEEP
>>         bool
>>         depends on SUSPEND || HIBERNATION || XEN_SAVE_RESTORE
>>         default y
>> we could get rid of XEN_SAVE_RESTORE as the suspend/resume code doesnt
>> depend on PM_SLEEP
>
> OK. Do you want to respin patch #2 without the #ifdefs in the drivers/base/...
> and provide a new patch (#4?) which will alter the Kconfig file appropriately?
>
>
yep. will do.
shriram

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