]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
Revert "iommu/io-pgtable: Avoid redundant TLB syncs"
authorRobin Murphy <robin.murphy@arm.com>
Thu, 6 Jul 2017 16:55:31 +0000 (17:55 +0100)
committerWill Deacon <will.deacon@arm.com>
Thu, 20 Jul 2017 09:30:27 +0000 (10:30 +0100)
commit2984f7f3bb5a82df763c2d81444573ed86f36eb8
tree048518e4ab2f43d356a8c5c84d9c876ed9dee923
parent98a8f63e56a0bdcf1d0af8d840d011ab90386684
Revert "iommu/io-pgtable: Avoid redundant TLB syncs"

The tlb_sync_pending flag was necessary for correctness in the Mediatek
M4U driver, but since it offered a small theoretical optimisation for
all io-pgtable users it was implemented as a high-level thing. However,
now that some users may not be using a synchronising lock, there are
several ways this flag can go wrong for them, and at worst it could
result in incorrect behaviour.

Since we've addressed the correctness issue within the Mediatek driver
itself, and fixing the optimisation aspect to be concurrency-safe would
be quite a headache (and impose extra overhead on every operation for
the sake of slightly helping one case which will virtually never happen
in typical usage), let's just retire it.

This reverts commit 88492a4700360a086e55d8874ad786105a5e8b0f.

Signed-off-by: Robin Murphy <robin.murphy@arm.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
drivers/iommu/io-pgtable.h