On Tue, Jan 17, 2012 at 12:05:03 -0700, Eric Blake wrote:
On 01/17/2012 08:47 AM, Jiri Denemark wrote:
> In the past we didn't reserve 0:0:2.0 PCI address if there was no video
> device assigned to a domain, which made it impossible to add a video
> device later on. So we fixed it (commit v0.9.0-37-g7b2cac1) by always
> reserving that address. However, that breaks existing domains without
> video devices that already have another device assigned to the
> problematic address.
>
> This patch reserves address 0:0:2.0 only in case it was not explicitly
> assigned to another device, which means libvirt will try to keep this
> address free and will not automatically assign it new devices. But
> existing domains for which older libvirt already assigned the address to
> a non-video device will keep working as they used to work before 0.9.1.
> Moreover, users who want to create a domain without a video device and
> use its address for another device may do so by explicitly configuring
> the PCI address in domain XML.
> ---
> src/qemu/qemu_command.c | 21 +++++++++++++++------
> 1 files changed, 15 insertions(+), 6 deletions(-)
ACK.
Nasty that we had a regression in ABI for that long.
Thanks, pushed.
Jirka