From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: Re: [PATCH] pvops: Make suspend work when CONFIG_SUSPEND=n Date: Fri, 4 Mar 2011 10:56:10 -0500 Message-ID: <20110304155610.GD8391@dumpdata.com> References: <1299238501.6552.295.camel@zakaz.uk.xensource.com> <1299239542.6552.302.camel@zakaz.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Shriram Rajagopalan Cc: Ian Campbell , Frank Pan , Jeremy Fitzhardinge , "xen-devel@lists.xensource.com" , Stefano Stabellini List-Id: xen-devel@lists.xenproject.org On Fri, Mar 04, 2011 at 07:42:00AM -0800, Shriram Rajagopalan wrote: > On Fri, Mar 4, 2011 at 3:52 AM, Ian Campbell wrote: > > On Fri, 2011-03-04 at 11:45 +0000, Frank Pan wrote: > >> > AFAIK the conclusion is that an approach which ensures both > >> > XEN_SAVE_RESTORE and SUSPEND (actually HIBERNATE after the above > >> > discussion) are enable when necessary (by making the former depend on > >> > the later) is what is going to be taken. > >> > >> That's good. How long will it be committed on pvops trees? > > > > I don't know. Stefano, Rafael and Shriram are taking care of it AFAIK. > > > Well, from my side, I am waiting on Stefano & Rafael. The discussion seemed > to have ended with "lets wait for the code to settle", though there seemed to be > no consensus regarding the need to enable HIBERNATE for XEN_SAVE_RESTORE > to work. > Someone suggested creating a new user visible hibernate symbol that would > solve this issue and make the main hibernate logic depend on this symbol rather > than the HIBERNATE symbol. I could certainly spin up a patch for that but nobody > seemed to have reached a conclusion. Please do. I was under the understanding that we were waiting for a victi^H^H^Hvolunteer to implement that. That was the only thing gatting your patchset going in.