On Thu, Jul 27, 2023 at 12:23:07AM -0700, Nicolin Chen wrote: > This is a preparatory change for ioas replacement support for access. > The replacement routine does an iopt_add_access() at a new IOAS first > and then iopt_remove_access() at the old IOAS upon the success of the > first call. However, the first call overrides the iopt_access_list_id > in the access struct, resulting in that id un-erased in the xarray. > > Add an iopt_access_list_id in iopt_remove_access, so the replacement > routine can save the id before it gets overwritten, and pass the id > in iopt_remove_access() for a proper cleanup. > > The existing callers should just pass in access->iopt_access_list_id. > > Signed-off-by: Nicolin Chen <nicolinc@xxxxxxxxxx> > --- > drivers/iommu/iommufd/device.c | 6 ++++-- > drivers/iommu/iommufd/io_pagetable.c | 6 +++--- > drivers/iommu/iommufd/iommufd_private.h | 3 ++- > 3 files changed, 9 insertions(+), 6 deletions(-) Reviewed-by: Jason Gunthorpe <jgg@xxxxxxxxxx> Jason
