[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv2 3/6] block: add a knob to disable multiwrite_m
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCHv2 3/6] block: add a knob to disable multiwrite_merge |
Date: |
Wed, 22 Oct 2014 12:29:44 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 |
On 10/22/2014 07:21 AM, Peter Lieven wrote:
> The block layer silently merges write requests since
> commit 40b4f539. This patch adds a knob to disable
> this feature as there has been some discussion lately
> if multiwrite is a good idea at all and as it falsifies
> benchmarks.
>
> Signed-off-by: Peter Lieven <address@hidden>
> Reviewed-by: Max Reitz <address@hidden>
>
> Signed-off-by: Peter Lieven <address@hidden>
Is the double s-o-b intentional?
> +++ b/hmp.c
> @@ -348,6 +348,10 @@ void hmp_info_block(Monitor *mon, const QDict *qdict)
>
> BlockdevDetectZeroesOptions_lookup[info->value->inserted->detect_zeroes]);
> }
>
> + if (!info->value->inserted->write_merging) {
> + monitor_printf(mon, " Write Merging: off\n");
> + }
Is it also worth printing something if write merging is enabled?
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCHv2 0/6] multiwrite patches for 2.2, Peter Lieven, 2014/10/22
- [Qemu-devel] [PATCHv2 4/6] hw/virtio-blk: add a constant for max number of merged requests, Peter Lieven, 2014/10/22
- [Qemu-devel] [PATCHv2 2/6] block: introduce bdrv_runtime_opts, Peter Lieven, 2014/10/22
- [Qemu-devel] [PATCHv2 3/6] block: add a knob to disable multiwrite_merge, Peter Lieven, 2014/10/22
- Re: [Qemu-devel] [PATCHv2 3/6] block: add a knob to disable multiwrite_merge,
Eric Blake <=
- [Qemu-devel] [PATCHv2 1/6] block: add accounting for merged requests, Peter Lieven, 2014/10/22
- [Qemu-devel] [PATCHv2 6/6] block: fix qemu-iotest reference output for test 067, Peter Lieven, 2014/10/22
- [Qemu-devel] [PATCHv2 5/6] block: add qemu-iotest for write-merge parameter, Peter Lieven, 2014/10/22