| From: David Hildenbrand <david@redhat.com> |
| Subject: vfio/pci: drop nth_page() usage within SG entry |
| Date: Mon, 1 Sep 2025 17:03:53 +0200 |
| |
| It's no longer required to use nth_page() when iterating pages within a |
| single SG entry, so let's drop the nth_page() usage. |
| |
| Link: https://lkml.kernel.org/r/20250901150359.867252-33-david@redhat.com |
| Signed-off-by: David Hildenbrand <david@redhat.com> |
| Reviewed-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com> |
| Reviewed-by: Alex Williamson <alex.williamson@redhat.com> |
| Reviewed-by: Brett Creeley <brett.creeley@amd.com> |
| Cc: Jason Gunthorpe <jgg@ziepe.ca> |
| Cc: Yishai Hadas <yishaih@nvidia.com> |
| Cc: Shameer Kolothum <shameerali.kolothum.thodi@huawei.com> |
| Cc: Kevin Tian <kevin.tian@intel.com> |
| Signed-off-by: Andrew Morton <akpm@linux-foundation.org> |
| --- |
| |
| drivers/vfio/pci/pds/lm.c | 3 +-- |
| drivers/vfio/pci/virtio/migrate.c | 3 +-- |
| 2 files changed, 2 insertions(+), 4 deletions(-) |
| |
| --- a/drivers/vfio/pci/pds/lm.c~vfio-pci-drop-nth_page-usage-within-sg-entry |
| +++ a/drivers/vfio/pci/pds/lm.c |
| @@ -151,8 +151,7 @@ static struct page *pds_vfio_get_file_pa |
| lm_file->last_offset_sg = sg; |
| lm_file->sg_last_entry += i; |
| lm_file->last_offset = cur_offset; |
| - return nth_page(sg_page(sg), |
| - (offset - cur_offset) / PAGE_SIZE); |
| + return sg_page(sg) + (offset - cur_offset) / PAGE_SIZE; |
| } |
| cur_offset += sg->length; |
| } |
| --- a/drivers/vfio/pci/virtio/migrate.c~vfio-pci-drop-nth_page-usage-within-sg-entry |
| +++ a/drivers/vfio/pci/virtio/migrate.c |
| @@ -53,8 +53,7 @@ virtiovf_get_migration_page(struct virti |
| buf->last_offset_sg = sg; |
| buf->sg_last_entry += i; |
| buf->last_offset = cur_offset; |
| - return nth_page(sg_page(sg), |
| - (offset - cur_offset) / PAGE_SIZE); |
| + return sg_page(sg) + (offset - cur_offset) / PAGE_SIZE; |
| } |
| cur_offset += sg->length; |
| } |
| _ |
