Integrated: 8343957: Rename ObjectMonitor::owner_from() and JavaThread::_lock_id

Patricio Chilano Mateo pchilanomate at openjdk.org
Wed Dec 4 21:28:47 UTC 2024


On Tue, 3 Dec 2024 19:10:55 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

> Please review this small renaming patch. During the review of JDK-8338383 there were some comments about improving the naming for `ObjectMonitor::owner_from()` and `JavaThread::_lock_id`. These originate from the changes introduced to inflated monitors, where we now record the `java.lang.Thread.tid` of the owner in the ObjectMonitor's `_owner` field instead of a `JavaThread*`. I renamed `_lock_id` as `_monitor_owner_id` and `owner_from()` as `owner_id_from()`.
> 
> Thanks,
> Patricio

This pull request has now been integrated.

Changeset: c113f82f
Author:    Patricio Chilano Mateo <pchilanomate at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/c113f82f78c7d9be1ac297aebfeb6051f0f904fb
Stats:     83 lines in 19 files changed: 0 ins; 2 del; 81 mod

8343957: Rename ObjectMonitor::owner_from() and JavaThread::_lock_id

Reviewed-by: coleenp, dholmes

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

PR: https://git.openjdk.org/jdk/pull/22524


More information about the serviceability-dev mailing list