On Wed, 6 Nov 2024 06:55:41 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> SendaoYan has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   1. change os-arch to linux-all; 2. delete 
>> jdk/jfr/event/gc/detailed/TestZPageAllocationEvent.java from Problemlist
>
> test/jdk/ProblemList.txt line 779:
> 
>> 777: java/lang/Thread/jni/AttachCurrentThread/AttachTest.java#id0    8343244 
>> linux-aarch64,linux-ppc64le
>> 778: java/lang/Thread/jni/AttachCurrentThread/AttachTest.java#id1    8343244 
>> linux-aarch64,linux-ppc64le
>> 779: jdk/jfr/event/gc/detailed/TestZPageAllocationEvent.java         8343244 
>> linux-aarch64,linux-ppc64le
> 
> Also IIUC the JFR test only failed with non-generational ZGC which has now 
> been removed.

The test of `jdk/jfr/event/gc/detailed/TestZPageAllocationEvent.java` was 
removed ZGC option `-XX:-ZGenerational` and `-XX:+ZGenerational`. I have no 
idea that the crash of 
[JDK-8343244](https://bugs.openjdk.org/browse/JDK-8343244) was trigger by 
`-XX:+UseZGC -XX:-ZGenerational` or just by `-XX:+UseZGC`.
Anyway, remove `jdk/jfr/event/gc/detailed/TestZPageAllocationEvent.java` from 
this Problemlist maybe a good option.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/21917#discussion_r1830546518

Reply via email to