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

Jiri Denemark jdenemar at redhat.com
Fri Nov 13 09:12:38 UTC 2015


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.
> 
> 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
  the right one (isa or pseries); we'd also need to check we don't have
  multiple panic devices with the same model
- 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




More information about the libvir-list mailing list