RFR: 8352256: ObjectSynchronizer::quick_notify misses JFR event notification path
Aleksey Shipilev
shade at openjdk.org
Tue Mar 18 17:47:12 UTC 2025
On Tue, 18 Mar 2025 12:02:04 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
> This looks good. At one point, because of this lack of encapsulation, I wanted us to get rid of quick_notify because I didn't think it was quicker. This improves the encapsulation too and we can figure out if it's useful later.
Yeah, I wondered about these "quick" functions myself when doing this. I guess there is a papercut performance optimization for not re-checking the owner and waitset, which is why I opted to do `ObjectMonitor::quick_*` entries that bypass those checks as well. But I am doubtful those checks matter.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24097#issuecomment-2734186068
More information about the hotspot-runtime-dev
mailing list