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

    but index must be larger than bus

but that's a separate issue.)

libvir-list mailing list

Reply via email to