[Tracepoint event]
jbd2:jbd2_start_commit [Tracepoint event]
......
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/util/parse-events.c | 212 -
1 file
are at
the same line
After this patch:
$perf list --list-opts
--raw-dump
$ -- the new line
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/perf.c | 1 +
tools/perf/util/parse-options.c | 2 ++
2 files changed, 3 insertions
://lkml.org/lkml/2015/2/15/75 for PATCH v2
--
Thanks,
Yunlong Song
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http
On 2015/2/13 22:52, Arnaldo Carvalho de Melo wrote:
Em Fri, Feb 13, 2015 at 09:11:53PM +0800, Yunlong Song escreveu:
Distinguish the output of 'perf list --list-opts' or 'perf --list-cmds'
with the next command prompt, which also happens in other cases (e.g.
record, report ...).
Example
Hi,
Found some functions to improve and bugs to fix in perf list.
Yunlong Song (7):
perf list: clean up the printing functions of hardware/software events
perf list: sort the output of 'perf list' to view more clearly
perf list: fix some inaccuracy problem when parsing the argument
perf
:jbd2_start_commit [Tracepoint event]
......
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-list.c | 2 -
tools/perf/util/parse-events.c | 166 +++--
2 files
are at
the same line
After this patch:
$perf list --list-opts
--raw-dump
$ -- the new line
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/perf.c | 1 +
tools/perf/util/parse-options.c | 2 ++
2 files changed, 3 insertions(+)
diff --git
-backend
stalled-cycles-frontend
As shown above, only the hw events are printed.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/Documentation/perf-list.txt | 6 ++
tools/perf/builtin-list.c | 21 -
2 files changed, 14 insertions(+), 13
the events of tracepoint_foo are printed.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-list.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/builtin-list.c b/tools/perf/builtin-list.c
index de5680e..003dec5 100644
--- a/tools/perf/builtin
Do not need print_events_type or __print_events_type for listing hw/sw
events, let print_symbol_events do its job instead. Moreover,
print_symbol_events can also handle event_glob and name_only.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-list.c | 6
[Hardware cache event]
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-list.c | 3 +++
tools/perf/util/parse-events.c | 5 -
2 files changed, 3 insertions(+), 5 deletions(-)
diff --git a/tools/perf/builtin-list.c b/tools/perf/builtin-list.c
index b81a62c..af5bd05
$
As shown above, the result prints nothing rather than a segmentation
fault. The null result means 'perf list' has no cmds to display at this
time.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/util/parse-options.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff
On 2015/2/13 22:55, Arnaldo Carvalho de Melo wrote:
Em Fri, Feb 13, 2015 at 09:11:54PM +0800, Yunlong Song escreveu:
Extend 'perf list --raw-dump' to 'perf list --raw-dump [hw|sw|cache
|tracepoint|pmu|event_glob]' in order to show the raw-dump of a
certain kind of events rather than all
On 2015/2/13 22:45, Arnaldo Carvalho de Melo wrote:
Em Fri, Feb 13, 2015 at 09:11:50PM +0800, Yunlong Song escreveu:
return;
+
+if (evt_num_known) {
+evt_list = zalloc(sizeof(char *) * evt_num);
+if (!evt_list)
+goto out_enomem
Do not need print_events_type or __print_events_type for listing hw/sw
events, let print_symbol_events do its job instead. Moreover,
print_symbol_events can also handle event_glob and name_only.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-list.c | 6
instructions stalled-cycles-backend
stalled-cycles-frontend
As shown above, only the hw events are printed.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/Documentation/perf-list.txt | 6 ++
tools/perf/builtin-list.c | 21 -
2 files changed, 14
above, only the events of tracepoint_foo are printed.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-list.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/tools/perf/builtin-list.c b/tools/perf/builtin-list.c
index ad8018e..2acbcf0 100644
Hi,
Found some functions to improve and bugs to fix in perf list.
Yunlong Song (5):
perf list: Sort the output of 'perf list' to view more clearly
perf list: Fix some inaccuracy problem when parsing the argument
perf list: Clean up the printing functions of hardware/software events
perf
--period --no-samples --no-buildid-cache --no-buildid --cgroup --delay
--uid --branch-any --branch-filter --weight --transaction --per-thread
--intr-regs
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/util/parse-options.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion
of perf trace can come out now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-trace.c| 7 ---
tools/perf/perf-completion.sh | 9 ++---
2 files changed, 10 insertions(+), 6 deletions(-)
diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c
node-store-misses
uncore_rbox_0/qpi1_filt_send/
branch-loads dTLB-load-misses
L1-icache-load-misses node-stores
uncore_rbox_0/qpi1_idle_filt/
...
As shown above, the events of record can come out now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
, the subsubcommands of perf timechart can come out now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-timechart.c | 7 ---
tools/perf/perf-completion.sh | 2 +-
2 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/tools/perf/builtin-timechart.c b/tools/perf
-completion.sh.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/perf-completion.sh | 36 ++--
1 file changed, 30 insertions(+), 6 deletions(-)
diff --git a/tools/perf/perf-completion.sh b/tools/perf/perf-completion.sh
index bdd4035..3ba80b2 100644
of perf help can come out now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-help.c | 17 ++---
tools/perf/perf-completion.sh | 2 +-
2 files changed, 15 insertions(+), 4 deletions(-)
diff --git a/tools/perf/builtin-help.c b/tools/perf/builtin-help.c
out.
After this patch:
$ perf kvm --verbose TAB
buildid-list diff recordreportstat
top
As shown above, the subsubcommands of perf kvm can come out now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/perf-completion.sh | 23
of perf data can come out now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-data.c | 6 --
tools/perf/perf-completion.sh | 2 +-
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/tools/perf/builtin-data.c b/tools/perf/builtin-data.c
index 155cf75
, the subcommands of perf can come out now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/perf-completion.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/perf-completion.sh b/tools/perf/perf-completion.sh
index 3e25d3e..7b98ae4 100644
--- a/tools/perf/perf
, the subsubcommands of perf script can come out now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-script.c | 5 +++--
tools/perf/perf-completion.sh | 2 +-
2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c
--pid
--running-time --transaction
As shown above, the result is exactly the options of record as we wished.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/perf-completion.sh | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/tools/perf/perf
kvm --TAB
--alloc--caller --input--line --raw-ip --sort
--verbose
As shown above, the options of perf kvm can come out now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/perf-completion.sh | 21 -
1 file changed, 12 insertions(+), 9
Hi,
Found some functions to improve and bugs to fix in perf bash
completion.
Yunlong Song (13):
perf tools: Fix the bash completion for listing options of perf
subcommand
perf tools: Fix the bash completion for listing subsubcommands of perf
subcommand
perf tools
test can come out now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/perf-completion.sh | 2 +-
tools/perf/tests/builtin-test.c | 5 +++--
2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/tools/perf/perf-completion.sh b/tools/perf/perf-completion.sh
index
above, the events of record can come out now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/perf-completion.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/perf-completion.sh b/tools/perf/perf-completion.sh
index 01ce841..4b58ac2 100644
Hi,
Found some functions to improve and bugs to fix in perf building.
Yunlong Song (2):
perf build: Use FEATURE-DUMP instead of PERF-FEATURES in the
.gitignore file
perf build: Add config/feature-checks/*.output to the .gitignore file
tools/perf/.gitignore
Since commit 4ae61202b31c (perf build: Rename PERF-FEATURES into
FEATURE-DUMP) renames PERF-FEATURES into FEATURE-DUMP, the .gitignore
file should also do this thing for consistency.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/.gitignore | 2 +-
1 file changed, 1 insertion
-stackprotector-all.make.output
config/feature-checks/test-sync-compare-and-swap.make.output
config/feature-checks/test-timerfd.make.output
config/feature-checks/test-zlib.make.output
After this patch:
$ git status
nothing to commit, working directory clean
Signed-off-by: Yunlong
-
...
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/util/annotate.c | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/tools/perf/util/annotate.c b/tools/perf/util/annotate.c
index 61bf912..0f9ebad 100644
--- a/tools/perf/util/annotate.c
+++ b
'
# Total weight : 188
# Sort order : (null)
#
...
After this patch:
$ perf mem report
# To display the perf.data header info, please use --header/--header-only
options.
#
# Samples: 18 of event 'cpu/mem-loads/pp'
# Total weight : 188
#
...
Signed-off-by: Yunlong Song yunlong.s
itself.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-sched.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
index 7fe3b3c..3261300 100644
--- a/tools/perf/builtin-sched.c
+++ b/tools/perf
( :5: 5), nr_events: 1
...
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-sched.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
index 20d887b..dd71481 100644
--- a/tools
:163540), nr_events: 1
task 1551 ( unknown: 0), nr_events: 10
...
As shown above, the result continues without any segmentation fault.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-sched.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions
Hi,
Found some functions to improve and bugs to fix in perf sched replay.
Yunlong Song (9):
perf sched replay: Use struct task_desc instead of struct task_task
for correct meaning
perf sched replay: Increase the MAX_PID value to fix assertion failure
problem
perf sched
3 ( :5: 5), nr_events: 1
...
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-sched.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
index a1893e8..c466104 100644
There is no struct task_task at all, thus it is a typo error in the
old commits, now fix it to what it should be in order to avoid
unnecessary misunderstanding.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-sched.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
...
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-sched.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
index c466104..20d887b 100644
--- a/tools/perf/builtin-sched.c
+++ b/tools
-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-sched.c | 31 ++-
1 file changed, 26 insertions(+), 5 deletions(-)
diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
index 3261300..5ab58c6 100644
--- a/tools/perf/builtin-sched.c
above, the -f option really works now.
Besides for replay, -f option can also work for latency and map.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-sched.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
to achieve more accurate results instead of the default
value 10.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-sched.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
index 7b7b798..5275bab
On 2015/3/31 22:25, David Ahern wrote:
On 3/31/15 7:46 AM, Yunlong Song wrote:
diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
index a1893e8..c466104 100644
--- a/tools/perf/builtin-sched.c
+++ b/tools/perf/builtin-sched.c
@@ -28,7 +28,7 @@
#define MAX_CPUS
On 2015/3/31 22:32, David Ahern wrote:
On 3/31/15 7:46 AM, Yunlong Song wrote:
diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
index c466104..20d887b 100644
--- a/tools/perf/builtin-sched.c
+++ b/tools/perf/builtin-sched.c
@@ -23,6 +23,7 @@
#include semaphore.h
On 2015/3/31 23:56, Arnaldo Carvalho de Melo wrote:
Em Tue, Mar 31, 2015 at 08:32:37AM -0600, David Ahern escreveu:
On 3/31/15 7:46 AM, Yunlong Song wrote:
- BUG_ON(pid = MAX_PID);
+ if (sched-pid_to_task == NULL) {
+ if (sysctl__read_int(kernel/pid_max, pid_max) 0
,symbol_daddr,dso_daddr,snoop,tlb,locked
#
...
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
Signed-off-by: Namhyung Kim namhy...@kernel.org
---
tools/perf/builtin-report.c | 2 +-
tools/perf/util/sort.h | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/tools/perf/builtin
On 2015/3/23 8:44, Namhyung Kim wrote:
Hi Yunlong,
On Thu, Mar 19, 2015 at 09:58:32PM +0800, Yunlong Song wrote:
When '--sort' is not set, 'perf mem report will print a null pointer as
the output value of sort order, so fix it.
Example:
Before this patch:
$ perf mem report
On 2015/2/19 1:56, Ingo Molnar wrote:
* Yunlong Song yunlong.s...@huawei.com wrote:
If the long_name of a 'struct option' is defined as NULL, --list-opts
will incorrectly print '--(null)' in its output. As a result, '--(null)'
will finally appear in the case of bash completion, e.g. 'perf
Do not need print_events_type or __print_events_type for listing hw/sw
events, let print_symbol_events do its job instead. Moreover,
print_symbol_events can also handle event_glob and name_only.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-list.c | 6
, for [PATCH v3 4/7],
--raw-dump without an argument still shows all the events.
Yunlong Song (7):
perf list: Sort the output of 'perf list' to view more clearly
perf list: Fix some inaccuracy problem when parsing the argument
perf list: Clean up the printing functions of hardware/software events
-- the required result
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/perf-completion.sh | 6 ++
tools/perf/perf.c | 27 +++
2 files changed, 29 insertions(+), 4 deletions(-)
diff --git a/tools/perf/perf-completion.sh b/tools/perf/perf
--period --no-samples --no-buildid-cache --no-buildid --cgroup --delay
--uid --branch-any --branch-filter --weight --transaction --per-thread
--intr-regs
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/util/parse-options.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion
are at
the same line
After this patch:
$perf list --list-opts
--raw-dump
$ -- the new line
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/perf.c | 1 +
tools/perf/util/parse-options.c | 2 ++
2 files changed, 3 insertions
[Tracepoint event]
jbd2:jbd2_start_commit [Tracepoint event]
......
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/util/parse-events.c | 212 -
1 file
instructions stalled-cycles-backend
stalled-cycles-frontend
As shown above, only the hw events are printed.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/Documentation/perf-list.txt | 6 ++
tools/perf/builtin-list.c | 21 -
2 files changed, 14
above, only the events of tracepoint_foo are printed.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-list.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/tools/perf/builtin-list.c b/tools/perf/builtin-list.c
index ad8018e..2acbcf0 100644
-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-data.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/builtin-data.c b/tools/perf/builtin-data.c
index 9705ba7..0385063 100644
--- a/tools/perf/builtin-data.c
+++ b/tools/perf/builtin-data.c
@@ -86,7 +86,7
On 2015/3/24 0:15, Arnaldo Carvalho de Melo wrote:
Em Mon, Mar 23, 2015 at 11:50:05AM +0800, Yunlong Song escreveu:
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
Signed-off-by: Namhyung Kim namhy...@kernel.org
Humm, this doesn't seem to apply (Namhyung's S-o-B), as this patch was
made
process?
--
Thanks,
Yunlong Song
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
On 2015/4/24 21:31, Yunlong Song wrote:
[Profiling Background]
Now we are profiling the performance of ext4 and f2fs on an eMMC card with
iozone,
we find a case that ext4 is better than f2fs in random write under the test of
iozone -s 262144 -r 64 -i 0 -i 2. We want to analyze the I/O delay
On 2015/4/24 21:49, David Ahern wrote:
On 4/24/15 7:31 AM, Yunlong Song wrote:
[Profiling Problem]
Although perf can record the events (with call stack) of a specified pid,
e.g. using
perf record -g iozone -s 262144 -r 64 -i 0 -i 2. But we find iozone is
interrupted
That command tells
On 2015/4/24 21:49, Yunlong Song wrote:
On 2015/4/24 21:31, Yunlong Song wrote:
[Profiling Background]
Now we are profiling the performance of ext4 and f2fs on an eMMC card with
iozone,
we find a case that ext4 is better than f2fs in random write under the test
of
iozone -s 262144 -r 64
On 2015/4/24 21:58, David Ahern wrote:
On 4/24/15 7:31 AM, Yunlong Song wrote:
Now we are profiling the performance of ext4 and f2fs on an eMMC card with
iozone,
we find a case that ext4 is better than f2fs in random write under the test
of
iozone -s 262144 -r 64 -i 0 -i 2. We want
On 2015/4/25 23:53, David Ahern wrote:
On 4/25/15 8:05 AM, Yunlong Song wrote:
On 2015/4/24 21:58, David Ahern wrote:
On 4/24/15 7:31 AM, Yunlong Song wrote:
Now we are profiling the performance of ext4 and f2fs on an eMMC card with
iozone,
we find a case that ext4 is better than f2fs
On 2015/5/6 5:53, Rabin Vincent wrote:
On Sat, Apr 25, 2015 at 09:53:57AM -0600, David Ahern wrote:
On 4/25/15 8:05 AM, Yunlong Song wrote:
But this only shows the system call like strace, but we want the call
stack of kernel functions in fact.
We haven't added the callchain option yet
On 2015/3/31 21:46, Yunlong Song wrote:
Hi,
Found some functions to improve and bugs to fix in perf sched replay.
Yunlong Song (9):
perf sched replay: Use struct task_desc instead of struct task_task
for correct meaning
perf sched replay: Increase the MAX_PID value to fix
On 2015/4/1 15:06, Yunlong Song wrote:
On 2015/3/31 23:56, Arnaldo Carvalho de Melo wrote:
Em Tue, Mar 31, 2015 at 08:32:37AM -0600, David Ahern escreveu:
On 3/31/15 7:46 AM, Yunlong Song wrote:
- BUG_ON(pid = MAX_PID);
+ if (sched-pid_to_task == NULL) {
+ if (sysctl__read_int
-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-record.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c
index cfdff50..5b0962a 100644
--- a/tools/perf/builtin-record.c
+++ b/tools/perf/builtin-record.c
@@ -731,6
On 2015/4/8 16:55, Peter Zijlstra wrote:
On Wed, Apr 08, 2015 at 12:02:28PM +0800, Yunlong Song wrote:
Commit 31a9883106cc (perf record: Add clockid parameter) used
CLOCK_MONOTONIC_RAW in the struct clockid_map clockids[], but the
CLOCK_MONOTONIC_RAW macro is not defined in older releases (e.g
On 2015/4/8 21:03, Arnaldo Carvalho de Melo wrote:
Em Wed, Apr 08, 2015 at 12:02:28PM +0800, Yunlong Song escreveu:
Commit 31a9883106cc (perf record: Add clockid parameter) used
CLOCK_MONOTONIC_RAW in the struct clockid_map clockids[], but the
CLOCK_MONOTONIC_RAW macro is not defined in older
Hi,
Enable perf commands to use perf file when it is not owned by current
user or root.
Yunlong Song (10):
perf tools: Support using -f to override perf.data file ownership for
evlist
perf tools: Support using -f to override perf.data file ownership for
inject
perf tools
process, the result shows 0 sample here.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-kvm.c | 2 ++
tools/perf/util/kvm-stat.h | 1 +
2 files changed, 3 insertions(+)
diff --git a/tools/perf/builtin-kvm.c b/tools/perf/builtin-kvm.c
index 643722f..1f9338f 100644
: 0x7fffeb9a0d00, ...
0.211 ( 0.004 ms): ls/47325 stat(filename: 0x7fffeb9a0d00, ...
0.220 ( 0.007 ms): ls/47325 open(filename: 0x7f312599e8ff, ...
...
...
As shown above, the -f option really works now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin
, the -f option really works now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-evlist.c | 2 ++
tools/perf/util/evsel.h | 1 +
2 files changed, 3 insertions(+)
diff --git a/tools/perf/builtin-evlist.c b/tools/perf/builtin-evlist.c
index 0f93f85..695ec5a 100644
: 437599
Total bytes allocated: 615472
Total bytes wasted on internal fragmentation: 177873
Internal fragmentation: 28.900259%
Cross CPU allocations: 6/1192
As shown above, the -f option really works now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-kmem.c | 9
;
svg width=1000 height=10110 version=1.1
xmlns=http://www.w3.org/2000/svg;
defs
style type=text/css
![CDATA[
rect { stroke-width: 1; }
...
...
As shown above, the -f option really works now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin
works now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-inject.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/perf/builtin-inject.c b/tools/perf/builtin-inject.c
index ea46df25..40a33d7 100644
--- a/tools/perf/builtin-inject.c
+++ b/tools/perf/builtin
: 63058 cycles: 8105a592
raise_softirq ([kernel.kallsyms])
ls 41298 2590086.567448:1706255 cycles:406ae0
[unknown] (/usr/bin/ls)
As shown above, the -f option really works now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin
...
...
As shown above, the -f option really works now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-lock.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/tools/perf/builtin-lock.c b/tools/perf/builtin-lock.c
index 7893a9b..d49c2ab 100644
--- a/tools/perf/builtin
/kcore:page_remove_rmap
As shown above, the -f option really works now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-mem.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tools/perf/builtin-mem.c b/tools/perf/builtin-mem.c
index b4dcf0b..675216e 100644
--- a/tools/perf/builtin
perf_stream_0
As shown above, the -f option really works now.
Signed-off-by: Yunlong Song yunlong.s...@huawei.com
---
tools/perf/builtin-data.c | 4 +++-
tools/perf/util/data-convert-bt.c | 3 ++-
tools/perf/util/data-convert-bt.h | 2 +-
3 files changed, 6 insertions(+), 3 deletions(-)
diff
add kbuild test") dynamically creates file llvm-src-kbuild.c
during the perf building. Add them to .gitignore.
Signed-off-by: Yunlong Song <yunlong.s...@huawei.com>
---
tools/perf/tests/.gitignore | 2 ++
1 file changed, 2 insertions(+)
create mode 100644 tools/perf/tests/.gitignor
/bpf/ also mistakes FEATURE-DUMP.libbpf for FEATURE-DUMP, which
causes that "make clean" will leave (or say create) FEATURE-DUMP.libbpf
rather than remove FEATURE-DUMP.libbpf file.
Signed-off-by: Yunlong Song <yunlong.s...@huawei.com>
---
tools/lib/bpf/.gitignore | 2 +-
tools/lib/bpf/
>
> Anyway, fixing it up, thanks for the patch!
>
> - Arnaldo
Yes, right. Thanks for your detailed explanation.
--
Thanks,
Yunlong Song
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kerne
On 2015/10/14 21:40, Arnaldo Carvalho de Melo wrote:
>
> Le 14 oct. 2015 10:33 AM, "Yunlong Song" <yunlong.s...@huawei.com
> <mailto:yunlong.s...@huawei.com>> a écrit :
>>
>> On 2015/10/14 11:10, Arnaldo Carvalho de Melo wrote:
>>
>> >
new dynamic tracepoints
See 'perf help COMMAND' for more information on a specific command.
As shown above, the options description really appears now.
Signed-off-by: Yunlong Song <yunlong.s...@huawei.com>
---
tools/perf/builtin-help.c | 2 +-
tools/perf/builtin.h
Hi,
Make some fixes to perf's help for consistency.
Yunlong Song (3):
perf help: Add options description to 'perf -h'
perf help: Change 'usage' to 'Usage' for consistency
perf help: Change the usage's stdout to stderr for consistency
tools/perf/builtin-help.c | 13
Capitalize 'usage' to make it consistent with all the other 'Usage' in the
codes, e.g., usage_builtin.
Signed-off-by: Yunlong Song <yunlong.s...@huawei.com>
---
tools/perf/builtin-help.c | 2 +-
tools/perf/util/parse-options.c | 4 ++--
2 files changed, 3 insertions(+), 3 del
Commit 7c422f5572667fef0db38d2046ecce69dcf0afc8 ("tools build: Build fixdep
helper from perf and basic libs") dynamically creates fixdep during the perf
building. Add it to .gitignore.
Signed-off-by: Yunlong Song <yunlong.s...@huawei.com>
---
tools/build/.gitignore | 1 +
1
The builtins use stderr everywhere to show the usage info when the opts
or cmds are incorrectly used, for consistency, change perf's stdout to
stderr to show its usage info when it is incorrectly (including no
command) used.
Signed-off-by: Yunlong Song <yunlong.s...@huawei.com>
---
tool
e existing [OPTIONS] in
the perf_usage_string[].
>>>
>>> const char perf_usage_string[] =
>>> - "perf [--version] [--help] [OPTIONS] COMMAND [ARGS]";
>>> + "perf [--version] [--help] [--list-opts] [OPTIONS] COMMAND [ARGS]";
--
Thanks
On 2015/10/15 16:51, Yunlong Song wrote:
> Commit 7c422f5572667fef0db38d2046ecce69dcf0afc8 ("tools build: Build fixdep
> helper from perf and basic libs") dynamically creates fixdep during the perf
> building. Add it to .gitignore.
>
> Signed-off-by: Yunlong Song
On 2015/10/19 23:29, Namhyung Kim wrote:
> Hi,
>
> On Thu, Oct 15, 2015 at 03:39:50PM +0800, Yunlong Song wrote:
>> Add options description to 'perf -h' to make it consistent with other
>> builtins
>> (e.g., 'perf stat -h').
>>
>> Example:
>>
>&g
ut to stdout, just like 'ls --help', what do you think?
>
> - Arnaldo
>
I think the reason that options do not show in 'perf -h' but show in
other builtins is not the different use of stdout or stderr. I will
send a patch to fix this instead.
--
Thanks,
Yunlong Song
--
To unsubscribe
1 - 100 of 716 matches
Mail list logo
