Re: [PATCH 1/6] mm/gup: perform check_dax_vmas only when FS_DAX is enabled

To: John Hubbard <jhubbard@xxxxxxxxxx>
Subject: Re: [PATCH 1/6] mm/gup: perform check_dax_vmas only when FS_DAX is enabled
From: Pavel Tatashin <pasha.tatashin@xxxxxxxxxx>
Date: Thu, 3 Dec 2020 09:52:26 -0500
Cc: LKML <linux-kernel@xxxxxxxxxxxxxxx>, linux-mm <linux-mm@xxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Vlastimil Babka <vbabka@xxxxxxx>, Michal Hocko <mhocko@xxxxxxxx>, David Hildenbrand <david@xxxxxxxxxx>, Oscar Salvador <osalvador@xxxxxxx>, Dan Williams <dan.j.williams@xxxxxxxxx>, Sasha Levin <sashal@xxxxxxxxxx>, Tyler Hicks <tyhicks@xxxxxxxxxxxxxxxxxxx>, Joonsoo Kim <iamjoonsoo.kim@xxxxxxx>, mike.kravetz@xxxxxxxxxx, Steven Rostedt <rostedt@xxxxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, Jason Gunthorpe <jgg@xxxxxxxx>, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Mel Gorman <mgorman@xxxxxxx>, Matthew Wilcox <willy@xxxxxxxxxxxxx>, David Rientjes <rientjes@xxxxxxxxxx>
In-reply-to: <be8546cc-bad3-d47e-dae7-0daca69c699c@nvidia.com>
References: <20201202052330.474592-1-pasha.tatashin@soleen.com> <20201202052330.474592-2-pasha.tatashin@soleen.com> <be8546cc-bad3-d47e-dae7-0daca69c699c@nvidia.com>

> Looks obviously correct, and the follow-up simplication is very nice.
>
> Reviewed-by: John Hubbard <jhubbard@xxxxxxxxxx>

Thank you,
Pasha

References:
- [PATCH 0/6] prohibit pinning pages in ZONE_MOVABLE
  - From: Pavel Tatashin
- [PATCH 1/6] mm/gup: perform check_dax_vmas only when FS_DAX is enabled
  - From: Pavel Tatashin
- Re: [PATCH 1/6] mm/gup: perform check_dax_vmas only when FS_DAX is enabled
  - From: John Hubbard