Re: [PATCH v1 10/11] drm/i915: track_pfn() -> "pfnmap tracking"

Lorenzo Stoakes
2025-04-28T20:24:01Z

On Fri, Apr 25, 2025 at 10:17:14AM +0200, David Hildenbrand wrote:
> track_pfn() does not exist, let's simply refer to it as "pfnmap
> tracking".
>
> Signed-off-by: David Hildenbrand <david@redhat.com>

Reviewed-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>

> ---
> drivers/gpu/drm/i915/i915_mm.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_mm.c b/drivers/gpu/drm/i915/i915_mm.c
> index 76e2801619f09..c33bd3d830699 100644
> --- a/drivers/gpu/drm/i915/i915_mm.c
> +++ b/drivers/gpu/drm/i915/i915_mm.c
> @@ -100,7 +100,7 @@ int remap_io_mapping(struct vm_area_struct *vma,
>
> GEM_BUG_ON((vma->vm_flags & EXPECTED_FLAGS) != EXPECTED_FLAGS);
>
> - /* We rely on prevalidation of the io-mapping to skip track_pfn(). */
> + /* We rely on prevalidation of the io-mapping to skip pfnmap tracking. */
> r.mm = vma->vm_mm;
> r.pfn = pfn;
> r.prot = __pgprot((pgprot_val(iomap->prot) & _PAGE_CACHE_MASK) |
> @@ -140,7 +140,7 @@ int remap_io_sg(struct vm_area_struct *vma,
> };
> int err;
>
> - /* We rely on prevalidation of the io-mapping to skip track_pfn(). */
> + /* We rely on prevalidation of the io-mapping to skip pfnmap tracking. */
> GEM_BUG_ON((vma->vm_flags & EXPECTED_FLAGS) != EXPECTED_FLAGS);
>
> while (offset >= r.sgt.max >> PAGE_SHIFT) {
> --
> 2.49.0
>