RFR(L): 8032410: compiler/uncommontrap/TestStackBangRbp.java times out on Solaris-Sparc V9

[Lindenmaier, Goetz]

Hi Roland, 

The change in deoptimization.cpp does not reflect what I fixed
in the patch I sent to you. 
You need to pass index==0 for test of top_frame.

Best regards,
  Goetz.


-----Original Message-----
From: hotspot-compiler-dev [mailto:hotspot-compiler-dev-bounces at openjdk.java.net] On Behalf Of Roland Westrelin
Sent: Monday, March 17, 2014 7:24 PM
To: Vladimir Kozlov
Cc: hotspot compiler; ppc-aix-port-dev at openjdk.java.net
Subject: Re: RFR(L): 8032410: compiler/uncommontrap/TestStackBangRbp.java times out on Solaris-Sparc V9

> Mikael matched code in interpreter in AbstractInterpreterGenerator::bang_stack_shadow_pages(). This way we consistently touching the same number of pages.
> 
> What I want to avoid is 8026775 situation when a middle page is not touched. My main concern now is that you changed the page where banging *starts* even so the variable is named bang_end:
> 
>  // This is how far the previous frame's stack banging extended.
>  const int bang_end_safe = bang_end;
> 
> But after 8026775 change (<=StackShadowPages) your changes seems right since StackShadowPages*page_size is already touched.
> 
> You may be touching +1 page but as we discussed above with StackYellowPages >=2 it should be fine.

Thanks, Vladimir.

Here is a new webrev:

http://cr.openjdk.java.net/~roland/8032410/webrev.02/

that should take all comments into account. In particular as suggested by Goetz I got rid of the templates. The native wrappers now bang to StackShadowPages+1.

Roland.