RFR (XS): 8144881: Various fixes to linux/sparc

Mikael Vidstedt mikael.vidstedt at oracle.com
Tue Dec 8 02:34:58 UTC 2015


Builds returned successfully. Let me know how you like the new webrev!

Cheers,
Mikael

> On Dec 7, 2015, at 15:16, Mikael Vidstedt <mikael.vidstedt at oracle.com> wrote:
> 
> 
> Good control question! Since these are build problems, and since building on other platforms works today, making corresponding changes to files for other platforms is not strictly necessary, but for consistency it's of course reasonable to do so anyway. Here's an updated webrev doing exactly that:
> 
> http://cr.openjdk.java.net/~mikael/webrevs/8144881/webrev.01/webrev/
> 
> Changes since webrev.00:
> 
> * Copyright year updated (*grumble*)
> * Changes to print sigflags in check_signal_handler in os_linux.cpp were also applied to the other os_*.cpp files (where applicable)
> * code/codeCache.hpp included in all os_cpu/vm/os_<os>_<cpu>.cpp files (where applicable)
> * Added return statement to other JVM_handle_*_signal (where applicable) - these functions could *really* use some additional cleanup/unification...
> * Also: I noticed that the rest of the code in os_linux_sparc.cpp uses "sigcontext", as opposed to "struct sigcontext", so I dropped the "struct"
> 
> Note: I'll have to build this again to make sure it still builds correctly across all the platforms, so consider this an optimistic webrev+review.
> 
> Cheers,
> Mikael
> 
> 
>> On 2015-12-07 14:19, Dmitry Samersoff wrote:
>> Mikael,
>> 
>> The changes looks good for me, but
>> 
>> Most of changes don't look as sparc-specific.
>> 
>> Do you need to update other platforms as well?
>> 
>> -Dmitry
>> 
>>> On 2015-12-08 00:53, Mikael Vidstedt wrote:
>>> Please review this small change which fixes a handful of places which
>>> currently lead to build failures on linux/sparc. The changes are all
>>> relatively straightforward and come in a few different flavors:
>>> 
>>> * Incorrect format specifiers
>>> * "reaches end of function returning value"
>>> * wrong type name (sigcontext_t -> struct sigcontext)
>>> * wrong variable name (addr -> pc)
>>> 
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8144881
>>> Webrev:
>>> http://cr.openjdk.java.net/~mikael/webrevs/8144881/webrev.00/webrev/
>>> 
>>> Cheers,
>>> Mikael
> 


More information about the ppc-aix-port-dev mailing list