RFR: 8227068: [Graal] MappedByteBuffer bulk access memory failures are not handled gracefully
Jamsheed
jamsheed.c.m at oracle.com
Fri Jul 12 14:26:22 UTC 2019
Hi Vladimir,
upstream Graal PR is approved and is merged to master (links in jbs). i
have applied the same patch on jdk-jdk
webrev: http://cr.openjdk.java.net/~jcm/8227068/webrev.01/
request for review.
Best regards,
Jamsheed
On 09/07/19 9:40 PM, Vladimir Kozlov wrote:
> Looks good. But I think you should go through upstream Graal PR first.
> You can push into jdk/jdk simultaneously but changes should match to
> upstream changes.
>
> thanks,
> Vladimir
>
> On 7/9/19 8:29 AM, Jamsheed wrote:
>> Hi,
>>
>> Request for review
>>
>> JDK-8191278 guarded c2 compiler Unsafe.copyMemory intrinsic for
>> invalid memory access failures. this was not done for graal compiler
>> intrinsic.
>>
>> this patch handles graal case.
>>
>> jbs: https://bugs.openjdk.java.net/browse/JDK-8227068
>>
>> webrev: http://cr.openjdk.java.net/~jcm/8227068/webrev.00/
>>
>> tested: graal unittests, mach5 (1-5, tests running)
>>
>> aot failing test: manually verified.
>>
>> Best regards,
>>
>> Jamsheed
>>
More information about the hotspot-compiler-dev
mailing list