qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Marcel Apfelbaum <marcel@redhat.com>
Cc: kraxel@redhat.com, quintela@redhat.com, seabios@seabios.org,
	qemu-devel@nongnu.org, agraf@suse.de, alex.williamson@redhat.com,
	kevin@koconnor.net, qemu-ppc@nongnu.org, hare@suse.de,
	imammedo@redhat.com, amit.shah@redhat.com, pbonzini@redhat.com,
	leon.alrae@imgtec.com, aurelien@aurel32.net, rth@twiddle.net
Subject: Re: [Qemu-devel] [PATCH v5 for-2.3 28/28] docs: Add PXB documentation
Date: Tue, 10 Mar 2015 18:42:37 +0100	[thread overview]
Message-ID: <20150310184149-mutt-send-email-mst@redhat.com> (raw)
In-Reply-To: <54FF19FA.8040808@redhat.com>

On Tue, Mar 10, 2015 at 06:21:14PM +0200, Marcel Apfelbaum wrote:
> On 03/10/2015 05:47 PM, Michael S. Tsirkin wrote:
> >On Tue, Mar 10, 2015 at 05:32:14PM +0200, Marcel Apfelbaum wrote:
> >>Signed-off-by: Marcel Apfelbaum <marcel@redhat.com>
> >>---
> >>  docs/pci_expander_bridge.txt | 52 ++++++++++++++++++++++++++++++++++++++++++++
> >>  1 file changed, 52 insertions(+)
> >>  create mode 100644 docs/pci_expander_bridge.txt
> >>
> >>diff --git a/docs/pci_expander_bridge.txt b/docs/pci_expander_bridge.txt
> >>new file mode 100644
> >>index 0000000..58bf7a8
> >>--- /dev/null
> >>+++ b/docs/pci_expander_bridge.txt
> >>@@ -0,0 +1,52 @@
> >>+PCI EXPANDER BRIDGE (PXB)
> >>+=========================
> >>+
> >>+Description
> >>+===========
> >>+PXB is a "light-weight" host bridge in the same PCI domain
> >>+as the main host bridge whose purpose is to enable
> >>+the main host bridge to support multiple PCI root buses.
> >>+It is implemented only for i440fx.
> >
> >BTW what makes it i440fx specific?
> >Also, what happens if you try to use it
> >with a different machine type?
> Is is i440fx specific, please look at patch 22/28.
> Also we have a specific check for i440fx, so CRS
> will not be emitted for other machine types.
> 
> Thanks,
> Marcel

In fact it won't work at all. Need to think about it,
maybe we can make it work more generally.
For CRS, should be possible to emit for q35 too?

> >
> >>+
> >>+As opposed to PCI-2-PCI bridge's secondary bus, PXB's bus
> >>+is a primary bus and can be associated with a NUMA node
> >>+(different from the main host bridge) allowing the guest OS
> >>+to recognize the proximity of a pass-through device to
> >>+other resources as RAM and CPUs.
> >>+
> >>+Usage
> >>+=====
> >>+A detailed command line would be:
> >>+
> >>+[qemu-bin + storage options]
> >>+-bios [seabios-dir]/out/bios.bin -L [seabios-dir]/out/
> >>+-m 2G
> >>+-object memory-backend-ram,size=1024M,policy=bind,host-nodes=0,id=ram-node0 -numa node,nodeid=0,cpus=0,memdev=ram-node0
> >>+-object memory-backend-ram,size=1024M,policy=interleave,host-nodes=0,id=ram-node1 -numa node,nodeid=1,cpus=1,memdev=ram-node1
> >>+-device pxb-device,id=bridge1,bus=pci.0,numa_node=1,bus_nr=4 -netdev user,id=nd-device e1000,bus=bridge1,addr=0x4,netdev=nd
> >>+-device pxb-device,id=bridge2,bus=pci.0,numa_node=0,bus_nr=8 -device e1000,bus=bridge2,addr=0x3
> >>+-device pxb-device,id=bridge3,bus=pci.0,bus_nr=40 -drive if=none,id=drive0,file=[img] -device virtio-blk-pci,drive=drive0,scsi=off,bus=bridge3,addr=1
> >>+
> >>+Here you have:
> >>+ - 2 NUMA nodes for the guest, 0 and 1. (both mapped to the same NUMA node in host, but you can and should put it in different host NUMA nodes)
> >>+ - a pxb host bridge attached to NUMA 1 with an e1000 behind it
> >>+ - a pxb host bridge attached to NUMA 0 with an e1000 behind it
> >>+ - a pxb host bridge not attached to any NUMA with a hard drive behind it.
> >>+
> >>+Implementation
> >>+==============
> >>+The PXB is composed by:
> >>+- HostBridge (TYPE_PXB_HOST)
> >>+  The host bridge allows to register and query the PXB's rPCI root bus in QEMU.
> >>+- PXBDev(TYPE_PXB_DEVICE)
> >>+  It is a regular PCI Device that resides on the piix host-bridge bus and its bus uses the same PCI domain.
> >>+  However, the bus behind is exposed through ACPI as a primary PCI bus and starts a new PCI hierarchy.
> >>+  The interrupts from devices behind the PXB are routed through this device the same as if it were a
> >>+  PCI-2-PCI bridge. The _PRT follows the i440fx model.
> >>+- PCIBridgeDev(TYPE_PCI_BRIDGE_DEV)
> >>+  Created automatically as part of init sequence.
> >>+  When adding a device to PXB it is attached to the bridge for two reasons:
> >>+  - Using the bridge will enable hotplug support
> >>+  - All the devices behind the bridge will use bridge's IO/MEM windows compacting
> >>+    the PCI address space.
> >>+
> >>--
> >>2.1.0

  reply	other threads:[~2015-03-10 17:43 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-10 15:31 [Qemu-devel] [PATCH v5 for-2.3 00/28] hw/pc: implement multiple primary busses for pc machines Marcel Apfelbaum
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 01/28] acpi: fix aml_equal term implementation Marcel Apfelbaum
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 02/28] acpi: add aml_or() term Marcel Apfelbaum
2015-03-11  1:17   ` Shannon Zhao
2015-03-11 13:07     ` Marcel Apfelbaum
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 03/28] acpi: add aml_add() term Marcel Apfelbaum
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 04/28] acpi: add aml_lless() term Marcel Apfelbaum
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 05/28] acpi: add aml_index() term Marcel Apfelbaum
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 06/28] acpi: add aml_shiftleft() term Marcel Apfelbaum
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 07/28] acpi: add aml_shiftright() term Marcel Apfelbaum
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 08/28] acpi: add aml_increment() term Marcel Apfelbaum
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 09/28] acpi: add aml_while() term Marcel Apfelbaum
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 10/28] hw/acpi: add support for multiple root busses Marcel Apfelbaum
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 11/28] hw/apci: add _PRT method for extra PCI " Marcel Apfelbaum
2015-03-10 16:41   ` Michael S. Tsirkin
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 12/28] hw/acpi: add _CRS method for extra " Marcel Apfelbaum
2015-03-10 15:38   ` Michael S. Tsirkin
2015-03-10 16:17     ` Marcel Apfelbaum
2015-03-10 15:31 ` [Qemu-devel] [PATCH v5 for-2.3 13/28] hw/acpi: remove from root bus 0 the crs resources used by other busses Marcel Apfelbaum
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 14/28] hw/pci: move pci bus related code to separate files Marcel Apfelbaum
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 15/28] hw/pci: made pci_bus_is_root a PCIBusClass method Marcel Apfelbaum
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 16/28] hw/pci: made pci_bus_num " Marcel Apfelbaum
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 17/28] hw/pci: introduce TYPE_PCI_MAIN_HOST_BRIDGE interface Marcel Apfelbaum
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 18/28] hw/pci: removed 'rootbus nr is 0' assumption from qmp_pci_query Marcel Apfelbaum
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 19/28] hw/pci: implement iteration over multiple host bridges Marcel Apfelbaum
2015-03-10 16:39   ` Michael S. Tsirkin
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 20/28] hw/pci: introduce PCI Expander Bridge (PXB) Marcel Apfelbaum
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 21/28] hw/pci: inform bios if the system has more than one pci bridge Marcel Apfelbaum
2015-03-10 16:36   ` Michael S. Tsirkin
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 22/28] hw/pci: piix - suport multiple host bridges Marcel Apfelbaum
2015-03-10 16:22   ` Michael S. Tsirkin
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 23/28] hw/pxb: add map_irq func Marcel Apfelbaum
2015-03-10 16:43   ` Michael S. Tsirkin
2015-03-16 12:11     ` Marcel Apfelbaum
2015-03-16 15:27       ` Michael S. Tsirkin
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 24/28] hw/pci_bus: add support for NUMA nodes Marcel Apfelbaum
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 25/28] hw/pxb: add numa_node parameter Marcel Apfelbaum
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 26/28] acpi: restrict the aml emission to PXB host bridges Marcel Apfelbaum
2015-03-10 15:41   ` Michael S. Tsirkin
2015-03-10 16:18     ` Marcel Apfelbaum
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 27/28] apci: fix PXB behaviour if used with unsupported BIOS Marcel Apfelbaum
2015-03-10 15:44   ` Michael S. Tsirkin
2015-03-10 16:19     ` Marcel Apfelbaum
2015-03-10 16:21       ` Michael S. Tsirkin
2015-03-10 15:32 ` [Qemu-devel] [PATCH v5 for-2.3 28/28] docs: Add PXB documentation Marcel Apfelbaum
2015-03-10 15:47   ` Michael S. Tsirkin
2015-03-10 16:21     ` Marcel Apfelbaum
2015-03-10 17:42       ` Michael S. Tsirkin [this message]
2015-03-16 12:16         ` Marcel Apfelbaum
2015-03-16 15:28           ` Michael S. Tsirkin
2015-03-16 15:47             ` Marcel Apfelbaum
2015-03-11 13:32 ` [Qemu-devel] [PATCH v5 for-2.3 00/28] hw/pc: implement multiple primary busses for pc machines Gerd Hoffmann
2015-03-11 13:44   ` Marcel Apfelbaum
2015-03-11 13:51     ` Gerd Hoffmann
2015-03-11 14:01       ` Marcel Apfelbaum
2015-03-11 14:02         ` Michael S. Tsirkin
2015-03-11 14:12 ` Gerd Hoffmann
2015-03-11 14:14   ` Marcel Apfelbaum

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150310184149-mutt-send-email-mst@redhat.com \
    --to=mst@redhat.com \
    --cc=agraf@suse.de \
    --cc=alex.williamson@redhat.com \
    --cc=amit.shah@redhat.com \
    --cc=aurelien@aurel32.net \
    --cc=hare@suse.de \
    --cc=imammedo@redhat.com \
    --cc=kevin@koconnor.net \
    --cc=kraxel@redhat.com \
    --cc=leon.alrae@imgtec.com \
    --cc=marcel@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    --cc=quintela@redhat.com \
    --cc=rth@twiddle.net \
    --cc=seabios@seabios.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).