On Wed, 2020-01-08 at 15:29 +0100, Daniel Schwierzeck wrote:

Am 08.01.20 um 04:00 schrieb Weijie Gao:

...

In U-Boot the exception vector base will be moved to top of memory, to be used to display register dump when exception occurs.

But some old linux kernel does not honor the base set in CP0_EBASE. A modified exception vector base will cause kernel crash.

This patch adds an option to enable reset exception vector base to 0x80000000 before booting linux kernel.

Signed-off-by: Weijie Gao weijie.gao@mediatek.com

arch/mips/Kconfig | 13 +++++++++++++ arch/mips/lib/bootm.c | 9 +++++++++ 2 files changed, 22 insertions(+)

diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig index a3ae603044..4688717593 100644 --- a/arch/mips/Kconfig +++ b/arch/mips/Kconfig @@ -287,6 +287,19 @@ config MIPS_RELOCATION_TABLE_SIZE

 If unsure, leave at the default value.

+config RESTORE_EXCEPTION_VECTOR_BASE

• bool "Restore exception vector base before booting linux kernel"
• default n
• help

•  In U-Boot the exception vector base will be moved to top of memory,
  

•  to be used to display register dump when exception occurs.
  

•  But some old linux kernel does not honor the base set in CP0_EBASE.
  

•  A modified exception vector base will cause kernel crash.
  

•  This option will set exception vector base to 0x80000000.
  

Does it make sense to make the base address configurable and let it default to 0x80000000?

...

•  If unsure, say N.
  

endmenu

menu "OS boot interface" diff --git a/arch/mips/lib/bootm.c b/arch/mips/lib/bootm.c index 8c0d7672f2..86ea082fd0 100644 --- a/arch/mips/lib/bootm.c +++ b/arch/mips/lib/bootm.c @@ -294,6 +294,15 @@ static void boot_jump_linux(bootm_headers_t *images) bootstage_report(); #endif

+#ifdef CONFIG_RESTORE_EXCEPTION_VECTOR_BASE

• /* Restore EBASE for compatibility */
• set_c0_status(ST0_BEV);
• execution_hazard_barrier();
• write_c0_ebase(KSEG0);
• clear_c0_status(ST0_BEV);
• execution_hazard_barrier();

+#endif

could you move this to a function trap_restore() in arch/mips/lib/traps.c?

Also prefer the following over #ifdef:

if (CONFIG_IS_ENABLED(RESTORE_EXCEPTION_VECTOR_BASE))
    trap_restore();

Alternatively you could simply define trap_restore() as empty static inline function if RESTORE_EXCEPTION_VECTOR_BASE is disabled.

...

if (images->ft_len) kernel(-2, (ulong)images->ft_addr, 0, 0); else

Hi Daniel,

0x80000000 is the power-on value of EBASE. I think there is no need to change it to a different value.

For the rest parts, I'll take your advice.

Best Regards,

Weijie