aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorYaowei Bai <bywxiaobai@163.com>2015-09-22 09:43:13 -0400
committerSteven Rostedt <rostedt@goodmis.org>2015-09-22 13:11:30 -0400
commitf0132c4e0d06046a324051c09bd094719b9216ac (patch)
tree0adbbf325b273be644f1f8001540611031f55462
parent1f93e4a96c9109378204c147b3eec0d0e8100fde (diff)
kernel/trace_probe: is_good_name can be boolean
This patch makes is_good_name return bool to improve readability due to this particular function only using either one or zero as its return value. No functional change. Link: http://lkml.kernel.org/r/1442929393-4753-2-git-send-email-bywxiaobai@163.com Signed-off-by: Yaowei Bai <bywxiaobai@163.com> Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
-rw-r--r--kernel/trace/trace_probe.h8
1 files changed, 4 insertions, 4 deletions
diff --git a/kernel/trace/trace_probe.h b/kernel/trace/trace_probe.h
index b98dee914542..f6398db09114 100644
--- a/kernel/trace/trace_probe.h
+++ b/kernel/trace/trace_probe.h
@@ -302,15 +302,15 @@ static nokprobe_inline void call_fetch(struct fetch_param *fprm,
302} 302}
303 303
304/* Check the name is good for event/group/fields */ 304/* Check the name is good for event/group/fields */
305static inline int is_good_name(const char *name) 305static inline bool is_good_name(const char *name)
306{ 306{
307 if (!isalpha(*name) && *name != '_') 307 if (!isalpha(*name) && *name != '_')
308 return 0; 308 return false;
309 while (*++name != '\0') { 309 while (*++name != '\0') {
310 if (!isalpha(*name) && !isdigit(*name) && *name != '_') 310 if (!isalpha(*name) && !isdigit(*name) && *name != '_')
311 return 0; 311 return false;
312 } 312 }
313 return 1; 313 return true;
314} 314}
315 315
316static inline struct event_file_link * 316static inline struct event_file_link *