[Public]
-----Original Message----- From: Thomas Monjalon <[email protected]> Sent: Thursday, May 27, 2021 7:40 PM To: Somalapuram, Amaranath <[email protected]>; David Marchand <[email protected]> Cc: dev <[email protected]>; Akhil Goyal <[email protected]>; Kumar, Ravi1 <[email protected]>; Song, Keesang <[email protected]> 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 <[email protected]> wrote: > > > > From: Amaranath Somalapuram <[email protected]> > > > > 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.
