From nobody Tue Apr 7 06:21:35 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 64474C4332F for ; Wed, 12 Oct 2022 21:57:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229841AbiJLV5X (ORCPT ); Wed, 12 Oct 2022 17:57:23 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60760 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229491AbiJLV5W (ORCPT ); Wed, 12 Oct 2022 17:57:22 -0400 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 652E610451A; Wed, 12 Oct 2022 14:57:21 -0700 (PDT) Received: from localhost.localdomain (unknown [76.135.50.127]) by linux.microsoft.com (Postfix) with ESMTPSA id 179C920F0F77; Wed, 12 Oct 2022 14:57:21 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 179C920F0F77 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1665611841; bh=cBwmnLdZr7+xCHkswbZxO11VOAXb4IRb/2OCDhqD//U=; h=From:To:Cc:Subject:Date:From; b=nujnO1WCLnkK/yPReail5/fe+pehA5rL172rcM82lvRtobSLGnke5+syFW3Q3VsXu zpuobY5doRb0jp5sPS7efPFbuhftxXGC2bIe6vFlvMNcGxW3iqHlVgOnbccu229v6u DrSf1k/WpWuPaF4/73FaGy3KQfmcp/slGT8SDcrY= From: Beau Belgrave To: rostedt@goodmis.org, mhiramat@kernel.org Cc: linux-trace-devel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] tracing: Track event ref in tracefs enable/disable Date: Wed, 12 Oct 2022 14:57:17 -0700 Message-Id: <20221012215717.10492-1-beaub@linux.microsoft.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" When events are enabled via the "enable" file within tracefs there is no get or put ref. Add these to ensure modules and dynamic events do not unload while the event is enabled via tracefs. Signed-off-by: Beau Belgrave --- kernel/trace/trace_events.c | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index 0356cae0cf74..f0b17adba1bc 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c @@ -634,6 +634,8 @@ static int __ftrace_event_enable_disable(struct trace_e= vent_file *file, } =20 call->class->reg(call, TRACE_REG_UNREGISTER, file); + + trace_event_put_ref(call); } /* If in SOFT_MODE, just set the SOFT_DISABLE_BIT, else clear it */ if (file->flags & EVENT_FILE_FL_SOFT_MODE) @@ -659,7 +661,7 @@ static int __ftrace_event_enable_disable(struct trace_e= vent_file *file, } =20 if (!(file->flags & EVENT_FILE_FL_ENABLED)) { - bool cmd =3D false, tgid =3D false; + bool cmd =3D false, tgid =3D false, put =3D false; =20 /* Keep the event disabled, when going to SOFT_MODE. */ if (soft_disable) @@ -677,14 +679,22 @@ static int __ftrace_event_enable_disable(struct trace= _event_file *file, set_bit(EVENT_FILE_FL_RECORDED_TGID_BIT, &file->flags); } =20 + if (!trace_event_try_get_ref(call)) + goto error_enabling; + + put =3D true; + ret =3D call->class->reg(call, TRACE_REG_REGISTER, file); if (ret) { +error_enabling: + pr_info("event trace: Could not enable event " + "%s\n", trace_event_name(call)); if (cmd) tracing_stop_cmdline_record(); if (tgid) tracing_stop_tgid_record(); - pr_info("event trace: Could not enable event " - "%s\n", trace_event_name(call)); + if (put) + trace_event_put_ref(call); break; } set_bit(EVENT_FILE_FL_ENABLED_BIT, &file->flags); base-commit: 6c0f39e87b6ab1a3009e3a49d3e6f6db8dc756a8 --=20 2.25.1