Andreas, Looks good for me!
-Dmitry On 2016-04-12 19:05, Andreas Eriksson wrote: > Hi, > > Please review this fix for 8153641: assert(thread_state == > _thread_in_native) failed: Assumed thread_in_native while heap dump > > Bug: https://bugs.openjdk.java.net/browse/JDK-8153641 > Webrev: http://cr.openjdk.java.net/~aeriksso/8153641/webrev.00/ > > The crash is because changes I did in JDK-8129419 > <https://bugs.openjdk.java.net/browse/JDK-8129419> do not work with > solaris logic used for UseVMInterruptibleIO in some cases. > This change reverts the changes I did to the solaris version of > os::write, and changes heapDumper.cpp to use ::write and ::close > directly, as it did before. > > Thanks, > Andreas -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the sources.