From: Gihun Chang <gihun.chang@samsung.com>
Date: Tue, 22 Nov 2016 05:50:20 +0000 (+0900)
Subject: Skip the  initialization of kernel event when atrace run with --append option.
X-Git-Tag: submit/tizen/20161123.094102~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=9f7f29b349f68efecca849d96b27c481b201d1e6;p=tools%2Fttrace.git

Skip the  initialization of kernel event when atrace run with --append option.

Change-Id: I49fcc426b831cb28b951d0133d2bd61f6c478a8f
Signed-off-by: Gihun Chang <gihun.chang@samsung.com>
---

diff --git a/src/atrace/atrace.cpp b/src/atrace/atrace.cpp
index c51825d..6293b06 100755
--- a/src/atrace/atrace.cpp
+++ b/src/atrace/atrace.cpp
@@ -407,7 +407,7 @@ static bool setTracingEnabled(bool enable)
 }
 
 // Clear the contents of the kernel trace.
-static bool clearTrace()
+static bool clearKernelTrace()
 {
     return truncateFile(k_tracePath);
 }
@@ -671,18 +671,17 @@ static bool setUpTrace()
 	}
 	printf("Tags: 0x%llx, excludedTags: 0x%llx\n", tags, excludedTags);
     }
+
     ok &= startTrace();
-	if(!g_append_trace) {
-		// For debug
-		// printf("\nclear the trace\n");
-		//
-	    ok &= clearTrace();
-	}
-    ok &= setTagsProperty(tags);
+    if(!g_append_trace) {
+        // Clear Kernel Trace
+        ok &= clearKernelTrace();
 
-    // Disable all the sysfs enables.  This is done as a separate loop from
-    // the enables to allow the same enable to exist in multiple categories.
-    ok &= disableKernelTraceEvents();
+        // Disable all the sysfs enables.  This is done as a separate loop from
+        // the enables to allow the same enable to exist in multiple categories.
+        ok &= disableKernelTraceEvents();
+    }
+    ok &= setTagsProperty(tags);
 
     // Enable all the sysfs enables that are in an enabled category.
     for (int i = 0; i < NELEM(k_categories); i++) {
@@ -932,7 +931,7 @@ static void showHelp(const char *cmd)
                     "  --async_dump    dump the current contents of circular trace buffer\n"
                     "  --async_stop    stop tracing and dump the current contents of circular\n"
                     "                    trace buffer\n"
-                    "  --append        append traces to the existing traces. do not clear the trace buffer\n"
+                    "  --append        append traces to the existing traces. do not clear the trace buffer\n and kernel trace events set"
 		    "  --backup        back up the existing traces to /tmp/trace.backup and then clear the trace buffer\n"
                     "  --list_categories\n"
                     "                  list the available tracing categories\n"
@@ -1113,7 +1112,7 @@ int main(int argc, char **argv)
             printf("\ntrace aborted.\n");
             fflush(stdout);
         }
-        clearTrace();
+        clearKernelTrace();
     } else if (!ok) {
         fprintf(stderr, "unable to start tracing\n");
     }