From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41346) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aLELs-0005Y4-4O for qemu-devel@nongnu.org; Mon, 18 Jan 2016 13:16:29 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aLELo-0005Tf-RD for qemu-devel@nongnu.org; Mon, 18 Jan 2016 13:16:28 -0500 Received: from mx1.redhat.com ([209.132.183.28]:39653) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aLELo-0005TZ-Lt for qemu-devel@nongnu.org; Mon, 18 Jan 2016 13:16:24 -0500 References: <1453130846-14064-1-git-send-email-marcel@redhat.com> From: Laszlo Ersek Message-ID: <569D2BF5.9050604@redhat.com> Date: Mon, 18 Jan 2016 19:16:21 +0100 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] hw/pci: ensure that only PCI/PCIe bridges can be attached to pxb/pxb-pcie devices List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell , Marcel Apfelbaum Cc: QEMU Developers , "Michael S. Tsirkin" On 01/18/16 19:08, Peter Maydell wrote: > On 18 January 2016 at 15:27, Marcel Apfelbaum wrote: >> PCI devices can't be plugged directly into PCI extra root bridges >> because their resources can't be computed by firmware before the ACPI >> tables are loaded. >> >> Signed-off-by: Marcel Apfelbaum >> --- >> >> Hi, >> >> This patch follows the discussion: >> https://lists.gnu.org/archive/html/qemu-devel/2016-01/msg01484.html > > Is it definitely the case that no current working command lines plug > PCI devices directly into these things (including on platforms that > don't have anything to do with ACPI at all) ? No clue about "pxb-pcie", but re: "pxb", the documentation and examples by Marcel (see: "docs/pci_expander_bridge.txt") will certainly continue working, with this patch place. And, that text file is authoritative for pxb, since Marcel (et al) wrote the code directly for the purposes described in the txt. (But I'll let Marcel answer too! :)) Thanks Laszlo