RFR (XS): JDK-8146093 [sparc only] compiler/interpreter/7116216/StackOverflow.java Program terminates with signal 11, Segmentation fault. in __1cLRegisterMap2t6MpnKJavaThread_b_v_ ()

[Daniel D. Daugherty]

On 4/4/16 2:31 AM, Frederic Parain wrote:
> Greetings,
>
> Please review this small fix in the stack overflow check
> for the SPARC platform. The check was not updated when
> JEP-270 has been pushed. The one line fix in file
> templateInterpreterGenerator_sparc.cpp adjusts the check
> to the new execution stack layout introduced by JEP-270.
>
> Changes in interpreterRuntime.[ch]pp files are just
> a piggybacked clean up to remove a dead method from the
> JEP-270 push.
>
> Webrev:
> http://cr.openjdk.java.net/~fparain/8146093/webrev.00/

src/share/vm/interpreter/interpreterRuntime.hpp
     No comments.

src/share/vm/interpreter/interpreterRuntime.cpp
     No comments; thanks for fixing the bad indent on next function.

src/cpu/sparc/vm/templateInterpreterGenerator_sparc.cpp
     No comments.

Thumbs up.

Dan


>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8146093
>
> The bug also triggered another type of failure which
> has been assigned to a different CR:
> https://bugs.openjdk.java.net/browse/JDK-8152675
>
> The fix has been tested against the two failures and
> resolves both of them.
>
> Testing:
>   JPRT with hotspot testset
>   manual testing of the two tests indicated in CRs,
>   with stress testing of several dozens hours each
>   (stack overflow bugs are often hard to reproduce)
>
> Thanks,
>
> Fred
>
>