From: Namhyung Kim Date: Fri, 7 Jun 2013 06:07:48 +0000 (+0900) Subject: tracing: Do not call kmem_cache_free() on allocation failure X-Git-Tag: v3.11-rc1~33^2~27 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=aaf6ac0f0871cb7fc0f28f3a00edf329bc7adc29;p=profile%2Fivi%2Fkernel-x86-ivi.git tracing: Do not call kmem_cache_free() on allocation failure There's no point calling it when _alloc() failed. Link: http://lkml.kernel.org/r/1370585268-29169-1-git-send-email-namhyung@kernel.org Signed-off-by: Namhyung Kim Signed-off-by: Steven Rostedt --- diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index db086f1..f57b015 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c @@ -97,7 +97,7 @@ static int __trace_define_field(struct list_head *head, const char *type, field = kmem_cache_alloc(field_cachep, GFP_TRACE); if (!field) - goto err; + return -ENOMEM; field->name = name; field->type = type; @@ -114,11 +114,6 @@ static int __trace_define_field(struct list_head *head, const char *type, list_add(&field->link, head); return 0; - -err: - kmem_cache_free(field_cachep, field); - - return -ENOMEM; } int trace_define_field(struct ftrace_event_call *call, const char *type,