RFR(S): 8229254: solaris_x64 build fails

David Holmes david.holmes at oracle.com
Mon Aug 12 02:47:17 UTC 2019


Hi Boris,

On 9/08/2019 11:14 pm, Boris Ulasevich wrote:
> Hi,
> 
> Please review the following simple change to make solaris_x64 build 
> compilable.
> 
> Syntax error was brought by JDK-8191278 [1]. The fix is evident.

Yes looks good.

> Missing constant issue occurred after JDK-8226238 [2]. I copied 
> workaround for the case of missing EM_AARCH64 constant in system elf.h 
> header from os_linux.cpp implementation.

I'm really not sure why we keep expanding a Solaris specific structure 
with entries for architectures the OS does not run on ...

That aside the fix seems fine - the value is somewhat arbitrary but 
doesn't conflict with anything AFAICS.

Thanks,
David
-----

> http://cr.openjdk.java.net/~bulasevich/8229254/webrev.00
> http://bugs.openjdk.java.net/browse/JDK-8229254
> 
> Testing done: tier1,tier2.
> 
> thanks,
> Boris
> 
> [1] MappedByteBuffer bulk access memory failures are not handled gracefully
> 
> http://mail.openjdk.java.net/pipermail/hotspot-dev/2019-June/038352.html
> [2] Improve error output and fix elf issues in os::dll_load
> 
> http://mail.openjdk.java.net/pipermail/hotspot-dev/2019-June/038597.html


More information about the hotspot-dev mailing list