Re: [PATCH] iommu/arm-smmu-v3: Ensure we sync STE when only changing config field

2017-10-06 Thread Auger Eric
Hi Will, On 05/10/2017 18:54, Robin Murphy wrote: > On 05/10/17 17:49, Will Deacon wrote: >> The SMMUv3 architecture permits caching of data structures deemed to be >> "reachable" by the SMU, which includes STEs marked as invalid. When >> transitioning an STE to a bypass/fault configuration at ini

Re: [PATCH] iommu/arm-smmu-v3: Ensure we sync STE when only changing config field

2017-10-05 Thread Robin Murphy
On 05/10/17 17:49, Will Deacon wrote: > The SMMUv3 architecture permits caching of data structures deemed to be > "reachable" by the SMU, which includes STEs marked as invalid. When > transitioning an STE to a bypass/fault configuration at init or detach > time, we mistakenly elide the CMDQ_OP_CFGI

[PATCH] iommu/arm-smmu-v3: Ensure we sync STE when only changing config field

2017-10-05 Thread Will Deacon
The SMMUv3 architecture permits caching of data structures deemed to be "reachable" by the SMU, which includes STEs marked as invalid. When transitioning an STE to a bypass/fault configuration at init or detach time, we mistakenly elide the CMDQ_OP_CFGI_STE operation in some cases, therefore potent