> +extern void unlock_page_private_2(struct page *page); No need for the extern. Otherwise this looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx> -- Linux-cachefs mailing list Linux-cachefs@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/linux-cachefs