Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934187AbbEOIle (ORCPT ); Fri, 15 May 2015 04:41:34 -0400 Received: from szxga02-in.huawei.com ([119.145.14.65]:30172 "EHLO szxga02-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933435AbbEOIl3 (ORCPT ); Fri, 15 May 2015 04:41:29 -0400 Message-ID: <5555B119.5040201@huawei.com> Date: Fri, 15 May 2015 16:40:57 +0800 From: "Wangnan (F)" User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Ingo Molnar CC: , , , , , , , , , , , , , , Subject: Re: [RFC PATCH v2 00/37] perf tools: introduce 'perf bpf' command to load eBPF programs. References: <1431676290-1230-1-git-send-email-wangnan0@huawei.com> <20150515080330.GA5277@gmail.com> In-Reply-To: <20150515080330.GA5277@gmail.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.111.66.109] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5441 Lines: 127 在 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/