[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20221228213212.628636-1-shacharr@google.com>
Date: Wed, 28 Dec 2022 21:32:12 +0000
From: Shachar Raindel <shacharr@...gle.com>
To: Alex Williamson <alex.williamson@...hat.com>,
Cornelia Huck <cohuck@...hat.com>
Cc: kvm@...r.kernel.org, linux-kernel@...r.kernel.org,
Shachar Raindel <shacharr@...gle.com>
Subject: [PATCH] Convert backwards goto into a while loop
The function vaddr_get_pfns used a goto retry structure to implement
retrying. This is discouraged by the coding style guide (which is
only recommending goto for handling function exits). Convert the code
to a while loop, making it explicit that the follow block only runs
when the pin attempt failed.
Signed-off-by: Shachar Raindel <shacharr@...gle.com>
---
drivers/vfio/vfio_iommu_type1.c | 27 ++++++++++++++-------------
1 file changed, 14 insertions(+), 13 deletions(-)
diff --git a/drivers/vfio/vfio_iommu_type1.c b/drivers/vfio/vfio_iommu_type1.c
index 23c24fe98c00..7f38d7fc3f62 100644
--- a/drivers/vfio/vfio_iommu_type1.c
+++ b/drivers/vfio/vfio_iommu_type1.c
@@ -570,27 +570,28 @@ static int vaddr_get_pfns(struct mm_struct *mm, unsigned long vaddr,
}
*pfn = page_to_pfn(pages[0]);
- goto done;
- }
+ } else
+ do {
+
+ /* This is not a normal page, lookup PFN for P2P DMA */
+ vaddr = untagged_addr(vaddr);
- vaddr = untagged_addr(vaddr);
+ vma = vma_lookup(mm, vaddr);
-retry:
- vma = vma_lookup(mm, vaddr);
+ if (!vma || !(vma->vm_flags & VM_PFNMAP))
+ break;
- if (vma && vma->vm_flags & VM_PFNMAP) {
- ret = follow_fault_pfn(vma, mm, vaddr, pfn, prot & IOMMU_WRITE);
- if (ret == -EAGAIN)
- goto retry;
+ ret = follow_fault_pfn(vma, mm, vaddr, pfn,
+ prot & IOMMU_WRITE);
+ if (ret)
+ continue; /* Retry for EAGAIN, otherwise bail */
- if (!ret) {
if (is_invalid_reserved_pfn(*pfn))
ret = 1;
else
ret = -EFAULT;
- }
- }
-done:
+ } while (ret == -EAGAIN);
+
mmap_read_unlock(mm);
return ret;
}
--
2.39.0.314.g84b9a713c41-goog
Powered by blists - more mailing lists