RFR: 8324969: C2: prevent elimination of unbalanced coarsened locking regions [v5]

Vladimir Kozlov kvn at openjdk.org
Thu Feb 15 05:45:25 UTC 2024


On Wed, 14 Feb 2024 19:38:01 GMT, Dean Long <dlong at openjdk.org> wrote:

>> Vladimir Kozlov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Additional changes
>
> My concern is that a group of unbalanced locks is added to coarsened_locks (they have different BoxLockNodes), but later some of those locks are removed by `remove_useless_coarsened_locks` or `remove_coarsened_lock`.  Then when `mark_unbalanced_boxes` finally runs, it incorrectly sees the group as balanced.
> 
> It is easy to check for unbalanced in `add_coarsened_locks` as the locks are added.  I think this is the most conservative approach.

@dean-long, I added verification code as I described in previous comment. Please look.
I am running tier1-5,xcopm,stress testing with it. So far did not hit new asserts.

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

PR Comment: https://git.openjdk.org/jdk/pull/17697#issuecomment-1945407681


More information about the hotspot-compiler-dev mailing list