From a7130cb8b6f610218c8f50d34445e35f329072df Mon Sep 17 00:00:00 2001 From: "Eunji, Lee" Date: Thu, 26 May 2016 16:01:51 +0900 Subject: [PATCH] add ttrace_exension_write() call to trace APIs Change-Id: Ia76b586c2ec8aae7482221a844c418971f6933e9 Signed-off-by: Eunji, Lee --- CMakeLists.txt | 2 +- packaging/atrace-bootup.sh | 2 +- packaging/ttrace.spec | 1 + src/ttrace.c | 63 +++++++++++++++++++++++++++++++++------------- ttrace.pc.in | 2 +- 5 files changed, 50 insertions(+), 20 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 1a1abf6..e5f4c4e 100755 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -46,7 +46,7 @@ SET(HEADERS_ttrace ttrace.h trace.h) INCLUDE(FindPkgConfig) -pkg_check_modules(pkg_ttrace REQUIRED dlog capi-base-common) +pkg_check_modules(pkg_ttrace REQUIRED dlog capi-base-common ttrace-extension-static) FOREACH(flag ${pkg_ttrace_CFLAGS}) SET(EXTRA_CFLAGS_common "${EXTRA_CFLAGS_common} ${flag}") ENDFOREACH(flag) diff --git a/packaging/atrace-bootup.sh b/packaging/atrace-bootup.sh index 7f07a3d..775b5d2 100755 --- a/packaging/atrace-bootup.sh +++ b/packaging/atrace-bootup.sh @@ -13,7 +13,7 @@ function HELP { CONF="/etc/ttrace.conf" SPACE=" " -COMMAND="atrace --async_start" +COMMAND="atrace --async_start --append" DEFTAGS="" NUMARGS=$# diff --git a/packaging/ttrace.spec b/packaging/ttrace.spec index b401717..7478a3f 100755 --- a/packaging/ttrace.spec +++ b/packaging/ttrace.spec @@ -12,6 +12,7 @@ BuildRequires: pkgconfig(dlog) BuildRequires: pkgconfig(zlib) BuildRequires: pkgconfig(capi-base-common) BuildRequires: pkgconfig(libsmack) +BuildRequires: pkgconfig(ttrace-extension-static) BuildRequires: cmake %define keepstatic 1 diff --git a/src/ttrace.c b/src/ttrace.c index 4fb4011..e89fc41 100755 --- a/src/ttrace.c +++ b/src/ttrace.c @@ -25,6 +25,7 @@ #include "ttrace.h" #include "trace.h" #include "dlog.h" +#include "ttrace-extension.h" #define ENABLE_TTRACE @@ -54,8 +55,12 @@ #define FD_INITIAL_VALUE -1 #define TRACE_FILE_NOT_EXIST -2 +#define EXT_DEACTIVATED 0 +#define EXT_ACTIVATED 1 + int g_trace_handle_fd = FD_INITIAL_VALUE; int g_enabled_tag_fd = FD_INITIAL_VALUE; +int g_extension_state = EXT_DEACTIVATED; static uint64_t dummy = 0; uint64_t *cur_enabled_tag = (void *)&dummy; @@ -65,6 +70,7 @@ static uint64_t traceInit() uint64_t *sm_for_enabled_tag; TTRACE_LOG("traceInit: %p %p", cur_enabled_tag, ((void *)&dummy)); + g_extension_state = EXT_ACTIVATED; if (cur_enabled_tag == ((void *)&dummy)) { g_enabled_tag_fd = open(ENABLED_TAG_FILE, O_RDONLY | O_CLOEXEC); if (g_enabled_tag_fd < 0) { @@ -96,8 +102,10 @@ static uint64_t traceInit() return 0; } } - TTRACE_LOG("traceInit:: cur_enabled_tag >> %u", *cur_enabled_tag); + + /* ttrace is ready. deactive g_extension_state */ + g_extension_state = EXT_DEACTIVATED; return *cur_enabled_tag; } @@ -121,35 +129,41 @@ static inline uint64_t isTagEnabled(uint64_t cur_tag) */ void traceBegin(uint64_t tag, const char *name, ...) { - if (isTagEnabled(tag)) { + if (isTagEnabled(tag) || g_extension_state == EXT_ACTIVATED) { char buf[MAX_LEN]; int len = 0, ret = 0; va_list ap; TTRACE_LOG("traceBegin:: write >> tag: %u tag_bit: %u", tag, *cur_enabled_tag); + va_start(ap, name); len = snprintf(buf, MAX_LEN, "B|%d|", getpid()); len += vsnprintf(buf + len, MAX_LEN - len, name, ap); va_end(ap); - ret = write(g_trace_handle_fd, buf, len); + + if (g_extension_state == EXT_ACTIVATED) ttrace_extension_write(buf, len); + else ret = write(g_trace_handle_fd, buf, len); + if (ret < 0) fprintf(stderr, "error writing, len: %d, ret: %d, errno: %d at traceBegin.\n", len, ret, errno); } -#ifdef TTRACE_DEBUG - else +#ifdef TTRACE_DEBUG + else { TTRACE_LOG("traceBegin:: disabled tag >> tag: %u tag_bit: %u", tag, *cur_enabled_tag); #endif - } void traceEnd(uint64_t tag) { - if (isTagEnabled(tag)) { + if (isTagEnabled(tag) || g_extension_state == EXT_ACTIVATED) { int ret = 0; char end = 'E'; TTRACE_LOG("traceEnd:: write>> tag: %u tag_bit: %u", tag, *cur_enabled_tag); - ret = write(g_trace_handle_fd, &end, 1); + + if (g_extension_state == EXT_ACTIVATED) ttrace_extension_write("E", 1); + else ret = write(g_trace_handle_fd, &end, 1); + if (ret < 0) fprintf(stderr, "error writing, len: %d, ret: %d, errno: %d at traceEnd.\n", 1, ret, errno); } @@ -169,7 +183,7 @@ void traceEnd(uint64_t tag) */ void traceAsyncBegin(uint64_t tag, int cookie, const char *name, ...) { - if (isTagEnabled(tag)) { + if (isTagEnabled(tag) || g_extension_state == EXT_ACTIVATED) { char buf[MAX_LEN]; int len = 0, ret = 0; va_list ap; @@ -180,7 +194,10 @@ void traceAsyncBegin(uint64_t tag, int cookie, const char *name, ...) len += vsnprintf(buf + len, MAX_LEN - len, name, ap); len += snprintf(buf + len, MAX_LEN - len, "|%d", cookie); va_end(ap); - ret = write(g_trace_handle_fd, buf, len); + + if (g_extension_state == EXT_ACTIVATED) ttrace_extension_write(buf, len); + else ret = write(g_trace_handle_fd, buf, len); + if (ret < 0) fprintf(stderr, "error writing, len: %d, ret: %d, errno: %d at traceAsyncBegin.\n", len, ret, errno); } @@ -193,7 +210,7 @@ void traceAsyncBegin(uint64_t tag, int cookie, const char *name, ...) void traceAsyncEnd(uint64_t tag, int cookie, const char *name, ...) { - if (isTagEnabled(tag)) { + if (isTagEnabled(tag) || g_extension_state == EXT_ACTIVATED) { char buf[MAX_LEN]; int len = 0, ret = 0; va_list ap; @@ -204,7 +221,10 @@ void traceAsyncEnd(uint64_t tag, int cookie, const char *name, ...) len += vsnprintf(buf + len, MAX_LEN - len, name, ap); len += snprintf(buf + len, MAX_LEN - len, "|%d", cookie); va_end(ap); - ret = write(g_trace_handle_fd, buf, len); + + if (g_extension_state == EXT_ACTIVATED) ttrace_extension_write(buf, len); + else ret = write(g_trace_handle_fd, buf, len); + if (ret < 0) fprintf(stderr, "error writing, len: %d, ret: %d, errno: %d at traceAsyncEnd.\n", len, ret, errno); } @@ -223,7 +243,7 @@ void traceAsyncEnd(uint64_t tag, int cookie, const char *name, ...) /* LCOV_EXCL_START */ void traceMark(uint64_t tag, const char *name, ...) { - if (isTagEnabled(tag)) { + if (isTagEnabled(tag) || g_extension_state == EXT_ACTIVATED) { char buf[MAX_LEN], end = 'E'; int len = 0, ret = 0; va_list ap; @@ -233,10 +253,16 @@ void traceMark(uint64_t tag, const char *name, ...) len = snprintf(buf, MAX_LEN, "B|%d|", getpid()); len += vsnprintf(buf + len, MAX_LEN - len, name, ap); va_end(ap); - ret = write(g_trace_handle_fd, buf, len); + + if (g_extension_state == EXT_ACTIVATED) ttrace_extension_write(buf, len); + else ret = write(g_trace_handle_fd, buf, len); + if (ret < 0) fprintf(stderr, "error writing, len: %d, ret: %d, errno: %d at traceMark.\n", len, ret, errno); - ret = write(g_trace_handle_fd, &end, 1); + + if (g_extension_state == EXT_ACTIVATED) ttrace_extension_write("E", 1); + else ret = write(g_trace_handle_fd, &end, 1); + if (ret < 0) fprintf(stderr, "error writing, len: %d, ret: %d, errno: %d at traceMark.\n", 1, ret, errno); } @@ -256,7 +282,7 @@ void traceMark(uint64_t tag, const char *name, ...) */ void traceCounter(uint64_t tag, int value, const char *name, ...) { - if (isTagEnabled(tag)) { + if (isTagEnabled(tag) || g_extension_state == EXT_ACTIVATED) { char buf[MAX_LEN]; int len = 0, ret = 0; va_list ap; @@ -267,7 +293,10 @@ void traceCounter(uint64_t tag, int value, const char *name, ...) len += vsnprintf(buf + len, MAX_LEN - len, name, ap); len += snprintf(buf + len, MAX_LEN - len, "|%d", value); va_end(ap); - ret = write(g_trace_handle_fd, buf, len); + + if (g_extension_state == EXT_ACTIVATED) ttrace_extension_write(buf, len); + else ret = write(g_trace_handle_fd, buf, len); + if (ret < 0) fprintf(stderr, "error writing, len: %d, ret: %d, errno: %d at traceCounter.\n", len, ret, errno); } diff --git a/ttrace.pc.in b/ttrace.pc.in index 709639e..5869688 100644 --- a/ttrace.pc.in +++ b/ttrace.pc.in @@ -8,6 +8,6 @@ includedir=@INCLUDEDIR@ Name: ttrace Description: T-trace for tizen Version: @VERSION@ -Requires: dlog +Requires: dlog ttrace-extension Libs: -L${libdir} -lttrace Cflags: -I${includedir} -- 2.7.4