Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1391245
| From | Arnaldo Carvalho de Melo <acme@kernel.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH perf/core v2 0/8] perf tools: Update strbuf to remove xrealloc |
| Date | 2016-04-29 17:20 +0200 |
| Message-ID | <rtin0-8fU-29@gated-at.bofh.it> (permalink) |
| References | <rtidl-8c3-29@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
Em Sat, Apr 30, 2016 at 12:09:41AM +0900, Masami Hiramatsu escreveu: > Hi Arnaldo, > > Here is the 2nd version of the strbuf update, and just rebased on > the latest perf/core :) In order to avoid confusion, I've added > v2 on tag. > > This patch series does refactoring strbuf and xrealloc related code > to remove xrealloc since it can call die() to exit immediately when > it hits any error. Instead of that, it should return error code to > the caller so that the caller can handle its error. > > Thus, at first, this changes the strbuf APIs to return error code > instead of die() immediately. And then changing API callers according > to the following rules. > - Check the return value of strbuf APIs and handle errors and, > - If the caller returns an error code (errno), it returns > the return value of strbuf APIs. > - If the caller just return -1 or NULl in error case, it also > returns -1 or NULL in case of strbuf error. > - If the caller can call die() directly, it also call die() > in case of strbuf error. > - Error checking patches are splitted for each subcommand, since > it will help review. > > This also removes xrealloc and ALLOC_GROW from libperf, so that no > one use it anymore. Thanks a lot, I'll try reviewing this this weekend, - Arnaldo > Thank you, > > --- > > Masami Hiramatsu (8): > perf: Rewrite strbuf not to die > perf probe: Check the return value of strbuf APIs > perf help: Make check_emacsclient_version to check strbuf APIs > perf: Make alias handler to check return value of strbuf > perf header: Make topology checkers to check return value of strbuf > perf pmu: Make pmu_formats_string to check return value of strbuf > perf help: Do not use ALLOC_GROW in add_cmd_list > perf tools: Remove xrealloc and ALLOC_GROW > > > tools/perf/builtin-help.c | 18 ++-- > tools/perf/perf.c | 8 +- > tools/perf/util/Build | 1 > tools/perf/util/cache.h | 19 ----- > tools/perf/util/dwarf-aux.c | 50 +++++------- > tools/perf/util/header.c | 31 +++++--- > tools/perf/util/help-unknown-cmd.c | 30 +++++-- > tools/perf/util/pmu.c | 10 +- > tools/perf/util/probe-event.c | 146 +++++++++++++++++++++--------------- > tools/perf/util/probe-finder.c | 30 ++++--- > tools/perf/util/quote.c | 36 +++++---- > tools/perf/util/quote.h | 2 > tools/perf/util/strbuf.c | 93 +++++++++++++++++------ > tools/perf/util/strbuf.h | 25 ++++-- > tools/perf/util/util.h | 6 - > tools/perf/util/wrapper.c | 29 ------- > 16 files changed, 291 insertions(+), 243 deletions(-) > delete mode 100644 tools/perf/util/wrapper.c > > -- > Masami Hiramatsu
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH perf/core v2 0/8] perf tools: Update strbuf to remove xrealloc Masami Hiramatsu <mhiramat@kernel.org> - 2016-04-29 17:10 +0200
[PATCH perf/core v2 2/8] perf probe: Check the return value of strbuf APIs Masami Hiramatsu <mhiramat@kernel.org> - 2016-04-29 17:20 +0200
Re: [PATCH perf/core v2 2/8] perf probe: Check the return value of strbuf APIs Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-05-06 01:50 +0200
Re: [PATCH perf/core v2 2/8] perf probe: Check the return value of strbuf APIs Masami Hiramatsu <mhiramat@kernel.org> - 2016-05-10 04:40 +0200
[PATCH perf/core v2 5/8] perf header: Make topology checkers to check return value of strbuf Masami Hiramatsu <mhiramat@kernel.org> - 2016-04-29 17:20 +0200
Re: [PATCH perf/core v2 5/8] perf header: Make topology checkers to check return value of strbuf Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-05-06 02:00 +0200
Re: [PATCH perf/core v2 5/8] perf header: Make topology checkers to check return value of strbuf Masami Hiramatsu <mhiramat@kernel.org> - 2016-05-10 05:00 +0200
[PATCH perf/core v2 8/8] perf tools: Remove xrealloc and ALLOC_GROW Masami Hiramatsu <mhiramat@kernel.org> - 2016-04-29 17:20 +0200
Re: [PATCH perf/core v2 8/8] perf tools: Remove xrealloc and ALLOC_GROW Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-05-06 02:00 +0200
[PATCH perf/core v2 3/8] perf help: Make check_emacsclient_version to check strbuf APIs Masami Hiramatsu <mhiramat@kernel.org> - 2016-04-29 17:20 +0200
Re: [PATCH perf/core v2 3/8] perf help: Make check_emacsclient_version to check strbuf APIs Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-05-06 02:00 +0200
[PATCH perf/core v2 4/8] perf: Make alias handler to check return value of strbuf Masami Hiramatsu <mhiramat@kernel.org> - 2016-04-29 17:20 +0200
Re: [PATCH perf/core v2 4/8] perf: Make alias handler to check return value of strbuf Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-05-06 02:00 +0200
[PATCH perf/core v2 6/8] perf pmu: Make pmu_formats_string to check return value of strbuf Masami Hiramatsu <mhiramat@kernel.org> - 2016-04-29 17:20 +0200
Re: [PATCH perf/core v2 6/8] perf pmu: Make pmu_formats_string to check return value of strbuf Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-05-06 02:00 +0200
[PATCH perf/core v2 7/8] perf help: Do not use ALLOC_GROW in add_cmd_list Masami Hiramatsu <mhiramat@kernel.org> - 2016-04-29 17:20 +0200
Re: [PATCH perf/core v2 7/8] perf help: Do not use ALLOC_GROW in add_cmd_list Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-05-06 02:00 +0200
Re: [PATCH perf/core v2 0/8] perf tools: Update strbuf to remove xrealloc Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-04-29 17:20 +0200
[PATCH perf/core v2 1/8] perf: Rewrite strbuf not to die Masami Hiramatsu <mhiramat@kernel.org> - 2016-04-29 17:20 +0200
Re: [PATCH perf/core v2 1/8] perf: Rewrite strbuf not to die Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-05-06 01:30 +0200
Re: [PATCH perf/core v2 1/8] perf: Rewrite strbuf not to die Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-05-06 01:50 +0200
Re: [PATCH perf/core v2 1/8] perf: Rewrite strbuf not to die Masami Hiramatsu <mhiramat@kernel.org> - 2016-05-10 03:10 +0200
csiph-web