From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42889) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZkIAv-0005sZ-Vx for qemu-devel@nongnu.org; Thu, 08 Oct 2015 16:52:34 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZkIAu-0000Ae-Nl for qemu-devel@nongnu.org; Thu, 08 Oct 2015 16:52:29 -0400 Received: from mx1.redhat.com ([209.132.183.28]:46810) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZkIAu-0000AY-GD for qemu-devel@nongnu.org; Thu, 08 Oct 2015 16:52:28 -0400 Date: Thu, 8 Oct 2015 23:52:22 +0300 From: "Michael S. Tsirkin" Message-ID: <20151008235057-mutt-send-email-mst@redhat.com> References: <1443720248-15482-1-git-send-email-marcandre.lureau@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <1443720248-15482-1-git-send-email-marcandre.lureau@redhat.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v7 00/24] vhost-user: add migration support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: marcandre.lureau@redhat.com Cc: haifeng.lin@huawei.com, thibaut.collet@6wind.com, jasowang@redhat.com, qemu-devel@nongnu.org, pbonzini@redhat.com On Thu, Oct 01, 2015 at 07:23:44PM +0200, marcandre.lureau@redhat.com wro= te: > From: Marc-Andr=E9 Lureau >=20 > Hi, >=20 > The following series implement shareable log for vhost-user to support > memory tracking during live migration. On qemu-side, the solution is > fairly straightfoward since vhost already supports the dirty log, only > vhost-user couldn't access the log memory until then. >=20 > The series includes "vhost user: Add live migration" patches from > Thibaut Collet. OK, it was a quiet week otherwise, so I decided to pick this patchset up and fix up the trivial issues myself. I probably won't be able to do this in the future though. > v6->v7: > - add migration blocker if memfd failed > - add doc about the qemu_memfd_alloc() helper > - (rebase on top of Michael pci branch) >=20 > v5->v6: > - rebased > - fix protocol feature mask update > - add a patch from Thibault Collet using enum for features, and > compute mask > - add unistd.h linux headers to help building memfd if missing on > build host. Hopefully will be useful for other syscalls some day > - reorder/merge patches to share-allocate the log only if needed > - split the memfd helper to allow to drop the fallback code > - allow to call qemu_memfd_free() even if alloc failed > - add some missing spaces >=20 > v4->v5: > - rebase on top of last Michael S. Tsirkin PULL request > - block live migration if !PROTOCOL_F_LOG_SHMFD > - wait for a reply after SET_LOG_BASE > - split vhost_set_log_base from the rest of vhost_call refactoring > - use a seperate global vhost_log_shm >=20 > v3->v4: > - add the proto negotiation & the migration series > - replace the varargs vhost_call() approach for callbacks > - only share-allocate when the backend needs it >=20 > v2->v3: > - changed some patch summary > - added migration tests > - added a patch to replace error message with a trace >=20 > The development branch I used is: > https://github.com/elmarco/qemu branch "vhost-user" >=20 > Marc-Andr=E9 Lureau (21): > configure: probe for memfd > linux-headers: add unistd.h > util: add linux-only memfd fallback > util: add memfd helpers > util: add fallback for qemu_memfd_alloc() > vhost: document log resizing > vhost: add vhost_set_log_base op > vhost-user: add vhost_user_requires_shm_log() > vhost: alloc shareable log > vhost-user: send log shm fd along with log_base > vhost-user: add a migration blocker > vhost: use a function for each call > vhost-user: document migration log > net: add trace_vhost_user_event > vhost: add migration block if memfd failed > vhost-user-test: move wait_for_fds() out > vhost-user-test: remove useless static check > vhost-user-test: wrap server in TestServer struct > vhost-user-test: learn to tweak various qemu arguments > vhost-user-test: add live-migration test > vhost-user-test: check ownership during migration >=20 > Thibaut Collet (3): > vhost user: add support of live migration > vhost user: add rarp sending after live migration for legacy guest > vhost-user: use an enum helper for features mask >=20 > configure | 19 + > docs/specs/vhost-user.txt | 63 ++- > hw/net/vhost_net.c | 35 +- > hw/scsi/vhost-scsi.c | 7 +- > hw/virtio/vhost-backend.c | 121 +++- > hw/virtio/vhost-user.c | 572 ++++++++++++------- > hw/virtio/vhost.c | 121 ++-- > include/hw/virtio/vhost-backend.h | 77 ++- > include/hw/virtio/vhost.h | 15 +- > include/net/vhost_net.h | 1 + > include/qemu/memfd.h | 26 + > linux-headers/asm-arm/unistd.h | 448 +++++++++++++++ > linux-headers/asm-arm64/kvm.h | 37 +- > linux-headers/asm-arm64/unistd.h | 16 + > linux-headers/asm-mips/unistd.h | 1063 ++++++++++++++++++++++++++++= ++++++++ > linux-headers/asm-powerpc/unistd.h | 392 +++++++++++++ > linux-headers/asm-s390/unistd.h | 404 ++++++++++++++ > linux-headers/asm-x86/unistd.h | 15 + > linux-headers/asm-x86/unistd_32.h | 377 +++++++++++++ > linux-headers/asm-x86/unistd_64.h | 330 +++++++++++ > linux-headers/asm-x86/unistd_x32.h | 319 +++++++++++ > net/vhost-user.c | 34 +- > scripts/update-linux-headers.sh | 7 +- > tests/vhost-user-test.c | 372 +++++++++++-- > trace-events | 3 + > util/Makefile.objs | 9 +- > util/memfd.c | 165 ++++++ > 27 files changed, 4713 insertions(+), 335 deletions(-) > create mode 100644 include/qemu/memfd.h > create mode 100644 linux-headers/asm-arm/unistd.h > create mode 100644 linux-headers/asm-arm64/unistd.h > create mode 100644 linux-headers/asm-mips/unistd.h > create mode 100644 linux-headers/asm-powerpc/unistd.h > create mode 100644 linux-headers/asm-s390/unistd.h > create mode 100644 linux-headers/asm-x86/unistd.h > create mode 100644 linux-headers/asm-x86/unistd_32.h > create mode 100644 linux-headers/asm-x86/unistd_64.h > create mode 100644 linux-headers/asm-x86/unistd_x32.h > create mode 100644 util/memfd.c >=20 > --=20 > 2.4.3