On Sun, 26 May 2019 13:44:04 +0200 "Thomas Meyer" <[email protected]> wrote:
> From [email protected] Sun May 26 00:13:26 2019 > Subject: [PATCH] vfio-pci/nvlink2: Use vma_pages function instead of explicit > computation > To: [email protected], [email protected], > [email protected] > Content-Type: text/plain; charset="UTF-8" > Mime-Version: 1.0 > Content-Transfer-Encoding: 8bit > X-Patch: Cocci > X-Mailer: DiffSplit > Message-ID: <[email protected]> > References: <[email protected]> > In-Reply-To: <[email protected]> > X-Serial-No: 1 > > Use vma_pages function on vma object instead of explicit computation. > > Signed-off-by: Thomas Meyer <[email protected]> > --- > > diff -u -p a/drivers/vfio/pci/vfio_pci_nvlink2.c > b/drivers/vfio/pci/vfio_pci_nvlink2.c > --- a/drivers/vfio/pci/vfio_pci_nvlink2.c > +++ b/drivers/vfio/pci/vfio_pci_nvlink2.c > @@ -161,7 +161,7 @@ static int vfio_pci_nvgpu_mmap(struct vf > > atomic_inc(&data->mm->mm_count); > ret = (int) mm_iommu_newdev(data->mm, data->useraddr, > - (vma->vm_end - vma->vm_start) >> PAGE_SHIFT, > + vma_pages(vma), > data->gpu_hpa, &data->mem); > > trace_vfio_pci_nvgpu_mmap(vdev->pdev, data->gpu_hpa, data->useraddr, Besides the formatting of this patch, there's already a pending patch with this same change: https://lkml.org/lkml/2019/5/16/658 I think the original must have bounced from lkml due the encoding, but I'll use that one since it came first, is slightly cleaner in wrapping the line following the change, and already has Alexey's R-b. Thanks, Alex
