[PATCH RFC 0/5] iommu/vt-d: Create a cache_tag_id to reduce argument passing
From: Joel Granados via B4 Relay
Date: Fri Jun 28 2024 - 08:28:16 EST
Does it make sense to replace 4 arguments (domain id, dev, pasid and
type) with just one (cache_tag_id) in the {un}assignment functions in
drivers/iommu/intel/cache.c?
In working with this file I made a few changes so I could understand it
a bit better (no functional modifications). I can keep going if ppl see
that it make sense.
Comments are greatly appreciated.
Signed-off-by: Joel Granados <j.granados@xxxxxxxxxxx>
---
Joel Granados (5):
iommu/vt-d: introduce cache_tag_id to the struct cache_tag
iommu/vt-d: Replace cache_tag_match with a tag comparison function
iommu/vt-d: Pass cache_tag_id to {un}assignment functions
iommu/vt-d: Use ats_enable as an argument
iommu/vt-d: Remove dev_iommu_priv_get call from domain_get_id_for_dev
drivers/iommu/intel/cache.c | 184 ++++++++++++++++++++++----------------------
drivers/iommu/intel/iommu.h | 24 +++---
2 files changed, 104 insertions(+), 104 deletions(-)
---
base-commit: bcace518c52fd7d786a9093711e84847db98e1ef
change-id: 20240628-jag-revisit_cache_tag_assign-1984d808369f
Best regards,
--
Joel Granados <j.granados@xxxxxxxxxxx>