DescriptionRevert of [Oilpan] Set stack limit using estimated sizes (Re-land) (patchset #3 id:40001 of https://codereview.chromium.org/927213002/)
Reason for revert:
Broke a heap test on Android Nexus4
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20(Nexus4)
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/27126
Original issue's description:
> Set the stack limit size based on estimated sizes.
>
> To check if the stack memory has enough room for recursive
> calls, we tried to allocate a large object and assumed the size.
> This CL estimate the real thread stack limit using APIs.
> On some platforms where such APIs do not work correctly,
> we set the limit using the old algorithm.
>
> ---
> This CL is re-land of https://codereview.chromium.org/910663002/
> with fixing a test.
> We needed a longer call chain of trace() to expect
> to use an alternative marking method, because we got wider
> stack memory area to go eagerly.
>
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=190388
TBR=oilpan-reviews@chromium.org,kouhei@chromium.org,peria@google.com,haraken@chromium.org,peria@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=190403
Patch Set 1 #
Created: 5 years, 10 months ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|