[PATCH 02/19] arm64: kernel: Include _AC definition in page.h
Mark Rutland
mark.rutland at arm.com
Mon Jan 18 02:05:20 PST 2016
On Fri, Jan 15, 2016 at 07:18:37PM +0000, Geoff Levand wrote:
> From: James Morse <james.morse at arm.com>
>
> From: James Morse <james.morse at 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.morse at arm.com>
> Acked-by: Pavel Machek <pavel at ucw.cz>
> Signed-off-by: Geoff Levand <geoff at infradead.org>
This is sensible even in isolation, so FWIW:
Acked-by: Mark Rutland <mark.rutland at 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
>
>
More information about the linux-arm-kernel
mailing list