Skip to content

perf report: Remove unnecessary check in annotate_browser_write()

Arnaldo Carvalho de Melo linux-arm-kernel lkml 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest23 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/23perf report: Remove unnecessary check in annotate_browser_write()tools/perf/ui/browsers/annotate.c+10 -140 replies
2/23perf annotate browser: Display titles in left frametools/perf/ui/browsers/annotate.c+21 -30 replies
3/23perf config: Invert an if statement to reduce nesting in cmd_config()tools/perf/builtin-config.c+23 -210 replies
4/23perf script: Allow adding and removing fieldstools/perf/Documentation/perf-script.txt tools/perf/builtin-script.c+42 -30 replies
5/23tools: Adopt __noreturn from kernel sourcestools/include/linux/compiler-gcc.h tools/perf/util/scripting-engines/trace-event-python.c tools/perf/util/usage.c +1 more+11 -100 replies
6/23tools: Adopt __printf from kernel sourcestools/include/linux/compiler-gcc.h tools/perf/arch/arm/util/cs-etm.c tools/perf/util/cache.h +5 more+21 -230 replies
7/23tools: Adopt __scanf from kernel sourcestools/include/linux/compiler-gcc.h tools/perf/util/pmu.h+3 -20 replies
8/23perf tools: Use __maybe_unused consistentlytools/perf/bench/numa.c tools/perf/jvmti/jvmti_agent.h tools/perf/jvmti/libjvmti.c +3 more+8 -110 replies
9/23tools: Adopt noinline from kernel sourcestools/include/linux/compiler-gcc.h tools/include/linux/compiler.h tools/perf/tests/bp_signal.c +2 more+13 -140 replies
10/23tools: Adopt __packed from kernel sourcestools/include/linux/compiler-gcc.h tools/perf/util/genelf_debug.c+5 -20 replies
11/23tools: Adopt __aligned from kernel sourcestools/include/linux/compiler-gcc.h tools/perf/util/evlist.h+3 -10 replies
12/23perf coresight: Remove superfluous check before usetools/perf/arch/arm/util/cs-etm.c+12 -130 replies
13/23perf intel-pt/bts: Remove unused SAMPLE_SIZE defines and bts priv arraytools/perf/arch/x86/util/intel-bts.c tools/perf/arch/x86/util/intel-pt.c tools/perf/util/intel-bts.c+0 -100 replies
14/23perf annotate: Return arch from symbol__disassemble() and save it in browsertools/perf/builtin-top.c tools/perf/ui/browsers/annotate.c tools/perf/ui/gtk/annotate.c +2 more+19 -60 replies
15/23perf test llvm: Avoid error when PROFILE_ALL_BRANCHES is settools/perf/tests/bpf-script-test-prologue.c+9 -00 replies
16/23perf script: Support -F brstack,dso and brstacksym,dsotools/perf/builtin-script.c+49 -120 replies
17/23perf script: Support -F brstackoff,dsotools/perf/Documentation/perf-script.txt tools/perf/builtin-script.c+55 -50 replies
18/23perf ftrace: Show error message when fails to set ftrace filestools/perf/builtin-ftrace.c+5 -20 replies
19/23perf ftrace: Move setup_pager before opening trace_pipetools/perf/builtin-ftrace.c+2 -20 replies
20/23perf ftrace: Add option for function filteringtools/perf/Documentation/perf-ftrace.txt tools/perf/builtin-ftrace.c+141 -60 replies
21/23perf ftrace: Add -D option for depth filtertools/perf/Documentation/perf-ftrace.txt tools/perf/builtin-ftrace.c+34 -00 replies
22/23perf config: Check error cases of {show_spec, set}_config()tools/perf/builtin-config.c+15 -20 replies
23/23perf config: Refactor the code using 'ret' variable in cmd_config()tools/perf/builtin-config.c+13 -170 replies

Reviews trailers per patch

5 of 23 patches have a Reviewed-by or Acked-by.

PatchMathieu PoirierMilian WolffJiri OlsaArnaldo Carvalho de MeloThomas-Mich RichterAdrian Hunter
1/23A-b
2/23A-b
3/23
4/23T-b A-b T-b
5/23
6/23
7/23
8/23
9/23
10/23
11/23
12/23A-b
13/23A-b
14/23
15/23Reported-and-Tested-by
16/23T-b
17/23T-b
18/23
19/23
20/23T-b
21/23T-b
22/23
23/23

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 6 reviewers.

Read the v1 thread