[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v5 8/8] block: Add blklogwrites
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-block] [PATCH v5 8/8] block: Add blklogwrites |
Date: |
Fri, 29 Jun 2018 18:31:36 +0200 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
Am 29.06.2018 um 18:02 hat Ari Sundholm geschrieben:
> On 06/29/2018 03:05 PM, Kevin Wolf wrote:
> > Am 19.06.2018 um 15:43 hat Ari Sundholm geschrieben:
> > > + s->cur_log_sector = 1;
> > > + s->nr_entries = 0;
> >
> > Would it be useful to implement a mode that appends to the log?
> >
> > In that case, you'd obviously use the sector size from the existing
> > superblock instead of allowing the user to specify something else.
> >
>
> Such a mode may indeed be useful. Thank you for the idea. Would it be OK to
> introduce this feature as a separate patch a bit later?
Yes, of course.
> > > +static BlockDriver bdrv_blk_log_writes = {
> > > + .format_name = "blklogwrites",
> > > + .protocol_name = "blklogwrites",
> >
> > This is for the blklogwrites:X:Y syntax, which is not supported, so it
> > should be removed.
> >
>
> Just protocol_name, I assume? Will remove, thanks.
Right, just protocol_name. format_name is always required.
Kevin
- [Qemu-block] [PATCH v5 5/8] hw/block/virtio-blk: Always apply block configuration to block driver, (continued)
- [Qemu-block] [PATCH v5 5/8] hw/block/virtio-blk: Always apply block configuration to block driver, Ari Sundholm, 2018/06/19
- [Qemu-block] [PATCH v5 1/8] block: Move two block permission constants to the relevant enum, Ari Sundholm, 2018/06/19
- [Qemu-block] [PATCH v5 4/8] hw/ide/qdev: Always apply block configuration to block driver, Ari Sundholm, 2018/06/19
- [Qemu-block] [PATCH v5 2/8] block: Add a mechanism for passing a block driver a block configuration, Ari Sundholm, 2018/06/19
- [Qemu-block] [PATCH v5 7/8] hw/block/fdc: Always apply block configuration to block driver, Ari Sundholm, 2018/06/19
- [Qemu-block] [PATCH v5 8/8] block: Add blklogwrites, Ari Sundholm, 2018/06/19
- Re: [Qemu-block] [PATCH v5 0/8] New block driver: blklogwrites, Ari Sundholm, 2018/06/26