
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>

</head>

<body dir="ltr">

<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;">

<p>To be honest, I'm not sure why Unsafe_getObject handles the G1 barrier for the referent field. I'd be happier claiming that "referent" is the JVM's field, don't mess with it.</p>

<p><br>

</p>

<p>BUT, <span>getObjectVolatile() is documented as being just like getObject(), + volatile. So I think there's not much choice.</span></p>

<p><span><br>

</span></p>

<p><span> - Derek</span><br>

</p>

<br>

<br>

<div style="color: rgb(0, 0, 0);">

<div>

<hr tabindex="-1" style="display:inline-block; width:98%">

<div id="x_divRplyFwdMsg" dir="ltr"><font style="font-size:11pt" face="Calibri, sans-serif" color="#000000"><b>From:</b> hotspot-runtime-dev <hotspot-runtime-dev-bounces@openjdk.java.net> on behalf of Doerr, Martin <martin.doerr@sap.com><br>

<b>Sent:</b> Friday, September 2, 2016 10:40 AM<br>

<b>To:</b> David Holmes; hotspot-runtime-dev@openjdk.java.net; hotspot-gc-dev@openjdk.java.net<br>

<b>Subject:</b> RE: RFR(XXS): 8165018: Missing memory barrier for PPC64 in Unsafe_GetObjectVolatile</font>

<div> </div>

</div>

</div>

<font size="2"><span style="font-size:10pt;">

<div class="PlainText">Hi,<br>

<br>

first of all, thanks, David, for the review and for sponsoring.<br>

<br>

I'm not sure if it's a bug that Unsafe_GetObjectVolatile does not perform the G1 barrier.<br>

One could argue that users of Unsafe should know what they are doing and reading the referent field by Unsafe_GetObjectVolatile may be unsupported.<br>

On the other hand, if somebody tries to do this, G1 may unexpectedly remove the java object to which the reference was returned by Unsafe_GetObjectVolatile.<br>

No clue if there's anybody out there who will ever run into such a bad situation.<br>

<br>

Thanks and best regards,<br>

Martin<br>

<br>

<br>

-----Original Message-----<br>

From: David Holmes [<a href="mailto:david.holmes@oracle.com">mailto:david.holmes@oracle.com</a>]

<br>

Sent: Freitag, 2. September 2016 07:04<br>

To: Doerr, Martin <martin.doerr@sap.com>; hotspot-runtime-dev@openjdk.java.net; hotspot-gc-dev@openjdk.java.net<br>

Subject: Re: RFR(XXS): 8165018: Missing memory barrier for PPC64 in Unsafe_GetObjectVolatile<br>

<br>

Hi Martin,<br>

<br>

Adding GC folk<br>

<br>

On 30/08/2016 8:51 PM, Doerr, Martin wrote:<br>

> Hi,<br>

><br>

> we found that a memory barrier for PPC64 is missing in the current Unsafe implementation. get_volatile already contains the memory barrier for "support_IRIW_for_not_multiple_copy_atomic_cpu". The same is needed in Unsafe_GetObjectVolatile.<br>

><br>

> Here's my webrev:<br>

> <a href="http://cr.openjdk.java.net/~mdoerr/8165018_UnsafePPC64/webrev.00/">http://cr.openjdk.java.net/~mdoerr/8165018_UnsafePPC64/webrev.00/</a><br>

<br>

That looks fine to me.<br>

<br>

> And while looking at it I wonder why Unsafe_GetObjectVolatile does not contain a G1 barrier like Unsafe_GetObject. Is it not possible to use the Volatile version to access the referent field of a Reference?<br>

<br>

That looks like a bug to me. :)<br>

<br>

> Please review. As it is shared code, I will need a sponsor, please.<br>

<br>

I can sponsor.<br>

<br>

Thanks,<br>

David<br>

<br>

> Best regards,<br>

> Martin<br>

><br>

</div>

</span></font></div>

</div>

</body>

</html>