From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:51880)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <paolo.bonzini@gmail.com>) id 1Wwl1c-0001rY-U4
	for qemu-devel@nongnu.org; Tue, 17 Jun 2014 00:29:45 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <paolo.bonzini@gmail.com>) id 1Wwl1T-000530-TO
	for qemu-devel@nongnu.org; Tue, 17 Jun 2014 00:29:36 -0400
Received: from mail-wi0-x232.google.com ([2a00:1450:400c:c05::232]:44568)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <paolo.bonzini@gmail.com>) id 1Wwl1T-00052u-Lo
	for qemu-devel@nongnu.org; Tue, 17 Jun 2014 00:29:27 -0400
Received: by mail-wi0-f178.google.com with SMTP id n15so5083836wiw.11
	for <qemu-devel@nongnu.org>; Mon, 16 Jun 2014 21:29:26 -0700 (PDT)
Sender: Paolo Bonzini <paolo.bonzini@gmail.com>
Message-ID: <539FC422.7010201@redhat.com>
Date: Tue, 17 Jun 2014 06:29:22 +0200
From: Paolo Bonzini <pbonzini@redhat.com>
MIME-Version: 1.0
References: <1402976701-10271-1-git-send-email-famz@redhat.com>
In-Reply-To: <1402976701-10271-1-git-send-email-famz@redhat.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH 0/6] virtio-blk: Unify request handling of
	dataplane
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Fam Zheng <famz@redhat.com>, qemu-devel@nongnu.org
Cc: Kevin Wolf <kwolf@redhat.com>, Stefan Hajnoczi <stefanha@redhat.com>

Il 17/06/2014 05:44, Fam Zheng ha scritto:
> This is based on top of my request converging series:
>
> [PATCH v4 0/9] virtio-blk: Converge VirtIOBlockRequest into VirtIOBlockReq
>
> Most of the request handlings are already the same now between dataplane and
> non-dataplane, except the missing IO accounting, error reporting and
> multiwrite. With this series, dataplane pulls in all of them by reusing
> non-dataplane handling code.
>
> Thread safety of error reporting relies on Paolo's series:
>
>     [PATCH 0/5] qemu-char/monitor: make monitor_puts thread safe
>
>     [PATCH v2 0/2] block: thread-safety patches for virtio-blk dataplane
>     rerror/werror

I left a comment on patch 5, and it is also missing a patch to create 
the restart BH in the right AioContext.

I suspect even more could be unified, but it can be left to 2.2.  This 
patch gets the features.

Paolo