Hi Mark,
Pretty strange to have a compiler bug like this, but I'm happy with the change. Did you raise it against the toolchain?
--Chris
-----Original Message----- From: Mark Brown [mailto:broonie@kernel.org] Sent: 14 October 2013 12:42 To: Liviu Dudau; Chris Redpath; Jon Medhurst; Alex Shi Cc: linaro-kernel@lists.linaro.org; Mark Brown Subject: [PATCH] smp: Don't use typedef to work around compiler issue with tracepoints
From: Mark Brown broonie@linaro.org
Having the typedef in place for the tracepoints causes compiler crashes in some situations. Just using void * directly avoids triggering the issue and should have no effect on the trace.
Signed-off-by: Mark Brown broonie@linaro.org
include/trace/events/smp.h | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/include/trace/events/smp.h b/include/trace/events/smp.h index c8abfd7..da0baf2 100644 --- a/include/trace/events/smp.h +++ b/include/trace/events/smp.h @@ -5,11 +5,10 @@ #define _TRACE_SMP_H
#include <linux/tracepoint.h> -typedef void (*__smp_call_func_t)(void *info);
DECLARE_EVENT_CLASS(smp_call_class,
TP_PROTO(__smp_call_func_t fnc),
TP_PROTO(void * fnc), TP_ARGS(fnc),@@ -35,7 +34,7 @@ DECLARE_EVENT_CLASS(smp_call_class, */ DEFINE_EVENT(smp_call_class, smp_call_func_entry,
TP_PROTO(__smp_call_func_t fnc),
TP_PROTO(void * fnc), TP_ARGS(fnc)); @@ -51,7 +50,7 @@ DEFINE_EVENT(smp_call_class, smp_call_func_entry, */ DEFINE_EVENT(smp_call_class, smp_call_func_exit,
TP_PROTO(__smp_call_func_t fnc),
TP_PROTO(void * fnc), TP_ARGS(fnc)); @@ -67,7 +66,7 @@ DEFINE_EVENT(smp_call_class, smp_call_func_exit, */ TRACE_EVENT(smp_call_func_send,
TP_PROTO(__smp_call_func_t func, int dest),
TP_PROTO(void * func, int dest), TP_ARGS(func, dest),-- 1.8.4.rc3
-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2557590 ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2548782