Re: [PATCH v3 14/26] conf: Add 'spapr-pci-host-bridge' controller model

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

 



On Sun, 2017-06-25 at 09:55 -0400, Laine Stump wrote:
> > Adding it to the virDomainControllerPCIModelName enumeration
> > is enough for existing code to handle it, so parsing and
> > formatting will work without further tweaking.
> > 
> > Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> 
> Reviewed-by: Laine Stump <laine@xxxxxxxxx>
> 
> (assuming you document it in one of these other patches)

How exactly would you like it to be documented?

I looked into it before sending out v3 and found out that
formatdomain.html already contains the following text:

  PCI controllers also have an optional subelement <model>
  with an attribute name. The name attribute holds the name
  of the specific device that qemu is emulating (e.g.
  "i82801b11-bridge") rather than simply the class of device
  ("dmi-to-pci-bridge", "pci-bridge"), which is set in the
  controller element's model attribute. In almost all cases,
  you should not manually add a <model> subelement to a
  controller, nor should you modify one that is automatically
  generated by libvirt. Since 1.2.19 (QEMU only).

So that about covers it, doesn't it? In fact, other existing
concrete controller models such as ioh3240 are not mentioned
at all.

-- 
Andrea Bolognani / Red Hat / Virtualization

--
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]
  Powered by Linux