From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55858) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZG5Cp-0004BP-G7 for qemu-devel@nongnu.org; Fri, 17 Jul 2015 08:57:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZG5Ck-0006GV-Bq for qemu-devel@nongnu.org; Fri, 17 Jul 2015 08:57:35 -0400 Received: from mail-wg0-x22b.google.com ([2a00:1450:400c:c00::22b]:36750) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZG5Ck-0006G4-4P for qemu-devel@nongnu.org; Fri, 17 Jul 2015 08:57:30 -0400 Received: by wgxm20 with SMTP id m20so81637728wgx.3 for ; Fri, 17 Jul 2015 05:57:29 -0700 (PDT) Sender: Paolo Bonzini References: <1435310564-9418-1-git-send-email-thibaut.collet@6wind.com> <1435310564-9418-2-git-send-email-thibaut.collet@6wind.com> <559FC31E.10600@redhat.com> <55A3221C.6090307@huawei.com> <55A867B5.9000100@redhat.com> From: Paolo Bonzini Message-ID: <55A8FBB6.5040607@redhat.com> Date: Fri, 17 Jul 2015 14:57:26 +0200 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH v4 1/1] vhost user: add support of live migration List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?UTF-8?Q?Marc-Andr=c3=a9_Lureau?= Cc: Linhaifeng , "Michael S. Tsirkin" , Thibaut Collet , jasowang@redhat.com, QEMU , stefanha@redhat.com On 17/07/2015 12:34, Marc-André Lureau wrote: > On Fri, Jul 17, 2015 at 4:25 AM, Paolo Bonzini wrote: > > But LOG_BASE makes little sense across processes, and LOG_FD is unused > > in QEMU, isn't it? So this patch is not enough to add support of live > > migration. > > You are right, LOG_BASE doesn't make much sense, and LOG_FD isn't > used, despite some code already there. Furthermore, the log is > allocated with regular malloc, hardly shareable. LOG_FD is implemented in the kernel drivers/vhost/vhost.c. It seems to be an eventfd-like mechanism to save on dirty bitmap scans. However, it's not well documented how to implement it in a correct way. In any case, live migration needs a new message type (like LOG_MMAP_FD) in the vhost-user protocol. Paolo