[xen stable-4.13] VT-d: don't lose errors when flushing TLBs on multiple IOMMUs

[patchbot@xxxxxxx]

commit 3f3ebda3ccbea1e8a5663693247017015fc79b68
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Thu Jul 15 09:43:11 2021 +0200
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Thu Jul 15 09:43:11 2021 +0200

    VT-d: don't lose errors when flushing TLBs on multiple IOMMUs
    
    While no longer an immediate problem with flushes no longer timing out,
    errors (if any) get properly reported by iommu_flush_iotlb_{dsi,psi}().
    Overwriting such an error with, perhaps, a success indicator received
    from another IOMMU will misguide callers. Record the first error, but
    don't bail from the loop (such that further necessary invalidation gets
    carried out on a best effort basis).
    
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
    Reviewed-by: Kevin Tian <kevin.tian@xxxxxxxxx>
    master commit: e7059776f9755b989a992d229c68c3d7778412be
    master date: 2021-06-24 16:30:06 +0200
---
 xen/drivers/passthrough/vtd/iommu.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/xen/drivers/passthrough/vtd/iommu.c 
b/xen/drivers/passthrough/vtd/iommu.c
index 2804d0b1b7..38719cf8ec 100644
--- a/xen/drivers/passthrough/vtd/iommu.c
+++ b/xen/drivers/passthrough/vtd/iommu.c
@@ -590,7 +590,7 @@ static int __must_check iommu_flush_iotlb(struct domain *d, 
dfn_t dfn,
     struct vtd_iommu *iommu;
     bool_t flush_dev_iotlb;
     int iommu_domid;
-    int rc = 0;
+    int ret = 0;
 
     /*
      * No need pcideves_lock here because we have flush
@@ -598,6 +598,8 @@ static int __must_check iommu_flush_iotlb(struct domain *d, 
dfn_t dfn,
      */
     for_each_drhd_unit ( drhd )
     {
+        int rc;
+
         iommu = drhd->iommu;
 
         if ( !test_bit(iommu->index, &hd->arch.iommu_bitmap) )
@@ -620,13 +622,12 @@ static int __must_check iommu_flush_iotlb(struct domain 
*d, dfn_t dfn,
                                        flush_dev_iotlb);
 
         if ( rc > 0 )
-        {
             iommu_flush_write_buffer(iommu);
-            rc = 0;
-        }
+        else if ( !ret )
+            ret = rc;
     }
 
-    return rc;
+    return ret;
 }
 
 static int __must_check iommu_flush_iotlb_pages(struct domain *d,
--
generated by git-patchbot for /home/xen/git/xen.git#stable-4.13