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 driver), > > > the other functions could not be cleaned when > > > hot-removing the whole slot. This patch adds > > > device per function in ACPI DSDT tables. > > > > > You can't unplug a single function. Guest surely knows that. > > Looking at guest code, it's clear that > at least a Linux guest doesn't know that. acpiphp_disable_slot function appears to eject all functions. > > > Have tested with linux/winxp/win7, hot-adding/hot-remving, > > > single/multiple function device, they are all fine. > > > Does not work for me (FC12 guest). As mentioned previously, Linux driver looks for function 0 when injection request is seen (see enable_device function in acpiphp_glue.c). > > What was not fine before? > > > > Have you looked at real HW that supports PCI hot plug DSDT? Does it > > looks the same? > > I recall I saw some examples like this on the net. > > > > > new acpi-dst.hex(332K): > > > http://amos-kong.rhcloud.com/pub/acpi-dsdt.hex > > > > > > Signed-off-by: Amos Kong <akong@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
