[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20240628-jag-revisit_cache_tag_assign-v1-0-a0c19063c983@samsung.com>
Date: Fri, 28 Jun 2024 14:27:38 +0200
From: Joel Granados via B4 Relay <devnull+j.granados.samsung.com@...nel.org>
To: David Woodhouse <dwmw2@...radead.org>,
Lu Baolu <baolu.lu@...ux.intel.com>, Joerg Roedel <joro@...tes.org>,
Will Deacon <will@...nel.org>, Robin Murphy <robin.murphy@....com>
Cc: iommu@...ts.linux.dev, linux-kernel@...r.kernel.org,
Joel Granados <j.granados@...sung.com>
Subject: [PATCH RFC 0/5] iommu/vt-d: Create a cache_tag_id to reduce
argument passing
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@...sung.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@...sung.com>
Powered by blists - more mailing lists