[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

RE: [PATCH v3 0/2] qemu: support shmem device migration



> On Fri, Jul 24, 2020 at 11:34:10AM +0800, Wang Xin wrote:
> >Shmem device support property role with 'master'(master=on) or
> >'peer'(master=off, default mode), which controls to copy the shared
> >memory on migration to the destination host or not. see
> >https://git.qemu.org/?p=qemu.git;a=blob_plain;f=docs/system/ivshmem.rst
> >;hb=HEAD
> >
> >
> >The 1st patch add attribute 'role', it based on Martin's old patch
> >https://www.redhat.com/archives/libvir-list/2016-September/msg00536.htm
> >l
> >
> >The 2nd patch remove shmem name check in migraiton, which allows user
> >change shared memory path on destination host.
> >
> >v3 -> v2:
> >  1. Correct description of shmem ‘role’ attribute. If there is any
> >     mistake in the description, please show me. Sorry for my poor
> >     English :)
> >  2. Remove the inappropriate suggestion in migration log.
> >  3. allow shmem name change in migration, instead add a new
> >     'mem-path' attribute.
> >
> >v2 -> v1:
> >  Fix ivshmem testcases failure.
> >
> >
> >Wang Xin (2):
> >  qemu: add support for shmem-{plain, doorbell} role
> >  conf: allow shmem name change in migration
> >
> 
> So as you read in the reviews I'm going to push this after the release.  But could you, please, send an update to the news
> after the release (or now, but mark it as for 6.7.0) so that it is also properly visible for the users?
> 
> Thanks a lot.

I'll send it after the v6.7.0 release.
My pleasure.

> 
> > docs/formatdomain.html.in                     | 13 ++++++-
> > docs/schemas/domaincommon.rng                 |  8 ++++
> > src/conf/domain_conf.c                        | 37 +++++++++++++++++--
> > src/conf/domain_conf.h                        | 10 +++++
> > src/libvirt_private.syms                      |  2 +
> > src/qemu/qemu_command.c                       | 17 ++++++++-
> > src/qemu/qemu_migration.c                     | 20 ++++++++--
> > .../qemuhotplug-ivshmem-plain-detach.xml      |  2 +-
> > .../qemuhotplug-ivshmem-plain.xml             |  2 +-
> > ...qemuhotplug-base-live+ivshmem-doorbell.xml |  2 +-
> > .../qemuhotplug-base-live+ivshmem-plain.xml   |  2 +-
> > .../shmem-plain-doorbell.args                 |  6 ++-
> > .../qemuxml2argvdata/shmem-plain-doorbell.xml |  4 +-
> > .../shmem-plain-doorbell.xml                  |  4 +-
> > tests/qemuxml2xmltest.c                       |  3 +-
> > 15 files changed, 110 insertions(+), 22 deletions(-)
> >
> >--
> >2.23.0
> >
> >
> >


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]