RFR: 8061467: Bad page size passed to setup_large_pages() on Solaris
erik.helin at oracle.com
Thu Nov 6 13:51:50 UTC 2014
thanks for handling this!
On 2014-11-05 16:14, Thomas Schatzl wrote:
> Hi all,
> since Erik is on vacation and this bug prevents a lot of tests from
> running, I took over and added a (simple) regression test case.
> jprt, Solaris numa machine showing the crash before, and no crash after
> the change, local testing, jprt test case runs
> Since the test is extremely simple (class with empty main method), I am
> going to keep Erik as the original author of this change as his change
> is the one that required the hard work.
> On Tue, 2014-11-04 at 13:09 +0100, Thomas Schatzl wrote:
>> 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).
>>> - 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?
More information about the hotspot-gc-dev