worker: set stack size for worker threads#26049

Closed

addaleax wants to merge 1 commit intonodejs:masterfrom

addaleax:worker-stack-overflow

Member

addaleax commented Feb 11, 2019 •

edited

Loading

~~(The first commit is #26037.)~~

This is so we can inform V8 about a known limit for the stack.

Otherwise, on some systems recursive functions may lead to
segmentation faults rather than “safe” failures.

Checklist

make -j4 test (UNIX), or vcbuild test (Windows) passes
tests and/or benchmarks are included
commit message follows commit guidelines

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready worker