Re: [EXTERNAL] Re: Question regarding VIOT proposal

2020-12-03 Thread Al Stone
On 03 Dec 2020 22:21, Yinghan Yang wrote: > Hi Jean, > > I'm sorry for the delayed response. I think the new "PCI range node" > description makes sense. Could you please make this change in the proposal? > > Other than that, the proposal looks good to go. > > Thanks, > Yinghan Jean, were you g

RE: [EXTERNAL] Re: Question regarding VIOT proposal

2020-12-03 Thread Yinghan Yang via iommu
Hi Jean, I'm sorry for the delayed response. I think the new "PCI range node" description makes sense. Could you please make this change in the proposal? Other than that, the proposal looks good to go. Thanks, Yinghan -Original Message- From: Jean-Philippe Brucker Sent: Friday, Novem

RE: [PATCH v13 07/15] iommu/smmuv3: Allow stage 1 invalidation with unmanaged ASIDs

2020-12-03 Thread Shameerali Kolothum Thodi
Hi Eric, > -Original Message- > From: kvmarm-boun...@lists.cs.columbia.edu > [mailto:kvmarm-boun...@lists.cs.columbia.edu] On Behalf Of Auger Eric > Sent: 01 December 2020 13:59 > To: wangxingang > Cc: Xieyingtai ; jean-phili...@linaro.org; > k...@vger.kernel.org; m...@kernel.org; j...@8b

[PATCH 2/3] iommu: Stop exporting alloc_iova_mem()

2020-12-03 Thread John Garry
It is not used outside iova.c Signed-off-by: John Garry --- drivers/iommu/iova.c | 3 +-- include/linux/iova.h | 6 -- 2 files changed, 1 insertion(+), 8 deletions(-) diff --git a/drivers/iommu/iova.c b/drivers/iommu/iova.c index 4803bd83447d..9fed0b040747 100644 --- a/drivers/iommu/iova.c

[PATCH 0/3] IOMMU: Some IOVA code tidy-up

2020-12-03 Thread John Garry
This series contains some minor tidy-ups by deleting an unreferenced function and unexporting some functions, highlighted by: https://lore.kernel.org/linux-iommu/6e09d847-fb7f-1ec1-02bf-f0c8b3158...@huawei.com/T/#med5a019f9d3835c162c16a48f34d05cc0111b0ca John Garry (3): iommu: Delete split_and_r

[PATCH 1/3] iommu: Delete split_and_remove_iova()

2020-12-03 Thread John Garry
Function split_and_remove_iova() has not been referenced since commit e70b081c6f37 ("iommu/vt-d: Remove IOVA handling code from the non-dma_ops path"), so delete it. Signed-off-by: John Garry --- drivers/iommu/iova.c | 41 - include/linux/iova.h | 10 -

[PATCH 3/3] iommu: Stop exporting free_iova_mem()

2020-12-03 Thread John Garry
It has no user outside iova.c Signed-off-by: John Garry --- drivers/iommu/iova.c | 3 +-- include/linux/iova.h | 5 - 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/iommu/iova.c b/drivers/iommu/iova.c index 9fed0b040747..f68fe4c0df28 100644 --- a/drivers/iommu/iova.c +

Re: [PATCH v8 0/9] iommu: I/O page faults for SMMUv3

2020-12-03 Thread Vivek Gautam
Hi Jean, On Thu, Nov 12, 2020 at 6:33 PM Jean-Philippe Brucker wrote: > > Add support for stall and PRI to the SMMUv3 driver, along with a common > I/O Page Fault handler. > > These patches were last sent as part of v7 of the larger SVA series [1]. > Main changes since v7: > * Dropped CONFIG_IOM

Re: [RESEND PATCH v3 0/4] iommu/iova: Solve longterm IOVA issue

2020-12-03 Thread John Garry
On 03/12/2020 06:04, Dmitry Safonov wrote: On Tue, 1 Dec 2020 at 21:50, Will Deacon wrote: On Tue, 17 Nov 2020 18:25:30 +0800, John Garry wrote: This series contains a patch to solve the longterm IOVA issue which leizhen originally tried to address at [0]. A sieved kernel log is at the follow

[PATCH AUTOSEL 5.9 22/39] arm-smmu-qcom: Ensure the qcom_scm driver has finished probing

2020-12-03 Thread Sasha Levin
From: John Stultz [ Upstream commit 72b55c96f3a5ae6e486c20b5dacf5114060ed042 ] Robin Murphy pointed out that if the arm-smmu driver probes before the qcom_scm driver, we may call qcom_scm_qsmmu500_wait_safe_toggle() before the __scm is initialized. Now, getting this to happen is a bit contrived

Re: [PATCH v13 05/15] iommu/smmuv3: Get prepared for nested stage support

2020-12-03 Thread Kunkun Jiang
On 2020/12/3 21:01, Auger Eric wrote: Hi Kunkun, On 12/3/20 1:32 PM, Kunkun Jiang wrote: Hi Eric, On 2020/11/18 19:21, Eric Auger wrote: When nested stage translation is setup, both s1_cfg and s2_cfg are set. We introduce a new smmu domain abort field that will be set upon guest stage1 conf

Re: [PATCH v13 05/15] iommu/smmuv3: Get prepared for nested stage support

2020-12-03 Thread Auger Eric
Hi Kunkun, On 12/3/20 1:32 PM, Kunkun Jiang wrote: > Hi Eric, > > On 2020/11/18 19:21, Eric Auger wrote: >> When nested stage translation is setup, both s1_cfg and >> s2_cfg are set. >> >> We introduce a new smmu domain abort field that will be set >> upon guest stage1 configuration passing. >> >

Re: [PATCH v13 05/15] iommu/smmuv3: Get prepared for nested stage support

2020-12-03 Thread Kunkun Jiang
Hi Eric, On 2020/11/18 19:21, Eric Auger wrote: When nested stage translation is setup, both s1_cfg and s2_cfg are set. We introduce a new smmu domain abort field that will be set upon guest stage1 configuration passing. arm_smmu_write_strtab_ent() is modified to write both stage fields in the

Re: [PATCH] iommu/io-pgtable: Remove tlb_flush_leaf

2020-12-03 Thread Will Deacon
On Wed, Nov 25, 2020 at 05:29:39PM +, Robin Murphy wrote: > The only user of tlb_flush_leaf is a particularly hairy corner of the > Arm short-descriptor code, which wants a synchronous invalidation to > minimise the races inherent in trying to split a large page mapping. > This is already far e