From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [SeaBIOS] [SeaBIOS PATCH 2/2] hotplug: Add device per func in ACPI DSDT tables Date: Mon, 19 Sep 2011 13:02:59 +0300 Message-ID: <20110919100257.GE4501@redhat.com> References: <20110919065338.22802.48911.stgit@t> <20110919095733.GO21417@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Amos Kong , seabios@seabios.org, kvm@vger.kernel.org, jasowang@redhat.com, alex williamson To: Gleb Natapov Return-path: Received: from mx1.redhat.com ([209.132.183.28]:51467 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753856Ab1ISKCK (ORCPT ); Mon, 19 Sep 2011 06:02:10 -0400 Content-Disposition: inline In-Reply-To: <20110919095733.GO21417@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: 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. > > Have tested with linux/winxp/win7, hot-adding/hot-remving, > > single/multiple function device, they are all fine. > > > 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