[OE-core] [PATCH 1/1] latencytop: Deprecate tracing_enabled for tracing_on
Rongqing Li
rongqing.li at windriver.com
Sat Jun 8 07:31:53 UTC 2013
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 at windriver.com wrote:
> From: "Roy.Li" <rongqing.li at 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 at 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 at 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
More information about the Openembedded-core
mailing list