488279 - on winmo the perf tests language/string/split.as and language/string/typed/split.as does not run

Status: VERIFIED FIXED

(Tamarin Graveyard :: Virtual Machine, defect, P3)

Description

[Dan Schaffer]

on windows mobile the language/string/split.as and language/string/typed/split.as exits with no output.   When I change the size to 10000 (from 50000) the test runs ok (in the emulator).   

The non-desktop size was set to 50000 also.  Should we lower the size? 

I skipped the test for now.

Comment 1

[Lars T Hansen]

Exit with no output never sounds like a bug, so the bug should be found and fixed before the test size is lowered (even if that might be the right thing to do).

Comment 2

[Dan Smith]

Dan, please confirm current behavior with OOM handling in place.

Comment 3

[Lars T Hansen]

Almost certainly an OOM issue, we're constructing a string with 28,550,000 characters which is then split apart into 4,100,000 separate strings; a small device is unlikely to handle that.  At 10,000 the volumes would be 5,710,000 and 820,000 respectively - much more likely.

Need to verify that the OOM handling signals OOM correctly, then reduce the test size on non-desktop.

Comment 4

[Dan Schaffer]

confirmed language/string/split.as and language/string/typed/split.as both output error: out of memory and exit with code=128.  I lowered the non-desktop config to 40,000 (from 50,000) and the test runs on the tg01 in < 2s.  provided patch with just the changes (the line endings for split were messed up in tamarin-redux).

Comment 5

[Dan Schaffer]

Attachment: patch to lower non-desktop split tests so they run on tg01

Comment 6

[Dan Schaffer]

moving target to future since the bug is not required for 10.1 release.

Comment 7

[Dan Schaffer]

pushed fix in 2928:50b9ce61ce2e

Comment 8

[Dan Schaffer]

verified tests are passing.