RFR: JDK-8176054 - [BACKOUT][REDO] G1 Needs pre barrier on dereference of weak JNI handles

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR: JDK-8176054 - [BACKOUT][REDO] G1 Needs pre barrier on dereference of weak JNI handles

jesper.wilhelmsson
Hi,

Can I please have a couple of reviews for this backout of JDK-8175085?

Bug: https://bugs.openjdk.java.net/browse/JDK-8176054

Webrev: http://cr.openjdk.java.net/~jwilhelm/8176054/webrev.00/

Thanks,
/Jesper


Reply | Threaded
Open this post in threaded view
|

Re: RFR: JDK-8176054 - [BACKOUT][REDO] G1 Needs pre barrier on dereference of weak JNI handles

Kim Barrett
> On Mar 2, 2017, at 11:51 AM, Jesper Wilhelmsson <[hidden email]> wrote:
>
> Hi,
>
> Can I please have a couple of reviews for this backout of JDK-8175085?
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8176054
>
> Webrev: http://cr.openjdk.java.net/~jwilhelm/8176054/webrev.00/
>
> Thanks,
> /Jesper

Looks good.  (sigh)

Reply | Threaded
Open this post in threaded view
|

Re: RFR: JDK-8176054 - [BACKOUT][REDO] G1 Needs pre barrier on dereference of weak JNI handles

jesper.wilhelmsson
Thanks Kim!
/Jesper


On 2017-03-02 22:23, Kim Barrett wrote:

>> On Mar 2, 2017, at 11:51 AM, Jesper Wilhelmsson <[hidden email]> wrote:
>>
>> Hi,
>>
>> Can I please have a couple of reviews for this backout of JDK-8175085?
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8176054
>>
>> Webrev: http://cr.openjdk.java.net/~jwilhelm/8176054/webrev.00/
>>
>> Thanks,
>> /Jesper
> Looks good.  (sigh)
>

Reply | Threaded
Open this post in threaded view
|

Re: RFR: JDK-8176054 - [BACKOUT][REDO] G1 Needs pre barrier on dereference of weak JNI handles

Mikael Gerdin
In reply to this post by Kim Barrett
Third time's the charm! Stay tuned for next week's episode of weak jni
handles...

/Mikael


On March 2, 2017 22:23:43 Kim Barrett <[hidden email]> wrote:

>> On Mar 2, 2017, at 11:51 AM, Jesper Wilhelmsson
>> <[hidden email]> wrote:
>>
>> Hi,
>>
>> Can I please have a couple of reviews for this backout of JDK-8175085?
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8176054
>>
>> Webrev: http://cr.openjdk.java.net/~jwilhelm/8176054/webrev.00/
>>
>> Thanks,
>> /Jesper
>
> Looks good.  (sigh)
>