From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49831) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Uoh70-0006vK-IY for qemu-devel@nongnu.org; Mon, 17 Jun 2013 17:37:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Uoh6x-0006LY-Dr for qemu-devel@nongnu.org; Mon, 17 Jun 2013 17:37:18 -0400 Received: from mx1.redhat.com ([209.132.183.28]:22819) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Uoh6x-0006LK-6q for qemu-devel@nongnu.org; Mon, 17 Jun 2013 17:37:15 -0400 Date: Tue, 18 Jun 2013 00:37:58 +0300 From: "Michael S. Tsirkin" Message-ID: <20130617213747.GA15872@redhat.com> References: <1370508534-12335-1-git-send-email-david@gibson.dropbear.id.au> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1370508534-12335-1-git-send-email-david@gibson.dropbear.id.au> Subject: Re: [Qemu-devel] [0/10] Clean up PCI code to allow for multiple root buses (v2) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: David Gibson Cc: aliguori@us.ibm.com, qemu-devel@nongnu.org On Thu, Jun 06, 2013 at 06:48:44PM +1000, David Gibson wrote: > The current PCI subsystem has kind of half-hearted support for > multiple independent root buses - aka PCI domains - in the form of the > PCIHostBus structure and its domain field. However, it doesn't quite > work because pci_host_bus_register() is always called with a domain of > 0. > > Worse, though, the whole concept of numbered domains isn't general > enough. Many platforms can have independent root buses (usually on > wholly independent host bridges), but only x86 gives them a > hardware-significant domain number, essentially as a hack to allow all > the separate config spaces to be accessed via the same IO ports. > Linux guests on other platforms will show domain numbers in lspci, but > these are purely guest assigned, so qemu won't know about them. > > This patch series, therefore, removes the broken-as-is domain concept > from qemu, and replaces it with a different way of handling multiple > root buses, based on a host bridge class method to provide a > identifier for the root bus. This hook is designed in such a way as > to allow a single bridge object to support mutiple root buses with > future changes, which will allow future implementations of x86 north > bridges with multiple domains to be supported correctly, and in way > that matches the existing practice for all external interfaces. Applied, thanks. If all's well, this shall go to Anthony in a couple of days. > v2: > * Rework concept of "primary" bus in response to Michael Tsirkin's > comments.