From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45818) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZIOCG-000727-DY for qemu-devel@nongnu.org; Thu, 23 Jul 2015 17:38:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZIOCD-0007gg-56 for qemu-devel@nongnu.org; Thu, 23 Jul 2015 17:38:32 -0400 Received: from e37.co.us.ibm.com ([32.97.110.158]:57922) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZIOCC-0007fW-Sk for qemu-devel@nongnu.org; Thu, 23 Jul 2015 17:38:29 -0400 Received: from /spool/local by e37.co.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Thu, 23 Jul 2015 15:38:28 -0600 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Michael Roth In-Reply-To: References: <1437566099-10004-1-git-send-email-lvivier@redhat.com> <1437675858-14070-1-git-send-email-lvivier@redhat.com> <1437684406.7562.62.camel@kernel.crashing.org> <20150724000923-mutt-send-email-mst@redhat.com> Message-ID: <20150723213819.15254.14516@loki> Date: Thu, 23 Jul 2015 16:38:19 -0500 Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH v2] pci: allow 0 address for PCI IO/MEM regions List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell , "Michael S. Tsirkin" Cc: Laurent Vivier , "qemu-ppc@nongnu.org" , QEMU Developers , David Gibson Quoting Peter Maydell (2015-07-23 16:24:20) > On 23 July 2015 at 22:19, Peter Maydell wrote: > > On 23 July 2015 at 22:10, Michael S. Tsirkin wrote: > >> On Thu, Jul 23, 2015 at 10:00:30PM +0100, Peter Maydell wrote: > >>> (Also, none of our PCI device models actually try to do > >>> the "BAR at zero means I won't respond" behaviour, which > >>> presumably they might do in real life.) > > > >> Maybe some devices do this, but I'm guessing not all of them, > >> since there's no hint in the pci spec that they should. > > > > I think this depends on which version of the spec you > > read. > = > Bikeshedding about ancient specs aside, I think it's the > bugs in the PC model's memory region priorities that > are the real reason the special case of zero is sticking > around. If we fixed those we should be able to drop it. What's the intended fix? That legacy/platform regions should hide any regions a guest attempts to map over it? > = > -- PMM >=20