On Mon, 29 Sep 2025 08:52:04 GMT, SendaoYan <[email protected]> wrote:

> Hi all,
> 
> We observed java/lang/Thread/virtual/Starvation.java intermittent timed out 
> on server with many CPU core number. The more CPU core number the 'runTest' 
> will create more pinned virtual threads. This PR compute an iteration count 
> which divide the CPU core number will make test run passed steady on server 
> with high CPUs.
> 
> Before apply the proposed patch, test run finish 257 seconds, after apply the 
> proposed patch test run finish 1.2 seconds on the 2 sockets system with 
> Intel(R) Xeon(R) Platinum 8480+. So I remove the 'timeout=800', I think the 
> default timeout 120 will be enough.

Hello @sendaoYan, the change looks OK to me.

Since the change also removes the timeout value from this test, please give a 
day or so to have this run in our CI to make sure there aren't any intermittent 
failures.

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

Marked as reviewed by jpai (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27543#pullrequestreview-3283022510

Reply via email to