[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v4 10/10] block/blklogwrites: Use the block devi
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-block] [PATCH v4 10/10] block/blklogwrites: Use the block device logical sector size when logging writes |
Date: |
Mon, 18 Jun 2018 17:36:44 +0200 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Fri 08 Jun 2018 02:32:28 PM CEST, Ari Sundholm wrote:
> The guest OS may perform writes which are aligned to the logical
> sector size instead of the physical one, so logging at this granularity
> records the writes performed on the block device most faithfully.
>
> Signed-off-by: Ari Sundholm <address@hidden>
> ---
> block/blklogwrites.c | 47 ++++++++++++++++++++++++++++++++---------------
> 1 file changed, 32 insertions(+), 15 deletions(-)
>
> diff --git a/block/blklogwrites.c b/block/blklogwrites.c
> index 216367f..decf5e5 100644
> --- a/block/blklogwrites.c
> +++ b/block/blklogwrites.c
> @@ -47,6 +47,8 @@ struct log_write_entry {
>
> typedef struct {
> BdrvChild *log_file;
> + uint32_t sectorsize;
> + uint32_t sectorbits;
> uint64_t cur_log_sector;
> uint64_t nr_entries;
> } BDRVBlkLogWritesState;
> @@ -67,6 +69,8 @@ static int blk_log_writes_open(BlockDriverState *bs, QDict
> *options, int flags,
> goto fail;
> }
>
> + s->sectorsize = BDRV_SECTOR_SIZE; /* May be updated later */
> + s->sectorbits = BDRV_SECTOR_BITS;
> s->cur_log_sector = 1;
> s->nr_entries = 0;
I haven't looked closely into this series so sorry if there's something
that I'm overlooking, but this caught my attention: why do you have a
patch that improves a driver that you introduced earlier in this same
series?
Berto
- [Qemu-block] [PATCH v4 01/10] block: Move two block permission constants to the relevant enum, (continued)
- [Qemu-block] [PATCH v4 01/10] block: Move two block permission constants to the relevant enum, Ari Sundholm, 2018/06/08
- [Qemu-block] [PATCH v4 06/10] hw/block/virtio-blk: Always apply block configuration to block driver, Ari Sundholm, 2018/06/08
- [Qemu-block] [PATCH v4 08/10] hw/block/fdc: Always apply block configuration to block driver, Ari Sundholm, 2018/06/08
- [Qemu-block] [PATCH v4 02/10] block: Add blklogwrites, Ari Sundholm, 2018/06/08
- [Qemu-block] [PATCH v4 05/10] hw/ide/qdev: Always apply block configuration to block driver, Ari Sundholm, 2018/06/08
- [Qemu-block] [PATCH v4 07/10] hw/block/nvme: Always apply block configuration to block driver, Ari Sundholm, 2018/06/08
- [Qemu-block] [PATCH v4 09/10] block/blklogwrites: Use block limits from the backend block configuration, Ari Sundholm, 2018/06/08
- [Qemu-block] [PATCH v4 03/10] block: Add a mechanism for passing a block driver a block configuration, Ari Sundholm, 2018/06/08
- [Qemu-block] [PATCH v4 10/10] block/blklogwrites: Use the block device logical sector size when logging writes, Ari Sundholm, 2018/06/08
- Re: [Qemu-block] [PATCH v4 10/10] block/blklogwrites: Use the block device logical sector size when logging writes,
Alberto Garcia <=
- Re: [Qemu-block] [PATCH v4 00/10] New block driver: blklogwrites, Ari Sundholm, 2018/06/14