Re: [PATCH] Add support for Xtensa architecture

On 05/12/2015 08:27 AM, Max Filippov wrote:
> Signed-off-by: Max Filippov <[email protected]>
> ---
>  libperf_events/operf_utils.h | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/libperf_events/operf_utils.h b/libperf_events/operf_utils.h
> index 32954cc..ad7e961 100644
> --- a/libperf_events/operf_utils.h
> +++ b/libperf_events/operf_utils.h
> @@ -190,4 +190,10 @@ void op_release_resources(void);
>  #define cpu_relax()    rmb()
>  #endif
>  
> +#ifdef __xtensa__
> +#include <asm/unistd.h>
> +#define rmb()		asm volatile("" ::: "memory")
> +#define cpu_relax()	asm volatile("" ::: "memory")
> +#endif
> +
>  #endif // OPERF_H_
> 


Hi Max,

Is this just to get the code to compile or are other patches required to support the xtensa processors?  There doesn't seem to be event files for the xtensa processors.  What does the output of ophelp and a run of "operf ls; opreport" look like?

-Will

------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud 
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y