Shriram Rajagopalan
2012-Feb-29 18:05 UTC
Re: [PATCH 0 of 3 RESEND V5] libxl: refactor suspend/resume code
Sorry for the empty preamble. something got messed up and one of the patches also didnt make it! Unfortunately, I forgot to include the last patch "libxl: resume instead of unpause on xl save -c" in this series. (message id: c7abecc14cceb1814033.1328251802@athos.nss.cs.ubc.ca ) It remains the same. Ian, could you please include that, when you commit this series ? Intro message: ------ This patch series refactors the suspend/resume code to minimize Remus specific code in libxl. There are a couple of trivial bug fixes too. Changes in V5: This series is just a resend, rebasing the patches to the latest tip. It depends on Stefano''s "V5: libxl: save/restore qemu physmap". Changes in V4: 1. Incorporated Ian Campbell''s comments on the suspend_cancel support patch. Changes in V3: 1. rebase patches based on Stefano''s patches use qmp_save instead of qmp_migrate 2. check if qemu moves to "running" state after resuming the device model 3. Moved comments on the co-operative suspend to libxl.h Changes in V2: 1. migrate code is refactored as save_config , create child, do_preamble instead of coaelscing them all into one single function. 2. More documentation for suspend_cancel parameter in domain_resume 3. Minor nits shriram _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Stefano Stabellini
2012-Mar-01 14:53 UTC
Re: [PATCH 0 of 3 RESEND V5] libxl: refactor suspend/resume code
On Wed, 29 Feb 2012, Shriram Rajagopalan wrote:> Sorry for the empty preamble. something got messed up and one of the patches also > didnt make it! Unfortunately, I forgot to include the last patch > > "libxl: resume instead of unpause on xl save -c" in this series. > (message id: c7abecc14cceb1814033.1328251802@athos.nss.cs.ubc.ca ) > It remains the same. > > Ian, could you please include that, when you commit this series ? > > Intro message: > ------ > > This patch series refactors the suspend/resume code to minimize > Remus specific code in libxl. There are a couple of trivial bug > fixes too. > > Changes in V5: > This series is just a resend, rebasing the patches to the latest tip. It depends on > Stefano''s "V5: libxl: save/restore qemu physmap".Small update: the promise of a soon review was reasserted. In any case be sure that your patch series is going to go in before the 4.2 freeze, one way or another. --8323329-1280384805-1330613596=:923 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel --8323329-1280384805-1330613596=:923--
Ian Jackson
2012-Mar-01 15:27 UTC
Re: [PATCH 0 of 3 RESEND V5] libxl: refactor suspend/resume code
Shriram Rajagopalan writes ("Re: [Xen-devel] [PATCH 0 of 3 RESEND V5] libxl: refactor suspend/resume code"):> This series is just a resend, rebasing the patches to the latest tip. It > depends on Stefano''s "V5: libxl: save/restore qemu physmap".Thanks. It''s unfortunate that this has been blocked so long. When it becomes unblocked (ie, when the qemu upstream part is in) I trust Stefano and you will remind me. Ian.