[libvirt] [PATCHv2 10/17] conf: new pci controller model "pcie-root-port"

John Ferlan jferlan at redhat.com
Sat Jul 25 00:32:46 UTC 2015


...
>>> @@ -18914,6 +18941,12 @@ virDomainControllerDefFormat(virBufferPtr buf,
>>>              if (def->opts.pciopts.chassisNr != -1)
>>>                  virBufferAsprintf(buf, " chassisNr='%d'",
>>>                                    def->opts.pciopts.chassisNr);
>>> +            if (def->opts.pciopts.chassis != -1)
>>> +                virBufferAsprintf(buf, " chassis='%d'",
>>> +                                  def->opts.pciopts.chassis);
>>> +            if (def->opts.pciopts.port != -1)
>>> +                virBufferAsprintf(buf, " port='0x%x'",
>>> +                                  def->opts.pciopts.port);
>> OH... And this is what I get for not reading ahead... So now my previous
>> comments are unfounded... <sigh>
> 
> Which comments are those? There was a comment in a previous patch about
> not needing the boolean "pciTarget" that wasn't valid, but your comments
> about the range were certainly valid.
> 

In patch 7 I indicated it wasn't necessary to have multiple lines and
the whole thing could be formatted in one line (when just chassisNr
existed).... But with adding more attributes, that's no longer valid.

John




More information about the libvir-list mailing list