On 10/31/19 4:49 PM, Ira Weiny wrote:
On Wed, Oct 30, 2019 at 03:49:30PM -0700, John Hubbard wrote:
Document the new pin_user_pages() and related calls and behavior.
Thanks to Jan Kara and Vlastimil Babka for explaining the 4 cases in this documentation. (I've reworded it and expanded on it slightly.)
As I said before I think this may be better in a previous patch where you reference it.
Yes, I'll merge this in with patch #5 ("mm/gup: introduce pin_user_pages*() and FOLL_PIN").
...
+TODO: There is also a special case when the pages are DAX pages: in addition to +the above flags, the caller needs something like a layout lease on the +associated file. This is yet to be implemented. When it is implemented, it's +expected that the lease will be a prerequisite to setting FOLL_LONGTERM.
For now we probably want to leave this note out until we figure out how this is going to work. Best to say something like:
Some pages, such as DAX pages, can't be pinned with longterm pins and will fail.
OK, that is better, I'll use that instead of the "TODO".
thanks,
John Hubbard NVIDIA