[dpdk-dev] [PATCH v3] crypto/ccp: enable IOMMU for CCP

Somalapuram, Amaranath Amaranath.Somalapuram at amd.com
Fri May 28 07:02:36 CEST 2021


[Public]



-----Original Message-----
From: Thomas Monjalon <thomas at monjalon.net> 
Sent: Thursday, May 27, 2021 7:40 PM
To: Somalapuram, Amaranath <Amaranath.Somalapuram at amd.com>; David Marchand <david.marchand at redhat.com>
Cc: dev <dev at dpdk.org>; Akhil Goyal <gakhil at marvell.com>; Kumar, Ravi1 <Ravi1.Kumar at amd.com>; Song, Keesang <Keesang.Song at amd.com>
Subject: Re: [dpdk-dev] [PATCH v3] crypto/ccp: enable IOMMU for CCP

[CAUTION: External Email]

27/05/2021 15:24, David Marchand:
> On Fri, Dec 25, 2020 at 9:06 AM <asomalap at amd.com> wrote:
> >
> > From: Amaranath Somalapuram <Amaranath.Somalapuram at amd.com>
> >
> > CCP use vdev framework, and vdev framework don’t support IOMMU.
> > Adding custom IOMMU support for AMD CCP driver.
>
> I am currently looking at pci bus patches/cleanups.
> I ended up looking at crypto/ccp.
> This driver code contains a lot of features duplicated with the pci bus.
>
> Why is the ccp driver not a PCI driver?

Indeed it looks abusing vdev.
We should drop all the code duplicating the PCI bus driver.
If nothing else is done, it would mean breaking the probing of this driver.

Adding more people in Cc list to have a fix before it is broken, thanks.

Enable IOMMU for vdev was not supported in DPDK.
I can remove all the duplicating code after I test the CCP with IOMMU.


More information about the dev mailing list