[SeaBIOS PATCH 2/2] hotplug: Add device per func in ACPI DSDT tables

2011-09-19 Thread Amos Kong
Only func 0 is registered to guest driver (we can only found func 0 in slot-funcs list of driver), the other functions could not be cleaned when hot-removing the whole slot. This patch adds device per function in ACPI DSDT tables. Have tested with linux/winxp/win7, hot-adding/hot-remving,

Re: [SeaBIOS PATCH 2/2] hotplug: Add device per func in ACPI DSDT tables

2011-09-19 Thread Michael S. Tsirkin
On Mon, Sep 19, 2011 at 12:36:44PM +0300, Michael S. Tsirkin wrote: On Mon, Sep 19, 2011 at 02:53:47PM +0800, Amos Kong wrote: Only func 0 is registered to guest driver (we can only found func 0 in slot-funcs list of driver), the other functions could not be cleaned when hot-removing the

Re: [SeaBIOS] [SeaBIOS PATCH 2/2] hotplug: Add device per func in ACPI DSDT tables

2011-09-19 Thread Gleb Natapov
On Mon, Sep 19, 2011 at 03:27:38AM -0400, Amos Kong wrote: Only func 0 is registered to guest driver (we can only found func 0 in slot-funcs list of driver), the other functions could not be cleaned when hot-removing the whole slot. This patch adds device per function in ACPI DSDT tables.

Re: [SeaBIOS] [SeaBIOS PATCH 2/2] hotplug: Add device per func in ACPI DSDT tables

2011-09-19 Thread Michael S. Tsirkin
On Mon, Sep 19, 2011 at 12:57:33PM +0300, Gleb Natapov wrote: On Mon, Sep 19, 2011 at 03:27:38AM -0400, Amos Kong wrote: Only func 0 is registered to guest driver (we can only found func 0 in slot-funcs list of driver), the other functions could not be cleaned when hot-removing the

Re: [SeaBIOS PATCH 2/2] hotplug: Add device per func in ACPI DSDT tables

2011-09-19 Thread Michael S. Tsirkin
On Mon, Sep 19, 2011 at 12:36:44PM +0300, Michael S. Tsirkin wrote: On Mon, Sep 19, 2011 at 02:53:47PM +0800, Amos Kong wrote: Only func 0 is registered to guest driver (we can only found func 0 in slot-funcs list of driver), the other functions could not be cleaned when hot-removing the

Re: [SeaBIOS] [SeaBIOS PATCH 2/2] hotplug: Add device per func in ACPI DSDT tables

2011-09-19 Thread Gleb Natapov
On Mon, Sep 19, 2011 at 01:02:59PM +0300, Michael S. Tsirkin wrote: On Mon, Sep 19, 2011 at 12:57:33PM +0300, Gleb Natapov wrote: On Mon, Sep 19, 2011 at 03:27:38AM -0400, Amos Kong wrote: Only func 0 is registered to guest driver (we can only found func 0 in slot-funcs list of

Re: [SeaBIOS] [SeaBIOS PATCH 2/2] hotplug: Add device per func in ACPI DSDT tables

2011-09-19 Thread Gleb Natapov
On Mon, Sep 19, 2011 at 01:12:30PM +0300, Gleb Natapov wrote: On Mon, Sep 19, 2011 at 01:02:59PM +0300, Michael S. Tsirkin wrote: On Mon, Sep 19, 2011 at 12:57:33PM +0300, Gleb Natapov wrote: On Mon, Sep 19, 2011 at 03:27:38AM -0400, Amos Kong wrote: Only func 0 is registered to

Re: [SeaBIOS] [SeaBIOS PATCH 2/2] hotplug: Add device per func in ACPI DSDT tables

2011-09-19 Thread Michael S. Tsirkin
On Mon, Sep 19, 2011 at 01:32:48PM +0300, Gleb Natapov wrote: I recall I saw some examples like this on the net. Checking real HW DSDT will validate that we are doing a right thing here. According to Microsoft own documentation they want _EJ0 for each function:

Re: [SeaBIOS] [SeaBIOS PATCH 2/2] hotplug: Add device per func in ACPI DSDT tables

2011-09-19 Thread Marcelo Tosatti
On Mon, Sep 19, 2011 at 01:02:59PM +0300, Michael S. Tsirkin wrote: On Mon, Sep 19, 2011 at 12:57:33PM +0300, Gleb Natapov wrote: On Mon, Sep 19, 2011 at 03:27:38AM -0400, Amos Kong wrote: Only func 0 is registered to guest driver (we can only found func 0 in slot-funcs list of

Re: [SeaBIOS] [SeaBIOS PATCH 2/2] hotplug: Add device per func in ACPI DSDT tables

2011-09-19 Thread Michael S. Tsirkin
On Mon, Sep 19, 2011 at 01:27:25PM -0300, Marcelo Tosatti wrote: On Mon, Sep 19, 2011 at 01:02:59PM +0300, Michael S. Tsirkin wrote: On Mon, Sep 19, 2011 at 12:57:33PM +0300, Gleb Natapov wrote: On Mon, Sep 19, 2011 at 03:27:38AM -0400, Amos Kong wrote: Only func 0 is registered to