RFR: 8282410: Remove SA ProcDebugger support

Kevin Walls kevinw at openjdk.org
Thu Aug 25 10:46:14 UTC 2022


On Thu, 25 Aug 2022 03:31:03 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

> SA has the following source directory:
> 
> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/debugger/proc
> 
> However, there are no references to any classes in this directory. There used to be a reference to ProcDebuggerLocal, but this reference was removed, along with the class, when solaris support was removed by [JDK-8244224](https://bugs.openjdk.org/browse/JDK-8244224). There appears to be no other need for anything in this directory now, and I believe it can be safely removed.

A lot of files 8-)
3 files in riscv64 were surprising to me, as they are new, from:
8276799: Implementation of JEP 422: Linux/RISC-V Port
But there are "non-proc" versions of the RISCV64 files, and I don't see any references to the /proc/ files that were added.
e.g. sun/jvm/hotspot/debugger/linux/riscv64/LinuxRISCV64CFrame uses RISCV64ThreadContext, and LinuxCDebugger uses RISCV64ThreadContext.
These might have been added due to honourably mirroring what other platforms had, although they aren't needed, so this looks good.

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

Marked as reviewed by kevinw (Committer).

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


More information about the serviceability-dev mailing list