RFR(S): 8071822: [TEST_BUG] Adapt collectorPolicy internal tests to support 64K pages

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Mon Feb 16 16:33:29 UTC 2015

Hi Jon,

thanks for looking at this change!

It's not that easy as  just aligning to vm_page_size, but I can use

Best regards,

From: hotspot-gc-dev [mailto:hotspot-gc-dev-bounces at openjdk.java.net] On Behalf Of Jon Masamitsu
Sent: Donnerstag, 12. Februar 2015 00:35
To: hotspot-gc-dev at openjdk.java.net
Subject: Re: RFR(S): 8071822: [TEST_BUG] Adapt collectorPolicy internal tests to support 64K pages


Could you have used page alignment to determine the
new expected sizes?

For example, instead of

 987     // Calculate what we expect the flag to be.

 988     flag_value = adapted_InitialHeapSize - MaxNewSize;

 989     verify_old_initial(flag_value);
expected_old_initial = align_size_up(flag_value, os::vm_page_size())

On 2/10/2015 7:43 AM, Lindenmaier, Goetz wrote:

could someone please have a look at this change?
It's really not that big.

GC-folks maybe?


From: Lindenmaier, Goetz
Sent: Freitag, 30. Januar 2015 10:36
To: hotspot-dev Source Developers
Subject: RFR(S): 8071822: [TEST_BUG] Adapt collectorPolicy internal tests to support 64K pages


this fixes a final problem with 64-bit page size in the jtreg tests.
It addresses the internal VM tests of the collector policies.

Those tests set a row of fixed flag values, and one special value to test.
Then they call the heap ergonomics and check the expected result of the
special value.

With 64K page size, the heap ergonomics adapt more values, breaking the tests.
Wrt. the adapted flag values the tests are correct though.

This change makes the expected values depend on the adapted flags, and the
tests pass. They only depend on adapted flags that are not subject of the
corresponding test.

Please review this test.  I please need a sponsor.


Best regards,

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/hotspot-gc-dev/attachments/20150216/92aec452/attachment.htm>

More information about the hotspot-gc-dev mailing list