[PATCH v2 0/7] perf kwork: A couple of fixes according to review comments

Yang Jihong posted 7 patches 3 years, 8 months ago
tools/perf/builtin-kwork.c                 | 67 +++++++++++++---------
tools/perf/util/bpf_skel/kwork_trace.bpf.c |  2 +-
2 files changed, 40 insertions(+), 29 deletions(-)
[PATCH v2 0/7] perf kwork: A couple of fixes according to review comments
Posted by Yang Jihong 3 years, 8 months ago
Some issues are fixed based on the comments of several reviewers:
1. Add '--synth task' option for record (note from Namhyung)
2. Replace hard-coded initialization of nr_tracepoints with ARRAY_SIZE macro (note from Namhyung)
3. Fix spelling mistake: "PRINT_TIME_UNIT_MESC_WIDTH" -> "PRINT_TIME_UNIT_MSEC_WIDTH" (note from Namhyung)
4. Fix spelling mistake: "Captuer" -> "Capture" (note from Colin)
5. Add some {} for multiline for/if blocks (note from Arnaldo)

Modifying based on the latest commit (9a0b36266f7a) at acme/perf/core:
git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git perf/core

Yang Jihong (7):
  perf kwork: Add '--synth task' option for record
  perf kwork: Replace hard-coded initialization of nr_tracepoints with
    ARRAY_SIZE macro
  perf kwork: Fix spelling mistake: "Captuer" -> "Capture"
  perf kwork: Fix spelling mistake: "PRINT_TIME_UNIT_MESC_WIDTH" ->
    "PRINT_TIME_UNIT_MSEC_WIDTH"
  perf kwork: Strdup function name in workqueue_work_init
  perf kwork: Expand capacity of bpf entries
  perf kwork: Add some {} for multiline for/if blocks

 tools/perf/builtin-kwork.c                 | 67 +++++++++++++---------
 tools/perf/util/bpf_skel/kwork_trace.bpf.c |  2 +-
 2 files changed, 40 insertions(+), 29 deletions(-)

-- 
2.30.GIT
Re: [PATCH v2 0/7] perf kwork: A couple of fixes according to review comments
Posted by Yang Jihong 3 years, 8 months ago
Hello,

PING.

Regards,
Jihong
.

On 2022/7/30 11:03, Yang Jihong wrote:
> Some issues are fixed based on the comments of several reviewers:
> 1. Add '--synth task' option for record (note from Namhyung)
> 2. Replace hard-coded initialization of nr_tracepoints with ARRAY_SIZE macro (note from Namhyung)
> 3. Fix spelling mistake: "PRINT_TIME_UNIT_MESC_WIDTH" -> "PRINT_TIME_UNIT_MSEC_WIDTH" (note from Namhyung)
> 4. Fix spelling mistake: "Captuer" -> "Capture" (note from Colin)
> 5. Add some {} for multiline for/if blocks (note from Arnaldo)
> 
> Modifying based on the latest commit (9a0b36266f7a) at acme/perf/core:
> git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git perf/core
> 
> Yang Jihong (7):
>    perf kwork: Add '--synth task' option for record
>    perf kwork: Replace hard-coded initialization of nr_tracepoints with
>      ARRAY_SIZE macro
>    perf kwork: Fix spelling mistake: "Captuer" -> "Capture"
>    perf kwork: Fix spelling mistake: "PRINT_TIME_UNIT_MESC_WIDTH" ->
>      "PRINT_TIME_UNIT_MSEC_WIDTH"
>    perf kwork: Strdup function name in workqueue_work_init
>    perf kwork: Expand capacity of bpf entries
>    perf kwork: Add some {} for multiline for/if blocks
> 
>   tools/perf/builtin-kwork.c                 | 67 +++++++++++++---------
>   tools/perf/util/bpf_skel/kwork_trace.bpf.c |  2 +-
>   2 files changed, 40 insertions(+), 29 deletions(-)
>