Re: [PATCH v2 1/2] conf: add xml element devices/pvpanic

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

 



On 12/03/2013 11:31 PM, Hu Tao wrote:

>>
>> pvpanic is a qemu term, but I could see the feasibility of other
>> hypervisors having a paravirt device with a sole purpose of notifying
>> the host about panics.  Do we want to come up with a more generic name?
> 
> Give it a generic name is easy, but what about attributes? different
> hypervisors may have different paravirt devices with different
> attributes, we can't just mix attributes of unrelated devices into one
> generic device. Make the devices concrete and accept/reject it if
> hypervisors recognize it or not is better.

Then do what we've done in the past - separate the presence of a type of
device from the driver-specific attributes, something like:

<devices>
  <panic>
    <driver type='qemu' ioport='0x505'/>
  </panic>
...

where we use the type='' field of <driver> to toggle between a
discriminated union of any other per-hypervisor specific attributes.  By
default, you don't need a <driver> subelement; requesting <panic/> is
sufficient to use the defaults for the current hypervisor.

Dan, do you have any thoughts on the best representation to use?  Or is
Hu's original proposal of:

  <pvpanic ioport='0x505'/>

sufficient?

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
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]