Subject: [PATCH 6/6] sh: oprofile: Use perf-events oprofile backend

From: [email-masked] (Matt Fleming)
Date: Mon, 27 Sep 2010 21:01:38 +0100

On Thu, Sep 16, 2010 at 04:32:54PM +0200, Robert Richter wrote:
> > diff --git a/arch/sh/kernel/perf_event.c b/arch/sh/kernel/perf_event.c
> > index 2cb9ad5..3c3fc9a 100644
> > --- a/arch/sh/kernel/perf_event.c
> > +++ b/arch/sh/kernel/perf_event.c
> > @@ -59,6 +59,14 @@ static inline int sh_pmu_initialized(void)
> >  	return !!sh_pmu;
> >  }
> >  
> > +const char *sh_pmu_name(void)
> > +{
> > +	if (!sh_pmu)
> > +		return NULL;
> > +
> > +	return sh_pmu->name;
> > +}
> 
> Couldn't we make this a generic function like perf_num_counters()?

Well, ARM doesn't have names as strings for its pmus currently. What's
more, ARM wouldn't use it; SH would be the only user of this function. I
don't think this one makes sense to be a generic function.