Thanks!

- Andreas

On 2016-02-10 18:38, Dmitry Samersoff wrote:
Andreas,

Looks good for me!

-Dmitry

On 2016-01-28 20:36, Andreas Eriksson wrote:
Hi,

Please review this backport of JDK-8129419
<https://bugs.openjdk.java.net/browse/JDK-8129419>.

Hotspot webrev:
http://cr.openjdk.java.net/~aeriksso/8129419/webrev.jdk8.00/hotspot/
JDK webrev: http://cr.openjdk.java.net/~aeriksso/8129419/webrev.jdk8.00/jdk/

There are three changes compared to JDK 9:

1) The hprof parser is in the JDK repo instead of top repo.
Otherwise these changes are the same.

2) On Solaris os::write has some UseVMInterruptibleIO logic that failed
because it assumed it was called by a JavaThread.
I changed it to skip UseVMInterruptibleIO logic if called by a
non-JavaThread.

3) The "heap dump file created" message were using sprintf and pragmas
because of legacy reasons.
Code after the change is the same as JDK 9 code.

Regards,
Andreas


Reply via email to