Re: [PATCH 1/1] latencytop: Deprecate tracing_enabled for tracing_on

[Rongqing Li <rongqing.li@windriver.com>]

I would like to send this patch to upstream, but the homelink of
latencytop seems deprecated, and I can not access.

These kernel commits can prove this patch is valid.

     Commit 6752ab4a9c30 "tracing: Deprecate tracing_enabled for tracing_on"
     Commit 0fb9656d "tracing: Make tracing_enabled be equal to tracing_on"
     Commit 02404baf1b47 "tracing: Remove deprecated tracing_enabled file"

Thanks.

-Roy


On 06/08/2013 03:22 PM, rongqing.li@windriver.com wrote:
> From: "Roy.Li" <rongqing.li@windriver.com>
>
> 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
>
> Signed-off-by: Roy.Li <rongqing.li@windriver.com>
> ---
>   .../latencytop-0.5/latencytop-fsync.patch          |   49 ++++++++++++++++++++
>   meta/recipes-kernel/latencytop/latencytop_0.5.bb   |    3 +-
>   2 files changed, 51 insertions(+), 1 deletion(-)
>   create mode 100644 meta/recipes-kernel/latencytop/latencytop-0.5/latencytop-fsync.patch
>
> 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
> index 0000000..9720cdb
> --- /dev/null
> +++ b/meta/recipes-kernel/latencytop/latencytop-0.5/latencytop-fsync.patch
> @@ -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();
> diff --git a/meta/recipes-kernel/latencytop/latencytop_0.5.bb b/meta/recipes-kernel/latencytop/latencytop_0.5.bb
> index 3e35bf9..dad0588 100644
> --- a/meta/recipes-kernel/latencytop/latencytop_0.5.bb
> +++ b/meta/recipes-kernel/latencytop/latencytop_0.5.bb
> @@ -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"
>

-- 
Best Reagrds,
Roy | RongQing Li