On Mon, Sep 14, 2020 at 04:53:25PM -0700, Ralph Campbell wrote:Not true. The page_free() callback records the page is free by setting
Since set_page_refcounted() is defined in mm_interal.h I would have to
move the definition to someplace like page_ref.h or have the drivers
cal init_page_count() or set_page_count() since get_page() calls
VM_BUG_ON_PAGE() if refcount == 0.
I'll move set_page_refcounted() since that is what the page allocator
uses and seems named for the purpose.
I don't think any of the three ->page_free instances even cares about
the page refcount.