RFR: 8237521: Memory Access API fixes for 32-bit

[Nick Gasson]

Hi Maurizio,

Yes, for jdk/jdk. Do I need another Reviewer?


Thanks,
Nick


On 01/22/20 20:46 pm, Maurizio Cimadamore wrote:
> Fixes look good to me - I assume this is for JDK 15, right?
>
> Maurizio
>
> On 22/01/2020 08:43, Nick Gasson wrote:
>> Hi,
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8237521
>> Webrev: http://cr.openjdk.java.net/~ngasson/8237521/webrev.01/
>>
>> This is a follow-up to JDK-8236634 which just contained changes to the 
>> tests to make them build/pass on 32-bit systems for JDK 14. This patch 
>> removes the test workarounds and fixes two things to make them pass:
>>
>> * In Unsafe_AllocateMemory0 handle the case where the aligned-up size 
>> overflows a 32-bit size_t and throw an OutOfMemoryError instead of 
>> calling os::malloc(0).
>>
>> * Change the value of Utils.MAX_ALIGN from 16 to 8 on 32-bit systems, 
>> as malloc can return addresses that are only 8-byte aligned.
>>
>> Tested jdk_foreign on x86_32, arm32, aarch64, x86_64.
>>
>>
>> Thanks,
>> Nick