lttng: build the tracepoint provider lib from .c files in repo

we should not have `TRACEPOINT_DEFINE` deinfed in both the application
we are tracing and the tracing provider library.

* tracing/*.c: reference the header file with path
* README.md: add instructions to add .c file
* CMakeLists.c:
  - compile the .c file in repo, not the one generated by
    lttng-gen-tp
  - no need to mark the generated header file as GENEREATED,
    they are marked so by default (they are listed in OUTPUT)
  - do not generate .c file anymore, they are not used.

Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/src/tracing/CMakeLists.txt b/src/tracing/CMakeLists.txt
index 8cde4fbc0344545baf874870c5463ffe5912967b..c434e62f595c74c0c576f507a88e2a4a39f8c28f 100644 (file)
--- a/src/tracing/CMakeLists.txt
+++ b/src/tracing/CMakeLists.txt
@@ -2,22 +2,20 @@
 # the source files include the tracing headers like
 # #include "tracing/oprequest.h". so better put them into
 # ${PROJECT_BINARY_DIR}/include, where acconfig.h is also located
-set(header_dir ${CMAKE_BINARY_DIR}/include/tracing)
+set(working_dir ${CMAKE_BINARY_DIR}/include)
+set(header_dir ${working_dir}/tracing)
 file(MAKE_DIRECTORY ${header_dir})
 
 file(GLOB tps "*.tp")
 foreach(tp ${tps})
   get_filename_component(name ${tp} NAME_WE)
   set(header ${header_dir}/${name}.h)
-  set(src ${header_dir}/${name}.c)
   add_custom_command(
-    OUTPUT ${header} ${src}
-    COMMAND ${LTTNG_GEN_TP} ${tp} -o tracing/${name}.h -o tracing/${name}.c
+    OUTPUT ${header}
+    COMMAND ${LTTNG_GEN_TP} ${tp} -o tracing/${name}.h
     DEPENDS ${tp}
-    WORKING_DIRECTORY ${CMAKE_BINARY_DIR}/include
-    COMMENT "generating ${header} ${src}")
-  set_source_files_properties(${header} ${src}
-    PROPERTIES GENERATED TRUE)
+    WORKING_DIRECTORY ${working_dir}
+    COMMENT "generating ${header}")
   add_custom_target(
     ${name}-tp
     DEPENDS ${header})
@@ -28,11 +26,8 @@ function(add_tracing_library name tracings version)
     get_filename_component(tp ${tp_file} NAME_WE)
     list(APPEND hdrs
       ${header_dir}/${tp}.h)
-    list(APPEND srcs
-      ${header_dir}/${tp}.h
-      ${header_dir}/${tp}.c)
   endforeach()
-  add_library(${name} SHARED ${srcs})
+  add_library(${name} SHARED ${hdrs} ${tp}.c)
   target_link_libraries(${name} ${LTTNGUST_LIBRARIES} ${CMAKE_DL_LIBS})
   string(REGEX MATCH "^[0-9]+" soversion ${version})
   set_target_properties(${name} PROPERTIES

diff --git a/src/tracing/README.md b/src/tracing/README.md
index 15f1ac60a4dc0ca27a51ed1be89373d909e0a744..dea1d20a2c5fe492a08988b026aacb8435fa656c 100644 (file)
--- a/src/tracing/README.md
+++ b/src/tracing/README.md
@@ -13,12 +13,17 @@ Then install as normal
     apt-get install lttng-tools liblttng-ust-dev
 
 Add/Update Provider
-================
+===================
 
 ## Create tracepoint definition file
 
 Add tracepoint definitions for the provider into a `.tp` file. Documentation
 on defining a tracepoint can be found in `man lttng-ust`. By convention files
 are named according to the logical sub-system they correspond to (e.g.
-`mutex.tp`, `pg.tp`). Place the `.tp` file into the `src/tracing` directory
+`mutex.tp`, `pg.tp`). And add a C source file to be compiled into the tracepoint
+provider shared object, in which `TRACEPOINT_DEFINE` should be defined. See
+[LTTng document](http://lttng.org/docs/#doc-dynamic-linking) for details.
+Place the `.tp` and the `.c` files into the `src/tracing` directory
 and modify the CMake file `src/tracing/CMakeLists.txt` accordingly.
+
+

diff --git a/src/tracing/librados.c b/src/tracing/librados.c
index 5b8d4d66a446faf566f5cfdfc234c819cfde17f2..ae25f3e3b0e8c01a16ade3de6b8c9dbeb4e4dfa2 100644 (file)
--- a/src/tracing/librados.c
+++ b/src/tracing/librados.c
@@ -3,4 +3,4 @@
 /*
  * The header containing our TRACEPOINT_EVENTs.
  */
-#include "librados.h"
+#include "tracing/librados.h"

diff --git a/src/tracing/librbd.c b/src/tracing/librbd.c
index 6e5977c4650b857eec8cdb27f1eb9e0e4595817d..c70bf8ef903c23160a2a2f544b48f2df5840f420 100644 (file)
--- a/src/tracing/librbd.c
+++ b/src/tracing/librbd.c
@@ -3,4 +3,4 @@
 /*
  * The header containing our TRACEPOINT_EVENTs.
  */
-#include "librbd.h"
+#include "tracing/librbd.h"

diff --git a/src/tracing/objectstore.c b/src/tracing/objectstore.c
index 21bbf271ac153ea7ae0ed26e2b38057212b7fd9e..c1ec0eb138467fc1bc53f025324f65e616756419 100644 (file)
--- a/src/tracing/objectstore.c
+++ b/src/tracing/objectstore.c
@@ -3,4 +3,4 @@
 /*
  * The header containing our TRACEPOINT_EVENTs.
  */
-#include "objectstore.h"
+#include "tracing/objectstore.h"

diff --git a/src/tracing/oprequest.c b/src/tracing/oprequest.c
index 02f1fc53d0527fed2e26c3d04a6573e6712f6749..1daa37f8335b47ff9ee416786169431e1a9f1ff9 100644 (file)
--- a/src/tracing/oprequest.c
+++ b/src/tracing/oprequest.c
@@ -3,4 +3,4 @@
 /*
  * The header containing our TRACEPOINT_EVENTs.
  */
-#include "oprequest.h"
+#include "tracing/oprequest.h"

diff --git a/src/tracing/osd.c b/src/tracing/osd.c
index ff70ddfff8739955537e9ccd7ce266dd12e3e022..5f6781491f0fbb02e211541475fccd262fb4e3d0 100644 (file)
--- a/src/tracing/osd.c
+++ b/src/tracing/osd.c
@@ -3,4 +3,4 @@
 /*
  * The header containing our TRACEPOINT_EVENTs.
  */
-#include "osd.h"
+#include "tracing/osd.h"

diff --git a/src/tracing/pg.c b/src/tracing/pg.c
index 661ebb7729bcacf059c33e43bad84029ab0c6aa3..1373ed5e4c1d142d668939de58abfa87dc82c060 100644 (file)
--- a/src/tracing/pg.c
+++ b/src/tracing/pg.c
@@ -3,4 +3,4 @@
 /*
  * The header containing our TRACEPOINT_EVENTs.
  */
-#include "pg.h"
+#include "tracing/pg.h"