RFR: 8255982: Extend BasicJMapTest to test with different GC Heap [v3]
Lin Zang
lzang at openjdk.java.net
Wed Nov 11 11:24:08 UTC 2020
On Wed, 11 Nov 2020 11:13:29 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Lin Zang has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Refine the test configuration.
>
> I think it is fine to omit Epsilon from this testing. There is another trick you can use: `@test id=serial` would name the tests appropriately, not just "id#":
>
> Running test 'jtreg:test/jdk/sun/tools/jmap/BasicJMapTest.java'
> Passed: sun/tools/jmap/BasicJMapTest.java#parallel
> Passed: sun/tools/jmap/BasicJMapTest.java#serial
> Passed: sun/tools/jmap/BasicJMapTest.java#z
> Passed: sun/tools/jmap/BasicJMapTest.java#g1
> Passed: sun/tools/jmap/BasicJMapTest.java#shenandoah
> Test results: passed: 5
>
> I also tested this patch with x86_32 that implicitly disables Z, and it passes. Also passes with TEST_VM_OPTS="-XX:+UseSerialGC", with only one config automatically selected.
>
> @iignatev might want to ack this, as our resident test overseer.
@shipilev Thanks for guidance! Added the id tag. -Lin
-------------
PR: https://git.openjdk.java.net/jdk/pull/1094
More information about the serviceability-dev
mailing list