From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Zijlstra Subject: Re: ftrace/perf_event leak Date: Wed, 01 Sep 2010 13:02:57 +0200 Message-ID: <1283338977.2059.938.camel@laptop> References: <4C7E11E5.1040402@redhat.com> <1283331868.2059.808.camel@laptop> <4C7E1C36.6070400@redhat.com> <4C7E1F12.8030304@cn.fujitsu.com> <4C7E2D1D.3080603@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT Cc: Li Zefan , Ingo Molnar , Frederic Weisbecker , Steven Rostedt , kvm-devel , Linux Kernel Mailing List To: Avi Kivity Return-path: Received: from casper.infradead.org ([85.118.1.10]:51875 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753832Ab0IALDd convert rfc822-to-8bit (ORCPT ); Wed, 1 Sep 2010 07:03:33 -0400 In-Reply-To: <4C7E2D1D.3080603@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On Wed, 2010-09-01 at 13:38 +0300, Avi Kivity wrote: > On 09/01/2010 12:38 PM, Li Zefan wrote: > > > > Then try this: > > Tested-by: Avi Kivity > Thanks, queued as: --- Subject: perf, trace: Fix module leak From: Li Zefan Date: Wed Sep 01 12:58:43 CEST 2010 Commit 1c024eca (perf, trace: Optimize tracepoints by using per-tracepoint-per-cpu hlist to track events) caused a module refcount leak. Tested-by: Avi Kivity Signed-off-by: Peter Zijlstra LKML-Reference: <4C7E1F12.8030304@cn.fujitsu.com> --- kernel/trace/trace_event_perf.c | 3 +++ 1 file changed, 3 insertions(+) Index: linux-2.6/kernel/trace/trace_event_perf.c =================================================================== --- linux-2.6.orig/kernel/trace/trace_event_perf.c +++ linux-2.6/kernel/trace/trace_event_perf.c @@ -91,6 +91,8 @@ int perf_trace_init(struct perf_event *p tp_event->class && tp_event->class->reg && try_module_get(tp_event->mod)) { ret = perf_trace_event_init(tp_event, p_event); + if (ret) + module_put(tp_event->mod); break; } } @@ -147,6 +149,7 @@ void perf_trace_destroy(struct perf_even } } out: + module_put(tp_event->mod); mutex_unlock(&event_mutex); }