On Sun, Jun 28, 2020 at 11:18 PM Kees Cook <keescook@xxxxxxxxxxxx> wrote: > > Fix a case of needless quotes in __section(), which Clang doesn't like. > > Acked-by: Will Deacon <will@xxxxxxxxxx> > Signed-off-by: Kees Cook <keescook@xxxxxxxxxxxx> Yep, I remember bugs from this. Probably should scan the kernel for other instances of this. +Joe for checkpatch.pl validation. Reviewed-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx> > --- > arch/arm64/mm/mmu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/mm/mmu.c b/arch/arm64/mm/mmu.c > index 1df25f26571d..dce024ea6084 100644 > --- a/arch/arm64/mm/mmu.c > +++ b/arch/arm64/mm/mmu.c > @@ -42,7 +42,7 @@ > u64 idmap_t0sz = TCR_T0SZ(VA_BITS); > u64 idmap_ptrs_per_pgd = PTRS_PER_PGD; > > -u64 __section(".mmuoff.data.write") vabits_actual; > +u64 __section(.mmuoff.data.write) vabits_actual; > EXPORT_SYMBOL(vabits_actual); > > u64 kimage_voffset __ro_after_init; > -- > 2.25.1 > -- Thanks, ~Nick Desaulniers