[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v2 2/3] block-jobs: Optionally unre
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v2 2/3] block-jobs: Optionally unregister live block operations |
Date: |
Thu, 31 Aug 2017 08:45:40 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 08/30/2017 12:24 PM, Eduardo Habkost wrote:
>> On Wed, Aug 30, 2017 at 01:01:41PM -0400, Jeff Cody wrote:
>>> From: Jeffrey Cody <address@hidden>
>>>
>>> If configured without live block operations enabled, unregister the
>>> live block operation commands.
>>>
>>> Signed-off-by: Jeff Cody <address@hidden>
>>> ---
>>> monitor.c | 16 ++++++++++++++++
>>> 1 file changed, 16 insertions(+)
>>>
>
>>
>> I suggest using the new mechanisms added by:
>>
>> [PATCH 00/26] qapi: add #if pre-processor conditions to generated code
>
> Those haven't landed yet, but as both series are proposed for 2.11, I
> indeed agree that basing this series on top of that one will be a bit
> cleaner.
Rebasing shouldn't be hard. However, we then have to hold it until the
QAPI series lands. I don't think holding is necessary, as the conflicts
between the two are obvious, and should be straightforward to resolve.