Re: [PATCH V2 2/2] tracing/probe: Show subsystem name in messages

From: Namhyung Kim
Date: Fri Feb 10 2017 - 11:08:29 EST


On Fri, Feb 10, 2017 at 10:23:06PM +0900, Masami Hiramatsu wrote:
> Show "trace_probe:", "trace_kprobe:" and "trace_uprobe:"
> headers for each warning/error/info message. This will
> help people to notice that kprobe/uprobe events caused
> those messages.
>
> Signed-off-by: Masami Hiramatsu <mhiramat@xxxxxxxxxx>
> ---
> kernel/trace/trace_kprobe.c | 1 +
> kernel/trace/trace_probe.c | 1 +
> kernel/trace/trace_uprobe.c | 1 +
> 3 files changed, 3 insertions(+)
>
> diff --git a/kernel/trace/trace_kprobe.c b/kernel/trace/trace_kprobe.c
> index d3729bd..5f688cc 100644
> --- a/kernel/trace/trace_kprobe.c
> +++ b/kernel/trace/trace_kprobe.c
> @@ -16,6 +16,7 @@
> * along with this program; if not, write to the Free Software
> * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
> */
> +#define pr_fmt(fmt) "trace_kprobe: " fmt
>
> #include <linux/module.h>
> #include <linux/uaccess.h>
> diff --git a/kernel/trace/trace_probe.c b/kernel/trace/trace_probe.c
> index 847c1e0..52478f0 100644
> --- a/kernel/trace/trace_probe.c
> +++ b/kernel/trace/trace_probe.c
> @@ -21,6 +21,7 @@
> * Copyright (C) IBM Corporation, 2010-2011
> * Author: Srikar Dronamraju
> */
> +#define pr_fmt(fmt) "trace_probe: " fmt
>
> #include "trace_probe.h"
>
> diff --git a/kernel/trace/trace_uprobe.c b/kernel/trace/trace_uprobe.c
> index e5445ab..2c6b2d0c 100644
> --- a/kernel/trace/trace_uprobe.c
> +++ b/kernel/trace/trace_uprobe.c
> @@ -17,6 +17,7 @@
> * Copyright (C) IBM Corporation, 2010-2012
> * Author: Srikar Dronamraju <srikar@xxxxxxxxxxxxxxxxxx>
> */
> +#define pr_fmt(fmt) "trace_kprobe: " fmt

s/kprobe/uprobe/ ?

Thanks,
Namhyung

>
> #include <linux/module.h>
> #include <linux/uaccess.h>
>