From af0669f0edbbcb8c17f7c2b919089485c8327f4f Mon Sep 17 00:00:00 2001 From: "Michael S. Tsirkin" Date: Tue, 14 Dec 2010 14:24:53 +0200 Subject: [PATCH] pci: don't use bus number in migration, stub out Using bus numbers in migration is clearly wrong as they are guest assigned. Not really sure what the right thing to do is, for now stick 0 in there so things keep working for non-nested setups, add a TODO. We also probably have to mark nested bridges as non-migrateable until this is fixed? Signed-off-by: Michael S. Tsirkin Acked-by: Alex Williamson --- hw/pci.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/hw/pci.c b/hw/pci.c index ef00d20d5f..0cb411720d 100644 --- a/hw/pci.c +++ b/hw/pci.c @@ -2014,7 +2014,10 @@ static char *pcibus_get_dev_path(DeviceState *dev) char path[16]; snprintf(path, sizeof(path), "%04x:%02x:%02x.%x", - pci_find_domain(d->bus), d->config[PCI_SECONDARY_BUS], + pci_find_domain(d->bus), + 0 /* TODO: need a persistent path for nested buses. + * Note: pci_bus_num(d->bus) is not right as it's guest + * assigned. */, PCI_SLOT(d->devfn), PCI_FUNC(d->devfn)); return strdup(path);