RE: [PATCH v10 10/13] iommu/arm-smmu-v3: Check for SVA features

2020-12-14 Thread Krishna Reddy
>> The Tegra Next Generation SOC uses arm-smmu-v3, but it doesn't have support >> for BTM. >> Do you have plan to get your earlier patch to handle invalidate >> notifications into upstream sometime soon? >> Can the dependency on BTM be relaxed with the patch? >> >> PATCH v9 13/13]

Re: [PATCH v10 10/13] iommu/arm-smmu-v3: Check for SVA features

2020-12-14 Thread Jean-Philippe Brucker
On Wed, Dec 09, 2020 at 07:49:09PM +, Krishna Reddy wrote: > Hi Jean, > > > Why is BTM mandated for SVA? I couldn't find this requirement in > > > SMMU spec (Sorry if I missed it or this got discussed earlier). But > > > if performance is the > > only concern here, > > > is it better just to

RE: [PATCH v10 10/13] iommu/arm-smmu-v3: Check for SVA features

2020-12-09 Thread Krishna Reddy
> > The Tegra Next Generation SOC uses arm-smmu-v3, but it doesn't have support > > for BTM. > > Do you have plan to get your earlier patch to handle invalidate > > notifications into upstream sometime soon? >Is that a limitation of the SMMU implementation, the interconnect or the

Re: [PATCH v10 10/13] iommu/arm-smmu-v3: Check for SVA features

2020-12-09 Thread Will Deacon
On Wed, Dec 09, 2020 at 07:49:09PM +, Krishna Reddy wrote: > > > Why is BTM mandated for SVA? I couldn't find this requirement in > > > SMMU spec (Sorry if I missed it or this got discussed earlier). But > > > if performance is the > > only concern here, > > > is it better just to allow it

RE: [PATCH v10 10/13] iommu/arm-smmu-v3: Check for SVA features

2020-12-09 Thread Krishna Reddy
Hi Jean, > > Why is BTM mandated for SVA? I couldn't find this requirement in > > SMMU spec (Sorry if I missed it or this got discussed earlier). But > > if performance is the > only concern here, > > is it better just to allow it with a warning rather than limiting > > SMMUs without > BTM? > >

RE: [PATCH v10 10/13] iommu/arm-smmu-v3: Check for SVA features

2020-09-24 Thread Shameerali Kolothum Thodi
.com; catalin.mari...@arm.com; > Suzuki K Poulose ; robin.mur...@arm.com; > zhangfei@linaro.org; w...@kernel.org > Subject: Re: [PATCH v10 10/13] iommu/arm-smmu-v3: Check for SVA features > > Hi Shameer, > > On Mon, Sep 21, 2020 at 08:59:39AM +, Shameer

Re: [PATCH v10 10/13] iommu/arm-smmu-v3: Check for SVA features

2020-09-24 Thread Jean-Philippe Brucker
Hi Shameer, On Mon, Sep 21, 2020 at 08:59:39AM +, Shameerali Kolothum Thodi wrote: > > +bool arm_smmu_sva_supported(struct arm_smmu_device *smmu) > > +{ > > + unsigned long reg, fld; > > + unsigned long oas; > > + unsigned long asid_bits; > > + u32 feat_mask = ARM_SMMU_FEAT_BTM | > >

RE: [PATCH v10 10/13] iommu/arm-smmu-v3: Check for SVA features

2020-09-21 Thread Shameerali Kolothum Thodi
Hi Jean, > -Original Message- > From: iommu [mailto:iommu-boun...@lists.linux-foundation.org] On Behalf Of > Jean-Philippe Brucker > Sent: 18 September 2020 11:19 > To: iommu@lists.linux-foundation.org; linux-arm-ker...@lists.infradead.org; > linux...@kvack.org > Cc: fenghua...@intel.com;