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

[Roland Westrelin]

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