Re: [BUGFIX][PATCH 3/4] hvm_save_one: return correct data.

[George Dunlap <george.dunlap@eu.citrix.com>]

On 12/20/2013 12:32 AM, Don Slutz wrote:
> Adding "George Dunlap" to list of people.
>
> George,
>   I think this is a bug fix that can make it into 4.4.  Some how I 
> forgot to include you directly.  Please let me know if pushing for 
> this is wrong.
>
> Bug fix is attached to:
>
> http://lists.xen.org/archives/html/xen-devel/2013-12/msg02412.html
>
> Thanks,
>    -Don Slutz
>
> On 12/17/13 05:40, Andrew Cooper wrote:
>> On 17/12/13 08:20, Jan Beulich wrote:
>>>>>> On 16.12.13 at 18:51, Don Slutz <dslutz@verizon.com> wrote:
>>>> I feel that the attached bugfix patch is simple enough to make it 
>>>> into 4.4
>>>> and also be back ported to stable branches.
>>> I would tend to agree on the 4.4 part of this, but afaict this is used
>>> for debugging purposes only, so I'm not so sure about backporting
>>> to stable branches. What is your rationale behind that request?
>>>
>>> Andrew, short of taking your intrusive (and not yet agreed upon)
>>> rework - would you agree Don's change at least is a sufficient
>>> improvement to include it irrespective of any intentions you might
>>> have with this code?
>>>
>>> Jan
>>>
>> Yes - Dons' patch would appear to fix up all the corner-cases I am aware
>> of in the hypercall.

It looks like this should only affect xenctx.  It is a bug fix, and 
while not a critical one, nonetheless an important one.  So I agree, it 
should be accepted.

Release-acked-by: George Dunlap <george.dunlap@eu.citrix.com>