From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40098) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dUxYA-0005DE-G4 for qemu-devel@nongnu.org; Tue, 11 Jul 2017 11:58:11 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dUxY7-0000NG-Cz for qemu-devel@nongnu.org; Tue, 11 Jul 2017 11:58:10 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49786) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dUxY7-0000Ms-5z for qemu-devel@nongnu.org; Tue, 11 Jul 2017 11:58:07 -0400 Date: Tue, 11 Jul 2017 18:57:54 +0300 From: "Michael S. Tsirkin" Message-ID: <20170711185430-mutt-send-email-mst@kernel.org> References: <1499413442-5053-1-git-send-email-mark.cave-ayland@ilande.co.uk> <20170711182954-mutt-send-email-mst@kernel.org> <1fc46dca-4fc5-a7cd-986e-1e11a64642f3@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1fc46dca-4fc5-a7cd-986e-1e11a64642f3@redhat.com> Subject: Re: [Qemu-devel] [PATCH 0/3] pci: allow PCI bus slots to be marked as reserved List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Marcel Apfelbaum Cc: Mark Cave-Ayland , qemu-devel@nongnu.org, armbru@redhat.com On Tue, Jul 11, 2017 at 06:51:50PM +0300, Marcel Apfelbaum wrote: > On 11/07/2017 18:31, Michael S. Tsirkin wrote: > > On Fri, Jul 07, 2017 at 08:43:59AM +0100, Mark Cave-Ayland wrote: > > > For some machines it is impossible to plug devices into a particular PCI bus > > > slot, e.g. for a real Ultra 5 there are 2 PCI bridges attached to the root > > > bus behind which all devices must be plugged. Ignoring this rule will cause > > > problems with interrupt routing since the interrupt numbers are calculated > > > based upon PCI bridge id and secondary PCI bus slot id. > > > > > > This patchset adds a new dev_reserved_mask property to PCIBus which is a > > > bitmask used to indicate whether PCI bus slots are reserved, i.e. they cannot > > > be used for hot or cold plugging on a particular PCI bus. > > > > > > Signed-off-by: Mark Cave-Ayland > > > > Hi Michael, > > > Could you please point me at series making use of this > > functionality? > > There is a discussion upstream, I am not aware of patches yet. > > https://lists.gnu.org/archive/html/qemu-devel/2017-07/msg00771.html > > Thanks, > Marcel It is probably a good idea to defer merging this patchset until there's an agreement on how it will be used. > > > Thanks! > > > > > > > > Mark Cave-Ayland (3): > > > pci: move check for existing devfn into new pci_bus_devfn_available() > > > helper > > > pci: add dev_reserved_mask property to PCIBus > > > pci: add reserved slot check to do_pci_register_device() > > > > > > hw/pci/pci.c | 34 ++++++++++++++++++++++++++++++---- > > > include/hw/pci/pci_bus.h | 1 + > > > 2 files changed, 31 insertions(+), 4 deletions(-) > > > > > > -- > > > 1.7.10.4