Patchwork [1/1] runqemu: set qemuarm memory size back to 128MB

login
register
mail settings
Submitter Laurentiu Palcu
Date Sept. 20, 2013, 8:06 a.m.
Message ID <313d83d8a64fc904a8c2a60f4aba603c793fa4dc.1379662642.git.laurentiu.palcu@intel.com>
Download mbox | patch
Permalink /patch/58417/
State Accepted
Commit 405be52d26d39d94c1682c1b3d3c93f7a9593bfd
Headers show

Comments

Laurentiu Palcu - Sept. 20, 2013, 8:06 a.m.
The following commit, 6ccd4d6, increased the RAM size for qemu machines
to 256MB due to some smart sanity tests failing on autobuilder because
more memory was needed.

Unfortunately this leads to various, potentially dangerous, issues like
the one observed during sudoku-savant project compilation:

collect: relinking
collect2: error: '_ZNK6sudoku5ClearINS_6SquareEEclERS1_' was assigned to
'board.rpo', but was not defined during recompilation, or vice versa
board.o:(.rodata+0x8): undefined reference to
`sudoku::Clear<sudoku::Square>::operator()(sudoku::Square&) const'
board.o:(.rodata+0x20): undefined reference to
`sudoku::Clear<sudoku::Sequence>::operator()(sudoku::Sequence&) const'
board.o:(.rodata+0x34): undefined reference to `typeinfo for
sudoku::Action<sudoku::Sequence>'
...AND THE LIST CONTINUES...
collect2: error: ld returned 1 exit status
make: *** [sudoku-savant] Error 1

After some tests, I found that the maximum amount of memory needed for
sudoku to compile properly is 146MB(!?!).

My attempts to create a simpler test case (using templates), in order to
replicate and isolate the issue failed. All the tests compiled just
fine.

So, my guess is that this problem is certainly memory related but the
cause might be hidden in any of the following: qemu versatile hw model,
in the kernel or, highly unlikely but not impossible, the toolchain
itself. The reason I don't really think the cause is in the toolchain is
the fact that the compilation completes just fine for 128MB on qemuarm but
also on other qemu machines (with 256MB of memory).

Since this issue might need lots of time to have a proper fix, I'll revert back
to using 128MB for qemuarm for the time being.

[YOCTO #5133]

Signed-off-by: Laurentiu Palcu <laurentiu.palcu@intel.com>
---
 scripts/runqemu-internal |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch

diff --git a/scripts/runqemu-internal b/scripts/runqemu-internal
index 485119f..74b0c35 100755
--- a/scripts/runqemu-internal
+++ b/scripts/runqemu-internal
@@ -48,7 +48,7 @@  else
             mem_size=256
             ;;
         "qemuarm")
-            mem_size=256
+            mem_size=128
             ;;
         "qemumicroblaze")
             mem_size=64