On Mon, Nov 02, 2020 at 08:03:16AM +0800, Fengfei Xi wrote:
> The names of functions xfs_buf_get_maps and _xfs_buf_free_pages
xfs_buf_get_maps? I don't see any changes for that function...?
--D
> can fully express their roles. So their comments are redundant.
> We could drop them entirely.
>
> Signed-off-by: Fengfei Xi <[email protected]>
> ---
> fs/xfs/xfs_buf.c | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/fs/xfs/xfs_buf.c b/fs/xfs/xfs_buf.c
> index 4e4cf91..2aeed30 100644
> --- a/fs/xfs/xfs_buf.c
> +++ b/fs/xfs/xfs_buf.c
> @@ -197,9 +197,6 @@
> return 0;
> }
>
> -/*
> - * Frees b_pages if it was allocated.
> - */
> static void
> xfs_buf_free_maps(
> struct xfs_buf *bp)
> @@ -297,9 +294,6 @@
> return 0;
> }
>
> -/*
> - * Frees b_pages if it was allocated.
> - */
> STATIC void
> _xfs_buf_free_pages(
> xfs_buf_t *bp)
> --
> 1.9.1
>