Re: RFR(S): JDK-8165496 assert(_exception_caught == false) failed: _exception_caught is out of phase

[serguei.spit...@oracle.com]

Hi Dmitry,

Sorry, I do not see how this fixes the problem.
What are you trying to solve by calling the set_exception_detected() 
conditionally?

The _exception_detected flag at that point has to be set anyway, right?

The root cause of this issue is that the assert is unreasonable and does 
not solve anything.
So that the assert has to be replaced with cleaning the 
_exception_caught flag.

Please, read my comment in the bug report.
I also thought that you were agree with this conclusion. :)

Thanks,
Serguei


On 10/21/16 01:42, Dmitry Samersoff wrote:

Everybody,

Please review a small modification of the fix for JDK-8134434:

http://cr.openjdk.java.net/~dsamersoff/JDK-8165496/webrev.04/

Its' possible that we come to rethrow_C when _exception_caught is
already cleared. We need not to set exception_detected in this
case.

-Dmitry

RFR(S): JDK-8165496 assert(_exception_caught == false) failed: _exception_caught is out of phase

[Dmitry Samersoff]

Everybody,

Please review a small modification of the fix for JDK-8134434:

http://cr.openjdk.java.net/~dsamersoff/JDK-8165496/webrev.04/

Its' possible that we come to rethrow_C when _exception_caught is
already cleared. We need not to set exception_detected in this
case.

-Dmitry

-- 
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.