qemu-devel
[Top][All Lists]
Advanced

[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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]