Re: issue in unplug qemu PCI devices

["Zhai, Edwin" <edwin.zhai@intel.com>]

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