Hi Nicolin, > -----Original Message----- > From: Nicolin Chen <[email protected]> > Sent: Wednesday, February 5, 2025 12:09 AM > To: Shameerali Kolothum Thodi > <[email protected]>; Eric Auger > <[email protected]> > Cc: [email protected]; Peter Maydell <[email protected]>; Jason > Gunthorpe <[email protected]>; Daniel P. Berrangé <[email protected]>; > [email protected]; [email protected]; Linuxarm > <[email protected]>; Wangzhou (B) <[email protected]>; > jiangkunkun <[email protected]>; Jonathan Cameron > <[email protected]>; [email protected] > Subject: Re: [RFC PATCH 0/5] hw/arm/virt: Add support for user-creatable > nested SMMUv3 > > On Tue, Feb 04, 2025 at 06:49:15PM +0100, Eric Auger wrote: > > > In summary, we will have the following series: > > > 1) HWPT uAPI patches in backends/iommufd.c (Zhenzhong or Shameer) > > > https://lore.kernel.org/qemu- > devel/[email protected] > prd11.prod.outlook.com/ > > > 2) vIOMMU uAPI patches in backends/iommufd.c (I will rebase/send) > > > for 1 and 2, are you taking about the "Add VIOMMU infrastructure > support > > " series in Shameer's branch: private-smmuv3-nested-dev-rfc-v1. > > Sorry I may instead refer to NVidia or Intel's branch but I am not sure > > about the last ones. > > That "vIOMMU infrastructure" is for 2, yes. > > For 1, it's inside the Intel's series: > "cover-letter: intel_iommu: Enable stage-1 translation for passthrough > device" > > So, we need to extract them out and make it separately.. > > > > 3) vSMMUv3 patches for HW-acc/nesting (Hoping Don/you could take > over) > > We can start sending it upstream assuming we have a decent test > environment. > > > > However in > > > https://lore.kernel.org/all/[email protected] > om/ > > > > Shameer suggested he may include it in his SMMU multi instance series. > > What do you both prefer? > > Sure, I think it's good to include those patches,
One of the feedback I received on my series was to rename "arm-smmuv3-nested" to "arm-smmuv3-accel" and possibly rename function names to include "accel' as well and move those functions to a separate "smmuv3-accel.c" file. I suppose that applies to the " Add HW accelerated nesting support for arm SMMUv3" series as well. Is that fine with you? Thanks, Shameer
