Commit 5983d1e7 authored by Nicolin Chen's avatar Nicolin Chen Committed by Jason Gunthorpe
Browse files

iommufd/selftest: Drop parent domain from mock_iommu_domain_nested 

There is no use of this parent domain. Delete the dead code.

Note that the s2_parent in struct mock_viommu will be a deadcode too. Yet,
keep it because it will be soon used by HW queue objects, i.e. no point in
adding it back and forth in such a short window. Besides, keeping it could
cover the majority of vIOMMU use cases where a driver-level structure will
be larger in size than the core structure.

Link: https://patch.msgid.link/r/0f155a7cd71034a498448fe4828fb4aaacdabf95.1749882255.git.nicolinc@nvidia.com


Signed-off-by: default avatarNicolin Chen <nicolinc@nvidia.com>
Reviewed-by: default avatarJason Gunthorpe <jgg@nvidia.com>
Reviewed-by: default avatarKevin Tian <kevin.tian@intel.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
parent 63141fa7

drivers/iommu/iommufd/selftest.c

+0 −3
Original line number Diff line number Diff line
@@ -138,7 +138,6 @@ to_mock_domain(struct iommu_domain *domain) 
struct mock_iommu_domain_nested {

	struct iommu_domain domain;

	struct mock_viommu *mock_viommu;

	struct mock_iommu_domain *parent;

	u32 iotlb[MOCK_NESTED_DOMAIN_IOTLB_NUM];

};
@@ -434,7 +433,6 @@ mock_domain_alloc_nested(struct device *dev, struct iommu_domain *parent, 
	mock_nested = __mock_domain_alloc_nested(user_data);

	if (IS_ERR(mock_nested))

		return ERR_CAST(mock_nested);

	mock_nested->parent = mock_parent;

	return &mock_nested->domain;

}
@@ -692,7 +690,6 @@ mock_viommu_alloc_domain_nested(struct iommufd_viommu *viommu, u32 flags, 
	if (IS_ERR(mock_nested))

		return ERR_CAST(mock_nested);

	mock_nested->mock_viommu = mock_viommu;

	mock_nested->parent = mock_viommu->s2_parent;

	return &mock_nested->domain;

}