Integrated: JDK-8260707: java/lang/instrument/PremainClass/InheritAgent0100.java times out

[Thomas Stuefe]

On Mon, 1 Feb 2021 10:38:30 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> Hi,
> 
> may I please have reviews for this trivial fix.
> 
> We see timeouts with this test on slow large (memory wise) AIX machines, as well as large core files.
> 
> This test is a negative test which tests that the VM corectly recognizes an error condition and aborts. Code goes through jni_FatalError()->os::abort(), writes a core and aborts the process.
> 
> No parameters are given for that VM invocation, so heap size depends on machine size. On my Linux box, the generated core takes about 500m. On AIX we see cores of 16G and more.
> 
> The difference between AIX and other platforms is that AIX does not have the notion of "committing" memory (we have no MAP_NORESERVE flag), so all mmapped memory counts toward the commit charge from the moment it is mapped. That makes core files on AIX annoyingly large. I'd expect a similar behavior on other platforms with overcommit disabled.
> 
> The fix is to run the test without CreateCoredumpOnCrash. For good measure, I also reduced the heap size to 128m.
> 
> Thanks to @ArnoZeller for figuring that one out.
> 
> 
> -----
> Tests: manual jtreg tests, verifying that no cores are written.

This pull request has now been integrated.

Changeset: d7b1fc59
Author:    Thomas Stuefe <stuefe at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/d7b1fc59
Stats:     5 lines in 3 files changed: 0 ins; 0 del; 5 mod

8260707: java/lang/instrument/PremainClass/InheritAgent0100.java times out

Co-authored-by: Arno Zeller <azeller at openjdk.org>
Reviewed-by: dholmes, sspitsyn, dcubed

-------------

PR: https://git.openjdk.java.net/jdk/pull/2332