All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/3] tracing: Modify is_delete, is_return from int to bool
@ 2012-04-09  9:11 ` Srikar Dronamraju
  0 siblings, 0 replies; 42+ messages in thread
From: Srikar Dronamraju @ 2012-04-09  9:11 UTC (permalink / raw)
  To: Peter Zijlstra, Ingo Molnar
  Cc: Andrew Morton, Linus Torvalds, Ananth N Mavinakayanahalli,
	Jim Keniston, LKML, Linux-mm, Oleg Nesterov, Andi Kleen,
	Christoph Hellwig, Steven Rostedt, Arnaldo Carvalho de Melo,
	Masami Hiramatsu, Thomas Gleixner, Anton Arapov

From: Srikar Dronamraju <srikar@linux.vnet.ibm.com>

is_delete and is_return can take utmost 2 values and are better of
being a boolean than a int. There are no functional changes.

Acked-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
---

Changelog:
(v5):
 - separated out as different patch on Masami's suggestion.

 kernel/trace/trace_kprobe.c |   16 ++++++++--------
 1 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/kernel/trace/trace_kprobe.c b/kernel/trace/trace_kprobe.c
index 580a05e..4f935f8 100644
--- a/kernel/trace/trace_kprobe.c
+++ b/kernel/trace/trace_kprobe.c
@@ -651,7 +651,7 @@ static struct trace_probe *alloc_trace_probe(const char *group,
 					     void *addr,
 					     const char *symbol,
 					     unsigned long offs,
-					     int nargs, int is_return)
+					     int nargs, bool is_return)
 {
 	struct trace_probe *tp;
 	int ret = -ENOMEM;
@@ -944,7 +944,7 @@ static int split_symbol_offset(char *symbol, unsigned long *offset)
 #define PARAM_MAX_STACK (THREAD_SIZE / sizeof(unsigned long))
 
 static int parse_probe_vars(char *arg, const struct fetch_type *t,
-			    struct fetch_param *f, int is_return)
+			    struct fetch_param *f, bool is_return)
 {
 	int ret = 0;
 	unsigned long param;
@@ -977,7 +977,7 @@ static int parse_probe_vars(char *arg, const struct fetch_type *t,
 
 /* Recursive argument parser */
 static int __parse_probe_arg(char *arg, const struct fetch_type *t,
-			     struct fetch_param *f, int is_return)
+			     struct fetch_param *f, bool is_return)
 {
 	int ret = 0;
 	unsigned long param;
@@ -1089,7 +1089,7 @@ static int __parse_bitfield_probe_arg(const char *bf,
 
 /* String length checking wrapper */
 static int parse_probe_arg(char *arg, struct trace_probe *tp,
-			   struct probe_arg *parg, int is_return)
+			   struct probe_arg *parg, bool is_return)
 {
 	const char *t;
 	int ret;
@@ -1162,7 +1162,7 @@ static int create_trace_probe(int argc, char **argv)
 	 */
 	struct trace_probe *tp;
 	int i, ret = 0;
-	int is_return = 0, is_delete = 0;
+	bool is_return = false, is_delete = false;
 	char *symbol = NULL, *event = NULL, *group = NULL;
 	char *arg;
 	unsigned long offset = 0;
@@ -1171,11 +1171,11 @@ static int create_trace_probe(int argc, char **argv)
 
 	/* argc must be >= 1 */
 	if (argv[0][0] == 'p')
-		is_return = 0;
+		is_return = false;
 	else if (argv[0][0] == 'r')
-		is_return = 1;
+		is_return = true;
 	else if (argv[0][0] == '-')
-		is_delete = 1;
+		is_delete = true;
 	else {
 		pr_info("Probe definition must be started with 'p', 'r' or"
 			" '-'.\n");


--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply related	[flat|nested] 42+ messages in thread

end of thread, other threads:[~2012-05-08  9:31 UTC | newest]

Thread overview: 42+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-04-09  9:11 [PATCH 1/3] tracing: Modify is_delete, is_return from int to bool Srikar Dronamraju
2012-04-09  9:11 ` Srikar Dronamraju
2012-04-09  9:11 ` [PATCH 2/3] tracing: Extract out common code for kprobes/uprobes trace events Srikar Dronamraju
2012-04-09  9:11   ` Srikar Dronamraju
2012-04-12 12:47   ` Masami Hiramatsu
2012-04-12 12:47     ` Masami Hiramatsu
2012-05-08  4:08   ` [tip:perf/uprobes] tracing: Extract out common code for kprobes/ uprobes " tip-bot for Srikar Dronamraju
2012-05-08  4:08     ` tip-bot for Srikar Dronamraju
2012-04-09  9:11 ` [PATCH 3/3] tracing: Provide trace events interface for uprobes Srikar Dronamraju
2012-04-09  9:11   ` Srikar Dronamraju
2012-04-11 10:30   ` [PATCH UPDATED " Srikar Dronamraju
2012-04-11 10:30     ` Srikar Dronamraju
2012-04-12 13:02     ` Masami Hiramatsu
2012-04-12 13:02       ` Masami Hiramatsu
2012-04-12 14:49       ` Srikar Dronamraju
2012-04-12 14:49         ` Srikar Dronamraju
2012-04-13  2:28         ` Masami Hiramatsu
2012-04-13  2:28           ` Masami Hiramatsu
2012-04-12 13:14     ` Steven Rostedt
2012-04-12 13:14       ` Steven Rostedt
2012-04-14 11:12       ` Ingo Molnar
2012-04-14 11:12         ` Ingo Molnar
2012-04-16  9:22         ` Srikar Dronamraju
2012-04-16  9:22           ` Srikar Dronamraju
2012-04-20  9:57         ` Srikar Dronamraju
2012-04-20  9:57           ` Srikar Dronamraju
2012-05-08  4:09     ` [tip:perf/uprobes] " tip-bot for Srikar Dronamraju
2012-05-08  4:09       ` tip-bot for Srikar Dronamraju
2012-05-08  4:12     ` [PATCH UPDATED 3/3] " Ingo Molnar
2012-05-08  4:12       ` Ingo Molnar
2012-05-08  8:30       ` Peter Zijlstra
2012-05-08  8:30         ` Peter Zijlstra
2012-05-08  8:50         ` Srikar Dronamraju
2012-05-08  8:50           ` Srikar Dronamraju
2012-05-08  9:20           ` Ingo Molnar
2012-05-08  9:20             ` Ingo Molnar
2012-05-08  9:19             ` Srikar Dronamraju
2012-05-08  9:19               ` Srikar Dronamraju
2012-04-12 12:42 ` [PATCH 1/3] tracing: Modify is_delete, is_return from int to bool Masami Hiramatsu
2012-04-12 12:42   ` Masami Hiramatsu
2012-05-08  4:07 ` [tip:perf/uprobes] " tip-bot for Srikar Dronamraju
2012-05-08  4:07   ` tip-bot for Srikar Dronamraju

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.