[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv6 17/17] block/raw: copy BlockLimits on raw_open
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCHv6 17/17] block/raw: copy BlockLimits on raw_open |
Date: |
Thu, 24 Oct 2013 11:06:25 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130923 Thunderbird/17.0.9 |
Il 24/10/2013 10:12, Peter Lieven ha scritto:
>> This must be moved before the introduction of BlockLimits in the iscsi
>> driver, or patches that use BlockLimits in block.c will not have any
>> effect.
> You are the first to mention this. I was thinking the whole series will
> be seen as once so it shouldn't matter.
In general, series should keep old functionality at all stages. This
helps when someone reports a regression, because we can ask them to
bisect and not have them burdened by problems in the middle of a series.
(It would also help you debugging things, if this series turned out to
have a bug).
After patch 10 of this series, an iSCSI array will stop receiving split
requests for large discards. This may introduce spurious failures.
I made the same remark on patch 11, but that patch alone is not enough
to restore this; you need this one too for patch 11 to have any effect.
So the correct order is patch 17 first, then patch 11, then patch 10.
In other word, remove code only after it has become dead.
Paolo
- [Qemu-devel] [PATCHv6 14/17] block/get_block_status: fix BDRV_BLOCK_ZERO for unallocated blocks, (continued)
- [Qemu-devel] [PATCHv6 14/17] block/get_block_status: fix BDRV_BLOCK_ZERO for unallocated blocks, Peter Lieven, 2013/10/24
- [Qemu-devel] [PATCHv6 13/17] block: introduce bdrv_make_zero, Peter Lieven, 2013/10/24
- [Qemu-devel] [PATCHv6 12/17] iscsi: add bdrv_co_write_zeroes, Peter Lieven, 2013/10/24
- [Qemu-devel] [PATCHv6 16/17] qemu-img: conditionally zero out target on convert, Peter Lieven, 2013/10/24
- [Qemu-devel] [PATCHv6 17/17] block/raw: copy BlockLimits on raw_open, Peter Lieven, 2013/10/24
- [Qemu-devel] [PATCHv6 15/17] qemu-img: add support for fully allocated images, Peter Lieven, 2013/10/24