On Fri, Jan 15, 2016 at 07:18:37PM +0000, Geoff Levand wrote: > From: James Morse <james.mo...@arm.com> > > From: James Morse <james.mo...@arm.com>
Not sure why this appears multiple times. > page.h uses '_AC' in the definition of PAGE_SIZE, but doesn't include > linux/const.h where this is defined. This produces build warnings when only > asm/page.h is included by asm code. > > Signed-off-by: James Morse <james.mo...@arm.com> > Acked-by: Pavel Machek <pa...@ucw.cz> > Signed-off-by: Geoff Levand <ge...@infradead.org> This is sensible even in isolation, so FWIW: Acked-by: Mark Rutland <mark.rutl...@arm.com> I note that for the !__ASSEMBLY__ portion we use current, READ_IMPLIES_EXEC, and some VM_* flags, without including the headers those are defined in. It might be worth fixing those up also. Mark. > --- > arch/arm64/include/asm/page.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm64/include/asm/page.h b/arch/arm64/include/asm/page.h > index 9b2f5a9..fbafd0a 100644 > --- a/arch/arm64/include/asm/page.h > +++ b/arch/arm64/include/asm/page.h > @@ -19,6 +19,8 @@ > #ifndef __ASM_PAGE_H > #define __ASM_PAGE_H > > +#include <linux/const.h> > + > /* PAGE_SHIFT determines the page size */ > /* CONT_SHIFT determines the number of pages which can be tracked together > */ > #ifdef CONFIG_ARM64_64K_PAGES > -- > 2.5.0 > > _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec