lkml.org 
[lkml]   [2015]   [Jan]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 21/26] perf trace: Fix error reporting for evsel pgfault constructor
    Date
    From: Arnaldo Carvalho de Melo <acme@redhat.com>

    In that case the only failure possible is not to have enough memory, as
    we are just creating the evsels, not trying to access any system
    facility such as debugfs files or syscalls.

    Cc: Adrian Hunter <adrian.hunter@intel.com>
    Cc: Borislav Petkov <bp@suse.de>
    Cc: David Ahern <dsahern@gmail.com>
    Cc: Don Zickus <dzickus@redhat.com>
    Cc: Frederic Weisbecker <fweisbec@gmail.com>
    Cc: Jiri Olsa <jolsa@redhat.com>
    Cc: Namhyung Kim <namhyung@kernel.org>
    Cc: Stephane Eranian <eranian@google.com>
    Link: http://lkml.kernel.org/n/tip-7k6asvfhiwiu2zs6o2oknchk@git.kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    ---
    tools/perf/builtin-trace.c | 13 +++++--------
    1 file changed, 5 insertions(+), 8 deletions(-)

    diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c
    index 684609d7a83d..eaaa540bf1f3 100644
    --- a/tools/perf/builtin-trace.c
    +++ b/tools/perf/builtin-trace.c
    @@ -2063,18 +2063,12 @@ static int trace__run(struct trace *trace, int argc, const char **argv)

    if ((trace->trace_pgfaults & TRACE_PFMAJ) &&
    perf_evlist__add_pgfault(evlist, PERF_COUNT_SW_PAGE_FAULTS_MAJ)) {
    - /*
    - * FIXME: This one needs better error handling, as by now we
    - * already checked that debugfs is mounted and that we have access to it,
    - * so probably the case is that something is busted wrt this specific
    - * software event, ditto for the next gotos to out_error_tp...
    - */
    - goto out_error_tp;
    + goto out_error_mem;
    }

    if ((trace->trace_pgfaults & TRACE_PFMIN) &&
    perf_evlist__add_pgfault(evlist, PERF_COUNT_SW_PAGE_FAULTS_MIN))
    - goto out_error_tp;
    + goto out_error_mem;

    if (trace->sched &&
    perf_evlist__add_newtp(evlist, "sched", "sched_stat_runtime",
    @@ -2225,6 +2219,9 @@ out_error:
    fprintf(trace->output, "%s\n", errbuf);
    goto out_delete_evlist;
    }
    +out_error_mem:
    + fprintf(trace->output, "Not enough memory to run!\n");
    + goto out_delete_evlist;
    }

    static int trace__replay(struct trace *trace)
    --
    1.9.3


    \
     
     \ /
      Last update: 2015-01-26 19:01    [W:2.893 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site