Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Wed, 6 Nov 2019 13:06:37 -0500
From:   "Michael S. Tsirkin" <mst@redhat.com>
To:     Alexey Kardashevskiy <aik@ozlabs.ru>
Cc:     Ram Pai <linuxram@us.ibm.com>, linuxppc-dev@lists.ozlabs.org,
        benh@kernel.crashing.org, david@gibson.dropbear.id.au,
        mpe@ellerman.id.au, paulus@ozlabs.org, mdroth@linux.vnet.ibm.com,
        hch@lst.de, andmike@us.ibm.com, sukadev@linux.vnet.ibm.com,
        ram.n.pai@gmail.com, cai@lca.pw, tglx@linutronix.de,
        bauerman@linux.ibm.com, linux-kernel@vger.kernel.org
Subject: Re: [RFC v1 0/2] Enable IOMMU support for pseries Secure VMs
Message-ID: <20191106130558-mutt-send-email-mst@kernel.org>
References: <1572902923-8096-1-git-send-email-linuxram@us.ibm.com>
 <265679db-9cb3-1660-0cf6-97f740b1b48b@ozlabs.ru>
MIME-Version: 1.0
In-Reply-To: <265679db-9cb3-1660-0cf6-97f740b1b48b@ozlabs.ru>
Content-Type: text/plain; charset=WINDOWS-1252
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Wed, Nov 06, 2019 at 12:59:50PM +1100, Alexey Kardashevskiy wrote:
>=20
>=20
> On 05/11/2019 08:28, Ram Pai wrote:
> > This patch series enables IOMMU support for pseries Secure VMs.
> >=20
> >=20
> > Tested using QEMU command line option:
> >=20
> >  "-device virtio-scsi-pci,id=3Dscsi0,bus=3Dpci.0,addr=3D0x4,
> >  =09iommu_platform=3Don,disable-modern=3Doff,disable-legacy=3Don"
> >  and=20
> >=20
> >  "-device virtio-blk-pci,scsi=3Doff,bus=3Dpci.0,
> >  =09addr=3D0x5,drive=3Ddrive-virtio-disk0,id=3Dvirtio-disk0,
> >  =09iommu_platform=3Don,disable-modern=3Doff,disable-legacy=3Don"
>=20
>=20
> Worth mentioning that SLOF won't boot with such devices as SLOF does not =
know about iommu_platform=3Don.

Shouldn't be hard to support: set up the iommu to allow everything
and ack the feature. Right?

> >=20
> > Ram Pai (2):
> >   powerpc/pseries/iommu: Share the per-cpu TCE page with the hypervisor=
.
> >   powerpc/pseries/iommu: Use dma_iommu_ops for Secure VMs aswell.
> >=20
> >  arch/powerpc/platforms/pseries/iommu.c | 30 ++++++++++++++++++--------=
----
> >  1 file changed, 18 insertions(+), 12 deletions(-)
> >=20
>=20
> --=20
> Alexey