On Wed, Aug 11, 2021 at 12:05:42AM -0700, John Hubbard wrote: > - if (unlikely(!try_get_page(page))) > + if (unlikely(try_get_compound_head(page, 1) == NULL)) Why not a simple ! instead of the == NULL? Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>