From 0220cb34a91658ecc3782ec1a4700dcb5ece37d8 Mon Sep 17 00:00:00 2001 From: He Zhe Date: Sun, 8 Jul 2018 20:14:43 -0700 Subject: scripts/runqemu: fix qemumips qemumips64 memory detection kernel panic $ runqemu qemumips64 core-image-minimal nographic qemuparams="-m 512" ... [ 0.000000] Call Trace: [ 0.000000] [] clear_page+0x0/0x128 [ 0.000000] [] get_page_from_freelist+0xab8/0xc00 [ 0.000000] [] __alloc_pages_nodemask+0xdc/0xf68 [ 0.000000] [] __get_free_pages+0x18/0x70 [ 0.000000] [] setup_zero_pages+0x1c/0xb8 [ 0.000000] [] mem_init+0x54/0xa0 [ 0.000000] [] start_kernel+0x204/0x4d8 [ 0.000000] [] kernel_entry+0x0/0x40 [ 0.000000] Code: 02002025 1000f8d9 8e634d7c <34860f80> cc9e0000 cc9e0020 cc9e0040 cc9e0060 cc9e0080 [ 0.000000] [ 0.000000] ---[ end trace 0000000000000000 ]--- [ 0.000000] Kernel panic - not syncing: Attempted to kill the idle task! [ 0.000000] ---[ end Kernel panic - not syncing: Attempted to kill the idle task! ... OE uses qemumips to simulate a Malta board by default. As upstream qemu introduced: https://git.qemu.org/?p=qemu.git;a=commit;h=94c2b6aff43cdfcfdfb552773a6b6b973a72ef0b The Malta board can support up to 2GiB of RAM which should be able to boot a Linux kernel built with CONFIG_HIGHMEM enabled and passing "-m 2048" to QEMU and appending the following kernel parameters: ... mem=256M@0x0 mem=256M@0x90000000 mem=1536M@0x20000000 ... But the following commit in kernel broke above mem=X@Y setting which added the memory as reserved memory area. https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=73fbc1eba7ffa3bf0ad12486232a8a1edb4e4411 ... commit 73fbc1eba7ffa3bf0ad12486232a8a1edb4e4411 Author: Marcin Nowakowski Date: Wed Nov 23 14:43:49 2016 +0100 MIPS: fix mem=X@Y commandline processing ... So remove `mem=*' to disable user-defined physical RAM map which let kernel itself caculates memory ranges. Author: Hongxu Jia [ Merge the two fixes for qemumips32 and qemumips64 into one patch, and make it support all mips cases ] Signed-off-by: He Zhe Signed-off-by: Richard Purdie --- scripts/runqemu | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/scripts/runqemu b/scripts/runqemu index de42d0f323..73d7d5818b 100755 --- a/scripts/runqemu +++ b/scripts/runqemu @@ -668,7 +668,10 @@ class BaseConfig(object): logger.info('QB_MEM is not set, use 512M by default') self.set('QB_MEM', '-m 512') - self.kernel_cmdline_script += ' mem=%s' % self.get('QB_MEM').replace('-m','').strip() + 'M' + mach = self.get('MACHINE') + if not mach.startswith('qemumips'): + self.kernel_cmdline_script += ' mem=%s' % self.get('QB_MEM').replace('-m','').strip() + 'M' + self.qemu_opt_script += ' %s' % self.get('QB_MEM') def check_tcpserial(self): -- cgit 1.2.3-korg