Re: [PATCH] iommu/arm-smmu-v3: sync the OVACKFLG to PRIQ consumer register

2018-07-27 Thread Zhangshaokun
On 2018/7/27 17:48, Will Deacon wrote: > On Fri, Jul 27, 2018 at 05:41:46PM +0800, Zhangshaokun wrote: >> I saw it in your trees, is it necessary to Cc stable version? > > I don't think so, given that we don't actually support PRI upstream. > Got it, thanks your reply. Shaokun > Will > >> O

Re: [PATCH] iommu/arm-smmu-v3: sync the OVACKFLG to PRIQ consumer register

2018-07-27 Thread Will Deacon
On Fri, Jul 27, 2018 at 05:41:46PM +0800, Zhangshaokun wrote: > I saw it in your trees, is it necessary to Cc stable version? I don't think so, given that we don't actually support PRI upstream. Will > On 2018/7/23 20:56, Shaokun Zhang wrote: > > From: Miao Zhong > > > > When PRI queue occurs

Re: [PATCH] iommu/arm-smmu-v3: sync the OVACKFLG to PRIQ consumer register

2018-07-27 Thread Zhangshaokun
Hi Will, I saw it in your trees, is it necessary to Cc stable version? Cheers, Shaokun On 2018/7/23 20:56, Shaokun Zhang wrote: > From: Miao Zhong > > When PRI queue occurs overflow, driver should update the OVACKFLG to > the PRIQ consumer register, otherwise subsequent PRI requests will not >

Re: [PATCH] iommu/arm-smmu-v3: sync the OVACKFLG to PRIQ consumer register

2018-07-23 Thread Zhongmiao
Hi, On 24/07/18 12:45, Zhongmiao wrote: > Yeah,I haven't tested smmu eventq overflow , so i'm not sure if there > is any problem with smmu eventq. However, the code shows that there is also > a problem after the smmu eventq overflows. ^_^……. >There really shouldn't be any problem with

Re: [PATCH] iommu/arm-smmu-v3: sync the OVACKFLG to PRIQ consumer register

2018-07-23 Thread Zhongmiao
Hi , >"Note: In terms of delivering events, a queue in an unacknowledged overflow >state does not behave any differently to normal queue state. If software were >to consume events and free space but leave overflow unacknowledged, new events >could be recorded." Yeah, I agree with you . But I

Re: [PATCH] iommu/arm-smmu-v3: sync the OVACKFLG to PRIQ consumer register

2018-07-23 Thread Jean-Philippe Brucker
Hi, On 23/07/18 15:43, Zhongmiao wrote: > Yeah,I haven't tested smmu eventq overflow , so i'm not sure if there > is any problem with smmu eventq. However, the code shows that there is also > a problem after the smmu eventq overflows. ^_^……. There really shouldn't be any problem with th

Re: [PATCH] iommu/arm-smmu-v3: sync the OVACKFLG to PRIQ consumer register

2018-07-23 Thread Zhongmiao
Hi Jean, On 23/07/18 22:24, Miao Zhong wrote: > From: Miao Zhong > > When PRI queue occurs overflow, driver should update the OVACKFLG to > the PRIQ consumer register, otherwise subsequent PRI requests will not > be processed. >Since the upstream driver doesn't enable PRI in endpoints, I'm n

Re: [PATCH] iommu/arm-smmu-v3: sync the OVACKFLG to PRIQ consumer register

2018-07-23 Thread Jean-Philippe Brucker
On 23/07/18 13:56, Shaokun Zhang wrote: > From: Miao Zhong > > When PRI queue occurs overflow, driver should update the OVACKFLG to > the PRIQ consumer register, otherwise subsequent PRI requests will not > be processed. Since the upstream driver doesn't enable PRI in endpoints, I'm not sure thi

[PATCH] iommu/arm-smmu-v3: sync the OVACKFLG to PRIQ consumer register

2018-07-23 Thread Shaokun Zhang
From: Miao Zhong When PRI queue occurs overflow, driver should update the OVACKFLG to the PRIQ consumer register, otherwise subsequent PRI requests will not be processed. Cc: Will Deacon Cc: Robin Murphy Signed-off-by: Miao Zhong --- drivers/iommu/arm-smmu-v3.c | 1 + 1 file changed, 1 inse