Date:	Fri, 15 May 2015 16:40:57 +0800
From:	"Wangnan (F)" <wangnan0@...wei.com>
To:	Ingo Molnar <mingo@...nel.org>
CC:	<paulus@...ba.org>, <a.p.zijlstra@...llo.nl>, <mingo@...hat.com>,
	<acme@...nel.org>, <namhyung@...nel.org>, <jolsa@...nel.org>,
	<adrian.hunter@...el.com>, <dsahern@...il.com>, <ast@...mgrid.com>,
	<daniel@...earbox.net>, <brendan.d.gregg@...il.com>,
	<masami.hiramatsu.pt@...achi.com>, <linux-kernel@...r.kernel.org>,
	<lizefan@...wei.com>, <pi3orama@....com>
Subject: Re: [RFC PATCH v2 00/37] perf tools: introduce 'perf bpf' command
 to load eBPF programs.



在 2015/5/15 16:03, Ingo Molnar 写道:
> Just a small stylistic side note:
>
>> Wang Nan (37):
>>    tools perf: set vmlinux_path__nr_entries to 0 in vmlinux_path__exit.
>>    tools lib traceevent: install libtraceevent.a into libdir.
>>    tools build: Allow other override features to check.
>>    tools include: add __aligned_u64 to types.h.
>>    tools lib bpf: introduce 'bpf' library to tools.
>>    tools lib bpf: allow set printing function.
>>    tools lib bpf: defines basic interface.
>>    tools lib bpf: open eBPF object file and do basic validation.
>>    tools lib bpf: check swap according to EHDR.
>>    tools lib bpf: iterater over elf sections to collect information.
>>    tools lib bpf: collect version and license from ELF.
>>    tools lib bpf: collect map definitions.
>>    tools lib bpf: collect config section in object.
>>    tools lib bpf: collect symbol table in object files.
>>    tools lib bpf: collect bpf programs from object files.
>>    tools lib bpf: collect relocation sections from object file.
>>    tools lib bpf: collect relocation instructions for each program.
>>    tools lib bpf: clean elf memory after loading.
>>    tools lib bpf: add bpf.c/h for common bpf operations.
>>    tools lib bpf: create maps needed by object file.
>>    tools lib bpf: relocation programs.
>>    tools lib bpf: introduce bpf_load_program to bpf.c.
>>    tools lib bpf: load bpf programs in object file into kernel.
>>    tools lib bpf: accessors of bpf_program.
>>    tools lib bpf: accessors for struct bpf_object.
>>    tools perf: Add new 'perf bpf' command.
>>    tools perf: make perf depend on libbpf.
>>    tools perf: add 'perf bpf record' subcommand.
>>    tools perf: add bpf-loader and open elf object files.
>>    tools perf: collect all bpf programs.
>>    tools perf: config probe points of eBPF programs during prepartion.
>>    tools perf bpf: probe at kprobe points.
>>    tools perf bpf: load eBPF object into kernel.
>>    tools perf: add a bpf_wrapper global flag.
>>    tools perf: add bpf_fd field to evsel and introduce new event syntax.
>>    tools perf: generate event argv.
>>    tools perf bpf: passes generated arguments to cmd_record.
> The titles of the changes have numerous problems and inconsistencies:
>
>   - use consistent capitalization, i.e.:
>
>      tools perf: Collect all bpf programs
>
>   - don't use period at the end, i.e.:
>
>      tools perf: Generate event argv
>
>   - use consistent present tense verbs, i.e.:
>
>      -  tools lib bpf: defines basic interface.
>      +  tools lib bpf: Define basic interface
>
>   - Always use verbs! I.e. these are bad:
>
>      -   tools perf: config probe points of eBPF programs during prepartion.
>      -   tools lib bpf: relocation programs.
>
>   - take a look at 'git log tools/perf' to see what the established
>     title style is. For example it's not 'tools perf' but 'perf tools',
>     etc.
>
> etc.
>
> There's not a single title in this series that has a proper title.
> This makes the shortlog a difficult read and gives a bad first
> impression. Please fix.

Sorry for the inconvenience. I have fixed titles in my local tree as 
follow. You will see the fixed title
in next version.

Thank you.

   perf tools: Set vmlinux_path__nr_entries to 0 in vmlinux_path__exit
   tools lib traceevent: Install libtraceevent.a into libdir
   tools: Allow other override features to check
   tools: Add __aligned_u64 to types.h
   tools lib bpf: Introduce 'bpf' library to tools
   tools lib bpf: Allow set printing function
   tools lib bpf: Define basic interface
   tools lib bpf: Open eBPF object file and do basic validation
   tools lib bpf: Check swap according to EHDR
   tools lib bpf: Iterater over elf sections to collect information
   tools lib bpf: Collect version and license from ELF
   tools lib bpf: Collect map definitions
   tools lib bpf: Collect config section in object
   tools lib bpf: Collect symbol table in object files
   tools lib bpf: Collect bpf programs from object files
   tools lib bpf: Collect relocation sections from object file
   tools lib bpf: Collect relocation instructions for each program
   tools lib bpf: Clean elf memory after loading
   tools lib bpf: Add bpf.c/h for common bpf operations
   tools lib bpf: Create maps needed by object file
   tools lib bpf: Relocate programs
   tools lib bpf: Introduce bpf_load_program to bpf.c
   tools lib bpf: Load bpf programs in object file into kernel
   tools lib bpf: Introduce accessors of bpf_program
   tools lib bpf: Introduce accessors for struct bpf_object
   perf tools: Add new 'perf bpf' command
   perf tools: Make perf depend on libbpf
   perf bpf: Add 'perf bpf record' subcommand
   perf bpf: Add bpf-loader and open elf object files
   perf bpf: Collect all bpf programs
   perf bpf: Parse probe points of eBPF programs during prepartion
   perf bpf: Probe at kprobe points
   perf bpf: Load eBPF object into kernel
   perf tools: Add a bpf_wrapper global flag
   perf tools: Add bpf_fd field to evsel and introduce new event syntax
   perf bpf: Generate event argv for other commands
   perf bpf: Pass generated arguments to cmd_record

> Thanks,
>
>      Ingo


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists