Re: [Qemu-devel] [PATCH for-2.4 0/3] Migration regressions with Xen.

[Stefano Stabellini]

On Thu, 30 Jul 2015, Stefano Stabellini wrote:
> On Thu, 30 Jul 2015, Juan Quintela wrote:
> > Anthony PERARD <address@hidden> wrote:
> > > This is a critical issue for Xen as migration either with the same version
> > > of QEMU, or from a previous version of QEMU is broken.
> > >
> > > Any suggestion on how to move forward?
> > 
> > Will send a pull requset tomorrow.
> > 
> > Thinking about creating a single function that is called for all needed
> > places, just to avoid this problem in the future.
> > 
> 
> Hi Juan,
> 
> thanks for looking into this!
> Do you have patches already we can look at and help you test to make
> sure they fix the issue?

Any updates?


> 
> > 
> > >
> > > On Tue, Jul 28, 2015 at 04:54:42PM +0100, Anthony PERARD wrote:
> > >> Hi,
> > >> 
> > >> We've spotted several regression which prevent migration with Xen using 
> > >> the
> > >> same version of QEMU or from a previous version of QEMU (tryied with 
> > >> 2.2).
> > >> 
> > >> Regression have been introduce by at least:
> > >> - df4b1024526cae3479da3492d6371fd4a7324a03
> > >>   migration: create new section to store global state
> > >> - 61964c23e5ddd5a33f15699e45ce126f879e3e33
> > >>   migration: Add configuration section
> > >> 
> > >> The first patch would fix the default case when we use the machine 
> > >> 'xenfv'.
> > >> 
> > >> But we can also use the machine 'pc' and I don't know how to fix this 
> > >> case.
> > >> The second and third patches are a step in the direction of fixing
> > >> migration with the machine 'pc'.
> > >> 
> > >> I have not look at save/restore for PV guest yet.
> > >> 
> > >> Thanks,
> > >> 
> > >> Anthony PERARD (3):
> > >>   migration: Fix regretion for xenfv machine.
> > >>   migration: Fix global state with Xen.
> > >>   migration: Add configuration section to vmstate with xen.
> > >> 
> > >>  hw/i386/pc_piix.c             | 3 +++
> > >>  include/migration/migration.h | 1 +
> > >>  migration/migration.c         | 7 +++++++
> > >>  migration/savevm.c            | 6 ++++++
> > >>  4 files changed, 17 insertions(+)
> > 
>