mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-02-25 21:02:22 +00:00
qemu: fix error log in qemuAssignPCIAddresses()
This error message was too specific, based on the incorrect assumption that any error was cause by auto-added bridges: failed to create PCI bridge on bus 2: too many devices with fixed addresses In practice you can't know if a bridge with an index <= the bus it's connecting to was added automatically, or if it was a mistake in explicit config, and the auto-add problem is going to be dealt with in a different way in an upcoming patch. The new message is this: PCI Controller at index 1 (0x01) has " bus='0x02', but bus must be <= index (note that index is given in both decimal and hex because it is formatted as decimal in the XML, but bus is formatted as hex, and displaying the hex value of index makes it easier to see the problem when index > 9 (which will often be the case with PCIe, since most controllers only have a single port, not 32 slots as with standard PCI)). Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1004593 (cherry picked from commit ff2126225df019566e4e580d92e69e9df3ca96c0)
This commit is contained in:
parent
68b5d04ded
commit
46fdaf5d9a
@ -1545,10 +1545,9 @@ qemuDomainAssignPCIAddresses(virDomainDefPtr def,
|
|||||||
if (cont->model == VIR_DOMAIN_CONTROLLER_MODEL_PCI_BRIDGE &&
|
if (cont->model == VIR_DOMAIN_CONTROLLER_MODEL_PCI_BRIDGE &&
|
||||||
idx <= addr->bus) {
|
idx <= addr->bus) {
|
||||||
virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
|
virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
|
||||||
_("failed to create PCI bridge "
|
_("PCI controller at index %d (0x%02x) has "
|
||||||
"on bus %d: too many devices with fixed "
|
"bus='0x%02x', but bus must be <= index"),
|
||||||
"addresses"),
|
idx, idx, addr->bus);
|
||||||
addr->bus);
|
|
||||||
goto cleanup;
|
goto cleanup;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user