latencytop: Deprecate tracing_enabled for tracing_on
authorRoy.Li <rongqing.li@windriver.com>
Sat, 8 Jun 2013 07:22:06 +0000 (15:22 +0800)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 13 Jun 2013 16:38:05 +0000 (17:38 +0100)
tracing_enabled should not be used, it is heavy weight and does not
do much in helping lower the overhead. see commmit (Deprecate
tracing_enabled for tracing_on 6752ab4a9c30d5411b2dfdb251a3f1cb18aae48)
in kernel

(From OE-Core rev: ee95048cf0c79dc52857dbc0e5da9363f2ad8b9b)

Signed-off-by: Roy.Li <rongqing.li@windriver.com>
Signed-off-by: Saul Wold <sgw@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/recipes-kernel/latencytop/latencytop-0.5/latencytop-fsync.patch [new file with mode: 0644]
meta/recipes-kernel/latencytop/latencytop_0.5.bb

diff --git a/meta/recipes-kernel/latencytop/latencytop-0.5/latencytop-fsync.patch b/meta/recipes-kernel/latencytop/latencytop-0.5/latencytop-fsync.patch
new file mode 100644 (file)
index 0000000..9720cdb
--- /dev/null
@@ -0,0 +1,49 @@
+linux: sysfs: use tracing_on as tracing_enabled is deprecated
+
+tracing_enabled use in the kernel is being deprecated as per commit
+6752ab4a9c30 [tracing: Deprecate tracing_enabled for tracing_on] in
+the linux mainline kernel. tracing_enabled use will generate a warning
+and may no longer function as expected, therefore move to use tracing_on.
+
+Upstream-status: Pending
+
+Signed-off-by: Dennis Hall <dennis.hall@windriver.com>
+
+Index: latencytop-0.5/fsync.c
+===================================================================
+--- latencytop-0.5.orig/fsync.c        2012-08-03 10:45:49.000000000 -0400
++++ latencytop-0.5/fsync.c     2012-08-03 10:48:39.000000000 -0400
+@@ -149,19 +149,19 @@
+  * cd /sys/kernel/debug/tracing
+  * echo fsync > current_tracer
+  * echo ftrace_printk > iter_ctrl 
+- * echo 1 > tracing_enabled
++ * echo 1 > tracing_on
+  */
+       ret = system("/bin/mount -t debugfs none /sys/kernel/debug/");
+       if (!ret) 
+               return -1;
+       write_to_file("/sys/kernel/debug/tracing/current_tracer", "fsync");     
+       write_to_file("/sys/kernel/debug/tracing/iter_ctrl", "ftrace_printk");  
+-      write_to_file("/sys/kernel/debug/tracing/tracing_enabled", "1");
++      write_to_file("/sys/kernel/debug/tracing/tracing_on", "1");
+ }
+ int disable_fsync_tracer(void)
+ {
+-      write_to_file("/sys/kernel/debug/tracing/tracing_enabled", "0");
++      write_to_file("/sys/kernel/debug/tracing/tracing_on", "0");
+ }
+@@ -339,8 +339,8 @@
+               if (curduration > 5)
+                       curduration = 5;
+               /* clear the ftrace buffer */
+-              write_to_file("/sys/kernel/debug/tracing/tracing_enabled", "0");
+-              write_to_file("/sys/kernel/debug/tracing/tracing_enabled", "1");
++              write_to_file("/sys/kernel/debug/tracing/tracing_on", "0");
++              write_to_file("/sys/kernel/debug/tracing/tracing_on", "1");
+               key = select(1, &rfds, NULL, NULL, &end);
+               parse_ftrace();
+               print_global_list();
index 3e35bf9..dad0588 100644 (file)
@@ -10,7 +10,8 @@ DEPENDS = "virtual/libintl ncurses glib-2.0  ${@base_contains('DISTRO_FEATURES',
 PR = "r3"
 
 SRC_URI = "http://www.latencytop.org/download/latencytop-${PV}.tar.gz \
-            file://latencytop-makefile.patch"
+            file://latencytop-makefile.patch \
+            file://latencytop-fsync.patch"
 
 SRC_URI[md5sum] = "73bb3371c6ee0b0e68e25289027e865c"
 SRC_URI[sha256sum] = "9e7f72fbea7bd918e71212a1eabaad8488d2c602205d2e3c95d62cd57e9203ef"