On Thu, 24 Oct 2024 01:42:36 GMT, Alex Menkov <amen...@openjdk.org> wrote:
> The fix cleans up code to support list of JVMTI breakpoints. > - classes required to supports cache of byte code pointers (GrowableElement, > GrowableCache, JvmtiBreakpointCache) are dropped; > - class JvmtiCurrentBreakpoints (JvmtiBreakpoints factory) is left as is, > dropped unused code; > - fixed race in JvmtiCurrentBreakpoints::get_jvmti_breakpoints() (fix for > JDK-8210637); > - JvmtiBreakpoint:JvmtiBreakpoint() + JvmtiBreakpoint::copy(JvmtiBreakpoint& > bp) are replaced with copy ctor; > - JvmtiBreakpoints::clearall_in_class_at_safepoint() is simplified to do a > single pass; > > Testing: tier1..tier6 This pull request has now been integrated. Changeset: 471f112b Author: Alex Menkov <amen...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/471f112bca715d04304cbe35c6ed63df8c7b7fee Stats: 352 lines in 2 files changed: 27 ins; 284 del; 41 mod 8342577: Clean up JVMTI breakpoint support 8210637: Race in JvmtiCurrentBreakpoints::get_jvmti_breakpoints Reviewed-by: cjplummer, sspitsyn ------------- PR: https://git.openjdk.org/jdk/pull/21675
