[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