passthrough/amd: Drop unnecessary lock lookup.
authorTim Deegan <tim@xen.org>
Thu, 12 Sep 2013 10:45:13 +0000 (11:45 +0100)
committerTim Deegan <tim@xen.org>
Tue, 17 Sep 2013 15:36:08 +0000 (16:36 +0100)
The lock's not used for anything, and AFAICT no locking is needed
since the IVRS tables are static after boot.

Coverity CID 1087199

Signed-off-by: Tim Deegan <tim@xen.org>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
xen/drivers/passthrough/amd/iommu_intr.c

index 831f92a732eb94acef111d09b01806799d150445..213f4d7bab6d782db0bfff13bbab17631be8e577 100644 (file)
@@ -443,7 +443,6 @@ static int update_intremap_entry_from_msi_msg(
      * devices.
      */
 
-    lock = get_intremap_lock(iommu->seg, alias_id);
     if ( ( req_id != alias_id ) &&
          get_ivrs_mappings(iommu->seg)[alias_id].intremap_table != NULL )
     {