[Skiboot] [PATCH v2] core/init: Assert when kernel not found

Michael Neuling mikey at neuling.org
Thu Mar 1 10:07:16 AEDT 2018


If the kernel doesn't load out of flash or there is nothing at
KERNEL_LOAD_BASE, we end up with an esoteric message as we try to
branch to out of skiboot into nothing

  [    0.007197688,3] INIT: ELF header not found. Assuming raw binary.
  [    0.014035267,5] INIT: Starting kernel at 0x0, fdt at 0x3044ad90 13029
  [    0.014042254,3] ***********************************************
  [    0.014069947,3] Fatal Exception 0xe40 at 0000000000000000
  [    0.014085574,3] CFAR : 00000000300051c4
  [    0.014090118,3] SRR0 : 0000000000000000 SRR1 : 0000000000000000
  [    0.014096243,3] HSRR0: 0000000000000000 HSRR1: 9000000000001000
  [    0.014102546,3] DSISR: 00000000         DAR  : 0000000000000000
  [    0.014108538,3] LR   : 00000000300144c8 CTR  : 0000000000000000
  [    0.014114756,3] CR   : 40002202         XER  : 00000000
  [    0.014120301,3] GPR00: 000000003001447c GPR16: 0000000000000000

This improves the message and asserts in this case:

[    0.014042685,5] INIT: Starting kernel at 0x0, fdt at 0x3044ad90 13049 bytes)
[    0.014049556,0] FATAL: Kernel is zeros, can't execute!
[    0.014054237,0] Assert fail: core/init.c:566:0
[    0.014060472,0] Aborting!

Signed-off-by: Michael Neuling <mikey at neuling.org>
Reviewed-by: Vaidyanathan Srinivasan <svaidy at linux.vnet.ibm.com>
---
v2:
  Move location of assert as suggest by Vaidy
---
 core/init.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/core/init.c b/core/init.c
index 5633a9d38f..a8edc3b862 100644
--- a/core/init.c
+++ b/core/init.c
@@ -559,6 +559,13 @@ void __noreturn load_and_boot_kernel(bool is_reboot)
 	debug_descriptor.state_flags |= OPAL_BOOT_COMPLETE;
 
 	fdt_set_boot_cpuid_phys(fdt, this_cpu()->pir);
+
+	/* Check there is something there before we branch to it */
+	if (*(uint32_t *)kernel_entry == 0) {
+		prlog(PR_EMERG, "FATAL: Kernel is zeros, can't execute!\n");
+		assert(0);
+	}
+
 	if (kernel_32bit)
 		start_kernel32(kernel_entry, fdt, mem_top);
 	start_kernel(kernel_entry, fdt, mem_top);
-- 
2.14.1



More information about the Skiboot mailing list