RFR: JDK-8318636: Add jcmd to print annotated process memory map [v3]

Thomas Stuefe stuefe at openjdk.org
Fri Oct 27 11:57:36 UTC 2023


On Fri, 27 Oct 2023 09:41:28 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

>> Thomas Stuefe has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains four commits:
>> 
>>  - Merge master and solve merge conflicts
>>  - small fixes
>>  - start from VM op; show more thread details
>>  - start
>
> src/hotspot/os/linux/memMapPrinter_linux.cpp line 34:
> 
>> 32: struct proc_maps_info_t {
>> 33:   unsigned long long from = 0;
>> 34:   unsigned long long to = 0;
> 
> Can we not use `uintptr_t` here? scanf directive: https://stackoverflow.com/questions/12228227/how-to-get-pointer-sized-integer-using-scanf

I'll try void* and %p instead.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16301#discussion_r1374465100


More information about the serviceability-dev mailing list