On Wed, 31 Mar 2021 06:56:23 GMT, Stefan Karlsson <stef...@openjdk.org> wrote:
> Add more logging to the LargeCopyWithMark.java test, to gather more > information when this test fails with OOME. > > The intention is to gather more info for JDK-8239089. > > I consider this patch trivial, and expect to push it after I've gotten one > Review. test/jdk/java/io/BufferedInputStream/LargeCopyWithMark.java line 29: > 27: * @summary BufferedInputStream calculates negative array size with large > 28: * streams and mark > 29: * @run main/othervm -Xmx4G -Xlog:gc,gc+heap,gc+ergo+heap > -XX:+CrashOnOutOfMemoryError -XX:+IgnoreUnrecognizedVMOptions > -XX:+G1ExitOnExpansionFailure LargeCopyWithMark Looks okay but I assume you can split this line to avoid having a 170+ line in this file, this helps with future side-by-side review. ------------- PR: https://git.openjdk.java.net/jdk/pull/3282
