From: George Dunlap <george.dunlap@eu.citrix.com>
To: Ian Campbell <Ian.Campbell@citrix.com>,
Andrew Cooper <andrew.cooper3@citrix.com>
Cc: Keir Fraser <keir@xen.org>, Jan Beulich <JBeulich@suse.com>,
Don Slutz <dslutz@verizon.com>,
Xen-devel <xen-devel@lists.xen.org>
Subject: Re: [RFC PATCH 0/3] Xen hvm saving improvements
Date: Mon, 16 Dec 2013 10:32:35 +0000 [thread overview]
Message-ID: <52AED6C3.2050308@eu.citrix.com> (raw)
In-Reply-To: <1387187061.20076.30.camel@kazak.uk.xensource.com>
On 12/16/2013 09:44 AM, Ian Campbell wrote:
> On Mon, 2013-12-16 at 02:17 +0000, Andrew Cooper wrote:
>> This series is *very* RFC at the moment - it is only compile tested. I would
>> very much appreciate review on the design. (and I shall see about sorting out
>> some dev testing tomorrow)
>>
>> The ultimate purpose is to fix that the fact that
>> XEN_DOMCTL_gethvmcontext_partial is very unreliable at providing the correct
>> subset of information.
>>
>> All the changes are supposed to be internal to Xen. There should be no
>> difference at all to the result of XEN_DOMCTL_gethvmcontext (which can be used
>> as a useful dev test for the validity of the changes).
>>
>> George: As for whether this is intended for 4.5, I have no idea. On the one
>> hand, it is fixing a hypercall which doesn't function correctly, but on the
>> other hand, it is a very large and complicated set of changes relevant to
>> migration. Then again, the majority of the changes were largely mechanical,
>> and there is a simple test to prove whether there are bugs or not. I shall
>> defer to others for comments on the risk/rewards of this series.
> IMHO this is obviously not 4.4 material at this stage. Apart from
> anything else we've been managing to release with these short comings
> for many years.
Indeed.
-George
next prev parent reply other threads:[~2013-12-16 10:32 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-16 2:17 [RFC PATCH 0/3] Xen hvm saving improvements Andrew Cooper
2013-12-16 2:17 ` [RFC PATCH 1/3] xen/hvm-save: Refactor HVM_REGISTER_SAVE_RESTORE Andrew Cooper
2013-12-16 9:33 ` Jan Beulich
2013-12-16 2:17 ` [RFC PATCH 2/3] xen/hvm-save: Extend hvm_save_handler to take an instance parameter Andrew Cooper
2013-12-16 2:17 ` [RFC PATCH 3/3] xen/hvm-save: Adjust calling of multi-instance save handlers Andrew Cooper
2013-12-17 15:01 ` Tim Deegan
2013-12-16 9:44 ` [RFC PATCH 0/3] Xen hvm saving improvements Ian Campbell
2013-12-16 10:32 ` George Dunlap [this message]
2013-12-16 17:50 ` Don Slutz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52AED6C3.2050308@eu.citrix.com \
--to=george.dunlap@eu.citrix.com \
--cc=Ian.Campbell@citrix.com \
--cc=JBeulich@suse.com \
--cc=andrew.cooper3@citrix.com \
--cc=dslutz@verizon.com \
--cc=keir@xen.org \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.