[libvirt] [PATCH v2 03/12] qemu: Introduce pr-manager-helper capability

Michal Privoznik mprivozn at redhat.com
Tue Mar 6 17:31:55 UTC 2018


On 03/02/2018 02:58 AM, John Ferlan wrote:
> 
> 
> On 02/21/2018 01:11 PM, Michal Privoznik wrote:
>> The capability tracks if qemu has pr-manager-helper object. At
>> this time don't actually detect if qemu has the capability. Not
>> just yet. Only after the code is written the feature will be
>> enabled.
>>
>> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
>> ---
>>  src/qemu/qemu_capabilities.c | 1 +
>>  src/qemu/qemu_capabilities.h | 1 +
>>  2 files changed, 2 insertions(+)
>>
> 
> Very strange to not see the replies or xml files adjusted here.
> 
> And why is patch 12 not merged in here?  Like it was in v1 where you got
> an ACK from Peter?

Because Peter also said that the patches should be ordered in such way
that after each patch nothing is broken. So I had two options:

a) rewrite all the patches from scratch, or
b) separate the capability detection into its own patch so effectively
there is no way to use the feature until the very last patch and thus
there's nothing to break. This was used fairly often historically when
we were introducing new features in multiple patches.

Michal




More information about the libvir-list mailing list