Re: RFR: 8274523: java/lang/management/MemoryMXBean/MemoryTest.java test should handle Shenandoah

2021-09-30 Thread Aleksey Shipilev
On Wed, 29 Sep 2021 17:56:00 GMT, Aleksey Shipilev  wrote:

> Currently it fails with:
> 
> 
> $ CONF=linux-x86_64-server-fastdebug make run-test 
> TEST=java/lang/management/MemoryMXBean/MemoryTest.java
> 
> STDERR:
> java.lang.RuntimeException: TEST FAILED: Number of heap pools = 1 but 
> expected <= 3 and >= 3
> 
> 
> Z already handles it with a special configuration, Shenandoah should do the 
> same. 
> 
> Additional testing:
>  - [x] Affected test now works for Shenandoah
>  - [x] Affected test still works for Z
>  - [x] Affected test still works for G1

Thanks!

-

PR: https://git.openjdk.java.net/jdk/pull/5758


Re: RFR: 8274523: java/lang/management/MemoryMXBean/MemoryTest.java test should handle Shenandoah

2021-09-29 Thread Chris Plummer
On Wed, 29 Sep 2021 17:56:00 GMT, Aleksey Shipilev  wrote:

> Currently it fails with:
> 
> 
> $ CONF=linux-x86_64-server-fastdebug make run-test 
> TEST=java/lang/management/MemoryMXBean/MemoryTest.java
> 
> STDERR:
> java.lang.RuntimeException: TEST FAILED: Number of heap pools = 1 but 
> expected <= 3 and >= 3
> 
> 
> Z already handles it with a special configuration, Shenandoah should do the 
> same. 
> 
> Additional testing:
>  - [x] Affected test now works for Shenandoah
>  - [x] Affected test still works for Z
>  - [x] Affected test still works for G1

Marked as reviewed by cjplummer (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/5758


Re: RFR: 8274523: java/lang/management/MemoryMXBean/MemoryTest.java test should handle Shenandoah

2021-09-29 Thread Mandy Chung
On Wed, 29 Sep 2021 17:56:00 GMT, Aleksey Shipilev  wrote:

> Currently it fails with:
> 
> 
> $ CONF=linux-x86_64-server-fastdebug make run-test 
> TEST=java/lang/management/MemoryMXBean/MemoryTest.java
> 
> STDERR:
> java.lang.RuntimeException: TEST FAILED: Number of heap pools = 1 but 
> expected <= 3 and >= 3
> 
> 
> Z already handles it with a special configuration, Shenandoah should do the 
> same. 
> 
> Additional testing:
>  - [x] Affected test now works for Shenandoah
>  - [x] Affected test still works for Z
>  - [x] Affected test still works for G1

Marked as reviewed by mchung (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/5758