Re: [PATCH 3/4] perf inject: add jitdump mmap injection support

[David Ahern <dsahern@gmail.com>]

On 2/10/15 4:42 PM, Stephane Eranian wrote:
> diff --git a/tools/perf/Documentation/perf-inject.txt b/tools/perf/Documentation/perf-inject.txt
> index dc7442c..237f195 100644
> --- a/tools/perf/Documentation/perf-inject.txt
> +++ b/tools/perf/Documentation/perf-inject.txt
> @@ -40,6 +40,17 @@ OPTIONS
>   	Merge sched_stat and sched_switch for getting events where and how long
>   	tasks slept. sched_switch contains a callchain where a task slept and
>   	sched_stat contains a timeslice how long a task slept.
> +-j::
> +--jit::
> +	Merge a jitdump file into the perf.data file by adding mmap records to
> +	cover jitted code and emit ELF images for each jitted function. The ELF
> +	images are saved in the same directory as the jidump. Use -E to suppress
> +	ELF images generation.
> +-E::
> +--jit-disable-elf::
> +	When used with -, it prevents creating the ELF images for each jitted
> +	function. Only the jitted code mmap records are injected into the perf.data
> +	file. Option as no effect when -j is not used.

s/as/has/. But it would better to avoid the double negative. Maye 
something like this instead:

     This option requires -j|--jit.

David