RFR: 8299518: HotSpotVirtualMachine shared code across different platforms [v5]

Chris Plummer cjplummer at openjdk.org
Mon Jan 9 20:43:55 UTC 2023


On Mon, 9 Jan 2023 13:25:34 GMT, Yi Yang <yyang at openjdk.org> wrote:

>> harmless refactor to share code across different platforms of VirtualMachineImpl:
>> 1. Shared code to process command response after requesting a command execution
>> 2. Read functionality in SocketInputStream can be reused
>
> Yi Yang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update src/jdk.attach/share/classes/sun/tools/attach/HotSpotVirtualMachine.java
>   
>   Co-authored-by: Andrey Turbanov <turbanoff at gmail.com>

src/jdk.attach/aix/classes/sun/tools/attach/VirtualMachineImpl.java line 61:

> 59:         int pid = Integer.parseInt(vmid);
> 60:         if (pid < 1) {
> 61:             throw new AttachNotSupportedException("Invalid process identifier -1");

Why the change to using -1 instead of `vmid`? It seems presumptuous that it is -1, and also seems less useful than printing `vmid`.

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

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


More information about the serviceability-dev mailing list