From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:54137)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1ZauLF-0000jj-Ef
	for qemu-devel@nongnu.org; Sat, 12 Sep 2015 19:36:22 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1ZauLD-0003Ch-UL
	for qemu-devel@nongnu.org; Sat, 12 Sep 2015 19:36:21 -0400
Message-ID: <1442099564.25110.58.camel@kernel.crashing.org>
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: Sun, 13 Sep 2015 09:12:44 +1000
In-Reply-To: <1442083038.8395.292.camel@oracle.com>
References: <1441133302-18950-1-git-send-email-knut.omang@oracle.com>
	<55E6EBE1.5070709@gmail.com> <1441199414.25731.115.camel@oracle.com>
	<1441210345.20355.295.camel@redhat.com>
	<1441233210.2668.90.camel@kernel.crashing.org>
	<1441258016.3260.126.camel@oracle.com>
	<1442083038.8395.292.camel@oracle.com>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH v2 0/2] intel_iommu: Add support for
 translation for devices behind bridges
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: Knut Omang <knut.omang@oracle.com>, Alex Williamson <alex.williamson@redhat.com>
Cc: Eduardo Habkost <ehabkost@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, Jan Kiszka <jan.kiszka@siemens.com>, Alexander Graf <agraf@suse.de>, qemu-devel@nongnu.org, Andreas =?ISO-8859-1?Q?F=E4rber?= <andreas.faerber@web.de>, qemu-ppc@nongnu.org, Le Tan <tamlokveer@gmail.com>, marcel@redhat.com, Paolo Bonzini <pbonzini@redhat.com>, David Gibson <david@gibson.dropbear.id.au>, Richard Henderson <rth@twiddle.net>

On Sat, 2015-09-12 at 20:37 +0200, Knut Omang wrote:
> As the thread went silent after our conclusions, I have made a second
> implementation for the Intel IOMMU according to this alternate scheme,
> It keeps the current API and handles the bus number resolution lazily
> within the IOMMU implementation, I will post the (single) patch as v3
> of this. 
> 
> Hopefully this is acceptable and can be leveraged to do a similar
> rework, or be abstracted as generic functionality (?) for the other
> architectures,..

Ah sorry, I meant to look at your email in more details and respond but
it fell through the cracks.

I'm happy to have a look at your work and see how it applies to me, you
can see my powernv code which also supports translation for devices
behind bridges here (but doesn't do as much caching as q35 does):

https://github.com/ozbenh/qemu/commit/4e0ed1002f98fd97aa7ca3a48c74933d0343dd42

Which depends on:

https://github.com/ozbenh/qemu/commit/facedeba8811985ca20ac3dbad5d07e1a10ea9b2

(Which I think Michael merged recently, I haven't checked).

Cheers,
Ben.