[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 1/2] block-backend: Introduce blk_drain() and re
|
From: |
Kevin Wolf |
|
Subject: |
Re: [Qemu-block] [PATCH 1/2] block-backend: Introduce blk_drain() and replace blk_drain_all() |
|
Date: |
Mon, 8 Jun 2015 13:03:12 +0200 |
|
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 03.06.2015 um 15:46 hat Alexander Yarygin geschrieben:
> Each call of the virtio_blk_reset() function calls blk_drain_all(),
> which works for all existing BlockDriverStates, while only one
> BlockDriverState needs to be drained.
>
> This patch introduces the blk_drain() function and replaces
> blk_drain_all() on it in virtio_blk_reset().
>
> Cc: Christian Borntraeger <address@hidden>
> Cc: Cornelia Huck <address@hidden>
> Cc: Kevin Wolf <address@hidden>
> Cc: Paolo Bonzini <address@hidden>
> Cc: Stefan Hajnoczi <address@hidden>
> Signed-off-by: Alexander Yarygin <address@hidden>
For the BlockBackend part:
Acked-by: Kevin Wolf <address@hidden>
The only thing that I could suggest is that if you respin, it would be
cleaner to split this into two parts (as small as they may be). That you
have an "and" in your subject line is a strong hint for this.
Kevin
- [Qemu-block] [PATCH RFC 0/2] Fix slow startup with many disks, Alexander Yarygin, 2015/06/03
- [Qemu-block] [PATCH 1/2] block-backend: Introduce blk_drain() and replace blk_drain_all(), Alexander Yarygin, 2015/06/03
- Re: [Qemu-block] [PATCH 1/2] block-backend: Introduce blk_drain() and replace blk_drain_all(), Paolo Bonzini, 2015/06/03
- Re: [Qemu-block] [PATCH 1/2] block-backend: Introduce blk_drain() and replace blk_drain_all(), Christian Borntraeger, 2015/06/03
- Re: [Qemu-block] [PATCH 1/2] block-backend: Introduce blk_drain() and replace blk_drain_all(), Christian Borntraeger, 2015/06/08
- Re: [Qemu-block] [Qemu-devel] [PATCH 1/2] block-backend: Introduce blk_drain() and replace blk_drain_all(), Stefan Hajnoczi, 2015/06/08
- Re: [Qemu-block] [PATCH 1/2] block-backend: Introduce blk_drain() and replace blk_drain_all(),
Kevin Wolf <=
- Re: [Qemu-block] [Qemu-devel] [PATCH 1/2] block-backend: Introduce blk_drain() and replace blk_drain_all(), Fam Zheng, 2015/06/10
- [Qemu-block] [PATCH 2/2] block: Let bdrv_drain_all() to call aio_poll() for each AioContext, Alexander Yarygin, 2015/06/03