Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a3339ff7 authored by Russell King's avatar Russell King Committed by Joonwoo Park
Browse files

ARM: poison the vectors page 



Fill the empty regions of the vectors page with an exception generating
instruction.  This ensures that any inappropriate branch to the vector
page is appropriately trapped, rather than just encountering some code
to execute.  (The vectors page was filled with zero before, which
corresponds with the "andeq r0, r0, r0" instruction - a no-op.)

Change-Id: I74c0105a2bcb3b78d9fc8b105a05c0fc269ba400
Cc: <stable@vger.kernel.org>
Acked-by Nicolas Pitre <nico@linaro.org>
Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
Signed-off-by: default avatarJoonwoo Park <joonwoop@codeaurora.org>
parent 1a19b198

arch/arm/kernel/traps.c

+10 −0
Original line number Diff line number Diff line
@@ -821,9 +821,19 @@ void __init early_trap_init(void *vectors_base) 
	extern char __vectors_start[], __vectors_end[];

	extern char __kuser_helper_start[], __kuser_helper_end[];

	int kuser_sz = __kuser_helper_end - __kuser_helper_start;

	unsigned i;



	vectors_page = vectors_base;



	/*

	 * Poison the vectors page with an undefined instruction.  This

	 * instruction is chosen to be undefined for both ARM and Thumb

	 * ISAs.  The Thumb version is an undefined instruction with a

	 * branch back to the undefined instruction.

	 */

	for (i = 0; i < PAGE_SIZE / sizeof(u32); i++)

		((u32 *)vectors_base)[i] = 0xe7fddef1;



	/*

	 * Copy the vectors, stubs and kuser helpers (in entry-armv.S)

	 * into the vector page, mapped at 0xffff0000, and ensure these