| Message ID | 1467695057-12431-11-git-send-email-benh@kernel.crashing.org (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
On Tue, 2016-05-07 at 05:03:46 UTC, Benjamin Herrenschmidt wrote: > Anything in early_setup() needs to be justified to be there, in > this case, we need the trampolines before we can take exceptions > and thus before we turn on the MMU. > > Also remove a pretty meaningless and misplaced debug message > > Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/63c254a501049f70c53aea6025 cheers
diff --git a/arch/powerpc/kernel/setup_64.c b/arch/powerpc/kernel/setup_64.c index e1b2e6f..a641753 100644 --- a/arch/powerpc/kernel/setup_64.c +++ b/arch/powerpc/kernel/setup_64.c @@ -276,10 +276,11 @@ void __init early_setup(unsigned long dt_ptr) /* Probe the machine type */ probe_machine(); + /* Setup the trampolines from the lowmem exception vectors + * to the kdump kernel when not using a relocatable kernel. + */ setup_kdump_trampoline(); - DBG("Found, Initializing memory management...\n"); - /* Initialize the hash table or TLB handling */ early_init_mmu();
Anything in early_setup() needs to be justified to be there, in this case, we need the trampolines before we can take exceptions and thus before we turn on the MMU. Also remove a pretty meaningless and misplaced debug message Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> --- arch/powerpc/kernel/setup_64.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)