[libvirt] [PATCHv3 0/4] Hyper-v crash feature support

[Dmitry Andreev]

On 13.11.2015 12:12, Jiri Denemark wrote:
> On Fri, Nov 13, 2015 at 11:30:30 +0300, Dmitry Andreev wrote:
>> A new Hyper-V cpu feature 'hv_crash' was added to QEMU. The feature
>> will become available in v2.5.0.
>>
>> This patch adds support for this feature.
>>n & Addressing -> Compose messages in HTML format for each ac
>> The previous version of this patch is
>> https://www.redhat.com/archives/libvir-list/2015-November/msg00400.html
>>
>> What is changed:
>>   * 'model' is an optional attribute and will not be added
>>     if it wasn't specified by the user [1/4]
>
> Ouch, there seems to be a misunderstanding here. I didn't want you to
> stop updating the model if it wasn't specified. The reason for adding a
> "default" model is:
>
> - XML parser takes the XML and leaves model as "default" in case it
>    wasn't specified in the XML
> - post parse callback in qemu driver changes the default panic model to
User may change guest arch or hv but he still want to have default
panic device. Are you sure we need to set panic model?
>    the right one (isa or pseries); we'd also need to check we don't have
>    multiple panic devices with the same model
One per type?
Should I use virDomainPanicDefPtr panics[VIR_DOMAIN_PANIC_MODEL_LAST]; 
or just send error from qemu_command if find multiple panic devices with 
the same model in array?
> - XML formatter only prints model if it is not default (won't happen
>    with qemu driver, of course)
>
>>   * docs text fixed [1/4] [3/4]
>>   * schema and code allows multiple panic devices [3/4]
>>   * more tests is added and all changes in old tests
>>     are removed [4/4]
>>
>> What isn't changed:
>>
>> I didn't add <code> tag around model values in docs because there
>> are many examples of "<ul><li> 'value' —" without a <code> tag.
>
> Fair enough.
>
> Jirka
>