iommu/vt-d: Create a cache_tag_id to reduce argument passing

[PATCH RFC 0/5] iommu/vt-d: Create a cache_tag_id to reduce argument passing

Joel Granados via B4 Relay posted 5 patches 1 year, 5 months ago

Download series mbox

drivers/iommu/intel/cache.c | 184 ++++++++++++++++++++++----------------------
drivers/iommu/intel/iommu.h |  24 +++---
2 files changed, 104 insertions(+), 104 deletions(-)

Expand all Fold all

[PATCH RFC 0/5] iommu/vt-d: Create a cache_tag_id to reduce argument passing

Posted by Joel Granados via B4 Relay 1 year, 5 months ago

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@samsung.com>
---
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@samsung.com>