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

[Mikael Vidstedt]

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
>>
>