[Qemu-devel] Re: [PATCH] pci: allow hotplug removal of cold-plugged devices

2010-11-16 Thread Michael S. Tsirkin
On Mon, Nov 15, 2010 at 11:23:44AM +0900, Isaku Yamahata wrote: Thank you for catching pcie part. The following fix is necessary. diff --git a/hw/pcie.c b/hw/pcie.c index 4df48b8..f461c1c 100644 --- a/hw/pcie.c +++ b/hw/pcie.c @@ -221,7 +221,7 @@ static int

[Qemu-devel] Re: [PATCH] pci: allow hotplug removal of cold-plugged devices

2010-11-15 Thread Cam Macdonell
On Sun, Nov 14, 2010 at 7:18 AM, Michael S. Tsirkin m...@redhat.com wrote: This patch fixes 5beb8ad503c88a76f2b8106c3b74b4ce485a60e1 which broke hotplug removal of cold plugged devices: - pass addition/removal state to hotplug callbacks - use that in piix and pcie This also fixes an assert

[Qemu-devel] Re: [PATCH] pci: allow hotplug removal of cold-plugged devices

2010-11-14 Thread Isaku Yamahata
Looks good. Thank you for catching pcie case. On Sun, Nov 14, 2010 at 04:18:04PM +0200, Michael S. Tsirkin wrote: This patch fixes 5beb8ad503c88a76f2b8106c3b74b4ce485a60e1 which broke hotplug removal of cold plugged devices: - pass addition/removal state to hotplug callbacks - use that in

[Qemu-devel] Re: [PATCH] pci: allow hotplug removal of cold-plugged devices

2010-11-14 Thread Isaku Yamahata
Thank you for catching pcie part. The following fix is necessary. diff --git a/hw/pcie.c b/hw/pcie.c index 4df48b8..f461c1c 100644 --- a/hw/pcie.c +++ b/hw/pcie.c @@ -221,7 +221,7 @@ static int pcie_cap_slot_hotplug(DeviceState *qdev, */ assert(PCI_FUNC(pci_dev-devfn) == 0); -if