On 3/28/23 20:16, Matthew Wilcox wrote:
...
+       if (!get_page_unless_zero(vmf->page))
+               return 0;

 From a folio point of view: what the hell are you doing here?  Tail
pages don't have individual refcounts; all the refcounts are actually

ohh, and I really should have caught that too. I plead spending too much
time recently in a somewhat more driver-centric mindset, and failing to
mentally shift gears properly for this case.

Sorry for missing that!

thanks,
--
John Hubbard
NVIDIA

Reply via email to