Re: [Qemu-devel] [PATCH v2 0/8] qemu-ga: add support for Windows

[Luiz Capitulino]

On Fri, 03 Feb 2012 11:23:05 -0600
Michael Roth <address@hidden> wrote:

> On 02/03/2012 10:45 AM, Luiz Capitulino wrote:
> > On Fri, 03 Feb 2012 10:37:25 -0600
> > Michael Roth<address@hidden>  wrote:
> >
> >> On 02/03/2012 08:18 AM, Luiz Capitulino wrote:
> >>> On Thu,  2 Feb 2012 13:58:52 -0600
> >>> Michael Roth<address@hidden>   wrote:
> >>>
> >>>> These patches apply on top of qemu.git master, and can also be obtained 
> >>>> from:
> >>>> git://github.com/mdroth/qemu.git qga-win32-v2
> >>>>
> >>>> Luiz/Gal, I decided not to roll the suspend/hibernate stuff into this 
> >>>> series
> >>>> since the s3 situation isn't fully sorted out yet. The file structure is 
> >>>> a
> >>>> little different now, posix/linux-specific stuff goes in 
> >>>> qga/commands-posix.c,
> >>>> win32-specific stuff in qga/commands-win32.c, but other than that it 
> >>>> should be
> >>>> a straightforward rebase if this gets merged first.
> >>>
> >>> I think I'll have to rebase my series on top of this one, when do you 
> >>> plan to
> >>> merge this?
> >>>
> >>
> >> Hopefully soon, was planning on waiting for the suspend/hibernate bits
> >> but we seem to be blocked on the s3 issues and I have other patches
> >> accumulating on top of win32 (hesitant to base those on master since
> >> this patchset does a lot of refactoring that might affect them), so I
> >> figured I'd push this for merge since it doesn't have any dependencies
> >> outside master.
> >
> > The S3 issues seem sorted to me, but I don't oppose having this series in 
> > first.
> >
> 
> Thanks, in retrospect I probably should've just gotten these out of the 
> way weeks ago since they'd immediately clobber git blame.
> 
> I'd been tracking Gerd's QMP wakeup series as the s3 resolution we need 
> for guest-suspend, is that still the case?

Yes. But now I remembered about a seabios bug with S3... Need to check if
it were already addressed.

> I guess those are coming 
> through your QMP queue?

Oh, as the QMP part is trivial I thought someone else would pick them up,
but I can do that.