RFR: 8265489: Stress test times out because of long ObjectSynchronizer::monitors_iterate(...) operation

[Daniel D.Daugherty]

On Thu, 19 Aug 2021 21:18:53 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

> monitors_iterate make several checks which often are true before filter monitor by a thread. It might take a lot of time when there are a lot of threads. So it makes sense to first check thread and only then other conditions.

Moving the thread check from the closure's do_monitor() call
into monitors_iterate() as early as possible is a good idea.

Do you have any measurements to show how much this helps?
I'm okay if you don't and I'd be happy waiting to see if it makes
a difference with some of those Tier8 timeouts...

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

Marked as reviewed by dcubed (Reviewer).

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