RFR: 8314029: Add file name parameter to Compiler.perfmap [v8]
Chris Plummer
cjplummer at openjdk.org
Mon Dec 18 20:33:54 UTC 2023
On Mon, 11 Dec 2023 22:41:56 GMT, Yi-Fan Tsai <duke at openjdk.org> wrote:
>> `jcmd Compiler.perfmap` uses the hard-coded file name for a perf map: `/tmp/perf-%d.map`. This change adds an optional argument for specifying a file name.
>>
>> `jcmd PID help Compiler.perfmap` shows the following usage.
>>
>>
>> Compiler.perfmap
>> Write map file for Linux perf tool.
>>
>> Impact: Low
>>
>> Syntax : Compiler.perfmap [<filename>]
>>
>> Arguments:
>> filename : [optional] Name of the map file (STRING, no default value)
>>
>>
>> The following section of man page is also updated. (`man -l src/jdk.jcmd/share/man/jcmd.1`)
>>
>>
>> Compiler.perfmap [arguments] (Linux only)
>> Write map file for Linux perf tool.
>>
>> Impact: Low
>>
>> arguments:
>>
>> · filename: (Optional) Name of the map file (STRING, no default value)
>>
>> If filename is not specified, a default file name is chosen using the pid of the target JVM process. For example, if the pid is 12345, then
>> the default filename will be /tmp/perf-12345.map.
>
> Yi-Fan Tsai has updated the pull request incrementally with one additional commit since the last revision:
>
> Update copyright of PerfMapTest
If we do settle on some additional changes, I think probably a follow-up CR would be cleaner than a BACKOUT and REDO.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/15871#issuecomment-1861563261
More information about the serviceability-dev
mailing list