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