Re: [Qemu-devel] [PATCH] Fix migration uint8 arrys handled

[Avi Kivity <avi@redhat.com>]

On 03/22/2011 12:23 PM, Stefan Berger wrote:
> On 03/22/2011 05:28 AM, Avi Kivity wrote:
>> On 03/22/2011 03:46 AM, Anthony Liguori wrote:
>>> On 03/21/2011 07:25 PM, Stefan Berger wrote:
>>>> On 03/15/2011 10:53 AM, Juan Quintela wrote:
>>>>> commit 82fa39b75181b730d6d4d09f443bd26bcfcd045c
>>>>>
>>>>> only contains half of the fix.  It forgots the save state fix for
>>>>> UINT8 indexes.
>>>>>
>>>>> Anthony, please apply, without this migration using hpet is broken.
>>>>> (only current user).
>>>> I have just been bisecting the code (from the tip) due to 
>>>> suspend/resume problems and it looks like commit 82fa39b7 is 
>>>> introducing the suspend/resume problem I am seeing (frozen screen).
>>>
>>> It's in tip now.
>>
>> Great, spent some lovely time bisecting and fixing it as well.
>>
> It doesn't work better now than it did before... Trying a 
> suspend/resume while in grub leaves me with a black screen upon resume...

Well, it fixed it for me (autotest migration tests).

Do you mean that 82fa39b7^ works but b784421ce4c doesn't?  What about 
b784421ce4c^ with 82fa39b7 reverted?  That will rule out some other bug.

-- 
error compiling committee.c: too many arguments to function