diff options
| author | Yanfei Xu <yanfei.xu@intel.com> | 2023-06-14 10:47:04 +0800 |
|---|---|---|
| committer | Joerg Roedel <jroedel@suse.de> | 2023-06-16 16:38:32 +0200 |
| commit | 3f13f72787bcd15792c01a34c2aed8bdf38397bb (patch) | |
| tree | e281e8aaf23652d41ad574b3548666c3e59ec676 /drivers/iommu | |
| parent | a0e9911ac14baf46d8a8bea322c5bd7b3845825c (diff) | |
| download | linux-stable-3f13f72787bcd15792c01a34c2aed8bdf38397bb.tar.gz linux-stable-3f13f72787bcd15792c01a34c2aed8bdf38397bb.tar.bz2 linux-stable-3f13f72787bcd15792c01a34c2aed8bdf38397bb.zip | |
iommu/vt-d: Remove two WARN_ON in domain_context_mapping_one()
Remove the WARN_ON(did == 0) as the domain id 0 is reserved and
set once the domain_ids is allocated. So iommu_init_domains will
never return 0.
Remove the WARN_ON(!table) as this pointer will be accessed in
the following code, if empty "table" really happens, the kernel
will report a NULL pointer reference warning at the first place.
Signed-off-by: Yanfei Xu <yanfei.xu@intel.com>
Link: https://lore.kernel.org/r/20230605112659.308981-3-yanfei.xu@intel.com
Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Diffstat (limited to 'drivers/iommu')
| -rw-r--r-- | drivers/iommu/intel/iommu.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/iommu/intel/iommu.c b/drivers/iommu/intel/iommu.c index e83fe243680b..4c0b7424c45e 100644 --- a/drivers/iommu/intel/iommu.c +++ b/drivers/iommu/intel/iommu.c @@ -1897,8 +1897,6 @@ static int domain_context_mapping_one(struct dmar_domain *domain, struct context_entry *context; int ret; - WARN_ON(did == 0); - if (hw_pass_through && domain_type_is_si(domain)) translation = CONTEXT_TT_PASS_THROUGH; @@ -1944,8 +1942,6 @@ static int domain_context_mapping_one(struct dmar_domain *domain, if (sm_supported(iommu)) { unsigned long pds; - WARN_ON(!table); - /* Setup the PASID DIR pointer: */ pds = context_get_sm_pds(table); context->lo = (u64)virt_to_phys(table->table) | |