Re: [PATCH v10 05/12] mm: HUGE_VMAP arch support cleanup

[Christophe Leroy]

Le 24/01/2021 à 12:40, Christoph Hellwig a écrit :
> > diff --git a/arch/arm64/include/asm/vmalloc.h b/arch/arm64/include/asm/vmalloc.h
> > index 2ca708ab9b20..597b40405319 100644
> > --- a/arch/arm64/include/asm/vmalloc.h
> > +++ b/arch/arm64/include/asm/vmalloc.h
> > @@ -1,4 +1,12 @@
> >   #ifndef _ASM_ARM64_VMALLOC_H
> >   #define _ASM_ARM64_VMALLOC_H
> >   
> > +#include <asm/page.h>
> > +
> > +#ifdef CONFIG_HAVE_ARCH_HUGE_VMAP
> > +bool arch_vmap_p4d_supported(pgprot_t prot);
> > +bool arch_vmap_pud_supported(pgprot_t prot);
> > +bool arch_vmap_pmd_supported(pgprot_t prot);
> > +#endif
>
> Shouldn't the be inlines or macros?  Also it would be useful
> if the architectures would not have to override all functions
> but just those that are it actually implements?
>
> Also lots of > 80 char lines in the patch.

Since https://github.com/linuxppc/linux/commit/bdc48fa11e46f867ea4d75fa59ee87a7f48be144
this 80 char limit is not strongly enforced anymore.

Allthough 80 is still the prefered limit, code is often more readable with a slightly longer single 
line that with lines splited.

Christophe