Re: [libvirt] [PATCH] qemu: Fix PCI address allocation

> When attaching a PCI device which doesn't explicitly set its PCI > address, libvirt allocates the address automatically. The problem is > that when checking which PCI address is unused, we only check for those > with slot number higher than the highest slot number ever used. > > Thus attaching/detaching such device several times in a row (31 is the > theoretical limit, less then 30 tries are enough in practise) makes any > further device attachment fail. Furthermore, attaching a device with > predefined PCI address to 0:0:31 immediately forbids attachment of any > PCI device without explicit address. > > This patch changes the logic so that we always check all PCI addresses > before we say there is no PCI address available. > - for (i = addrs->nextslot ; i <= QEMU_PCI_ADDRESS_LAST_SLOT ; i++) { > + for (i = 0 ; i <= QEMU_PCI_ADDRESS_LAST_SLOT ; i++) { Don't we still want to start from the last slot, and wrap around the search only when we reach the end, rather than always starting from 0? I'm thinking particularly about compatibility with older qemu, where always starting from 0 risks interleaving new assignments among the pre-assigned slots, and may end up allocating slots in a way that throws off Windows. Or am I worried about a non-issue?