diff options
| author | Keqian Zhu <zhukeqian1@huawei.com> | 2020-12-07 12:57:58 +0100 |
|---|---|---|
| committer | Will Deacon <will@kernel.org> | 2020-12-08 14:56:30 +0100 |
| commit | f12e0d22903e8fb653168efa67ae3308712ea97e (patch) | |
| tree | 5dc3fea8e7fbd6700aaf808f0a85222dd4c9a0b6 /drivers/iommu/io-pgtable-arm-v7s.c | |
| parent | iommu: Improve the performance for direct_mapping (diff) | |
| download | linux-f12e0d22903e8fb653168efa67ae3308712ea97e.tar.xz linux-f12e0d22903e8fb653168efa67ae3308712ea97e.zip | |
iommu: Defer the early return in arm_(v7s/lpae)_map
Although handling a mapping request with no permissions is a
trivial no-op, defer the early return until after the size/range
checks so that we are consistent with other mapping requests.
Signed-off-by: Keqian Zhu <zhukeqian1@huawei.com>
Link: https://lore.kernel.org/r/20201207115758.9400-1-zhukeqian1@huawei.com
Signed-off-by: Will Deacon <will@kernel.org>
Diffstat (limited to 'drivers/iommu/io-pgtable-arm-v7s.c')
| -rw-r--r-- | drivers/iommu/io-pgtable-arm-v7s.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/iommu/io-pgtable-arm-v7s.c b/drivers/iommu/io-pgtable-arm-v7s.c index a688f22cbe3b..359b96b0fa3e 100644 --- a/drivers/iommu/io-pgtable-arm-v7s.c +++ b/drivers/iommu/io-pgtable-arm-v7s.c @@ -522,14 +522,14 @@ static int arm_v7s_map(struct io_pgtable_ops *ops, unsigned long iova, struct io_pgtable *iop = &data->iop; int ret; - /* If no access, then nothing to do */ - if (!(prot & (IOMMU_READ | IOMMU_WRITE))) - return 0; - if (WARN_ON(iova >= (1ULL << data->iop.cfg.ias) || paddr >= (1ULL << data->iop.cfg.oas))) return -ERANGE; + /* If no access, then nothing to do */ + if (!(prot & (IOMMU_READ | IOMMU_WRITE))) + return 0; + ret = __arm_v7s_map(data, iova, paddr, size, prot, 1, data->pgd, gfp); /* * Synchronise all PTE updates for the new mapping before there's |