[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-hotspot-runtime-dev
Subject:    Re: RFR 8234613: JavaThread can escape back to Java from an ongoing handshake
From:       Patricio Chilano <patricio.chilano.mateo () oracle ! com>
Date:       2019-11-26 12:47:00
Message-ID: 1bb5086d-5467-b1f7-b2f1-ea73445b6a1c () oracle ! com
[Download RAW message or body]

Thanks Robbin!

Patricio
On 11/26/19 6:01 AM, Robbin Ehn wrote:
> Thanks Patricio!
> 
> /Robbin
> 
> On 11/25/19 7:58 PM, Patricio Chilano wrote:
> > Hi Robbin,
> > 
> > On 11/25/19 4:14 AM, Robbin Ehn wrote:
> > > Hi Patricio,
> > > 
> > > On 2019-11-22 19:25, Patricio Chilano wrote:
> > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8234613
> > > > Webrev: http://cr.openjdk.java.net/~pchilanomate/8234613/v01/webrev/
> > > 
> > > Thanks, I think this is good and easy to backport!
> > > You might as well add native to the assert.
> > Added!
> > 
> > > We should revisit this when we have time.
> > > There are two polls and four transition in this code, which is more 
> > > complicated
> > > than I like.
> > Agree, I think it could be simplified.
> > 
> > Here is v2:
> > http://cr.openjdk.java.net/~pchilanomate/8234613/v02/webrev/ 
> > <http://cr.openjdk.java.net/~pchilanomate/8234613/v02/webrev/src/hotspot/share/runtime/handshake.cpp.udiff.html>
> >  
> > Thanks for looking at this Robbin!
> > 
> > Patricio
> > > /Robbin
> > > 
> > > > 
> > > > Thanks,
> > > > Patricio
> > 


[prev in list] [next in list] [prev in thread] [next in thread]