including serviceability-dev.
On 12/1/20 9:08 PM, Coleen Phillimore wrote:
On 12/1/20 9:04 PM, David Holmes wrote:
On Tue, 1 Dec 2020 14:16:14 GMT, Coleen Phillimore
<cole...@openjdk.org> wrote:
Give an error message rather than logging the error and then
crashing later because the JVM can't detect stack overflow. In a
resource exhausted situation, thread creation is also failing.
This is the vm_exit_out_of_memory message printed:
`$ java -XX:+UseNewCode -version
[0.003s][warning][os,thread] Attempt to protect stack guard pages
failed (0x00007f606b249000-0x00007f606b24d000).
There is insufficient memory for the Java Runtime Environment to
continue.
Native memory allocation (mprotect) failed to protect 16384 bytes
for memory to guard stack pages
An error report file with more information is saved as:
/16mprotect/hs_err_pid30596.log`
`
Coleen Phillimore has updated the pull request with a new target
base due to a merge or a rebase. The incremental webrev excludes the
unrelated changes brought in by the merge/rebase. The pull request
contains three additional commits since the last revision:
- Merge branch 'master' into mprotect
- Made resexhaused001.004 manual tests. You can't reliably run
these tests.
- 8253916: ResourceExhausted/resexhausted001 crashes on Linux-x64
I'm okay with the functional change here, to do vm_exit_oom when
guard page creation fails.
I'm not sure about the test change. It was only problem-listed on
Linux but now is "manual" on all platforms and so will now be
excluded from automated testing on all platforms.
I didn't think these tests would be reliable anywhere, but maybe they
were on Windows. What do you suggest?
Thanks for reviewing,
Coleen
-------------
Marked as reviewed by dholmes (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/1540