RFR: JDK-8213199: GC abstraction for Assembler::needs_explicit_null_check()

Andrew Dinn adinn at redhat.com
Wed Nov 7 16:34:19 UTC 2018


Hi Roman,

On 07/11/18 15:23, Roman Kennke wrote:
> Hi Erik,
> 
>> The si_addr is always void*, so the cast to void* is redundant.
>> Otherwise, looks good.
>> Don't need to see another webrev for that.
> 
> Right. Thanks again for reviewing and helping.
> 
> I'll push it through jdk/submit while waiting for another review. Andrew?
Just one issue regarding the comment in assembler.cpp

 319     // The first page after heap_base is unmapped and
 320     // the 'offset' is equal to [heap_base + offset] for
 321     // narrow oop implicit null checks.

That's not the general case. It's only potentially true for narrow_oops
read out of fields when Matcher::narrow_oop_use_complex_address returns
true.

I would prefer for these specifics to be made clear as understanding it
and tracking it down was not straightforward. So, how about

    // A SEGV can legitimately happen in C2 code at address
    // (heap_base + offset) if  Matcher::narrow_oop_use_complex_address
    // is configured to allow narrow oops field loads to be implicitly
    // null checked

Otherwise all good.

regards,


Andrew Dinn
-----------



More information about the hotspot-dev mailing list