[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/3] dataplane: add query-blockstats support
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 0/3] dataplane: add query-blockstats support |
Date: |
Wed, 18 Jun 2014 19:10:21 +0800 |
On Tue, Jun 10, 2014 at 5:00 PM, Paolo Bonzini <address@hidden> wrote:
> Il 10/06/2014 09:29, Stefan Hajnoczi ha scritto:
>
>> This series adds blockstats support to virtio-blk data-plane and protects
>> query-blockstats from races with the dataplane IOThread.
>>
>> Stefan Hajnoczi (3):
>> dataplane: add bdrv_acct_*() accounting
>> block: make bdrv_query_stats() static
>> block: acquire AioContext is qmp_query_blockstats()
>>
>> block/qapi.c | 6 +++++-
>> hw/block/dataplane/virtio-blk.c | 10 ++++++++++
>> include/block/qapi.h | 1 -
>> 3 files changed, 15 insertions(+), 2 deletions(-)
>>
>
> ACK to patches 2 and 3.
>
> Regarding patch 1 it's nice that the change is so trivial so I'm not
> objecting to the patch.
>
> However, Fam's patches for VirtIOBlockReq{,uest} unification provide another
> way to achieve this. If we could drop do_rdwr_cmd in favor of
> virtio_blk_handle_read and virtio_blk_handle_write, we would get for free
> both blockstats and rerror/werror support. What do you think?
Fam has sent out equivalent patches to my series that are based on the
unified dataplane/non-dataplane virtio code.
NACK, let's use Fam's patches instead.
Stefan