Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH 0/1] runqemu: set qemuarm memory size back to 128MB
@ 2013-09-20  8:06 Laurentiu Palcu
  2013-09-20  8:06 ` [PATCH 1/1] " Laurentiu Palcu
  0 siblings, 1 reply; 2+ messages in thread
From: Laurentiu Palcu @ 2013-09-20  8:06 UTC (permalink / raw)
  To: openembedded-core

Hi,

I investigated this issue(#5133) for some time but failed to find the root
cause, so far.  I reverted the memory back to 128MB so that it can be tested on
AB to see if the smart sanity tests (which was the cause for this increase)
pass. Apparently, those tests failed only on x86 qemu machines. This patch is
only a workaround, if a proper solution cannot be found in time for the
release.

However, if there are people on this list that could help on tracking this
down, feel free to reply to this. You can also find more info about the issue
itself in the commit message.

Thanks,
Laurentiu

The following changes since commit 853bc53cd58a621918f0e5ce662dba263d1befb4:

  qemu: Fix broken patch (2013-09-18 21:56:28 +0100)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib lpalcu/b5133_sudoku_savant_issue
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=lpalcu/b5133_sudoku_savant_issue

Laurentiu Palcu (1):
  runqemu: set qemuarm memory size back to 128MB

 scripts/runqemu-internal |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
1.7.9.5



^ permalink raw reply	[flat|nested] 2+ messages in thread

* [PATCH 1/1] runqemu: set qemuarm memory size back to 128MB
  2013-09-20  8:06 [PATCH 0/1] runqemu: set qemuarm memory size back to 128MB Laurentiu Palcu
@ 2013-09-20  8:06 ` Laurentiu Palcu
  0 siblings, 0 replies; 2+ messages in thread
From: Laurentiu Palcu @ 2013-09-20  8:06 UTC (permalink / raw)
  To: openembedded-core

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(-)

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
-- 
1.7.9.5



^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-09-20  8:06 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-20  8:06 [PATCH 0/1] runqemu: set qemuarm memory size back to 128MB Laurentiu Palcu
2013-09-20  8:06 ` [PATCH 1/1] " Laurentiu Palcu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox