From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43241) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bSGLQ-00038K-VS for qemu-devel@nongnu.org; Wed, 27 Jul 2016 00:21:24 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bSGLP-0003Z8-KX for qemu-devel@nongnu.org; Wed, 27 Jul 2016 00:21:20 -0400 Received: from mx1.redhat.com ([209.132.183.28]:45828) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bSGLP-0003Yz-EP for qemu-devel@nongnu.org; Wed, 27 Jul 2016 00:21:19 -0400 Date: Wed, 27 Jul 2016 07:21:15 +0300 From: "Michael S. Tsirkin" Message-ID: <20160727072039-mutt-send-email-mst@kernel.org> References: <1467873289-14220-1-git-send-email-saxenap.ltc@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v3 0/2] vhost-user: Extend protocol to receive replies on any command. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?iso-8859-1?Q?Marc-Andr=E9?= Lureau Cc: Prerna , Prerna Saxena , Anil Kumar Boggarapu , QEMU , Felipe Franciosi On Mon, Jul 25, 2016 at 02:27:18PM +0400, Marc-Andr=E9 Lureau wrote: > Hi >=20 > On Mon, Jul 25, 2016 at 10:41 AM, Prerna wrote: > > > > > > On Thu, Jul 7, 2016 at 12:04 PM, Prerna Saxena > > wrote: > >> > >> From: Prerna Saxena > >> > >> The current vhost-user protocol requires the client to send response= s to > >> only a > >> few commands. For the remaining commands, it is impossible for QEMU = to > >> know the > >> status of the requested operation -- ie, did it succeed? If so, by w= hat > >> time? > >> > >> This is inconvenient, and can also lead to races. As an example: > >> [..snip..] > >> > >> References: > >> v1 : https://lists.gnu.org/archive/html/qemu-devel/2016-06/msg07152.= html > >> v2 : https://lists.gnu.org/archive/html/qemu-devel/2016-07/msg00048.= html > >> > >> > >> Prerna Saxena (2): > >> vhost-user: Introduce a new protocol feature REPLY_ACK. > >> vhost-user: Attempt to fix a race with set_mem_table. > >> > >> docs/specs/vhost-user.txt | 44 +++++++++++++++ > >> hw/virtio/vhost-user.c | 133 > >> ++++++++++++++++++++++++++++++---------------- > >> 2 files changed, 130 insertions(+), 47 deletions(-) > >> > > > > Ping ! > > Michael, MarcAndre, Did you have a chance to look at this patch serie= s? > > >=20 > That's not going to make it in 2.7 I am afraid. It's a bugfix so - depends on how quickly can comments be addressed. --=20 MST