lkml.org 
[lkml]   [2009]   [Aug]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[PATCH 2/2] trace_events: fix the include file dependencies
    The TRACE_EVENT depend on the include/linux/tracepoint.h first
    and include/trace/ftrace.h later, if we include the ftrace.h early,
    It'll occur building error, like blow:

    Both define TRACE_EVENT in trace_a.h and trace_b.h, if we include
    those in .c file, like this:

    #define CREATE_TRACE_POINTS
    include <trace/events/trace_a.h>
    include <trace/events/trace_b.h>

    There are can't work, because the TRACE_EVENT has re-defined by
    the previous .h file

    Reported-by: Wei Yongjun <yjwei@cn.fujitsu.com>
    Signed-off-by: Xiao Guangrong <xiaoguangrong@cn.fujitsu.com>
    ---
    include/linux/tracepoint.h | 3 +--
    include/trace/define_trace.h | 1 +
    2 files changed, 2 insertions(+), 2 deletions(-)

    diff --git a/include/linux/tracepoint.h b/include/linux/tracepoint.h
    index 5984ed0..8170985 100644
    --- a/include/linux/tracepoint.h
    +++ b/include/linux/tracepoint.h
    @@ -180,6 +180,7 @@ static inline void tracepoint_synchronize_unregister(void)
    }

    #define PARAMS(args...) args
    +#endif

    #ifndef TRACE_EVENT
    /*
    @@ -287,5 +288,3 @@ static inline void tracepoint_synchronize_unregister(void)
    #define TRACE_EVENT(name, proto, args, struct, assign, print) \
    DECLARE_TRACE(name, PARAMS(proto), PARAMS(args))
    #endif
    -
    -#endif
    diff --git a/include/trace/define_trace.h b/include/trace/define_trace.h
    index 76e93bf..202cecd 100644
    --- a/include/trace/define_trace.h
    +++ b/include/trace/define_trace.h
    @@ -52,6 +52,7 @@
    #include <trace/ftrace.h>
    #endif

    +#undef TRACE_EVENT
    #undef TRACE_HEADER_MULTI_READ

    /* Only undef what we defined in this file */
    --
    1.6.1.2



    \
     
     \ /
      Last update: 2009-08-25 08:09    [W:0.021 / U:90.620 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site