On Wed, 8 May 2024 20:23:21 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:

> A "leaf" failure is more specific than a "rank order" failure, so it is 
> better to report it first. Each "leaf" failure is also a "rank order" failure 
> (AFAICS).

It's not just a matter of which is reported first. Even if you swap the order 
of your two loops you get the same result. The problem is the "rank" loop does 
not check if any of the leaf monitors are already held. I think to fix that it 
would have to iterate up to NUM_DEBUG_RAW_MONITORS, which means there is 
overlap with the range that the "leaf" loop iterators over.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19044#discussion_r1594593936

Reply via email to