|
|
|
|
|
|
|
|
|
|
xen-devel
Re: [Xen-devel] issue in unplug qemu PCI devices
Stefano,
I have tested the latest ioemu, your 2 fixes really work.
Thanks!
Stefano Stabellini wrote:
On Wed, 17 Feb 2010, Edwin Zhai wrote:
Stefano,
Thanks for your fix. I'm in leave now and will have a test after back.
BTW, do you think we need fix issue 2? You know some OSV will install
and load the platform_pci module in guest by default. If end user
doesn't config valid PV device in config file, guest would crash after
unplugging emulated device. Is it necessary that we check if having a
valid backend in xenstore before unplug related emulated device?
You are right, qemu has to honor disable_pf for ioports as well.
---
diff --git a/hw/pc.c b/hw/pc.c
index 129e9d9..9375951 100644
--- a/hw/pc.c
+++ b/hw/pc.c
@@ -1047,9 +1047,10 @@ vga_bios_error:
#endif /* !CONFIG_DM */
if (pci_enabled) {
disable_pf = xenstore_parse_disable_pf_config();
- if (disable_pf != 1)
+ if (disable_pf != 1) {
pci_xen_platform_init(pci_bus);
- platform_fixed_ioport_init();
+ platform_fixed_ioport_init();
+ }
}
for(i = 0; i < MAX_SERIAL_PORTS; i++) {
--
best rgds,
edwin
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
|
|