RFR: 8319873: Add windows implementation for jcmd System.map and System.dump_map [v4]
Thomas Stuefe
stuefe at openjdk.org
Wed Aug 21 15:57:04 UTC 2024
On Wed, 21 Aug 2024 14:54:40 GMT, Simon Tooke <stooke at openjdk.org> wrote:
>> This is a port of [JDK-8318636](https://github.com/openjdk/jdk/pull/16301) to Windows.
>>
>> System.map and System.dump_map are implemented using the Windows API and provide roughly the same information in the same format. Most of the heavy lifting was implemented by @tstuefe in #16301 - this PR adds the Windows implementation and enables the common code for Windows 64 bit.
>>
>> [Sample output (with NMT enabled)](https://github.com/user-attachments/files/16663332/vm_memory_map_760.txt)
>
> Simon Tooke has updated the pull request incrementally with one additional commit since the last revision:
>
> debug copilation error fix
Good! I like the new logic in get_protect_string.
Needs a second reviewer. @jdksjolen maybe?
-------------
Marked as reviewed by stuefe (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/20597#pullrequestreview-2251262463
PR Comment: https://git.openjdk.org/jdk/pull/20597#issuecomment-2302431286
More information about the serviceability-dev
mailing list