From mboxrd@z Thu Jan 1 00:00:00 1970 From: mark.rutland@arm.com (Mark Rutland) Date: Wed, 15 Feb 2017 10:56:12 +0000 Subject: [PATCH] arm64: fix warning about swapper_pg_dir overflow In-Reply-To: <20170214212718.2792194-1-arnd@arndb.de> References: <20170214212718.2792194-1-arnd@arndb.de> Message-ID: <20170215105612.GB31733@leverpostej> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Feb 14, 2017 at 10:27:01PM +0100, Arnd Bergmann wrote: > With 4 levels of 16KB pages, we get this warning about the fact that we are > copying a whole page into an array that is declared as having only two pointers > for the top level of the page table: > > arch/arm64/mm/mmu.c: In function 'paging_init': > arch/arm64/mm/mmu.c:528:2: error: 'memcpy' writing 16384 bytes into a region of size 16 overflows the destination [-Werror=stringop-overflow=] > > This is harmless since we actually reserve a whole page in the definition of the > array that comes from, and just the extern declaration is short. The pgdir > is initialized to zero either way, so copying the actual entries here seems > like the best solution. > > Signed-off-by: Arnd Bergmann Acked-by: Mark Rutland Thanks, Mark. > --- > 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 2131521ddc24..b805c017f789 100644 > --- a/arch/arm64/mm/mmu.c > +++ b/arch/arm64/mm/mmu.c > @@ -525,7 +525,7 @@ void __init paging_init(void) > * To do this we need to go via a temporary pgd. > */ > cpu_replace_ttbr1(__va(pgd_phys)); > - memcpy(swapper_pg_dir, pgd, PAGE_SIZE); > + memcpy(swapper_pg_dir, pgd, PGD_SIZE); > cpu_replace_ttbr1(lm_alias(swapper_pg_dir)); > > pgd_clear_fixmap(); > -- > 2.9.0 >