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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





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' &mdash;" without a <code> tag.

Fair enough.

Jirka


--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]