RFR: 8061467: Bad page size passed to setup_large_pages() on Solaris

Thomas Schatzl thomas.schatzl at oracle.com
Tue Nov 4 12:09:37 UTC 2014


Hi Erik,

On Tue, 2014-10-28 at 19:10 +0100, Erik Helin wrote:
> Hi all,
> 
> this small patch fixes a call to Solaris::setup_large_pages that does 
> not check if the `alignment_hint` parameter is a valid page size (the 
> only such call left).
> 
> Webrev:
> http://cr.openjdk.java.net/~ehelin/8061467/webrev.00/
> 
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8061467
> 
> Testing:
> - Verified that the fix solves the issue on a Solaris machine that
>    supported NUMA.
> - JPRT

  looks good. Could you add a simple regression test like the one
suggested in the CR?

Thanks,
  Thomas




More information about the hotspot-gc-dev mailing list