[Libvirt-ci] Passed: libvirt/libvirt#839 (master - 71d56a3)

Travis CI builds at travis-ci.org
Sat Jan 20 07:48:59 UTC 2018


Build Update for libvirt/libvirt
-------------------------------------

Build: #839
Status: Passed

Duration: 14 hours, 33 minutes, and 10 seconds
Commit: 71d56a3 (master)
Author: Jim Fehlig
Message: nodedev: Fix failing to parse PCI address for non-PCI network devices

Commit 8708ca01c added virNetDevSwitchdevFeature() to check if a network
device has Switchdev capabilities. virNetDevSwitchdevFeature() attempts
to retrieve the PCI device associated with the network device, ignoring
non-PCI devices. It does so via the following call chain

  virNetDevSwitchdevFeature()->virNetDevGetPCIDevice()->
  virPCIGetDeviceAddressFromSysfsLink()

For non-PCI network devices (qeth, Xen vif, etc),
virPCIGetDeviceAddressFromSysfsLink() will report an error when
virPCIDeviceAddressParse() fails. virPCIDeviceAddressParse() also
logs an error. After commit 8708ca01c there are now two errors reported
for each non-PCI network device even though the errors are harmless.

To avoid the errors, introduce virNetDevIsPCIDevice() and use it in
virNetDevGetPCIDevice() before attempting to retrieve the associated
PCI device. virNetDevIsPCIDevice() uses the 'subsystem' property of the
device to determine if it is PCI. See the sysfs rules in kernel
documentation for more details

https://www.kernel.org/doc/html/latest/admin-guide/sysfs-rules.html

View the changeset: https://github.com/libvirt/libvirt/compare/72adaf2f1050...71d56a397925

View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/330911283?utm_source=email&utm_medium=notification

--

You can configure recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvirt-ci/attachments/20180120/248fa675/attachment.htm>


More information about the Libvirt-ci mailing list