On Tue, 24 Oct 2017 14:35:35 +0100 Will Deacon <will.dea...@arm.com> wrote:
> On Tue, Oct 24, 2017 at 03:04:04AM -0500, Kim Phillips wrote: > > Introduce new tools/perf/arch/*/util/evsel.c:perf_evsel__suppl_strerror() > > so each arch can start to customize usability for its h/w PMU drivers. > > > > Signed-off-by: Kim Phillips <kim.phill...@arm.com> > > --- > > tools/perf/arch/x86/util/Build | 1 + > > tools/perf/arch/x86/util/evsel.c | 24 ++++++++++++++++++++++++ > > tools/perf/util/evsel.c | 21 +++++++++++++++------ > > tools/perf/util/evsel.h | 2 ++ > > 4 files changed, 42 insertions(+), 6 deletions(-) > > create mode 100644 tools/perf/arch/x86/util/evsel.c > > This looks sensible to me, although it's difficult to justify all of the > parameters to perf_evsel__suppl_strerror judging by this patch alone. I looked at possibly reducing the number of parameters, but since both evsel and target elements are used to evaluate the condition the error occurred in, and err, msg, and size are required for basic strerror functioning, they all look to stay: see the shared strerror function to see how they're all used. Thanks, Kim
