From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:57006)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1ZRRhS-0007UT-1s
	for qemu-devel@nongnu.org; Mon, 17 Aug 2015 17:12:10 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1ZRRhO-0003lr-IG
	for qemu-devel@nongnu.org; Mon, 17 Aug 2015 17:12:09 -0400
Received: from gate.crashing.org ([63.228.1.57]:42593)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1ZRRhO-0003l8-8S
	for qemu-devel@nongnu.org; Mon, 17 Aug 2015 17:12:06 -0400
Message-ID: <1439845921.2416.27.camel@kernel.crashing.org>
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: Tue, 18 Aug 2015 07:12:01 +1000
In-Reply-To: <20150817215008-mutt-send-email-mst@redhat.com>
References: <1436051955.3948.29.camel@kernel.crashing.org>
	<1439804354.2416.10.camel@kernel.crashing.org>
	<20150817215008-mutt-send-email-mst@redhat.com>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH] pci: Fix pci_device_iommu_address_space()
 bus propagation
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: qemu-devel@nongnu.org

On Mon, 2015-08-17 at 21:50 +0300, Michael S. Tsirkin wrote:
> On Mon, Aug 17, 2015 at 07:39:14PM +1000, Benjamin Herrenschmidt 
> wrote:
> > On Sun, 2015-07-05 at 09:19 +1000, Benjamin Herrenschmidt wrote:
> > > he current code walks up the bus tree for an iommu, however it 
> > > passes
> > > to the iommu_fn() callback the bus/devfn of the immediate child 
> > > of
> > > the level where the callback was found, rather than the original
> > > bus/devfn where the search started from.
> > 
> > Hi Michael ! Any comment on this ? I'd like to post my series for 
> > "bare
> > metal" power8 support and the iommu implementation relies on this 
> > to
> > work.
> 
> I've picked this up for my next pull request.
> Thanks, and sorry about the delay.

No worries, I understand "busy" :-) Thanks for picking it up.

Cheers,
Ben.