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
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
On 13.11.2015 12:12, Jiri Denemark wrote: panic device. Are you sure we need to set panic model? 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