diff options
author | 2025-07-31 20:01:27 -0700 | |
---|---|---|
committer | 2025-08-22 08:41:20 +0200 | |
commit | 685ca577b408ffd9c5a4057a2acc0cd3e6978b36 (patch) | |
tree | 15fa147ee1c94b822566893aa4a8e12280ec8f30 /tools/perf/scripts/python/Perf-Trace-Util/lib | |
parent | iommu/tegra241-cmdqv: Fix missing cpu_to_le64 at lvcmdq_err_map (diff) | |
download | wireguard-linux-685ca577b408ffd9c5a4057a2acc0cd3e6978b36.tar.xz wireguard-linux-685ca577b408ffd9c5a4057a2acc0cd3e6978b36.zip |
iommu/arm-smmu-v3: Fix smmu_domain->nr_ats_masters decrement
The arm_smmu_attach_commit() updates master->ats_enabled before calling
arm_smmu_remove_master_domain() that is supposed to clean up everything
in the old domain, including the old domain's nr_ats_masters. So, it is
supposed to use the old ats_enabled state of the device, not an updated
state.
This isn't a problem if switching between two domains where:
- old ats_enabled = false; new ats_enabled = false
- old ats_enabled = true; new ats_enabled = true
but can fail cases where:
- old ats_enabled = false; new ats_enabled = true
(old domain should keep the counter but incorrectly decreased it)
- old ats_enabled = true; new ats_enabled = false
(old domain needed to decrease the counter but incorrectly missed it)
Update master->ats_enabled after arm_smmu_remove_master_domain() to fix
this.
Fixes: 7497f4211f4f ("iommu/arm-smmu-v3: Make changing domains be hitless for ATS")
Cc: stable@vger.kernel.org
Signed-off-by: Nicolin Chen <nicolinc@nvidia.com>
Acked-by: Will Deacon <will@kernel.org>
Reviewed-by: Jason Gunthorpe <jgg@nvidia.com>
Reviewed-by: Pranjal Shrivastava <praan@google.com>
Link: https://lore.kernel.org/r/20250801030127.2006979-1-nicolinc@nvidia.com
Signed-off-by: Joerg Roedel <joerg.roedel@amd.com>
Diffstat (limited to 'tools/perf/scripts/python/Perf-Trace-Util/lib')
0 files changed, 0 insertions, 0 deletions