144677 2015-05-06 02:12:05 -0700 New stress/new-largeish-contiguous-array-with-size.js fails on 32 bit platforms 2015-05-07 11:10:32 -0700 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) Unspecified Unspecified RESOLVED DUPLICATE 144609 P2 Normal --- 108645 1 ossy webkit-unassigned cgarcia clopez fpizlo mark.lam ossy oldest_to_newest 1092117 0 ossy 2015-05-06 02:12:05 -0700 New stress/new-largeish-contiguous-array-with-size.js introduced in https://trac.webkit.org/changeset/183847 and fails on 32 bit platforms. It failed once on GTK 32 bit bot: - https://build.webkit.org/builders/GTK%20Linux%2032-bit%20Release/builds/52461 - stress/new-largeish-contiguous-array-with-size.js.default: Exception: Error: heap too big before forced GC: 108104384 It fails persistently on EFL 32 bit ARM bots since it is introduced: - stress/new-largeish-contiguous-array-with-size.js.default: Exception: Error: heap too big before forced GC: 119708840 - stress/new-largeish-contiguous-array-with-size.js.default: Exception: Error: heap too big before forced GC: 124108928 - stress/new-largeish-contiguous-array-with-size.js.default: Exception: Error: heap too big before forced GC: 129308808 - stress/new-largeish-contiguous-array-with-size.js.default: Exception: Error: heap too big before forced GC: 120108656 ... Can it be a real bug or a bug in the test? 1092118 1 ossy 2015-05-06 02:13:37 -0700 and the same failure on the GTK ARM bot 1092641 2 fpizlo 2015-05-07 10:52:14 -0700 (In reply to comment #0) > New stress/new-largeish-contiguous-array-with-size.js > introduced in https://trac.webkit.org/changeset/183847 > and fails on 32 bit platforms. > > It failed once on GTK 32 bit bot: > - > https://build.webkit.org/builders/GTK%20Linux%2032-bit%20Release/builds/52461 > - stress/new-largeish-contiguous-array-with-size.js.default: Exception: > Error: heap too big before forced GC: 108104384 > > It fails persistently on EFL 32 bit ARM bots since it is introduced: > - stress/new-largeish-contiguous-array-with-size.js.default: Exception: > Error: heap too big before forced GC: 119708840 > - stress/new-largeish-contiguous-array-with-size.js.default: Exception: > Error: heap too big before forced GC: 124108928 > - stress/new-largeish-contiguous-array-with-size.js.default: Exception: > Error: heap too big before forced GC: 129308808 > - stress/new-largeish-contiguous-array-with-size.js.default: Exception: > Error: heap too big before forced GC: 120108656 > ... > > Can it be a real bug or a bug in the test? I think that the test might be wrong. I was assuming that 100MB is the largest that our minimum heap size would get. But I'm wrong: static inline size_t minHeapSize(HeapType heapType, size_t ramSize) { if (heapType == LargeHeap) return min(largeHeapSize, ramSize / 4); return smallHeapSize; } So, if you're on a machine with tons of heap, we'll be happy to give you 1/4 RAM headroom. This actually seems super sketchy anyways and maybe we should fix that - but for now it means that as written, my test is incorrect. I'll investigate. 1092647 3 fpizlo 2015-05-07 11:10:21 -0700 I will fix this bug as part of https://bugs.webkit.org/show_bug.cgi?id=144609. 1092648 4 fpizlo 2015-05-07 11:10:32 -0700 *** This bug has been marked as a duplicate of bug 144609 ***