On Fri, 22 Sep 2023 02:48:57 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> `jcmd Compiler.perfmap` uses the hard-coded file name for a perf map: 
>> `/tmp/perf-%d.map`. This change adds an option for specifying a file name.
>> 
>> The help message of Compiler.perfmap:
>> 
>> Compiler.perfmap
>> Write map file for Linux perf tool.
>> 
>> Impact: Low
>> 
>> Syntax : Compiler.perfmap [options]
>> 
>> Options: (options must be specified using the <key> or <key>=<value> syntax)
>>         filename : [optional] Name of the map file (STRING, no default value)
>
> src/jdk.jcmd/share/man/jcmd.1 line 1:
> 
>> 1: .\" Copyright (c) 2012, 2023, Oracle and/or its affiliates. All rights 
>> reserved.
> 
> The actual markdown source for this file needs to be updated with these 
> changes. Those sources are not open-source unfortunately. Please either 
> coordinate to get the sources updated with an Oracle developer as part of 
> this PR (they will integrate the internal part), or else please defer this to 
> a subtask and let an Oracle developer update the source and output at the 
> same time. Thanks.

I filed JDK-8320556 to update the closed source. It's assigned to me. I'll do 
the update after these changes are pushed. In the meantime I'll make sure the 
current jcmd.l changes are correct and match the closed changes I'll be making.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15871#discussion_r1401151891

Reply via email to