RFR: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed

2024-01-11 Thread sendaoYan
Reviewed-by: Yi Yang - Commit messages: - 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed Changes: https://git.openjdk.org/jdk/pull/17386/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=17386

Re: RFR: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed

2024-01-12 Thread sendaoYan
On Fri, 12 Jan 2024 03:31:37 GMT, sendaoYan wrote: > Reviewed-by: Yi Yang The test case before this PR has a maximum heap of 64MB and applies for 8M of memory each time in the for loop. When applying for memory for the sixth time, it was killed by the docker container because of

Re: RFR: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed [v2]

2024-01-22 Thread sendaoYan
> 8323640: [TESTBUG]testMemoryFailCount in > jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because > OOM killed sendaoYan has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compar

Withdrawn: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed

2024-01-22 Thread sendaoYan
On Fri, 12 Jan 2024 03:31:37 GMT, sendaoYan wrote: > 8323640: [TESTBUG]testMemoryFailCount in > jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because > OOM killed This pull request has been closed without being integrated. - PR: https://git.op

RFR: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed

2024-01-22 Thread sendaoYan
8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed - Commit messages: - 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed

Re: RFR: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed

2024-01-22 Thread sendaoYan
On Mon, 22 Jan 2024 09:31:43 GMT, sendaoYan wrote: > 8323640: [TESTBUG]testMemoryFailCount in > jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because > OOM killed The test case before this PR has a maximum heap of 64MB and applies for 8M of memory each time i

Re: RFR: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed

2024-01-22 Thread sendaoYan
On Mon, 22 Jan 2024 15:03:18 GMT, Severin Gehwolf wrote: > `1k` increments for a total of `512k` times seems overkill. Are you sure > that's needed to make the test pass? How about `1MB` increments for a total > of `512` times? When the docker serivice work normally on the test machine, this t

Re: RFR: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed [v2]

2024-01-22 Thread sendaoYan
> 8323640: [TESTBUG]testMemoryFailCount in > jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because > OOM killed sendaoYan has updated the pull request incrementally with one additional commit since the last revision: 8323640: [TESTBUG]testMemoryFailCoun

Re: RFR: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed [v3]

2024-01-23 Thread sendaoYan
> 8323640: [TESTBUG]testMemoryFailCount in > jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because > OOM killed sendaoYan has updated the pull request incrementally with one additional commit since the last revision: 8323640: [TESTBUG]testMemoryFailCoun

Re: RFR: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed [v3]

2024-01-23 Thread sendaoYan
On Tue, 23 Jan 2024 13:04:43 GMT, sendaoYan wrote: >> 8323640: [TESTBUG]testMemoryFailCount in >> jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail >> because OOM killed > > sendaoYan has updated the pull request incrementally with one additional

Integrated: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed

2024-01-23 Thread sendaoYan
On Mon, 22 Jan 2024 09:31:43 GMT, sendaoYan wrote: > 8323640: [TESTBUG]testMemoryFailCount in > jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because > OOM killed This pull request has now been integrated. Changeset: 791b427f Author:sendaoYan Committer

Re: RFR: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed [v3]

2024-04-08 Thread SendaoYan
On Tue, 23 Jan 2024 13:04:43 GMT, SendaoYan wrote: >> 8323640: [TESTBUG]testMemoryFailCount in >> jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail >> because OOM killed > > SendaoYan has updated the pull request incrementally with one additional

RFR: 8331466: Problemlist serviceability/dcmd/gc/RunFinalizationTest.java on macos-aarch64

2024-05-01 Thread SendaoYan
Hi, GHA [runner](https://github.com/sendaoYan/jdk-ysd/actions/runs/8881868940/job/24386063136) shows that serviceability/dcmd/gc/RunFinalizationTest.java intermittent fail on macos-aarch64. This testcase should be problemlist. Only change the ProblemList, no risk. - Commit

Re: RFR: 8331466: Problemlist serviceability/dcmd/gc/RunFinalizationTest.java on macos-aarch64

2024-05-05 Thread SendaoYan
On Sat, 4 May 2024 10:22:06 GMT, Serguei Spitsyn wrote: > Looks okay to me. @sspitsyn Thanks. - PR Comment: https://git.openjdk.org/jdk/pull/19033#issuecomment-2094774327

Re: RFR: 8331466: Problemlist serviceability/dcmd/gc/RunFinalizationTest.java on macos-aarch64 [v2]

2024-05-07 Thread SendaoYan
> Hi, > GHA > [runner](https://github.com/sendaoYan/jdk-ysd/actions/runs/8881868940/job/24386063136) > shows that serviceability/dcmd/gc/RunFinalizationTest.java intermittent fail > on macos-aarch64. The testcase has been problemlisted on > linux-all,windows-x64,aix-ppc64.

Re: RFR: 8331466: Problemlist serviceability/dcmd/gc/RunFinalizationTest.java on macos-aarch64 [v2]

2024-05-07 Thread SendaoYan
On Tue, 7 May 2024 17:43:02 GMT, Chris Plummer wrote: >> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> problemlist serviceability/dcmd/gc/RunFinalizationTest.java on generic-all > > test/hots

Re: RFR: 8331466: Problemlist serviceability/dcmd/gc/RunFinalizationTest.java on macos-aarch64 [v2]

2024-05-08 Thread SendaoYan
On Wed, 8 May 2024 07:37:52 GMT, Serguei Spitsyn wrote: >> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> problemlist serviceability/dcmd/gc/RunFinalizationTest.java on generic-all > > Marked as

Re: RFR: 8331466: Problemlist serviceability/dcmd/gc/RunFinalizationTest.java on macos-aarch64 [v2]

2024-05-08 Thread SendaoYan
On Wed, 8 May 2024 16:39:33 GMT, Chris Plummer wrote: >> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> problemlist serviceability/dcmd/gc/RunFinalizationTest.java on generic-all > > Marked as r

Re: RFR: 8331466: Problemlist serviceability/dcmd/gc/RunFinalizationTest.java on generic-all [v2]

2024-05-09 Thread SendaoYan
On Thu, 9 May 2024 21:26:07 GMT, Daniel D. Daugherty wrote: > I've updated the bug's sysnopsis: > > s/macos-aarch64/generic-all/ > > The fastest way to update the PR's title is with `/issue JDK-8331466` @dcubed-ojdk Thanks. - PR Comment: https://git.openjdk.org/jdk/pull/19033#is

Integrated: 8331466: Problemlist serviceability/dcmd/gc/RunFinalizationTest.java on generic-all

2024-05-13 Thread SendaoYan
On Wed, 1 May 2024 14:14:22 GMT, SendaoYan wrote: > Hi, > GHA > [runner](https://github.com/sendaoYan/jdk-ysd/actions/runs/8881868940/job/24386063136) > shows that serviceability/dcmd/gc/RunFinalizationTest.java intermittent fail > on macos-aarch64. The testcase has been p

Re: RFR: 8331466: Problemlist serviceability/dcmd/gc/RunFinalizationTest.java on generic-all [v2]

2024-05-13 Thread SendaoYan
On Wed, 8 May 2024 01:19:05 GMT, SendaoYan wrote: >> Hi, >> GHA >> [runner](https://github.com/sendaoYan/jdk-ysd/actions/runs/8881868940/job/24386063136) >> shows that serviceability/dcmd/gc/RunFinalizationTest.java intermittent >> fail on macos-aarch64. The t

RFR: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count

2024-05-26 Thread SendaoYan
Hi all, ObjectMonitorUsage.java failed with `unexpected waiter_count` after [JDK-8328083](https://bugs.openjdk.org/browse/JDK-8328083) on linux x86_32. It should be predicated with `@requires vm.continuations` to be skipped. Additional testing: - [x] linux x86_32, test has been skiped. - [x] l

Re: RFR: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count

2024-05-26 Thread SendaoYan
On Sun, 26 May 2024 11:26:50 GMT, Alan Bateman wrote: > That would mean it's not tested. I suspect the > java_lang_VirtualThread::is_instance checks will need to be changed to test > with is_a(vmClasses::BaseVirtualThread_klass()) to allow for the alternative > implementation. Do you mean cha

Re: RFR: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count

2024-05-26 Thread SendaoYan
On Sun, 26 May 2024 16:43:50 GMT, Alan Bateman wrote: > > > That would mean it's not tested. I suspect the > > > java_lang_VirtualThread::is_instance checks will need to be changed to > > > test with is_a(vmClasses::BaseVirtualThread_klass()) to allow for the > > > alternative implementation.

Re: RFR: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count [v2]

2024-05-27 Thread SendaoYan
x86_32, test has been skiped. > - [x] linux x86_64, test still work. SendaoYan has updated the pull request incrementally with one additional commit since the last revision: 1. java_lang_VirtualThread::is_instance(thread_oop) -> thread_oop->is_a(vmClasses::BaseVirtualThread_klass

Re: RFR: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count [v3]

2024-05-29 Thread SendaoYan
e this PR, which has been recorded > in [JDK-8333140](https://bugs.openjdk.org/browse/JDK-8333140) > - [ ] linux x86_64 run all testcases in serviceability/jvmti, all testcases > run successed. > > [x64.log](https://github.com/openjdk/jdk/files/15480081/x64.log) > [x86.log](https://

Re: RFR: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count [v2]

2024-05-29 Thread SendaoYan
On Thu, 30 May 2024 00:17:04 GMT, Serguei Spitsyn wrote: >> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 1. java_lang_VirtualThread::is_instance(thread_oop) -> >> thread_oop->is_a(vmCl

Re: RFR: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count [v2]

2024-05-29 Thread SendaoYan
On Thu, 30 May 2024 00:18:14 GMT, Serguei Spitsyn wrote: > The fix looks good in general but I've inlined one suggestion. Thank you for your correction. The code has been updated accordingly your suggest. I need redo the additional testing to verify it. - PR Comment: https://git.o

Re: RFR: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count [v3]

2024-05-29 Thread SendaoYan
On Thu, 30 May 2024 01:13:20 GMT, SendaoYan wrote: >> Hi all, >> ObjectMonitorUsage.java failed with `unexpected waiter_count` after >> [JDK-8328083](https://bugs.openjdk.org/browse/JDK-8328083) on linux x86_32. >> There are two changes in this PR

RFR: 8333353: Delete extra empty line in CodeBlob.java

2024-05-31 Thread SendaoYan
Hi all, This trivial fix, delete the extra empty line before `getOopMaps` function in `src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/code/CodeBlob.java` file. No risk. - Commit messages: - 853: Delete extra empty line in CodeBlob.java Changes: https://git.openjdk.org/

Re: RFR: 8333353: Delete extra empty line in CodeBlob.java

2024-05-31 Thread SendaoYan
On Fri, 31 May 2024 12:29:59 GMT, SendaoYan wrote: > Hi all, > This trivial fix, delete the extra empty line before `getOopMaps` function > in `src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/code/CodeBlob.java` > file. > No risk. The GHA test runner report a failure

Re: RFR: 8333353: Delete extra empty line in CodeBlob.java

2024-05-31 Thread SendaoYan
On Fri, 31 May 2024 16:30:55 GMT, Chris Plummer wrote: > Approved and trivial. Thanks for the review and approved. - PR Comment: https://git.openjdk.org/jdk/pull/19499#issuecomment-2142650762

Re: RFR: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count [v3]

2024-05-31 Thread SendaoYan
On Thu, 30 May 2024 06:36:09 GMT, SendaoYan wrote: >> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> change from java_lang_VirtualThread::is_instance(thread_oop) to >> hread_oop->is_a(vmClass

Integrated: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count

2024-06-01 Thread SendaoYan
On Sun, 26 May 2024 09:27:00 GMT, SendaoYan wrote: > Hi all, > ObjectMonitorUsage.java failed with `unexpected waiter_count` after > [JDK-8328083](https://bugs.openjdk.org/browse/JDK-8328083) on linux x86_32. > There are two changes in this PR: > 1. I

Re: RFR: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count [v3]

2024-06-01 Thread SendaoYan
On Thu, 30 May 2024 01:13:20 GMT, SendaoYan wrote: >> Hi all, >> ObjectMonitorUsage.java failed with `unexpected waiter_count` after >> [JDK-8328083](https://bugs.openjdk.org/browse/JDK-8328083) on linux x86_32. >> There are two changes in this PR

Integrated: 8333353: Delete extra empty line in CodeBlob.java

2024-06-03 Thread SendaoYan
On Fri, 31 May 2024 12:29:59 GMT, SendaoYan wrote: > Hi all, > This trivial fix, delete the extra empty line before `getOopMaps` function > in `src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/code/CodeBlob.java` > file. > No risk. This pull request has now been integra

Re: RFR: 8333353: Delete extra empty line in CodeBlob.java

2024-06-03 Thread SendaoYan
On Fri, 31 May 2024 12:29:59 GMT, SendaoYan wrote: > Hi all, > This trivial fix, delete the extra empty line before `getOopMaps` function > in `src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/code/CodeBlob.java` > file. > No risk. Thanks all for the rev

RFR: 8333477: Delete extra empty spaces in Makefiles

2024-06-04 Thread SendaoYan
Hi all, This PR several extra empty spaces and extra empty lines in several Makefiles. It's trivial fix, no risk. Thanks. - Commit messages: - 8333477: Delete extra empty spaces in Makefiles Changes: https://git.openjdk.org/jdk/pull/19537/files Webrev: https://webrevs.openjdk.

Re: RFR: 8333477: Delete extra empty spaces in Makefiles

2024-06-04 Thread SendaoYan
On Tue, 4 Jun 2024 07:47:46 GMT, SendaoYan wrote: > Hi all, > This PR several extra empty spaces and extra empty lines in several > Makefiles. It's trivial fix, no risk. > > Thanks. > /label build Thanks. - PR Comment: https://git.openjdk.org/jdk/

Re: RFR: 8333477: Delete extra empty spaces in Makefiles

2024-06-04 Thread SendaoYan
On Tue, 4 Jun 2024 07:47:46 GMT, SendaoYan wrote: > Hi all, > This PR several extra empty spaces and extra empty lines in several > Makefiles. It's trivial fix, no risk. > > Thanks. Thanks for the review. Thanks all for the review. - PR Comment: https:/

Re: RFR: 8333477: Delete extra empty spaces in Makefiles [v2]

2024-06-07 Thread SendaoYan
> Hi all, > This PR several extra empty spaces and extra empty lines in several > Makefiles. It's trivial fix, no risk. > > Thanks. SendaoYan has updated the pull request incrementally with one additional commit since the last revision: delete extra empty trailing bla

Re: RFR: 8333477: Delete extra empty spaces in Makefiles [v2]

2024-06-07 Thread SendaoYan
On Thu, 6 Jun 2024 17:49:08 GMT, Chen Liang wrote: >> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> delete extra empty trailing blank line in >> test/jdk/java/rmi/reliability/benchmark/bench/

Re: RFR: 8333477: Delete extra empty spaces in Makefiles [v2]

2024-06-07 Thread SendaoYan
On Fri, 7 Jun 2024 07:29:39 GMT, SendaoYan wrote: >> Hi all, >> This PR several extra empty spaces and extra empty lines in several >> Makefiles. It's trivial fix, no risk. >> >> Thanks. > > SendaoYan has updated the pull request incrementally wit

Re: RFR: 8333477: Delete extra empty spaces in Makefiles [v2]

2024-06-07 Thread SendaoYan
On Fri, 7 Jun 2024 12:53:46 GMT, Chen Liang wrote: >> No, it's an extra newline. A file should end with a newline but one is >> enough. > > As confusing as they are, unfortunately GitHub UI does not render extra > trailing newlines. This is the only one I could find with grepWin. I find the ex

Integrated: 8333477: Delete extra empty spaces in Makefiles

2024-06-07 Thread SendaoYan
On Tue, 4 Jun 2024 07:47:46 GMT, SendaoYan wrote: > Hi all, > This PR several extra empty spaces and extra empty lines in several > Makefiles. It's trivial fix, no risk. > > Thanks. This pull request has now been integrated. Changeset: d130d2f4 Author:SendaoYa

Re: RFR: 8333477: Delete extra empty spaces in Makefiles [v2]

2024-06-07 Thread SendaoYan
On Fri, 7 Jun 2024 07:29:39 GMT, SendaoYan wrote: >> Hi all, >> This PR several extra empty spaces and extra empty lines in several >> Makefiles. It's trivial fix, no risk. >> >> Thanks. > > SendaoYan has updated the pull request incrementally wit

RFR: 8334771: [TESTBUG] Run TestDockerMemoryMetrics.java with -Xcomp fails exitValue = 137

2024-06-24 Thread SendaoYan
Hi all, After [JDK-8294960](https://bugs.openjdk.org/browse/JDK-8294960), the footprint memory usage increased significantly when run the testcase with -Xcomp jvm options, then cause the testcase was killed by docker by OOM. Maybe the footprint memory usage increased was inevitable, so I thin

RFR: 8335137: Disable the SUSPEND_NONE mode of BreakpointOnClassPrepare.java

2024-06-25 Thread SendaoYan
Hi all, The newly added test `com/sun/jdi/BreakpointOnClassPrepare.java` fails at `SUSPEND_NONE` mode. To make this test less noisy, should we disable the SUSPEND_NONE mode for now. - Commit messages: - 8335137: Disable the SUSPEND_NONE mode of BreakpointOnClassPrepare.java Changes

Re: RFR: 8335137: Disable the SUSPEND_NONE mode of BreakpointOnClassPrepare.java

2024-06-26 Thread SendaoYan
On Wed, 26 Jun 2024 06:36:05 GMT, SendaoYan wrote: > Hi all, > The newly added test `com/sun/jdi/BreakpointOnClassPrepare.java` fails at > `SUSPEND_NONE` mode. > To make this test less noisy, should we disable the SUSPEND_NONE mode for now. > @sendaoYan I have a PR out to f

Withdrawn: 8335137: Disable the SUSPEND_NONE mode of BreakpointOnClassPrepare.java

2024-06-26 Thread SendaoYan
On Wed, 26 Jun 2024 06:36:05 GMT, SendaoYan wrote: > Hi all, > The newly added test `com/sun/jdi/BreakpointOnClassPrepare.java` fails at > `SUSPEND_NONE` mode. > To make this test less noisy, should we disable the SUSPEND_NONE mode for now. This pull request has been closed w

Re: RFR: 8323640: [TESTBUG]testMemoryFailCount in jdk/internal/platform/docker/TestDockerMemoryMetrics.java always fail because OOM killed [v3]

2024-07-02 Thread SendaoYan
On Tue, 2 Jul 2024 20:49:05 GMT, Dean Long wrote: > Why does 8M trigger the OOM Killer, but 1M does not? 8M trigger the OOM killer on some environments, maybe there are some test machines that 8M trigger the OOM exception rather than OOM killer. The intention of change `8M chunks per iteration`

RFR: 8336284: Test TestClhsdbJstackLock.java/TestJhsdbJstackLock.java fails with -Xcomp after JDK-8335743

2024-07-11 Thread SendaoYan
Hi all, Test TestClhsdbJstackLock.java/TestJhsdbJstackLock.java fails with -Xcomp after [JDK-8335743](https://bugs.openjdk.org/browse/JDK-8335743). I think the new failures was testcase bug. The change has been verified by -Xmixed/-Xcomp(c2)/-Xcomp -XX:TieredStopAtLevel=1(c1), only change the t

Re: RFR: 8336284: Test TestClhsdbJstackLock.java/TestJhsdbJstackLock.java fails with -Xcomp after JDK-8335743

2024-07-11 Thread SendaoYan
On Fri, 12 Jul 2024 03:24:42 GMT, SendaoYan wrote: > Hi all, > Test TestClhsdbJstackLock.java/TestJhsdbJstackLock.java fails with -Xcomp > after [JDK-8335743](https://bugs.openjdk.org/browse/JDK-8335743). I think the > new failures was testcase bug. > > https://github.co

Integrated: 8336284: Test TestClhsdbJstackLock.java/TestJhsdbJstackLock.java fails with -Xcomp after JDK-8335743

2024-07-12 Thread SendaoYan
On Fri, 12 Jul 2024 03:24:42 GMT, SendaoYan wrote: > Hi all, > Test TestClhsdbJstackLock.java/TestJhsdbJstackLock.java fails with -Xcomp > after [JDK-8335743](https://bugs.openjdk.org/browse/JDK-8335743). I think the > new failures was testcase bug. > > https://github.co

Re: RFR: 8336284: Test TestClhsdbJstackLock.java/TestJhsdbJstackLock.java fails with -Xcomp after JDK-8335743

2024-07-12 Thread SendaoYan
On Fri, 12 Jul 2024 03:24:42 GMT, SendaoYan wrote: > Hi all, > Test TestClhsdbJstackLock.java/TestJhsdbJstackLock.java fails with -Xcomp > after [JDK-8335743](https://bugs.openjdk.org/browse/JDK-8335743). I think the > new failures was testcase bug. > > https://github.co

Re: RFR: 8336284: Test TestClhsdbJstackLock.java/TestJhsdbJstackLock.java fails with -Xcomp after JDK-8335743

2024-07-12 Thread SendaoYan
On Fri, 12 Jul 2024 08:27:47 GMT, Alex Menkov wrote: > > Fix the testcase bug, I think it's not need the 2rd reviewer. > > My bad for hasty sponsorship. General rules are applied to test fixes - 2 > reviewers and at least 24 hours for review. Sorry for hasty integrate, I will pay attention nex

RFR: 8336691: Test LongArgTest.java intermittent fails java.lang.NoClassDefFoundError: jdk/test/lib/Utils

2024-07-17 Thread SendaoYan
Hi all, The testcase `serviceability/attach/LongArgTest.java` intermittent fails `java.lang.NoClassDefFoundError: jdk/test/lib/Utils`. Jtreg doesn't automatically compile `jdk/test/lib/Utils.class` and `jdk/test/lib/apps/LingeredApp.class` etc.. Maybe it's a jtreg framework bug. I think it's ne

Re: RFR: 8334771: [TESTBUG] Run TestDockerMemoryMetrics.java with -Xcomp fails exitValue = 137

2024-07-17 Thread SendaoYan
On Mon, 24 Jun 2024 16:16:29 GMT, SendaoYan wrote: > Hi all, > After [JDK-8294960](https://bugs.openjdk.org/browse/JDK-8294960), the > footprint memory usage increased significantly when run the testcase with > -Xcomp jvm options, then cause the testcase was killed by d

Re: RFR: 8334771: [TESTBUG] Run TestDockerMemoryMetrics.java with -Xcomp fails exitValue = 137

2024-07-17 Thread SendaoYan
On Thu, 18 Jul 2024 05:52:51 GMT, Jaikiran Pai wrote: > So I think we should have this increase in memory reviewed by @asotona or > someone familiar in that area, before deciding whether these tests should be > changed. Okey. - PR Comment: https://git.openjdk.org/jdk/pull/19864#i

Re: RFR: 8336691: Test LongArgTest.java intermittent fails java.lang.NoClassDefFoundError: jdk/test/lib/Utils [v2]

2024-07-18 Thread SendaoYan
On Thu, 18 Jul 2024 04:57:27 GMT, Jaikiran Pai wrote: >> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> delete @build LongArgTest > > test/hotspot/jtreg/serviceability/attach/LongArgTest.java

Re: RFR: 8336691: Test LongArgTest.java intermittent fails java.lang.NoClassDefFoundError: jdk/test/lib/Utils [v2]

2024-07-18 Thread SendaoYan
jtreg framework bug. > I think it's necessory to compile `jdk.test.lib.Utils` and > `jdk.test.lib.apps.LingeredApp` explicitly. > Only change the testcase, the change has been verified, no risk. SendaoYan has updated the pull request incrementally with one additional commit si

Re: RFR: 8334771: [TESTBUG] Run TestDockerMemoryMetrics.java with -Xcomp fails exitValue = 137

2024-07-18 Thread SendaoYan
On Thu, 18 Jul 2024 08:45:43 GMT, Serguei Spitsyn wrote: > Looks okay. I agree this needs to be reviewed by @asotona . Thanks for the review. I will wait reviewed by @asotona before integrate. - PR Comment: https://git.openjdk.org/jdk/pull/19864#issuecomment-2236003849

Re: RFR: 8334771: [TESTBUG] Run TestDockerMemoryMetrics.java with -Xcomp fails exitValue = 137

2024-07-18 Thread SendaoYan
On Mon, 24 Jun 2024 16:16:29 GMT, SendaoYan wrote: > Hi all, > After [JDK-8294960](https://bugs.openjdk.org/browse/JDK-8294960), the > footprint memory usage increased significantly when run the testcase with > -Xcomp jvm options, then cause the testcase was killed by d

Re: RFR: 8334771: [TESTBUG] Run TestDockerMemoryMetrics.java with -Xcomp fails exitValue = 137

2024-07-18 Thread SendaoYan
On Thu, 18 Jul 2024 11:42:17 GMT, SendaoYan wrote: >> Hi all, >> After [JDK-8294960](https://bugs.openjdk.org/browse/JDK-8294960), the >> footprint memory usage increased significantly when run the testcase with >> -Xcomp jvm options, then cause the testcase was

Integrated: 8334771: [TESTBUG] Run TestDockerMemoryMetrics.java with -Xcomp fails exitValue = 137

2024-07-19 Thread SendaoYan
On Mon, 24 Jun 2024 16:16:29 GMT, SendaoYan wrote: > Hi all, > After [JDK-8294960](https://bugs.openjdk.org/browse/JDK-8294960), the > footprint memory usage increased significantly when run the testcase with > -Xcomp jvm options, then cause the testcase was killed by d

Re: RFR: 8334771: [TESTBUG] Run TestDockerMemoryMetrics.java with -Xcomp fails exitValue = 137

2024-07-19 Thread SendaoYan
On Mon, 24 Jun 2024 16:16:29 GMT, SendaoYan wrote: > Hi all, > After [JDK-8294960](https://bugs.openjdk.org/browse/JDK-8294960), the > footprint memory usage increased significantly when run the testcase with > -Xcomp jvm options, then cause the testcase was killed by d

RFR: 8339701: Test TestPinCaseWithCFLH.java fails with -Xcomp

2024-09-08 Thread SendaoYan
Hi all, The newly added testcase `serviceability/jvmti/vthread/TestPinCaseWithCFLH/TestPinCaseWithCFLH.java` fails with `-Xcomp` jvm option. The jvm option `-Xcomp` will make `BackgroundCompilation` false. In this test, disable `BackgroundCompilation` will make `instrumentation` in `premain`

Re: RFR: 8339701: Test TestPinCaseWithCFLH.java fails with -Xcomp

2024-09-09 Thread SendaoYan
On Mon, 9 Sep 2024 05:11:44 GMT, David Holmes wrote: > I think we need to understand why this happens, as I don't see this being a > test issue. I assume the test needs to run under a none-US locale for the > problem to arise? The locale of test enviroment is `en_US.UTF-8`, and this test also

Re: RFR: 8339701: Test TestPinCaseWithCFLH.java fails with -Xcomp

2024-09-09 Thread SendaoYan
server-release/images/jdk/bin/java`, so I think I am testing an images build. bash configure --with-jobs=32 --prefix=/tmp/tone/run/jtreg/jdk-repo/install-release --verbose --with-debug-level=release --enable-unlimited-crypto --with-vendor-name=yansendao --with-vendor-url=https://github.com/sen

Re: RFR: 8339701: Test TestPinCaseWithCFLH.java fails with -Xcomp

2024-09-09 Thread SendaoYan
On Mon, 9 Sep 2024 08:37:58 GMT, Alan Bateman wrote: > Can you say if there are any other modifications to the JDK that is being > tested? I'm wondering if anyone else can duplicate this. No, there is no any modification of the tested JDK. This failure only occur on specific environments, seem

Re: RFR: 8339701: Test TestPinCaseWithCFLH.java fails with -Xcomp

2024-09-09 Thread SendaoYan
On Sun, 8 Sep 2024 15:30:21 GMT, SendaoYan wrote: > Hi all, > The newly added testcase > `serviceability/jvmti/vthread/TestPinCaseWithCFLH/TestPinCaseWithCFLH.java` > fails with `-Xcomp` jvm option. The jvm option `-Xcomp` will make > `BackgroundCompilation` false. In thi

Re: RFR: 8339701: Test TestPinCaseWithCFLH.java fails with -Xcomp

2024-09-09 Thread SendaoYan
On Mon, 9 Sep 2024 09:18:24 GMT, Alan Bateman wrote: > I assume the sleep or System.gc is just delaying the install of the > transformer. I think this issue will require more investigation. The "main" > method hasn't run so I assume the issue is nothing to do with virtual threads > and the dia

Re: RFR: 8341246: Test com/sun/tools/attach/PermissionTest.java fails access denied after JDK-8327114 [v3]

2024-10-04 Thread SendaoYan
On Wed, 2 Oct 2024 15:44:58 GMT, SendaoYan wrote: >> Hi all, >> Test `com/sun/tools/attach/PermissionTest.java` fails access denied after >> [JDK-8327114](https://bugs.openjdk.org/browse/JDK-8327114). This testcase >> need the `readlink` permission of file `/proc/sel

RFR: 8342646: JTREG_TEST_THREAD_FACTORY in testing.md should be TEST_THREAD_FACTORY

2024-10-19 Thread SendaoYan
Hi all, In [make/RunTests.gmk](https://github.com/openjdk/jdk/blob/master/make/RunTests.gmk#L208), the keyword is 'TEST_THREAD_FACTORY'. So the below test command will print error: make test TEST=test/jdk/java/math/BigInteger/TestValueExact.java CONF=linux-x86_64-server-release JTREG="JTREG_TE

Re: RFR: 8342646: JTREG_TEST_THREAD_FACTORY in testing.md should be TEST_THREAD_FACTORY [v2]

2024-10-21 Thread SendaoYan
nue. Stop. > > > So I think we should fix the document bug in `doc/testing.md` to avoid take > the same mistake. Trivial fix, no risk. SendaoYan has updated the pull request incrementally with two additional commits since the last revision: - JTREG="TEST_THREAD_FACTORY=Virtual&

Re: RFR: 8342646: JTREG_TEST_THREAD_FACTORY in testing.md should be TEST_THREAD_FACTORY [v2]

2024-10-21 Thread SendaoYan
On Mon, 21 Oct 2024 12:48:55 GMT, Erik Joelsson wrote: >> SendaoYan has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - JTREG="TEST_THREAD_FACTORY=Virtual" >> - Use JTREG="JTREG_TEST_THREAD_

Re: RFR: 8342646: JTREG_TEST_THREAD_FACTORY in testing.md should be TEST_THREAD_FACTORY [v2]

2024-10-22 Thread SendaoYan
On Mon, 21 Oct 2024 13:12:55 GMT, SendaoYan wrote: >> Hi all, >> In >> [make/RunTests.gmk](https://github.com/openjdk/jdk/blob/master/make/RunTests.gmk#L208), >> the keyword is 'TEST_THREAD_FACTORY'. >> >> So the below test command will print e

Integrated: 8342646: JTREG_TEST_THREAD_FACTORY in testing.md should be TEST_THREAD_FACTORY

2024-10-22 Thread SendaoYan
On Sun, 20 Oct 2024 02:11:11 GMT, SendaoYan wrote: > Hi all, > In > [make/RunTests.gmk](https://github.com/openjdk/jdk/blob/master/make/RunTests.gmk#L208), > the keyword is 'TEST_THREAD_FACTORY'. > > So the below test command will print error: > > make test

RFR: 8343490: Update copyright year for JDK-8341692

2024-11-04 Thread SendaoYan
Hi all, The copyright year of some files which has been changed by [JDK-8341692](https://bugs.openjdk.org/browse/JDK-8341692) wasn't update correctly. This PR update the copyright year of [JDK-8341692](https://bugs.openjdk.org/browse/JDK-8341692). Trivial fix, no risk. - Commit m

RFR: 8343505: Problemlist AttachTest.java and TestZPageAllocationEvent.java

2024-11-05 Thread SendaoYan
Hi all, To make less CI noisy, before the root cause of [JDK-8343244](https://bugs.openjdk.org/browse/JDK-8343244) has fixed, should we problemlist the failure tests, include AttachTest.java and TestZPageAllocationEvent.java - Commit messages: - 8343244: Problemlist AttachTest.jav

Re: RFR: 8336691: Test LongArgTest.java intermittent fails java.lang.NoClassDefFoundError: jdk/test/lib/Utils [v2]

2024-11-09 Thread SendaoYan
On Thu, 18 Jul 2024 07:30:47 GMT, SendaoYan wrote: >> Hi all, >> The testcase `serviceability/attach/LongArgTest.java` intermittent fails >> `java.lang.NoClassDefFoundError: jdk/test/lib/Utils`. Jtreg doesn't >> automatically compile `jdk/test/lib/Utils.clas

RFR: 8341246: Test com/sun/tools/attach/PermissionTest.java fails access denied

2024-09-30 Thread SendaoYan
Hi all, Test `com/sun/tools/attach/PermissionTest.java` fails access denied after [JDK-8327114](https://bugs.openjdk.org/browse/JDK-8327114). This testcase need the `readlink` permission of file `/proc/self/ns/mnt` after [JDK-8327114](https://bugs.openjdk.org/browse/JDK-8327114). So this PR add

Re: RFR: 8341246: Test com/sun/tools/attach/PermissionTest.java fails access denied after JDK-8327114 [v2]

2024-10-01 Thread SendaoYan
-8327114). > So this PR add `readlink` permission to make this test work normally. > Before this PR, test run failed, after this PR, test run success. Test fix > only, no risk. SendaoYan has updated the pull request incrementally with one additional commit since the last revision: mer

Re: RFR: 8341246: Test com/sun/tools/attach/PermissionTest.java fails access denied after JDK-8327114 [v2]

2024-10-01 Thread SendaoYan
On Mon, 30 Sep 2024 17:05:08 GMT, Severin Gehwolf wrote: >> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> merge two comments enclosed with /* ... */ on two lines > > test/jdk/com/sun/tools/at

Re: RFR: 8341246: Test com/sun/tools/attach/PermissionTest.java fails access denied after JDK-8327114 [v2]

2024-10-02 Thread SendaoYan
On Tue, 1 Oct 2024 15:38:56 GMT, SendaoYan wrote: >> Hi all, >> Test `com/sun/tools/attach/PermissionTest.java` fails access denied after >> [JDK-8327114](https://bugs.openjdk.org/browse/JDK-8327114). This testcase >> need the `readlink` permission of file `/proc/sel

Re: RFR: 8341246: Test com/sun/tools/attach/PermissionTest.java fails access denied after JDK-8327114 [v2]

2024-10-02 Thread SendaoYan
On Tue, 1 Oct 2024 21:13:35 GMT, David Holmes wrote: >> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> merge two comments enclosed with /* ... */ on two lines > > test/jdk/com/sun/tools/at

Re: RFR: 8341246: Test com/sun/tools/attach/PermissionTest.java fails access denied after JDK-8327114 [v2]

2024-10-02 Thread SendaoYan
On Wed, 2 Oct 2024 15:36:23 GMT, Daniel D. Daugherty wrote: > @sendaoYan - This failure is quite noisy in the Oracle CI. We see many > failures in Tier[567]. When will you be integrating your fix? Sorry, these days are statutory holidays in China, so my PR processing is quite slow.

Re: RFR: 8341246: Test com/sun/tools/attach/PermissionTest.java fails access denied after JDK-8327114 [v3]

2024-10-02 Thread SendaoYan
-8327114). > So this PR add `readlink` permission to make this test work normally. > Before this PR, test run failed, after this PR, test run success. Test fix > only, no risk. SendaoYan has updated the pull request incrementally with one additional commit since the last revision

Re: RFR: 8341246: Test com/sun/tools/attach/PermissionTest.java fails access denied after JDK-8327114 [v3]

2024-10-02 Thread SendaoYan
On Wed, 2 Oct 2024 15:44:58 GMT, SendaoYan wrote: >> Hi all, >> Test `com/sun/tools/attach/PermissionTest.java` fails access denied after >> [JDK-8327114](https://bugs.openjdk.org/browse/JDK-8327114). This testcase >> need the `readlink` permission of file `/proc/sel

Integrated: 8341246: Test com/sun/tools/attach/PermissionTest.java fails access denied after JDK-8327114

2024-10-02 Thread SendaoYan
On Mon, 30 Sep 2024 16:11:27 GMT, SendaoYan wrote: > Hi all, > Test `com/sun/tools/attach/PermissionTest.java` fails access denied after > [JDK-8327114](https://bugs.openjdk.org/browse/JDK-8327114). This testcase > need the `readlink` permission of file `/proc/self/ns/mnt` a

Re: RFR: 8343505: Problemlist AttachTest.java and TestZPageAllocationEvent.java [v3]

2024-11-06 Thread SendaoYan
On Wed, 6 Nov 2024 06:53:34 GMT, David Holmes wrote: >> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> change os-arch to generic-all > > test/jdk/ProblemList.txt line 779: > >> 777: j

Re: RFR: 8343505: Problemlist AttachTest.java and TestZPageAllocationEvent.java [v3]

2024-11-06 Thread SendaoYan
> Hi all, > To make less CI noisy, before the root cause of > [JDK-8343244](https://bugs.openjdk.org/browse/JDK-8343244) has fixed, should > we problemlist the failure tests, include AttachTest.java and > TestZPageAllocationEvent.java SendaoYan has updated the pull request inc

Re: RFR: 8343505: Problemlist AttachTest.java and TestZPageAllocationEvent.java [v2]

2024-11-06 Thread SendaoYan
On Wed, 6 Nov 2024 06:55:41 GMT, David Holmes 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/TestZPageAllocat

Re: RFR: 8343505: Problemlist AttachTest.java and TestZPageAllocationEvent.java [v2]

2024-11-06 Thread SendaoYan
> Hi all, > To make less CI noisy, before the root cause of > [JDK-8343244](https://bugs.openjdk.org/browse/JDK-8343244) has fixed, should > we problemlist the failure tests, include AttachTest.java and > TestZPageAllocationEvent.java SendaoYan has updated the pull request inc

Re: RFR: 8343505: Problemlist AttachTest.java and TestZPageAllocationEvent.java [v4]

2024-11-06 Thread SendaoYan
> Hi all, > To make less CI noisy, before the root cause of > [JDK-8343244](https://bugs.openjdk.org/browse/JDK-8343244) has fixed, should > we problemlist the failure tests, include AttachTest.java and > TestZPageAllocationEvent.java SendaoYan has updated the pull request inc

Re: RFR: 8343505: Problemlist AttachTest.java and TestZPageAllocationEvent.java [v2]

2024-11-06 Thread SendaoYan
On Wed, 6 Nov 2024 08:19:42 GMT, Alan Bateman 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/TestZPageAllocat

Re: RFR: 8343505: Problemlist java/lang/Thread/jni/AttachCurrentThread/AttachTest.java [v4]

2024-11-06 Thread SendaoYan
On Thu, 7 Nov 2024 01:23:32 GMT, Leonid Mesnik wrote: > Please update bug/PR summary to reflect actual changes. Thanks for the remind. The bug and PR summary has been updated. - PR Comment: https://git.openjdk.org/jdk/pull/21917#issuecomment-2461149994

Integrated: 8343505: Problemlist java/lang/Thread/jni/AttachCurrentThread/AttachTest.java

2024-11-06 Thread SendaoYan
On Wed, 6 Nov 2024 02:39:24 GMT, SendaoYan wrote: > Hi all, > To make less CI noisy, before the root cause of > [JDK-8343244](https://bugs.openjdk.org/browse/JDK-8343244) has fixed, should > we problemlist the failure tests, include AttachTest.java and > TestZPageAllocationE

Re: RFR: 8343505: Problemlist java/lang/Thread/jni/AttachCurrentThread/AttachTest.java [v4]

2024-11-06 Thread SendaoYan
On Wed, 6 Nov 2024 08:39:08 GMT, SendaoYan wrote: >> Hi all, >> To make less CI noisy, before the root cause of >> [JDK-8343244](https://bugs.openjdk.org/browse/JDK-8343244) has fixed, should >> we problemlist the failure tests, include AttachTest.java and >>

Integrated: 8344903: Improve error handling TestJhsdbJstackPrintVMLocks.java

2024-11-24 Thread SendaoYan
On Sat, 23 Nov 2024 13:08:13 GMT, SendaoYan wrote: > Hi all, > The newly added test > `test/hotspot/jtreg/serviceability/sa/TestJhsdbJstackPrintVMLocks.java` > intermittent fails `the return value of > "jdk.test.lib.apps.LingeredApp.getProcess()" is null`. This PR c

  1   2   >