Re: [PATCH v9 1/2] fs: New zonefs file system

From: Damien Le Moal
Date: Tue Jan 28 2020 - 23:14:06 EST


On Tue, 2020-01-28 at 16:34 +0100, Markus Elfring wrote:
> â
> > +++ b/fs/zonefs/super.c
> â
> > +out:
> > + kunmap(page);
> > +out_free:
> > + __free_page(page);
>
> Would you like to reconsider your name selection for such labels?
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/coding-style.rst?id=b0be0eff1a5ab77d588b76bd8b1c92d5d17b3f73#n460
>
> Change possibility:
>
> +unmap:
> + kunmap(page);
> +free_page:
> + __free_page(page);
>

Fixed. Thanks !

>
> Regards,
> Markus

--
Damien Le Moal
Western Digital Research