Re: [PATCH v3 02/18] dax: Create a range version of dax_layout_busy_page()

From: Vivek Goyal
Date: Thu Aug 20 2020 - 10:29:50 EST


On Thu, Aug 20, 2020 at 02:58:55PM +0200, Jan Kara wrote:
[..]
> > /**
> > - * dax_layout_busy_page - find first pinned page in @mapping
> > + * dax_layout_busy_page_range - find first pinned page in @mapping
> > * @mapping: address space to scan for a page with ref count > 1
>
> Please document additional function arguments in the kernel-doc comment.
>
> Otherwise the patch looks good so feel free to add:
>
> Reviewed-by: Jan Kara <jack@xxxxxxx>
>
> after fixing this nit.
>

Hi Jan

Thanks for the review. Here is the updated patch. I also captured your
Reviewed-by.