Re: [PATCH] perf mem: Document uncorrect and missing options

[Jiri Olsa]

On Sun, Apr 22, 2018 at 04:29:06PM +0900, Sangwon Hong wrote:

SNIP

>  -C::
> ---cpu-list::
> - Restrict dump of raw samples to those provided via this option. Note 
> that the same
> - option can be passed in record mode. It will be interpreted the same 
> way as perf
> - record.
> +--cpu=::
> + Monitor only on the list of CPUs provided. Multiple CPUs can be 
> provided as a
> +comma-separated list with no space: 0,1. Ranges of CPUs are 
> specified with -: 0-2. Default
> +is to monitor all CPUS.
> +-U::
> +--hide-unresolved::
> + Only display entries resolved to a symbol.
> +
> +-p::
> +--phys-data::
> + Record/Report sample physical addresses
> +
> +RECORD OPTIONS
> +--
> +-e::
> +--event ::
> + Event selector. Use 'perf mem record -e list' to list available events. 
> 

extra white space at the end of above line.. 
other than that it looks ok, thanks for doing this

Acked-by: Jiri Olsa 

thanks,
jirka

Re: [PATCH] perf mem: Document uncorrect and missing options

[Jiri Olsa]

On Sun, Apr 22, 2018 at 04:29:06PM +0900, Sangwon Hong wrote:

SNIP

>  -C::
> ---cpu-list::
> - Restrict dump of raw samples to those provided via this option. Note 
> that the same
> - option can be passed in record mode. It will be interpreted the same 
> way as perf
> - record.
> +--cpu=::
> + Monitor only on the list of CPUs provided. Multiple CPUs can be 
> provided as a
> +comma-separated list with no space: 0,1. Ranges of CPUs are 
> specified with -: 0-2. Default
> +is to monitor all CPUS.
> +-U::
> +--hide-unresolved::
> + Only display entries resolved to a symbol.
> +
> +-p::
> +--phys-data::
> + Record/Report sample physical addresses
> +
> +RECORD OPTIONS
> +--
> +-e::
> +--event ::
> + Event selector. Use 'perf mem record -e list' to list available events. 
> 

extra white space at the end of above line.. 
other than that it looks ok, thanks for doing this

Acked-by: Jiri Olsa 

thanks,
jirka

[PATCH] perf mem: Document uncorrect and missing options

[Sangwon Hong]

Signed-off-by: Sangwon Hong 
---
 tools/perf/Documentation/perf-mem.txt | 41 +--
 1 file changed, 29 insertions(+), 12 deletions(-)

diff --git a/tools/perf/Documentation/perf-mem.txt 
b/tools/perf/Documentation/perf-mem.txt
index b021141..139f30f 100644
--- a/tools/perf/Documentation/perf-mem.txt
+++ b/tools/perf/Documentation/perf-mem.txt
@@ -28,29 +28,46 @@ OPTIONS
 ...::
Any command you can specify in a shell.
 
+-i::
+--input=::
+   Input file name.
+
 -f::
 --force::
Don't do ownership validation
 
 -t::
---type=::
+--type=::
Select the memory operation type: load or store (default: load,store)
 
 -D::
---dump-raw-samples=::
+--dump-raw-samples::
Dump the raw decoded samples on the screen in a format that is easy to 
parse with
one sample per line.
 
 -x::
---field-separator::
+--field-separator=::
Specify the field separator used when dump raw samples (-D option). By 
default,
The separator is the space character.
 
 -C::
---cpu-list::
-   Restrict dump of raw samples to those provided via this option. Note 
that the same
-   option can be passed in record mode. It will be interpreted the same 
way as perf
-   record.
+--cpu=::
+   Monitor only on the list of CPUs provided. Multiple CPUs can be 
provided as a
+comma-separated list with no space: 0,1. Ranges of CPUs are specified 
with -: 0-2. Default
+is to monitor all CPUS.
+-U::
+--hide-unresolved::
+   Only display entries resolved to a symbol.
+
+-p::
+--phys-data::
+   Record/Report sample physical addresses
+
+RECORD OPTIONS
+--
+-e::
+--event ::
+   Event selector. Use 'perf mem record -e list' to list available events. 

 
 -K::
 --all-kernel::
@@ -60,12 +77,12 @@ OPTIONS
 --all-user::
Configure all used events to run in user space.
 
---ldload::
-   Specify desired latency for loads event.
+-v::
+--verbose::
+   Be more verbose (show counter open errors, etc)
 
--p::
---phys-data::
-   Record/Report sample physical addresses
+--ldlat ::
+   Specify desired latency for loads event.
 
 SEE ALSO
 
-- 
2.7.4

[PATCH] perf mem: Document uncorrect and missing options

[Sangwon Hong]

Signed-off-by: Sangwon Hong 
---
 tools/perf/Documentation/perf-mem.txt | 41 +--
 1 file changed, 29 insertions(+), 12 deletions(-)

diff --git a/tools/perf/Documentation/perf-mem.txt 
b/tools/perf/Documentation/perf-mem.txt
index b021141..139f30f 100644
--- a/tools/perf/Documentation/perf-mem.txt
+++ b/tools/perf/Documentation/perf-mem.txt
@@ -28,29 +28,46 @@ OPTIONS
 ...::
Any command you can specify in a shell.
 
+-i::
+--input=::
+   Input file name.
+
 -f::
 --force::
Don't do ownership validation
 
 -t::
---type=::
+--type=::
Select the memory operation type: load or store (default: load,store)
 
 -D::
---dump-raw-samples=::
+--dump-raw-samples::
Dump the raw decoded samples on the screen in a format that is easy to 
parse with
one sample per line.
 
 -x::
---field-separator::
+--field-separator=::
Specify the field separator used when dump raw samples (-D option). By 
default,
The separator is the space character.
 
 -C::
---cpu-list::
-   Restrict dump of raw samples to those provided via this option. Note 
that the same
-   option can be passed in record mode. It will be interpreted the same 
way as perf
-   record.
+--cpu=::
+   Monitor only on the list of CPUs provided. Multiple CPUs can be 
provided as a
+comma-separated list with no space: 0,1. Ranges of CPUs are specified 
with -: 0-2. Default
+is to monitor all CPUS.
+-U::
+--hide-unresolved::
+   Only display entries resolved to a symbol.
+
+-p::
+--phys-data::
+   Record/Report sample physical addresses
+
+RECORD OPTIONS
+--
+-e::
+--event ::
+   Event selector. Use 'perf mem record -e list' to list available events. 

 
 -K::
 --all-kernel::
@@ -60,12 +77,12 @@ OPTIONS
 --all-user::
Configure all used events to run in user space.
 
---ldload::
-   Specify desired latency for loads event.
+-v::
+--verbose::
+   Be more verbose (show counter open errors, etc)
 
--p::
---phys-data::
-   Record/Report sample physical addresses
+--ldlat ::
+   Specify desired latency for loads event.
 
 SEE ALSO
 
-- 
2.7.4