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.

> 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 > > >