From: Tim Deegan <tim@xen.org>
Date: Thu, 12 Sep 2013 10:45:13 +0000 (+0100)
Subject: passthrough/amd: Drop unnecessary lock lookup.
X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~6332
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=554ded9d9f50c01ae05415b575a6b172c4023279;p=xen.git

passthrough/amd: Drop unnecessary lock lookup.

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>
---

diff --git a/xen/drivers/passthrough/amd/iommu_intr.c b/xen/drivers/passthrough/amd/iommu_intr.c
index 831f92a732..213f4d7bab 100644
--- a/xen/drivers/passthrough/amd/iommu_intr.c
+++ b/xen/drivers/passthrough/amd/iommu_intr.c
@@ -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 )
     {