> +extern void unlock_page_private_2(struct page *page);No need for the extern.Otherwise this looks good:Reviewed-by: Christoph Hellwig <hch@lst.de>