On Mon, 28 Oct 2024 23:21:14 GMT, Dean Long <dl...@openjdk.org> wrote:
>> What are we counting now with MaskFillerForNativeFrame that we weren't >> counting before this change? in MaskFillerForNative::set_one. > > So it sounds like the adjustment at line 119 is a bug fix, but what I don't > understand is why we weren't seeing problems before. Something in this PR > exposed the need for this change. > What are we counting now with MaskFillerForNativeFrame that we weren't > counting before this change? in MaskFillerForNative::set_one. > The number of oops in the parameter's for this native method. For Object.wait() we have only one, the j.l.Object reference. But for synchronized native methods there could be more. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21565#discussion_r1819908946
