On 07/07/2014 04:57 PM, Mathieu Desnoyers wrote: > Merged into master and stable-2.4, thanks!
Thanks ! Hmmm ... but what to do about static inline function cds_list_empty (included via lttng/ust-tracepoint-event.h -> urcu/rculist.h -> urcu/list.h). Would you accept a patch that introduces rcu_cds_notrace in urcu/compiler.h + respective changes in urcu/*.h ? -- Paul > > Mathieu > > ----- Original Message ----- >> From: "Paul Woegerer" <paul_woege...@mentor.com> >> To: lttng-dev@lists.lttng.org, "mathieu desnoyers" >> <mathieu.desnoy...@efficios.com> >> Cc: "Paul Woegerer" <paul_woege...@mentor.com> >> Sent: Monday, July 7, 2014 10:04:21 AM >> Subject: [PATCH lttng-ust] Add lttng_ust_notrace to static inline functions >> >> The static inline functions >> __tracepoint_provider_mismatch_<TRACEPOINT_PROVIDER> and >> __tracepoint_provider_check_<TRACEPOINT_PROVIDER> are missing the >> lttng_ust_notrace attribute to prevent them from getting instrumented >> with -finstrument-functions (if e.g. defined in CFLAGS). This patch is >> adding the attribute to those functions. >> >> Signed-off-by: Paul Woegerer <paul_woege...@mentor.com> >> --- >> include/lttng/ust-tracepoint-event.h | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/include/lttng/ust-tracepoint-event.h >> b/include/lttng/ust-tracepoint-event.h >> index d12e8bb..8895310 100644 >> --- a/include/lttng/ust-tracepoint-event.h >> +++ b/include/lttng/ust-tracepoint-event.h >> @@ -75,6 +75,8 @@ >> /* Reset all macros within TRACEPOINT_EVENT */ >> #include <lttng/ust-tracepoint-event-reset.h> >> >> +static inline lttng_ust_notrace >> +void _TP_COMBINE_TOKENS(__tracepoint_provider_mismatch_, >> TRACEPOINT_PROVIDER)(void); >> static inline >> void _TP_COMBINE_TOKENS(__tracepoint_provider_mismatch_, >> TRACEPOINT_PROVIDER)(void) >> { >> @@ -88,6 +90,8 @@ void _TP_COMBINE_TOKENS(__tracepoint_provider_mismatch_, >> TRACEPOINT_PROVIDER)(vo >> #define TRACEPOINT_EVENT_INSTANCE(_provider, _template, _name, _args) >> \ >> __tracepoint_provider_mismatch_##_provider(); >> >> +static inline lttng_ust_notrace >> +void _TP_COMBINE_TOKENS(__tracepoint_provider_check_, >> TRACEPOINT_PROVIDER)(void); >> static inline >> void _TP_COMBINE_TOKENS(__tracepoint_provider_check_, >> TRACEPOINT_PROVIDER)(void) >> { >> -- >> 2.0.1 >> >> > -- Paul Woegerer, SW Development Engineer Sourcery Analyzer <http://go.mentor.com/sourceryanalyzer> Mentor Graphics, Embedded Software Division _______________________________________________ lttng-dev mailing list lttng-dev@lists.lttng.org http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev