On 04/04/2018 04:54 AM, Andrea Bolognani wrote: > On Tue, 2018-04-03 at 22:44 -0400, Laine Stump wrote: > >> In the meantime, if I remove that code (and don't apply any of your >> patches) then a pure pcie domain *can* be successfully edited to add a >> single pci-bridge (as long as you specify an index, *and* there is an >> unused index of a smaller value), but what ends up in the >> "proper/validated" config is a pci-bridge that is plugged directly into >> a pcie-root-port (which is also wrong, but silently so). >> >> So I guess I reserve my judgement until I see what happens with your >> entire series applied, which I'll do tomorrow morning. > Looking forward to that :) > So it turns out that once all of your patches are applied, the error is the same before or after this code is removed, so I think it may not be executed at all. Since it doesn't solve the issue it originally was intended to solve, and removing it makes no functional difference, I have no problem with you removing it (it would be nice if we could figure out a way to log a more informative error than: unsupported configuration: PCI controller at index 4 (0x04) has bus='0x04', but index must be larger than bus but that's a separate issue.) -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
