RFR (S) 8202705: ARM32 build crashes on long JavaThread offsets

Aleksey Shipilev shade at redhat.com
Mon Jun 4 12:10:28 UTC 2018


On 06/04/2018 01:58 PM, Boris Ulasevich wrote:
> Hello all,
> 
> Please review this patch to allow ARM32 MacroAssembler to handle updated JavaThread offsets:
>   http://cr.openjdk.java.net/~bulasevich/8202705/webrev.01/
>   https://bugs.openjdk.java.net/browse/JDK-8202705

Looks okay, but Rthread becomes misnomer in the middle of the method.

Maybe like this?

 // Borrow the Rthread for alloc counter
 Register Ralloc = Rthread;
 Rthread = NULL;
 add(Ralloc, Ralloc, in_bytes(JavaThread::allocated_bytes_offset());

 ...
 <work with Ralloc>
 ...

 // Unborrow the Rthread
 sub(Ralloc, Ralloc, in_bytes(JavaThread::allocated_bytes_offset()
 Rthread = Ralloc;
 Ralloc = NULL;

-Aleksey

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20180604/e28f9cf0/signature.asc>


More information about the hotspot-gc-dev mailing list