RFR: 8339289: Parameter size mismatch between client and VM sides of the Attach API - Windows [v4]

[Kevin Walls]

On Tue, 15 Oct 2024 22:31:46 GMT, Alex Menkov <amenkov at openjdk.org> wrote:

>> The fix improves Attch API protocol and implements updated protocol on windows; shared code is ready to implement updated protocol support on other platforms.
>> More detailed explanations on the 1st comment.
>> 
>> Testing: tier1,tier2,tier3,tier4,hs-tier5-svc
>>   manually tested backward compatibility (old tools can attach to current VMs, current tools can attach to older VMs) on Windows with jdk21u and jdk8u.
>
> Alex Menkov has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - updated comment
>  - feedback

src/hotspot/share/services/attachListener.cpp line 406:

> 404:   { "printflag",        print_flag },
> 405:   { "jcmd",             jcmd },
> 406:   { "getVersion",       get_version },

It's a bit of a nit, but "dumpheap" and other existing commands never use caps but the new "getVersion" does?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20782#discussion_r1814896928