From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 41E9519D89E for ; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603606; cv=none; b=TJb5QJVnShIarN+9KH7/GCvcs/zbsIzkyucrd3TZR4Gr5Su3WIxapWL/OoqcZzL5jctMFsKOUxY5n4J4fNIlIuchs8gS5+BXEAcJn8xI5WvzSKsT1y4Hl5iSAGTKxsJWWZtRIxXFSEK5u/tieLsb4y49ZI+W8VWtMHY+OasHod0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603606; c=relaxed/simple; bh=PZDLQ4C/EmmsD13YnFkR+qcWRk8KIkgZhUjFW3ZYJ2o=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=Az29FkbjaPQNfBWIJ6PDi+tcW2XSGdbPLGQQ9bMuxHAvvlL0vxzm7jEm9w3GjiPFTxt6HTQDWMBREk+KZXk7Xb262GLizuRFxdz8XnVBqKBVUaXIPwZ3cL7mnMOQvk8Sb6wEGDDFHEcQIckFGWR9BAXCGEYfMWO8Ld0PBQhKYmc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=fWqwrXJZ; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="fWqwrXJZ" Received: by smtp.kernel.org (Postfix) with ESMTPSA id CC1EAC4CEF7; Mon, 9 Feb 2026 02:20:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603605; bh=PZDLQ4C/EmmsD13YnFkR+qcWRk8KIkgZhUjFW3ZYJ2o=; h=Date:From:To:Cc:Subject:References:From; b=fWqwrXJZxK+UnNNm1iu4Ai+Z1vjr4WKSRy8Bb4FPo9kknEvUKX3vJvWBJj2WMzo6K OTMLhagu7L/8E65x/rpODPRd2UZ6HkfWRf9W2T4RyEOft8gT+FVC5HuObO8gNGgfDf sFhGnwbX57ExJTCfxfkMXAgB0xtsiUYTvW3STHLvHvDPHouMR2wSNy9JNdyQXg1Fzd TPQiZB+OqKX2P09gU4Xz4C+122Qj3JsWbalugaprEu2mM2cS/7du7pjiXYMJdtV3mt ghys3ItiP65xwWMb4W/tTuiY4cT5vDjG2bKaC45ufQJjG85tAGURRhJyRaFuzc1pIe tzZcAK3JFnqkA== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtQ-0000000Aa0Q-1RNl; Sun, 08 Feb 2026 21:20:52 -0500 Message-ID: <20260209022052.199521041@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:27 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 01/20] tracing: Add kerneldoc to trace_event_buffer_reserve() References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt Add a appropriate kerneldoc to trace_event_buffer_reserve() to make it easier to understand how that function is used. Cc: Masami Hiramatsu Cc: Mathieu Desnoyers Link: https://patch.msgid.link/20260130103745.1126e4af@gandalf.local.home Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace_events.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index 4972e1a2b5f3..af6d1fe5cab7 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c @@ -649,6 +649,22 @@ bool trace_event_ignore_this_pid(struct trace_event_fi= le *trace_file) } EXPORT_SYMBOL_GPL(trace_event_ignore_this_pid); =20 +/** + * trace_event_buffer_reserve - reserve space on the ring buffer for an ev= ent + * @fbuffer: information about how to save the event + * @trace_file: the instance file descriptor for the event + * @len: The length of the event + * + * The @fbuffer has information about the ring buffer and data will + * be added to it to be used by the call to trace_event_buffer_commit(). + * The @trace_file is the desrciptor with information about the status + * of the given event for a specific trace_array instance. + * The @len is the length of data to save for the event. + * + * Returns a pointer to the data on the ring buffer or NULL if the + * event was not reserved (event was filtered, too big, or the buffer + * simply was disabled for write). + */ void *trace_event_buffer_reserve(struct trace_event_buffer *fbuffer, struct trace_event_file *trace_file, unsigned long len) --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4352419DF4F for ; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603606; cv=none; b=Y0Y4Td0Ri4uAqrrRdry+2qK/GvU6ESCVafOi5afcazDRYJybcAobOHxjcPnKkGQ0fCkj3R+C86fXokhpPxEc/pSK+hYXa7v+kaxWQoapIyLQqepMb16H99X+1ePXNTNQrCDPVzYr7P981GzBelQO0vXSTSUgGcQuQ2O6sznw32w= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603606; c=relaxed/simple; bh=AWEaG16o84JaXpRCpDBz6Nkc1K2OnEIu8E2ZHVN4rRU=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=ngLdzrJH19sn3823dTZtDwu7ZyLJCyWEHYlEvHmyk6z/YlW5a5YDhVTNrfHkxWGJnSlNTE5kCidPY8Dj/36ZCaSANMEG6LTv4WQY6YyqXAIuINit0sw9n6GvK0GAlT/OZ8fH690mw5VAecOz0AHfZ7Q1DsPNIlEz6Orr+4yw5kE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=Qd+j/hsf; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="Qd+j/hsf" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 114CEC16AAE; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603606; bh=AWEaG16o84JaXpRCpDBz6Nkc1K2OnEIu8E2ZHVN4rRU=; h=Date:From:To:Cc:Subject:References:From; b=Qd+j/hsf09Cl85qUYSOG6bfz1Fxg9kk54LJ3l/i6FiPWOewLbeVSIelvG7vyXlwRm 6G8VBxrkAb5BmQKo+8xXIKIQpZqy/bpmdIYTm4GWm97MqWjxr0GvOhLoOMzkNEfu64 tSZ0QEVxF1fSbNHIyjjf4bAVQGRDew6wV8mwb18xo7XlR9aBgl0Z0eCIS6Wq5cFG6e wDJxXH42urt/eIFhcysSiEf/O2nN+8pLQ597v5+YiWVJoMLeE2CLDtR6YWCTjYT2oE MHafJUFoBQMbLrAf3HNisf81FBUMPyvrEMjpN2LyweXIeXVwhlR2VKN/G8o3lMRBlc EXR9PuTEO8CYg== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtQ-0000000Aa0v-29WF; Sun, 08 Feb 2026 21:20:52 -0500 Message-ID: <20260209022052.364650128@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:28 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton , Yaxiong Tian Subject: [for-next][PATCH 02/20] tracing: Rename `eval_map_wq` and allow other parts of tracing use it References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Yaxiong Tian The eval_map_work_func() function, though queued in eval_map_wq, holds the trace_event_sem read-write lock for a long time during kernel boot. This causes blocking issues for other functions. Rename eval_map_wq to trace_init_wq and make it global, thereby allowing other parts of tracing to schedule work on this queue asynchronously and avoiding blockage of the main boot thread. Link: https://patch.msgid.link/20260204015344.162818-1-tianyaxiong@kylinos.= cn Suggested-by: Steven Rostedt Acked-by: Masami Hiramatsu (Google) Signed-off-by: Yaxiong Tian Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 18 +++++++++--------- kernel/trace/trace.h | 1 + 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 396d59202438..8c0f3cfd196b 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -10785,7 +10785,7 @@ int tracing_init_dentry(void) extern struct trace_eval_map *__start_ftrace_eval_maps[]; extern struct trace_eval_map *__stop_ftrace_eval_maps[]; =20 -static struct workqueue_struct *eval_map_wq __initdata; +struct workqueue_struct *trace_init_wq __initdata; static struct work_struct eval_map_work __initdata; static struct work_struct tracerfs_init_work __initdata; =20 @@ -10801,15 +10801,15 @@ static int __init trace_eval_init(void) { INIT_WORK(&eval_map_work, eval_map_work_func); =20 - eval_map_wq =3D alloc_workqueue("eval_map_wq", WQ_UNBOUND, 0); - if (!eval_map_wq) { - pr_err("Unable to allocate eval_map_wq\n"); + trace_init_wq =3D alloc_workqueue("trace_init_wq", WQ_UNBOUND, 0); + if (!trace_init_wq) { + pr_err("Unable to allocate trace_init_wq\n"); /* Do work here */ eval_map_work_func(&eval_map_work); return -ENOMEM; } =20 - queue_work(eval_map_wq, &eval_map_work); + queue_work(trace_init_wq, &eval_map_work); return 0; } =20 @@ -10818,8 +10818,8 @@ subsys_initcall(trace_eval_init); static int __init trace_eval_sync(void) { /* Make sure the eval map updates are finished */ - if (eval_map_wq) - destroy_workqueue(eval_map_wq); + if (trace_init_wq) + destroy_workqueue(trace_init_wq); return 0; } =20 @@ -10980,9 +10980,9 @@ static __init int tracer_init_tracefs(void) if (ret) return 0; =20 - if (eval_map_wq) { + if (trace_init_wq) { INIT_WORK(&tracerfs_init_work, tracer_init_tracefs_work_func); - queue_work(eval_map_wq, &tracerfs_init_work); + queue_work(trace_init_wq, &tracerfs_init_work); } else { tracer_init_tracefs_work_func(NULL); } diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h index 69e7defba6c6..bb68539c64b7 100644 --- a/kernel/trace/trace.h +++ b/kernel/trace/trace.h @@ -769,6 +769,7 @@ extern cpumask_var_t __read_mostly tracing_buffer_mask; extern unsigned long nsecs_to_usecs(unsigned long nsecs); =20 extern unsigned long tracing_thresh; +extern struct workqueue_struct *trace_init_wq __initdata; =20 /* PID filtering */ =20 --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5D2321F7541 for ; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603606; cv=none; b=JSdxHisoK17yYh+WDjboFo7ow+hUS3M0snjeLSDRcvAB8Ch25YEPEswaI0ff7/j6Wb+OVxt9tUdQQppKvJ0DHN66jhr3wggBu5+TNlZB6pY7tsKAw1D8r+LyT5Q7n6JQ172zJpFqIOAnM3r3SH1kynomNxsr87MM7+ICjUcUm3Q= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603606; c=relaxed/simple; bh=8ecH07bGMwHC4uD+yk/3x8YQ1DXfG2vjCSTS00AqnUQ=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=e/fhFgsbXlRPUcoMPZdjn0td8bEBhDg1GI8tLhfeMWu+QOUnHT/T4XEN9maOu0RXXlggYOEI6408PP13ThqNjd9PVOdGJqfZ7Lpt/TGA90GYEPpPl9bk4YczQ3RU60Bfa5NI9/FG35ZhXwdaRMys72aVuGSJoB+tyxbVdO1pKvU= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=AgUWTPRd; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="AgUWTPRd" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 3B89BC2BC87; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603606; bh=8ecH07bGMwHC4uD+yk/3x8YQ1DXfG2vjCSTS00AqnUQ=; h=Date:From:To:Cc:Subject:References:From; b=AgUWTPRdGZTyTPDoeNr8qeFJTTf8Xd8or/kegeZBFktjYKo4EbYu9QMjJeaHdUPU2 n/LtC+Rxv2SnRc9ztyZ7L4XiXu0vT2AcRWMvrphRKXl7ee2EgaOI3qakW/7tbxLd9u 0W4SSpLdCma6LKCMSu9NXmJvBnoCiGq/6yFE9N5v/Zu3gB7YV1yqQGwtKXQikPo+TH pWIKG44Fsf0SvKGNxu4fLBq2nRVWJXrK0ESLZJxyCBqY8QXLFgpiAL/z08+7003KTN aPyytymRiw97qnIBib5nqrT51kKndXPn/H+aPWr8Ie3lGpbaXMxg9uH9IJoLSycCyE i0TTIB5VaYmpQ== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtQ-0000000Aa1P-2rZD; Sun, 08 Feb 2026 21:20:52 -0500 Message-ID: <20260209022052.531796944@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:29 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton , Yaxiong Tian Subject: [for-next][PATCH 03/20] blktrace: Make init_blk_tracer() asynchronous References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Yaxiong Tian The init_blk_tracer() function causes significant boot delay as it waits for the trace_event_sem lock held by trace_event_update_all(). Specifically, its child function register_trace_event() requires this lock, which is occupied for an extended period during boot. To resolve this, the execution of primary init_blk_tracer() is moved to the trace_init_wq workqueue, allowing it to run asynchronously, and prevent blocking the main boot thread. Link: https://patch.msgid.link/20260204015353.163331-1-tianyaxiong@kylinos.= cn Acked-by: Masami Hiramatsu (Google) Signed-off-by: Yaxiong Tian Signed-off-by: Steven Rostedt (Google) --- kernel/trace/blktrace.c | 23 ++++++++++++++++++++++- 1 file changed, 22 insertions(+), 1 deletion(-) diff --git a/kernel/trace/blktrace.c b/kernel/trace/blktrace.c index d031c8d80be4..d611cd1f02ef 100644 --- a/kernel/trace/blktrace.c +++ b/kernel/trace/blktrace.c @@ -1832,7 +1832,9 @@ static struct trace_event trace_blk_event =3D { .funcs =3D &trace_blk_event_funcs, }; =20 -static int __init init_blk_tracer(void) +static struct work_struct blktrace_works __initdata; + +static int __init __init_blk_tracer(void) { if (!register_trace_event(&trace_blk_event)) { pr_warn("Warning: could not register block events\n"); @@ -1852,6 +1854,25 @@ static int __init init_blk_tracer(void) return 0; } =20 +static void __init blktrace_works_func(struct work_struct *work) +{ + __init_blk_tracer(); +} + +static int __init init_blk_tracer(void) +{ + int ret =3D 0; + + if (trace_init_wq) { + INIT_WORK(&blktrace_works, blktrace_works_func); + queue_work(trace_init_wq, &blktrace_works); + } else { + ret =3D __init_blk_tracer(); + } + + return ret; +} + device_initcall(init_blk_tracer); =20 static int blk_trace_remove_queue(struct request_queue *q) --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 754CE2367D9 for ; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603606; cv=none; b=cvq+7MZVSVbBDb1EgjHN2SfGzYR7C5mS28sfgV4PspdaWn+p4f3wFPYtbqIsmSmtBmuua3w3gAAD4BhGVdAp57c76Gn/ncPQD1b9Wn59AGPrlLV4ioSM8IP3M+B9N4B99KTVKrMXMQVN4wc4zaqlQHrCYhBx0ZKCfjF9ihEfVVI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603606; c=relaxed/simple; bh=0wK/CVCCdN7PgklUZddDM9+V5/kap6yrYmHmD4HpuN4=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=VFNly4VUYBee5IOyEke5Du8HhxUo9mN2JsXHgtoWlQ1uqmE7xalrm1B8lZtyqSKWwIcFsu97q2ceaLlf2X8YA+0nVj00AB6tdtVRI79s7sO1h1HDgzdfcjf1sjcLh7flNbPy4mvgwSnVacOFBGHHHJ/rA7/cdcTjX78XiMUPhew= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=Qn4nm4ld; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="Qn4nm4ld" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5355AC19425; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603606; bh=0wK/CVCCdN7PgklUZddDM9+V5/kap6yrYmHmD4HpuN4=; h=Date:From:To:Cc:Subject:References:From; b=Qn4nm4ldmAtd6Dh7L25IUguz6WE1ABoZurj7NuTbi5h5XcAZuGK5XRUBFnMLjX0Pf Et3/M21s9+s/PR8hRTWZ6C1+O3CZ/wh3HPkDGsJGpon0yMhsgypQ97oqjH7Lx0FZwL 6OSVttMHM/HiqT+kv10EyeM2nELITgAcVtRgRbnnpUBRMVO3y05ThYw0dGP0z8S9Dp jRGQKX0KpSOu+/Elk6vpjX49AoF1VXq7ZzZJHCGKgA8np2rXiv23wpDeX2u6/HWCJq ZsWG2xC/IDR+9LJEri8JZJl+xktLmm6+PIVPGCtsjrzIbp55iJJ+Ob4YhiTCsmopOz jTCoRxUaOK8vA== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtQ-0000000Aa1t-3Yjo; Sun, 08 Feb 2026 21:20:52 -0500 Message-ID: <20260209022052.701967355@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:30 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton , Yaxiong Tian Subject: [for-next][PATCH 04/20] tracing/kprobes: Skip setup_boot_kprobe_events() when no cmdline event References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Yaxiong Tian When the 'kprobe_event=3D' kernel command-line parameter is not provided, there is no need to execute setup_boot_kprobe_events(). This change optimizes the initialization function init_kprobe_trace() by skipping unnecessary work and effectively prevents potential blocking that could arise from contention on the event_mutex lock in subsequent operations. Link: https://patch.msgid.link/20260204015401.163748-1-tianyaxiong@kylinos.= cn Acked-by: Masami Hiramatsu (Google) Signed-off-by: Yaxiong Tian Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace_kprobe.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/kernel/trace/trace_kprobe.c b/kernel/trace/trace_kprobe.c index 9953506370a5..89d2740f7bb5 100644 --- a/kernel/trace/trace_kprobe.c +++ b/kernel/trace/trace_kprobe.c @@ -2048,6 +2048,10 @@ static __init int init_kprobe_trace(void) trace_create_file("kprobe_profile", TRACE_MODE_READ, NULL, NULL, &kprobe_profile_ops); =20 + /* If no 'kprobe_event=3D' cmd is provided, return directly. */ + if (kprobe_boot_events_buf[0] =3D=3D '\0') + return 0; + setup_boot_kprobe_events(); =20 return 0; --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 90BB927B357 for ; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603606; cv=none; b=ZwMi0ApGuAuzMaGWOcSZl1Uz9hdEWsW00CW2NfQospcZXCZLUBLwSHp2HZjhbyMJVVSLXsfarTurOzpap8WjlMIGJL3azMEdfBMKP948bta5JGsNYVATAH+WjmXRu/eHpyyeRfKoOYu4Hz+aXGat0vPgJef6oXWhrXN3rcfWCnY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603606; c=relaxed/simple; bh=pFV9tnmcckb4RxVzF95qloyI9IWFNLRdN8Kik0fkAG8=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=hxQa3h+oDjqYWMEQgMrpq6WkZgotVmeYFVpI9yvP7G7ZjpwfC2FuwOt3G5KKyNXCEeT/rdAtPTs3K0flxjk1zIcZY0WkpTipiOLdmoHEIcCq1czz9j1pE/GduBKmklV7jJS4G7E9eXvHhc+StxIeIc8hMHuphpLsKX4u6rnnJBY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=O+sqx7VI; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="O+sqx7VI" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 71413C19424; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603606; bh=pFV9tnmcckb4RxVzF95qloyI9IWFNLRdN8Kik0fkAG8=; h=Date:From:To:Cc:Subject:References:From; b=O+sqx7VI+Wp2SH1sCy/BE8/uZxOyNP2OOJyecI35/kPIisGiVaK/1wXtb9dhzZOmL e6/+suD3b6o1ZvitVtjMq4Z87L7Hkc2ubBB+39m/cb4TpWd1ozzSSNZNEnHJFkhOA9 bbDn7psef0kqliune241uAsk9L2J7w5KJNrffbPp6iq7BSlJO9Yl3xSi0pcLkOGxJs EyNVi/ZVLw2UrQEF7J4vCiUwB7D2rGjNJ7kUSfpH3owKVabb8jTPENeY3eOoMs/eGi HPjqDvYM+OkkjTjCul3gBWrMWGqzk78vhu07IBom46No9ktghInoRoUbItcFWOpExb jgZHoPF1FPMmA== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtR-0000000Aa2N-03MB; Sun, 08 Feb 2026 21:20:53 -0500 Message-ID: <20260209022052.868107499@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:31 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton , Tom Zanussi Subject: [for-next][PATCH 05/20] tracing: Have all triggers expect a file parameter References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt When the triggers were first created, they may not have had a file parameter passed to them and things needed to be done generically. But today, all triggers have a file parameter passed to them. Remove the generic code and add a "if (WARN_ON_ONCE(!file))" to each trigger. Cc: Mathieu Desnoyers Acked-by: Masami Hiramatsu (Google) Reviewed-by: Tom Zanussi Link: https://patch.msgid.link/20260206101351.609d8906@gandalf.local.home Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace_events_trigger.c | 62 +++++++++++------------------ 1 file changed, 24 insertions(+), 38 deletions(-) diff --git a/kernel/trace/trace_events_trigger.c b/kernel/trace/trace_event= s_trigger.c index 06b75bcfc7b8..7fa26327c9c7 100644 --- a/kernel/trace/trace_events_trigger.c +++ b/kernel/trace/trace_events_trigger.c @@ -1347,18 +1347,13 @@ traceon_trigger(struct event_trigger_data *data, { struct trace_event_file *file =3D data->private_data; =20 - if (file) { - if (tracer_tracing_is_on(file->tr)) - return; - - tracer_tracing_on(file->tr); + if (WARN_ON_ONCE(!file)) return; - } =20 - if (tracing_is_on()) + if (tracer_tracing_is_on(file->tr)) return; =20 - tracing_on(); + tracer_tracing_on(file->tr); } =20 static bool @@ -1368,13 +1363,11 @@ traceon_count_func(struct event_trigger_data *data, { struct trace_event_file *file =3D data->private_data; =20 - if (file) { - if (tracer_tracing_is_on(file->tr)) - return false; - } else { - if (tracing_is_on()) - return false; - } + if (WARN_ON_ONCE(!file)) + return false; + + if (tracer_tracing_is_on(file->tr)) + return false; =20 if (!data->count) return false; @@ -1392,18 +1385,13 @@ traceoff_trigger(struct event_trigger_data *data, { struct trace_event_file *file =3D data->private_data; =20 - if (file) { - if (!tracer_tracing_is_on(file->tr)) - return; - - tracer_tracing_off(file->tr); + if (WARN_ON_ONCE(!file)) return; - } =20 - if (!tracing_is_on()) + if (!tracer_tracing_is_on(file->tr)) return; =20 - tracing_off(); + tracer_tracing_off(file->tr); } =20 static bool @@ -1413,13 +1401,11 @@ traceoff_count_func(struct event_trigger_data *data, { struct trace_event_file *file =3D data->private_data; =20 - if (file) { - if (!tracer_tracing_is_on(file->tr)) - return false; - } else { - if (!tracing_is_on()) - return false; - } + if (WARN_ON_ONCE(!file)) + return false; + + if (!tracer_tracing_is_on(file->tr)) + return false; =20 if (!data->count) return false; @@ -1481,10 +1467,10 @@ snapshot_trigger(struct event_trigger_data *data, { struct trace_event_file *file =3D data->private_data; =20 - if (file) - tracing_snapshot_instance(file->tr); - else - tracing_snapshot(); + if (WARN_ON_ONCE(!file)) + return; + + tracing_snapshot_instance(file->tr); } =20 static int @@ -1570,10 +1556,10 @@ stacktrace_trigger(struct event_trigger_data *data, { struct trace_event_file *file =3D data->private_data; =20 - if (file) - __trace_stack(file->tr, tracing_gen_ctx_dec(), STACK_SKIP); - else - trace_dump_stack(STACK_SKIP); + if (WARN_ON_ONCE(!file)) + return; + + __trace_stack(file->tr, tracing_gen_ctx_dec(), STACK_SKIP); } =20 static int --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B65102C0268 for ; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603606; cv=none; b=bc6WYDYvSea5pgygIbI/m5biYZiHC0RWMcRlzcZmxWADVwIelOGnXp1AjhJY2pZNcQtnaIdVSDjzekVluzei9CF2G+GF33cguMUgHXeLJC+bKdTPz5cdAt5UU0JtEisKMifFaL7KCZ8LACr+zHap3Z+RIAw5zO5m7/o/AZ/93r8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603606; c=relaxed/simple; bh=Dl06F221z0jmsIOVowrDNHFWjewEVHhfiLn5CxoMs9U=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=Q69oEHCbGQQc2B9M6mhvIBZrG//7H52MPApqj5bfTbrRkG1Ejj2oRb4EW5RlrEd0gf4kykOPh5x3D6PCJBLniPm0WfZ2eeKFqTOtWQnDmtIV191hBWY7FMbbQtq2xBgn5cu6rYhcMsrNgQr9XMZX4e0Tsjdnm9tAGCaHUvbHYqg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=rXvrYSRW; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="rXvrYSRW" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 96DA7C2BC9E; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603606; bh=Dl06F221z0jmsIOVowrDNHFWjewEVHhfiLn5CxoMs9U=; h=Date:From:To:Cc:Subject:References:From; b=rXvrYSRWq5puKx7IXFY0zSfoSGQ+AVO3vWjo1OYDhmhe2xuZg2oa6fTw6SAu5eitp B4d/ZrgryujMByrZ0vz2IyILgBIbNZzlmX5ZdRMHzVxmQDktTiN3t7x38WNO/rWYXf NBH7HFkGU8FzMivOmoLcTZIk5B9MwqzVwDC2z5BCmM9te7xOFRoLleCg7xsdiXLgA8 Cz4tn4+vFSXGSzal+G07P+ZY23oI4aPTsSyRPsM22bSD+MoKNtxMFfTXWIliieIz/z j94sb085zB48KoVeaA1nIdqw9OudJDTqe67mRPrt6+dDI8gNBiUGTI06oLcE45Nn+W ks6NrLsWpOkwA== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtR-0000000Aa2r-0k4t; Sun, 08 Feb 2026 21:20:53 -0500 Message-ID: <20260209022053.029722129@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:32 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 06/20] tracing: Move tracing_set_filter_buffering() into trace_events_hist.c References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt The function tracing_set_filter_buffering() is only used in trace_events_hist.c. Move it to that file and make it static. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260206195936.617080218@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 20 -------------------- kernel/trace/trace.h | 1 - kernel/trace/trace_events_hist.c | 20 ++++++++++++++++++++ 3 files changed, 20 insertions(+), 21 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 8c0f3cfd196b..702ef851db45 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -7891,26 +7891,6 @@ u64 tracing_event_time_stamp(struct trace_buffer *bu= ffer, struct ring_buffer_eve return ring_buffer_event_time_stamp(buffer, rbe); } =20 -/* - * Set or disable using the per CPU trace_buffer_event when possible. - */ -int tracing_set_filter_buffering(struct trace_array *tr, bool set) -{ - guard(mutex)(&trace_types_lock); - - if (set && tr->no_filter_buffering_ref++) - return 0; - - if (!set) { - if (WARN_ON_ONCE(!tr->no_filter_buffering_ref)) - return -EINVAL; - - --tr->no_filter_buffering_ref; - } - - return 0; -} - struct ftrace_buffer_info { struct trace_iterator iter; void *spare; diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h index bb68539c64b7..31fb137e1c66 100644 --- a/kernel/trace/trace.h +++ b/kernel/trace/trace.h @@ -476,7 +476,6 @@ extern struct trace_array *trace_array_find(const char = *instance); extern struct trace_array *trace_array_find_get(const char *instance); =20 extern u64 tracing_event_time_stamp(struct trace_buffer *buffer, struct ri= ng_buffer_event *rbe); -extern int tracing_set_filter_buffering(struct trace_array *tr, bool set); extern int tracing_set_clock(struct trace_array *tr, const char *clockstr); =20 extern bool trace_clock_in_ns(struct trace_array *tr); diff --git a/kernel/trace/trace_events_hist.c b/kernel/trace/trace_events_h= ist.c index 0fc641461be5..e6f449f53afc 100644 --- a/kernel/trace/trace_events_hist.c +++ b/kernel/trace/trace_events_hist.c @@ -6531,6 +6531,26 @@ static bool existing_hist_update_only(char *glob, return updated; } =20 +/* + * Set or disable using the per CPU trace_buffer_event when possible. + */ +static int tracing_set_filter_buffering(struct trace_array *tr, bool set) +{ + guard(mutex)(&trace_types_lock); + + if (set && tr->no_filter_buffering_ref++) + return 0; + + if (!set) { + if (WARN_ON_ONCE(!tr->no_filter_buffering_ref)) + return -EINVAL; + + --tr->no_filter_buffering_ref; + } + + return 0; +} + static int hist_register_trigger(char *glob, struct event_trigger_data *data, struct trace_event_file *file) --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E16682C21CC for ; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; cv=none; b=QzOMr3S5h4rHK7L70GbubS1nSFydh0W0pS8zGfOdOgkkilUhyWHJT6/rMpXB2tMQrpHVx7Co6AaUAsePk3ayYf24x15XcesAro5BY+Enci44pclK+Mbrv44QNRAwTIm+fLAx30GD7zkQNd5d69EY5nczJaxP+a/rTw9fz1ivbN0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; c=relaxed/simple; bh=xUwMGF4Jzywu6ornQqphdCr2uHMPD7KLVnBeYSdAgR4=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=XJKl8gzgbYIWcFAVWfYxqVXGHCzShNRp96mi3beDU4WrMh6qemVpsb6gqa1y7ubPT4YfdfDUIKblBzOdexYvs/dZFrQR32Zxrg8/+BtuLSxSHgZ73hF77JTC+PWoruLQ0smGGj/CYZwLYIKsEezJh1cgsHmyFato6882BL7ruqA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=bAUBuMDT; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="bAUBuMDT" Received: by smtp.kernel.org (Postfix) with ESMTPSA id C584DC2BC86; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603606; bh=xUwMGF4Jzywu6ornQqphdCr2uHMPD7KLVnBeYSdAgR4=; h=Date:From:To:Cc:Subject:References:From; b=bAUBuMDTnGWXn3iKwUAD3VWdzhKigyvDQR9LCBarLc8DbXyWg6FKw9YwFOI1I8jJW vE2R7bTZ2dk2ZIVNW843sF0qOoauy/J1zBPdgwlZKplNWD3wTlXdeGlP/97089asxA qEu5ljX/dxF0oFx5Zqym9uwFIa3TJlFZHNzSUOSz1eyitdpl9X6nPQ57L52RtXk0It SDznQXg1wpe68ArKfwQ+0LkOStZhYAJ+UZX3Tgs6EBi3qHkjJthICuhN9Bh882CB27 U4aXSqbmDEB2f4NK1442grE+c/46AfYfgi0Xm2ZoyG1KoYshqeH+NXxvRPpj3J0cMa giVm8XnQL9muQ== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtR-0000000Aa3L-1SjY; Sun, 08 Feb 2026 21:20:53 -0500 Message-ID: <20260209022053.197351792@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:33 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 07/20] tracing: Clean up use of trace_create_maxlat_file() References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt In trace.c, the function trace_create_maxlat_file() is defined behind the #ifdef CONFIG_TRACER_MAX_TRACE block. The #else part defines it as: #define trace_create_maxlat_file(tr, d_tracer) \ trace_create_file("tracing_max_latency", TRACE_MODE_WRITE, \ d_tracer, tr, &tracing_max_lat_fops) But the one place that it it used has: #ifdef CONFIG_TRACER_MAX_TRACE trace_create_maxlat_file(tr, d_tracer); #endif Which is pointless and also wrong! It only gets created when both CONFIG_TRACE_MAX_TRACE and CONFIG_FS_NOTIFY is defined, but the file itself should not be dependent on CONFIG_FS_NOTIFY. Always create that file when TRACE_MAX_TRACE is defined regardless if FS_NOTIFY is or is not. Cc: Mathieu Desnoyers Acked-by: Masami Hiramatsu (Google) Link: https://patch.msgid.link/20260207191101.0e014abd@robin Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 38 ++++++++++++++++---------------------- 1 file changed, 16 insertions(+), 22 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 702ef851db45..d02c4004c718 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -1912,10 +1912,7 @@ static ssize_t trace_seq_to_buffer(struct trace_seq = *s, void *buf, size_t cnt) unsigned long __read_mostly tracing_thresh; =20 #ifdef CONFIG_TRACER_MAX_TRACE -static const struct file_operations tracing_max_lat_fops; - #ifdef LATENCY_FS_NOTIFY - static struct workqueue_struct *fsnotify_wq; =20 static void latency_fsnotify_workfn(struct work_struct *work) @@ -1932,17 +1929,6 @@ static void latency_fsnotify_workfn_irq(struct irq_w= ork *iwork) queue_work(fsnotify_wq, &tr->fsnotify_work); } =20 -static void trace_create_maxlat_file(struct trace_array *tr, - struct dentry *d_tracer) -{ - INIT_WORK(&tr->fsnotify_work, latency_fsnotify_workfn); - init_irq_work(&tr->fsnotify_irqwork, latency_fsnotify_workfn_irq); - tr->d_max_latency =3D trace_create_file("tracing_max_latency", - TRACE_MODE_WRITE, - d_tracer, tr, - &tracing_max_lat_fops); -} - __init static int latency_fsnotify_init(void) { fsnotify_wq =3D alloc_workqueue("tr_max_lat_wq", @@ -1967,14 +1953,22 @@ void latency_fsnotify(struct trace_array *tr) */ irq_work_queue(&tr->fsnotify_irqwork); } +#endif /* !LATENCY_FS_NOTIFY */ =20 -#else /* !LATENCY_FS_NOTIFY */ - -#define trace_create_maxlat_file(tr, d_tracer) \ - trace_create_file("tracing_max_latency", TRACE_MODE_WRITE, \ - d_tracer, tr, &tracing_max_lat_fops) +static const struct file_operations tracing_max_lat_fops; =20 +static void trace_create_maxlat_file(struct trace_array *tr, + struct dentry *d_tracer) +{ +#ifdef LATENCY_FS_NOTIFY + INIT_WORK(&tr->fsnotify_work, latency_fsnotify_workfn); + init_irq_work(&tr->fsnotify_irqwork, latency_fsnotify_workfn_irq); #endif + tr->d_max_latency =3D trace_create_file("tracing_max_latency", + TRACE_MODE_WRITE, + d_tracer, tr, + &tracing_max_lat_fops); +} =20 /* * Copy the new maximum trace into the separate maximum-trace @@ -2109,7 +2103,9 @@ update_max_tr_single(struct trace_array *tr, struct t= ask_struct *tsk, int cpu) __update_max_tr(tr, tsk, cpu); arch_spin_unlock(&tr->max_lock); } - +#else /* !CONFIG_TRACER_MAX_TRACE */ +static inline void trace_create_maxlat_file(struct trace_array *tr, + struct dentry *d_tracer) { } #endif /* CONFIG_TRACER_MAX_TRACE */ =20 struct pipe_wait { @@ -10664,9 +10660,7 @@ init_tracer_tracefs(struct trace_array *tr, struct = dentry *d_tracer) =20 create_trace_options_dir(tr); =20 -#ifdef CONFIG_TRACER_MAX_TRACE trace_create_maxlat_file(tr, d_tracer); -#endif =20 if (ftrace_create_function_files(tr, d_tracer)) MEM_FAIL(1, "Could not allocate function filter files"); --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 13A6B2D131A for ; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; cv=none; b=MJQHLzHX8zVOxifEBnGUnmKxvqxxm74TrQssC1AVGui4390u2aEXyFR7hj405S5wTVIZClBLtK3ZD5ma0WuStPmGxp9EZiWL+ZZtoug3nHC+oUaI3ZwwKk+8fBsr8IDJcgrl6/qMaFdzyLNwCFxg80NHTVBojTGFBtfXNOPlWVA= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; c=relaxed/simple; bh=G5l2nINDy7C/S1ObXYbobZa/juMWBtTnmfcabjwLGdI=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=JFEe+Y8g5q7ls19SpXMdG1+y/I4yhNVNd8Q9/jSZR+51Fw/Wsnr90wAAnH21wGt27Y3Vy5QYBSY609W4okBUs7s/0HCFJO2+zGiJeP3E0Pc4cFriUpR+k0kVI1oVsmr4pB2CasdVDNruh0JfRvT6EepAqtj3jjCa7pbxB7km9Yk= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=R2lCglqj; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="R2lCglqj" Received: by smtp.kernel.org (Postfix) with ESMTPSA id E9405C4CEF7; Mon, 9 Feb 2026 02:20:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603607; bh=G5l2nINDy7C/S1ObXYbobZa/juMWBtTnmfcabjwLGdI=; h=Date:From:To:Cc:Subject:References:From; b=R2lCglqjPRW9tCyrFyg/sfaHJLwDeNTfcsvYcYZU9X/MPZXcDfRJYKXb61qN8tzVG aTttlPmHywga8Y1Z1z4SZoSWcZBDajHYr+zjikhXX2iQ21zf8Q44Bj/WHN5j+cJISR Uchv/luV3CphBtuXrnWO+SylKypxHf6b7sQogqRZlRy1UJF/Q5HWZIFTEMLkV58BEs qw/aTSFU5N5qqO2tkY586K4V5uSSjI8AJjosVwoF8ePTdi4hY8+o4No5V/KVseNqY1 ciby57zCB/B7X34lV+fOfa/PTsxxhlpKcDfDZvnddqugNolvR8gYNOl/4b83Vq89Y6 s1c1PEx86jc/A== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtR-0000000Aa3r-29Bc; Sun, 08 Feb 2026 21:20:53 -0500 Message-ID: <20260209022053.368331001@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:34 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 08/20] tracing: Make tracing_disabled global for tracing system References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt The tracing_disabled variable is set to one on boot up to prevent some parts of tracing to access the tracing infrastructure before it is set up. It also can be set after boot if an anomaly is discovered. It is currently a static variable in trace.c and can be accessed via a function call trace_is_disabled(). There's really no reason to use a function call as the tracing subsystem should be able to access it directly. By making the variable accessed directly, code can be moved out of trace.c without adding overhead of a function call to see if tracing is disabled or not. Make tracing_disabled global and remove the tracing_is_disabled() helper function. Also add some "unlikely()"s around tracing_disabled where it's checked in hot paths. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208032449.483690153@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 13 ++++--------- kernel/trace/trace.h | 3 ++- kernel/trace/trace_events.c | 2 +- kernel/trace/trace_kprobe.c | 2 +- 4 files changed, 8 insertions(+), 12 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index d02c4004c718..1ff40c88e75c 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -114,7 +114,7 @@ DEFINE_PER_CPU(bool, trace_taskinfo_save); * of the tracer is successful. But that is the only place that sets * this back to zero. */ -static int tracing_disabled =3D 1; +int tracing_disabled =3D 1; =20 cpumask_var_t __read_mostly tracing_buffer_mask; =20 @@ -3423,7 +3423,7 @@ int __trace_array_vprintk(struct trace_buffer *buffer, unsigned int trace_ctx; char *tbuffer; =20 - if (tracing_disabled) + if (unlikely(tracing_disabled)) return 0; =20 /* Don't pollute graph traces with trace_vprintk internals */ @@ -4765,11 +4765,6 @@ int tracing_open_generic(struct inode *inode, struct= file *filp) return 0; } =20 -bool tracing_is_disabled(void) -{ - return (tracing_disabled) ? true: false; -} - /* * Open and update trace_array ref count. * Must have the current trace_array passed to it. @@ -7609,7 +7604,7 @@ tracing_mark_write(struct file *filp, const char __us= er *ubuf, unsigned long ip; char *buf; =20 - if (tracing_disabled) + if (unlikely(tracing_disabled)) return -EINVAL; =20 if (!(tr->trace_flags & TRACE_ITER(MARKERS))) @@ -7689,7 +7684,7 @@ tracing_mark_raw_write(struct file *filp, const char = __user *ubuf, ssize_t written =3D -ENODEV; char *buf; =20 - if (tracing_disabled) + if (unlikely(tracing_disabled)) return -EINVAL; =20 if (!(tr->trace_flags & TRACE_ITER(MARKERS))) diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h index 31fb137e1c66..433705bef480 100644 --- a/kernel/trace/trace.h +++ b/kernel/trace/trace.h @@ -657,6 +657,8 @@ trace_buffer_iter(struct trace_iterator *iter, int cpu) return iter->buffer_iter ? iter->buffer_iter[cpu] : NULL; } =20 +extern int tracing_disabled; + int tracer_init(struct tracer *t, struct trace_array *tr); int tracing_is_enabled(void); void tracing_reset_online_cpus(struct array_buffer *buf); @@ -668,7 +670,6 @@ int tracing_release_generic_tr(struct inode *inode, str= uct file *file); int tracing_open_file_tr(struct inode *inode, struct file *filp); int tracing_release_file_tr(struct inode *inode, struct file *filp); int tracing_single_release_file_tr(struct inode *inode, struct file *filp); -bool tracing_is_disabled(void); bool tracer_tracing_is_on(struct trace_array *tr); void tracer_tracing_on(struct trace_array *tr); void tracer_tracing_off(struct trace_array *tr); diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index af6d1fe5cab7..61fe01dce7a6 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c @@ -2268,7 +2268,7 @@ static int subsystem_open(struct inode *inode, struct= file *filp) struct event_subsystem *system =3D NULL; int ret; =20 - if (tracing_is_disabled()) + if (unlikely(tracing_disabled)) return -ENODEV; =20 /* Make sure the system still exists */ diff --git a/kernel/trace/trace_kprobe.c b/kernel/trace/trace_kprobe.c index 89d2740f7bb5..061658518605 100644 --- a/kernel/trace/trace_kprobe.c +++ b/kernel/trace/trace_kprobe.c @@ -2083,7 +2083,7 @@ static __init int kprobe_trace_self_tests_init(void) struct trace_kprobe *tk; struct trace_event_file *file; =20 - if (tracing_is_disabled()) + if (unlikely(tracing_disabled)) return -ENODEV; =20 if (tracing_selftest_disabled) --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3DC852D481F for ; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; cv=none; b=G7BOSafKTvgwIlxeu86XWtShLmHnQugCb1gvrYYebJtFDVGJLcgvPQkPAMFw5b7/acwixI/sI0rtbkWh+jsZ5BdIpHISm90BCX+vlkIyBflTDLkJhqVaQsa7J+VcuaZAHUOp1tplMdmxSSQQM2p/7JN33vGvEJ9ZvearV5EV1tM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; c=relaxed/simple; bh=Fuq3MVg+HWZWakjXsFW/WTIKLxaZ7Ml7kyax6qOut98=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=Mi2LjvkcklW4VFe+FcLtogwEzRqSj3onRmxaH9Ch5OAQGeD86dvm6zOaICFlMvS2LRsvasHNmUIqpGomOdqDXuLFZZuNYXirf/TmXNhMQ8o55lksyd02VLKwcrWgFfxw41e5fHu/cBHMi0Sh4eBPG2DW1eTLG1czUpRUOIkiHbM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=HvxamJRs; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="HvxamJRs" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 197D4C19425; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603607; bh=Fuq3MVg+HWZWakjXsFW/WTIKLxaZ7Ml7kyax6qOut98=; h=Date:From:To:Cc:Subject:References:From; b=HvxamJRsHU1R7IfcF3pt8oAoXvNm5zA3ZiTNjWE+eiAYdyf60fEfnfeCGco55/QjE cGhQHk12S8xuhGmVjLWvs4fbNigPlpdTR9TUHRcWe6I20c11RsTNZJ7gyApPmsPazA gjneenq4qlbv7/dNjStEsZDoGqOFgBpyu6ssx2DasFgPsd0sOR9Lepp6zE9FLtr0gt mAV7AUGi8pQy9zaAbYisavrAvHfvtk/kWXz+89O0vlqVbZPd3xEoSeUgxHbwaJOCYt lbuq01zvSfJRdzMeFS8dELqbZ3YhkPJMUCLZj+nlc3vlzB3INvqhCf3zZVW7HMHPuq d6bVFc08lTF8Q== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtR-0000000Aa4M-2ofI; Sun, 08 Feb 2026 21:20:53 -0500 Message-ID: <20260209022053.530777281@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:35 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 09/20] tracing: Make tracing_selftest_running global to the tracing subsystem References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt The file trace.c has become a catchall for most things tracing. Start making it smaller by breaking out various aspects into their own files. Make the variable tracing_selftest_running global so that it can be used by other files in the tracing subsystem and trace.c can be split up. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208032449.648932796@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 3 +-- kernel/trace/trace.h | 2 ++ 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 1ff40c88e75c..f040ee4fe101 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -67,7 +67,7 @@ * insertions into the ring-buffer such as trace_printk could occurred * at the same time, giving false positive or negative results. */ -static bool __read_mostly tracing_selftest_running; +bool __read_mostly tracing_selftest_running; =20 /* * If boot-time tracing including tracers/events via kernel cmdline @@ -83,7 +83,6 @@ void __init disable_tracing_selftest(const char *reason) } } #else -#define tracing_selftest_running 0 #define tracing_selftest_disabled 0 #endif =20 diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h index 433705bef480..19cffc7b5852 100644 --- a/kernel/trace/trace.h +++ b/kernel/trace/trace.h @@ -863,6 +863,7 @@ extern int trace_selftest_startup_nop(struct tracer *tr= ace, struct trace_array *tr); extern int trace_selftest_startup_branch(struct tracer *trace, struct trace_array *tr); +extern bool __read_mostly tracing_selftest_running; /* * Tracer data references selftest functions that only occur * on boot up. These can be __init functions. Thus, when selftests @@ -875,6 +876,7 @@ static inline void __init disable_tracing_selftest(cons= t char *reason) } /* Tracers are seldom changed. Optimize when selftests are disabled. */ #define __tracer_data __read_mostly +#define tracing_selftest_running 0 #endif /* CONFIG_FTRACE_STARTUP_TEST */ =20 extern void *head_page(struct trace_array_cpu *data); --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 74AE12D7384 for ; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; cv=none; b=O7yWef5JMsz/sD5CnZIqbpIrzJifzVvpl2LbPDDiHlKQ0eg41X9cH9eUTQZl0dpa+yUjPCzpDDLGdLLRsGp+yFYnrk2AueYsQf1dxXpRf98Bc0CZfC4GziiSh7RX9msWbTK3Vj5dOW2ubvcI7h3FkERGvaodGZjf0Vo6Ik+Dwec= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; c=relaxed/simple; bh=KMxvfazyQcUqTvhjfQ4HMsuXQB1IjxvgdUjPavDCpl0=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=Ks1wNDlrsgAfeip/alsMzi/adKyQmFMTeiwDIL6i/4Go78TUAiWau2Dvkil9Z8QWwifhE8x+aQGy/2wemc0tAX1ZJuNI3Fc705vJdvKdV9kZOSd3rIumcDWlELapOcH85kXj4Mhsb7byXWLUot2Qt6v2unAceAnJ4eelYUMf0lU= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=O3vH92J5; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="O3vH92J5" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4F86BC4CEF7; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603607; bh=KMxvfazyQcUqTvhjfQ4HMsuXQB1IjxvgdUjPavDCpl0=; h=Date:From:To:Cc:Subject:References:From; b=O3vH92J5LNLxXIDvTIeMaG5BDNuKvFCj7r2+yiz3zJqEwDxhwE/cZTBi7U0sFjW4C YuD3FBwOGFvczg2okhI7+DLdKSjGEzRZ2ERf7On4zlvg5R9VtEBG+3cYj5HvJvGPVC Pen+HtRKxdEsWUi7s6lMbBtGcPhwnyOjSAk+i6+ApXeaew1BrL5RUkETsc/Ldqp/ZK HL0AYk0gjKM1uK7jJDhU2wzL6L8CCrDyTsbcegQ4Y2yzy6lFh4YzhjkwBTB83120Na bCfVmDG2/kZ317k0i2NQIywjdrDGpVLl/MU3NCo6enfdwFZcpPY7JshAIMUgfstlUG zhvoiA/Fd3kZw== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtR-0000000Aa4r-3V0c; Sun, 08 Feb 2026 21:20:53 -0500 Message-ID: <20260209022053.691489583@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:36 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 10/20] tracing: Move __trace_buffer_{un}lock_*() functions to trace.h References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt The file trace.c has become a catchall for most things tracing. Start making it smaller by breaking out various aspects into their own files. Move the __always_inline functions __trace_buffer_lock_reserve(), __trace_buffer_unlock_commit() and trace_event_setup() into trace.h. The trace.c file will be split up and these functions will be used in more than one of these files. As they are already __always_inline they can easily be moved into the trace.h header file. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208032449.813550600@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 42 ------------------------------------------ kernel/trace/trace.h | 41 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 41 insertions(+), 42 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index f040ee4fe101..55cd0c774886 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -1058,30 +1058,6 @@ static inline void ftrace_trace_stack(struct trace_a= rray *tr, =20 #endif =20 -static __always_inline void -trace_event_setup(struct ring_buffer_event *event, - int type, unsigned int trace_ctx) -{ - struct trace_entry *ent =3D ring_buffer_event_data(event); - - tracing_generic_entry_update(ent, type, trace_ctx); -} - -static __always_inline struct ring_buffer_event * -__trace_buffer_lock_reserve(struct trace_buffer *buffer, - int type, - unsigned long len, - unsigned int trace_ctx) -{ - struct ring_buffer_event *event; - - event =3D ring_buffer_lock_reserve(buffer, len); - if (event !=3D NULL) - trace_event_setup(event, type, trace_ctx); - - return event; -} - void tracer_tracing_on(struct trace_array *tr) { if (tr->array_buffer.buffer) @@ -1109,24 +1085,6 @@ void tracing_on(void) } EXPORT_SYMBOL_GPL(tracing_on); =20 - -static __always_inline void -__buffer_unlock_commit(struct trace_buffer *buffer, struct ring_buffer_eve= nt *event) -{ - __this_cpu_write(trace_taskinfo_save, true); - - /* If this is the temp buffer, we need to commit fully */ - if (this_cpu_read(trace_buffered_event) =3D=3D event) { - /* Length is in event->array[0] */ - ring_buffer_write(buffer, event->array[0], &event->array[1]); - /* Release the temp buffer */ - this_cpu_dec(trace_buffered_event_cnt); - /* ring_buffer_unlock_commit() enables preemption */ - preempt_enable_notrace(); - } else - ring_buffer_unlock_commit(buffer); -} - int __trace_array_puts(struct trace_array *tr, unsigned long ip, const char *str, int size) { diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h index 19cffc7b5852..c2beabe96952 100644 --- a/kernel/trace/trace.h +++ b/kernel/trace/trace.h @@ -1568,6 +1568,47 @@ char *trace_user_fault_read(struct trace_user_buf_in= fo *tinfo, const char __user *ptr, size_t size, trace_user_buf_copy copy_func, void *data); =20 +static __always_inline void +trace_event_setup(struct ring_buffer_event *event, + int type, unsigned int trace_ctx) +{ + struct trace_entry *ent =3D ring_buffer_event_data(event); + + tracing_generic_entry_update(ent, type, trace_ctx); +} + +static __always_inline struct ring_buffer_event * +__trace_buffer_lock_reserve(struct trace_buffer *buffer, + int type, + unsigned long len, + unsigned int trace_ctx) +{ + struct ring_buffer_event *event; + + event =3D ring_buffer_lock_reserve(buffer, len); + if (event !=3D NULL) + trace_event_setup(event, type, trace_ctx); + + return event; +} + +static __always_inline void +__buffer_unlock_commit(struct trace_buffer *buffer, struct ring_buffer_eve= nt *event) +{ + __this_cpu_write(trace_taskinfo_save, true); + + /* If this is the temp buffer, we need to commit fully */ + if (this_cpu_read(trace_buffered_event) =3D=3D event) { + /* Length is in event->array[0] */ + ring_buffer_write(buffer, event->array[0], &event->array[1]); + /* Release the temp buffer */ + this_cpu_dec(trace_buffered_event_cnt); + /* ring_buffer_unlock_commit() enables preemption */ + preempt_enable_notrace(); + } else + ring_buffer_unlock_commit(buffer); +} + static inline void __trace_event_discard_commit(struct trace_buffer *buffer, struct ring_buffer_event *event) --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 77F862D738E for ; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; cv=none; b=J+5+v64IxdGzScZ7LUhPNi2sVohfhcSAg+thpErVKOXV+KyaBi4fcN53wA3ZpyvhYReg6gPQa3sYCplAJiWidiGs3Vj1YU1jk748cy9okiYJnOkL5xFMt2vD2wAB9awS3AU77BgCNDpUk5MhrHlZWfmNExU5hnbA/J2L+6J0H+I= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; c=relaxed/simple; bh=dekH0pbbsKuZXJAh6/ik9exOyvu3HaNrCaWGRiCcxt4=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=t4p0WXXMZKVrTGIwE811Crq6HRZ4WCQHVQZWNpHi9QHlRZj16ImD2cogwZf3F7Fg/RkaUC3t3FfV+6QGUmLpJAyj8GKBkDGpktN5xjJdwDuUy9xB/iReGrTiM5V7kqBT47JK5emYa8IytjO04g/p/9xoit8TU/eOpiq1rODhU7A= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=NsS1cgtz; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="NsS1cgtz" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5AE5CC16AAE; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603607; bh=dekH0pbbsKuZXJAh6/ik9exOyvu3HaNrCaWGRiCcxt4=; h=Date:From:To:Cc:Subject:References:From; b=NsS1cgtzOYd2mq3zuR5bhrgDkGFGFhWH1X5n+q9+T10RtN1yMTrF94v9PrGv5vHvl R4Prz2GV8z0AEjArETLT/Nbv5/nxBGEH87I5ce2qAuW7NaZCkr+no/YkSaNqn5BIut iYTa9dWugqSTIBmMV9VTHf2qDYE86wBqKWJUz1YF79hJTu88uxoGr7nOqYtdOfkFQ8 1hpwycCoPgxdvuy8p/h9eZsVsnPwLoP1IltKhLEHMpkb7k7FlZaFNHOYTN+g88OsBY oi4XB/REIOiyolfeDQywLbzEeZjK1k2tVkOmBMPtOzaU1+/4Vw5JuThkVtoXoh04fE 02vEQD2X0uezQ== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtR-0000000Aa5M-4BEN; Sun, 08 Feb 2026 21:20:53 -0500 Message-ID: <20260209022053.852007796@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:37 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 11/20] tracing: Move ftrace_trace_stack() out of trace.c and into trace.h References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt The file trace.c has become a catchall for most things tracing. Start making it smaller by breaking out various aspects into their own files. Make ftrace_trace_stack() into a static inline that tests if stack tracing is enabled and if so to call __ftrace_trace_stack() to do the stack trace. This keeps the test inlined in the fast paths and only does the function call if stack tracing is enabled. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208032449.974218132@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 45 ++++---------------------------------------- kernel/trace/trace.h | 31 ++++++++++++++++++++++++++++++ 2 files changed, 35 insertions(+), 41 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 55cd0c774886..a515b5241391 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -1032,32 +1032,6 @@ static inline void trace_access_lock_init(void) =20 #endif =20 -#ifdef CONFIG_STACKTRACE -static void __ftrace_trace_stack(struct trace_array *tr, - struct trace_buffer *buffer, - unsigned int trace_ctx, - int skip, struct pt_regs *regs); -static inline void ftrace_trace_stack(struct trace_array *tr, - struct trace_buffer *buffer, - unsigned int trace_ctx, - int skip, struct pt_regs *regs); - -#else -static inline void __ftrace_trace_stack(struct trace_array *tr, - struct trace_buffer *buffer, - unsigned int trace_ctx, - int skip, struct pt_regs *regs) -{ -} -static inline void ftrace_trace_stack(struct trace_array *tr, - struct trace_buffer *buffer, - unsigned long trace_ctx, - int skip, struct pt_regs *regs) -{ -} - -#endif - void tracer_tracing_on(struct trace_array *tr) { if (tr->array_buffer.buffer) @@ -2964,10 +2938,10 @@ struct ftrace_stacks { static DEFINE_PER_CPU(struct ftrace_stacks, ftrace_stacks); static DEFINE_PER_CPU(int, ftrace_stack_reserve); =20 -static void __ftrace_trace_stack(struct trace_array *tr, - struct trace_buffer *buffer, - unsigned int trace_ctx, - int skip, struct pt_regs *regs) +void __ftrace_trace_stack(struct trace_array *tr, + struct trace_buffer *buffer, + unsigned int trace_ctx, + int skip, struct pt_regs *regs) { struct ring_buffer_event *event; unsigned int size, nr_entries; @@ -3050,17 +3024,6 @@ static void __ftrace_trace_stack(struct trace_array = *tr, trace_clear_recursion(bit); } =20 -static inline void ftrace_trace_stack(struct trace_array *tr, - struct trace_buffer *buffer, - unsigned int trace_ctx, - int skip, struct pt_regs *regs) -{ - if (!(tr->trace_flags & TRACE_ITER(STACKTRACE))) - return; - - __ftrace_trace_stack(tr, buffer, trace_ctx, skip, regs); -} - void __trace_stack(struct trace_array *tr, unsigned int trace_ctx, int skip) { diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h index c2beabe96952..605ee23f3262 100644 --- a/kernel/trace/trace.h +++ b/kernel/trace/trace.h @@ -2279,6 +2279,37 @@ static inline void sanitize_event_name(char *name) *name =3D '_'; } =20 +#ifdef CONFIG_STACKTRACE +void __ftrace_trace_stack(struct trace_array *tr, + struct trace_buffer *buffer, + unsigned int trace_ctx, + int skip, struct pt_regs *regs); + +static __always_inline void ftrace_trace_stack(struct trace_array *tr, + struct trace_buffer *buffer, + unsigned int trace_ctx, + int skip, struct pt_regs *regs) +{ + if (!(tr->trace_flags & TRACE_ITER(STACKTRACE))) + return; + + __ftrace_trace_stack(tr, buffer, trace_ctx, skip, regs); +} +#else +static inline void __ftrace_trace_stack(struct trace_array *tr, + struct trace_buffer *buffer, + unsigned int trace_ctx, + int skip, struct pt_regs *regs) +{ +} +static inline void ftrace_trace_stack(struct trace_array *tr, + struct trace_buffer *buffer, + unsigned long trace_ctx, + int skip, struct pt_regs *regs) +{ +} +#endif + /* * This is a generic way to read and write a u64 value from a file in trac= efs. * --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BD3652DAFA9 for ; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; cv=none; b=ZMYxE1AYd9eJWnkQjxbkWGob9lmutw7Jur2CCA3/p4G/XX/DPFQb4mCZBn2oa8ys3vhp0+qa/2yk1z/WdEfedWDktiHGoYOUSwkSLBnAEjceYOoCmpBLulXff+yF0WaGGZ2CtMJwlfxiwp8jFKDaOGwKJjode0U9yRj5SdMP7Ug= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; c=relaxed/simple; bh=A0mk7oGtNvIZopDwfm6Bwp2uQhi2hrR1Ie3nN5yg274=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=pgGZqhG0S0qIl62iFs4mWGVy0B+xyrIiJmDwrSrGK7QZvixP79JnwG1vrwtc2Q3SGAXGlSy0Z8q7JGgr8iqCiNK8dJuIYC2vOqgZ+s9idk2Q8F4X2D9tbHbiaNJRS2Qhz4RoTNdLYNEFnsFK44lQO7xPKp+CIid/y4yn9oBv/sQ= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=q2JDcTx1; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="q2JDcTx1" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 93539C19425; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603607; bh=A0mk7oGtNvIZopDwfm6Bwp2uQhi2hrR1Ie3nN5yg274=; h=Date:From:To:Cc:Subject:References:From; b=q2JDcTx1r9bxbITzu68XZ0H0EyC8mMJ8UxE/MCQtRu/uQ9Jng4K2Zx35QxV3CmwU2 JQ4GWqMTJAhRH+06w3HXIrW5ZlvERbVy8uAG7Sx2iYrVQ+7PdKSpM3tMdtPZNu0lqQ 0t2xnIkYBqNTwP11yZjbEfJedJMJ4fXpNxTsGp2zs6GTxKbkSozWh9U/TYjesMthmm rg0uVjWhY6NQ8pqbIOxXpv9RETuPjZyvVnY4m62FjSedgugmgjsSGCBCbZHWzyJ8za Ql+iEdYThlb/skR2KZpBV0orZOWfF++0bbr/QVLJuFiFAMsNTXYrnb6jEYRYoZtdlU tt3LfZJ6lmi9w== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtS-0000000Aa5r-0fSR; Sun, 08 Feb 2026 21:20:54 -0500 Message-ID: <20260209022054.014681150@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:38 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 12/20] tracing: Make printk_trace global for tracing system References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt The printk_trace is used to determine which trace_array trace_printk() writes to. By making it a global variable among the tracing subsystem it will allow the trace_printk functions to be moved out of trace.c and still have direct access to that variable. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208032450.144525891@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 2 +- kernel/trace/trace.h | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index a515b5241391..4a73822e2603 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -534,7 +534,7 @@ static struct trace_array global_trace =3D { .trace_flags =3D TRACE_DEFAULT_FLAGS, }; =20 -static struct trace_array *printk_trace =3D &global_trace; +struct trace_array *printk_trace =3D &global_trace; =20 /* List of trace_arrays interested in the top level trace_marker */ static LIST_HEAD(marker_copies); diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h index 605ee23f3262..921e4daa2825 100644 --- a/kernel/trace/trace.h +++ b/kernel/trace/trace.h @@ -482,6 +482,8 @@ extern bool trace_clock_in_ns(struct trace_array *tr); =20 extern unsigned long trace_adjust_address(struct trace_array *tr, unsigned= long addr); =20 +extern struct trace_array *printk_trace; + /* * The global tracer (top) should be the first trace array added, * but we check the flag anyway. --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D73022DC322 for ; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; cv=none; b=i+2jbk1tVajQoCf/0m2L2LBaRGFUgQvx5M+KPn7dOQdxBFxR0qVFU3zDJ1RaBsPYykZSfZPP3SXLAszBrDMCRvNvlKutSYGqtoskOt8PmA4AhgL+Qi5w2FQBDpBq8oCT3rRbWheBBnemF7w++1iIlEu/g+djTYrKhMiQnJV+IbE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603607; c=relaxed/simple; bh=pUTg/sEmFc2m5VRwG4r8m/Nn6E+oF36FMnyGMPKOPvs=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=mJL4qL7iTkE/bws3vTatufEZvWU9vgqL4Ag2ZpllW8Yd+RvjK4lIDis+a/DIbWeZcjV6FwIaUSfjV/SxjM5Il5blgA4oeC8CLhXyKiXTVpFUXxLhc/y9duiW3Xwj7NfXpNPAHAEaocPufpfwPA6g394u4kszqAIG96F9O2knFcE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=YlT9ehuR; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="YlT9ehuR" Received: by smtp.kernel.org (Postfix) with ESMTPSA id BAF0BC4CEF7; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603607; bh=pUTg/sEmFc2m5VRwG4r8m/Nn6E+oF36FMnyGMPKOPvs=; h=Date:From:To:Cc:Subject:References:From; b=YlT9ehuRmSdTz4UnCCylNg4LPRUhpwLQaFtmdgrMkFJRx45wcpUsv6yii1Z9Nc7zx bHOof5p4fMSHcbhh/Y5+/bJ4fYh6y08L6QytsGZsO9TfKkTWsc4OGpOYeIQQBEo03t T0GoXb0wTJtTFfnpsyWLgZS1bgfARvm31iEL7yfBHhE/CWyt795SYsDlqUJU6WCFzx FoHuLhG7CLwnQwHJFL8CQ5YRGRSggCVV6a6rZGvVhHE9DLTN9C027MR5JErHkCmRLt rAyEk4CFkArtGLjA/3irafiuBSNZZvKFqabatugB9hMjYGjMpYwK/1QKG9QKlx43Sd r3qYw0KoWeaIQ== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtS-0000000Aa6M-1KmF; Sun, 08 Feb 2026 21:20:54 -0500 Message-ID: <20260209022054.178002010@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:39 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 13/20] tracing: Make tracing_update_buffers() take NULL for global_trace References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt The trace.c file has become a dumping ground for all tracing code and has become quite large. In order to move the trace_printk functions out of it these functions can not access global_trace directly, as that is something that needs to stay static in trace.c. Have tracing_update_buffers() take NULL for its trace_array to denote it should work on the global_trace top level trace_array allows that function to be used outside of trace.c and still update the global_trace trace_array. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208032450.318864210@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 4a73822e2603..601b6f622391 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -3234,7 +3234,7 @@ void trace_printk_init_buffers(void) pr_warn("**********************************************************\n"); =20 /* Expand the buffers to set size */ - if (tracing_update_buffers(&global_trace) < 0) + if (tracing_update_buffers(NULL) < 0) pr_err("Failed to expand tracing buffers for trace_printk() calls\n"); else buffers_allocated =3D 1; @@ -6186,6 +6186,9 @@ int tracing_update_buffers(struct trace_array *tr) { int ret =3D 0; =20 + if (!tr) + tr =3D &global_trace; + guard(mutex)(&trace_types_lock); =20 update_last_data(tr); --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EAE822DC76C for ; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603608; cv=none; b=PmfOpsr5nd+aVoWIHheJwtFiWgx4vyutC+8wyI6waisynt92ihIbTZSLNyTYeOaYBBEZ9j/aYwMw/go6FTuiLuGr2KbAqHg/5hymII1QsIEA1o0Ug6+TmLIPlsmB1pcS6vo7/hK3xUv7F1j+Gvr5XZUXHWb3LCSiM63OVtRuMKM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603608; c=relaxed/simple; bh=RvoQoNJDeWhZ6lz/n7VlEi4cb9/I1H8gqj6touL4uYw=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=OaZU0NCHEXl6Xs2axNtpg9XehoH2lGlZfKCOmh4NdgjPYeikqjncvG8t8cJNjr9QooZlvWEYx6JS+zzAcG8M+Qbijpv2Vw1MqBG2iYASIWcjOEAy6QyYiezE/ya7ik3NVz5vqZGaCYuiyGYE62sw14BbnVFBnqzaUVidqwPZMKM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=ALQzW3JJ; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="ALQzW3JJ" Received: by smtp.kernel.org (Postfix) with ESMTPSA id D1425C19424; Mon, 9 Feb 2026 02:20:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603607; bh=RvoQoNJDeWhZ6lz/n7VlEi4cb9/I1H8gqj6touL4uYw=; h=Date:From:To:Cc:Subject:References:From; b=ALQzW3JJLvVdB7N/wctLA8nKUquojPqAGQcPP/xHYHrVO5Tog3YTScT0jMvW0rds7 czX/4/RtTEzl4mFmo/QybLyrg6b5oW8hlpbsNREg2kS9Ab6Iidq0tEfBDepK67l7Y0 KFR6XHPkh+06NXuhRFSe+KnlFqspol9RDsCmsE2EsLZ2RLMoBDtw9XuBFzNBCJiqyf qo0WRMzoQI1YElnjGAF/R/TAtcN75fMUrfXNw+qNYnwV3TohA528FVfZdQgZxjzAiM S7zOB2vhzHtcki/82GpMk2Z1hAQemQw29UC6gvXc1BZ2kUnxUnmmHmesFjNnLt+cZd 9xoVxz2w6fcLg== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtS-0000000Aa6r-22lI; Sun, 08 Feb 2026 21:20:54 -0500 Message-ID: <20260209022054.339187156@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:40 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 14/20] tracing: Have trace_printk functions use flags instead of using global_trace References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt The trace.c file has become a dumping ground for all tracing code and has become quite large. In order to move the trace_printk functions out of it these functions can not access global_trace directly, as that is something that needs to stay static in trace.c. Instead of testing the trace_array tr pointer to &global_trace, test the tr->flags to see if TRACE_ARRAY_FL_GLOBAL set. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208032450.491116245@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 601b6f622391..f4ae80564615 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -1071,7 +1071,8 @@ int __trace_array_puts(struct trace_array *tr, unsign= ed long ip, if (!(tr->trace_flags & TRACE_ITER(PRINTK))) return 0; =20 - if (unlikely(tracing_selftest_running && tr =3D=3D &global_trace)) + if (unlikely(tracing_selftest_running && + (tr->flags & TRACE_ARRAY_FL_GLOBAL))) return 0; =20 if (unlikely(tracing_disabled)) @@ -3386,7 +3387,7 @@ int __trace_array_vprintk(struct trace_buffer *buffer, int trace_array_vprintk(struct trace_array *tr, unsigned long ip, const char *fmt, va_list args) { - if (tracing_selftest_running && tr =3D=3D &global_trace) + if (tracing_selftest_running && (tr->flags & TRACE_ARRAY_FL_GLOBAL)) return 0; =20 return __trace_array_vprintk(tr->array_buffer.buffer, ip, fmt, args); @@ -3422,7 +3423,7 @@ int trace_array_printk(struct trace_array *tr, return -ENOENT; =20 /* This is only allowed for created instances */ - if (tr =3D=3D &global_trace) + if (tr->flags & TRACE_ARRAY_FL_GLOBAL) return 0; =20 if (!(tr->trace_flags & TRACE_ITER(PRINTK))) @@ -3449,7 +3450,7 @@ int trace_array_init_printk(struct trace_array *tr) return -ENOENT; =20 /* This is only allowed for created instances */ - if (tr =3D=3D &global_trace) + if (tr->flags & TRACE_ARRAY_FL_GLOBAL) return -EINVAL; =20 return alloc_percpu_trace_buffer(); --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2F86F2DEA94 for ; Mon, 9 Feb 2026 02:20:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603608; cv=none; b=CpA4Ei0vNkPplNU8rZVUI88QK3Y4JC76olJV36udejZaRUWy+eTuPsfgBQlG7kamHZjxDAchD/DwdjkQFwYa4Q5AHqa4cTTzZg4Q3EzYT2SQwvijhDo3M9n0niX6z/LqyHR7JSh2AJxux+nv1fbFyYDG+INvD/X0h6ix+JNsDxM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603608; c=relaxed/simple; bh=H07E2wwO7Ir39hqPQXrL8iCbRhtNnVVdqS/KykLxLDw=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=Llbc3epyme3kzkv4G35gfZocxpjzQXw4pkqXJIVnCnyBVrPypjt22tQ581myHQgZdbaHeUwpxSjNlkkiBGCKCrCsxVJYR/kzD5OMJ2t1U7ZoZVRGs9drAd60OLxEEKDQQWosSFYypwNyvRwqG5lxs8BDvpSIDOoHPrcRZ0ls4Q4= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=gNJmVW1a; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="gNJmVW1a" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 13588C2BC86; Mon, 9 Feb 2026 02:20:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603608; bh=H07E2wwO7Ir39hqPQXrL8iCbRhtNnVVdqS/KykLxLDw=; h=Date:From:To:Cc:Subject:References:From; b=gNJmVW1agnXOqNkLJMjzC5VazjhgPoIPny9s+wilruU4J9artpbIhZ5V0USc8lnqG SzSkBOQ31qABBJw4Cx6uVOn4oVfpGcm5vKr2YYVbAP0mmeQR3qS38TJ+x9W1z+TVB6 blrMeoTF8Vu60tvCDcUPefZlKK8QyY/igMxaZU0MWMb2s2SPoEpE+BcjU0bPCXix1O ffE6e8C4LuAtD/pWy0fWNmLriVaoaCKk8Xzr3+KDQMtjfSDn+/YFz5rJNNFDfjBJXn cpDLKQkaNXgxoFXk5mxKfEicDq1uPbgM+vs62F9YYDc/Ruv4lIvC0kIbz9Ocj2D6gD y4fLSo6+7+xyg== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtS-0000000Aa7M-2kBe; Sun, 08 Feb 2026 21:20:54 -0500 Message-ID: <20260209022054.507484055@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:41 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 15/20] tracing: Use system_state in trace_printk_init_buffers() References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt The function trace_printk_init_buffers() is used to expand tha trace_printk buffers when trace_printk() is used within the kernel or in modules. On kernel boot up, it holds off from starting the sched switch cmdline recorder, but will start it immediately when it is added by a module. Currently it uses a trick to see if the global_trace buffer has been allocated or not to know if it was called by module load or not. But this is more of a hack, and can not be used when this code is moved out of trace.c. Instead simply look at the system_state and if it is running then it is know that it could only be called by module load. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208032450.660237094@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index f4ae80564615..4066c33674e7 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -3243,10 +3243,9 @@ void trace_printk_init_buffers(void) /* * trace_printk_init_buffers() can be called by modules. * If that happens, then we need to start cmdline recording - * directly here. If the global_trace.buffer is already - * allocated here, then this was called by module code. + * directly here. */ - if (global_trace.array_buffer.buffer) + if (system_state =3D=3D SYSTEM_RUNNING) tracing_start_cmdline_record(); } EXPORT_SYMBOL_GPL(trace_printk_init_buffers); --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7CD152E36F8 for ; Mon, 9 Feb 2026 02:20:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603608; cv=none; b=gHZOTf8h6jUAajx2R5zbt6zTGBrPJpTu2NQpZRNCyo8tgb6rPfgKiwUaB8sSoRsJ/uzQyO0U3Hba+/ijUtAtXTfwwbB+TZVWeywysioRWP0GTyYSpofwTk3gbmDbGT/aAPMXQH20lwBwFG9S9Obpfi8RhpqupxBOTwxWJceH8sY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603608; c=relaxed/simple; bh=MzKJvl+bOtRCmHCzI8fmm0MNJWKHYd8jZ1m9JWDShnc=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=dfBFf5k6XHJf8PzPplDk/akthesO/6gkbYV4o8R5vH20nsuRZ9VEctQU0KKT0AK41s2OKMwvRoGH4evdENA7yVkC3BVb1EjTHH6zG4gQ+HKeFz5CLWICv0vmrvNx+DgU2NNB2xH7QuvwibzG3Z45xtIkGk5SRR0qExrEE+BIvV4= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=ByreglAP; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="ByreglAP" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 3BD0BC4CEF7; Mon, 9 Feb 2026 02:20:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603608; bh=MzKJvl+bOtRCmHCzI8fmm0MNJWKHYd8jZ1m9JWDShnc=; h=Date:From:To:Cc:Subject:References:From; b=ByreglAPAc/wkojAXblzj01vzvi76ZNPc+7aRsfkidqrLx/jX2IfYqwKvfSZ7zedu ibzrcEoTpG4sSu9LmEE9XSF9vO64bgyAVU0FMCkEHDOwsPwB5xjcwe/2Qv7fSs2/28 3v6goVUsjPu5PEkARXxIjCct4k4EoR6b4FBMuvGWKnKJ7xN3EDzK15fEMftnaI2/Kg ot+k/btTAyCNHyu7r+7OrOQ8AYip9V8MM9x7k9v+n3PHVxMr1RHntu9PHGnwTY2eDJ mW0bsMIJsUuw3ug1PSDPc2mv1xtvQDVcoBsAp8zeLmqDpouSDE5kfGhEhiajbcJ64C 6FmkglI3Urc7g== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtS-0000000Aa7r-3RIS; Sun, 08 Feb 2026 21:20:54 -0500 Message-ID: <20260209022054.674149000@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:42 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 16/20] tracing: Move trace_printk functions out of trace.c and into trace_printk.c References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt The file trace.c has become a catchall for most things tracing. Start making it smaller by breaking out various aspects into their own files. Move the functions associated to the trace_printk operations out of trace.c= and into trace_printk.c. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208032450.828744197@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 431 ------------------------------------ kernel/trace/trace.h | 1 + kernel/trace/trace_printk.c | 431 ++++++++++++++++++++++++++++++++++++ 3 files changed, 432 insertions(+), 431 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 4066c33674e7..5812b830c1fa 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -539,17 +539,6 @@ struct trace_array *printk_trace =3D &global_trace; /* List of trace_arrays interested in the top level trace_marker */ static LIST_HEAD(marker_copies); =20 -static __always_inline bool printk_binsafe(struct trace_array *tr) -{ - /* - * The binary format of traceprintk can cause a crash if used - * by a buffer from another boot. Force the use of the - * non binary version of trace_printk if the trace_printk - * buffer is a boot mapped ring buffer. - */ - return !(tr->flags & TRACE_ARRAY_FL_BOOT); -} - static void update_printk_trace(struct trace_array *tr) { if (printk_trace =3D=3D tr) @@ -1059,108 +1048,6 @@ void tracing_on(void) } EXPORT_SYMBOL_GPL(tracing_on); =20 -int __trace_array_puts(struct trace_array *tr, unsigned long ip, - const char *str, int size) -{ - struct ring_buffer_event *event; - struct trace_buffer *buffer; - struct print_entry *entry; - unsigned int trace_ctx; - int alloc; - - if (!(tr->trace_flags & TRACE_ITER(PRINTK))) - return 0; - - if (unlikely(tracing_selftest_running && - (tr->flags & TRACE_ARRAY_FL_GLOBAL))) - return 0; - - if (unlikely(tracing_disabled)) - return 0; - - alloc =3D sizeof(*entry) + size + 2; /* possible \n added */ - - trace_ctx =3D tracing_gen_ctx(); - buffer =3D tr->array_buffer.buffer; - guard(ring_buffer_nest)(buffer); - event =3D __trace_buffer_lock_reserve(buffer, TRACE_PRINT, alloc, - trace_ctx); - if (!event) - return 0; - - entry =3D ring_buffer_event_data(event); - entry->ip =3D ip; - - memcpy(&entry->buf, str, size); - - /* Add a newline if necessary */ - if (entry->buf[size - 1] !=3D '\n') { - entry->buf[size] =3D '\n'; - entry->buf[size + 1] =3D '\0'; - } else - entry->buf[size] =3D '\0'; - - __buffer_unlock_commit(buffer, event); - ftrace_trace_stack(tr, buffer, trace_ctx, 4, NULL); - return size; -} -EXPORT_SYMBOL_GPL(__trace_array_puts); - -/** - * __trace_puts - write a constant string into the trace buffer. - * @ip: The address of the caller - * @str: The constant string to write - * @size: The size of the string. - */ -int __trace_puts(unsigned long ip, const char *str, int size) -{ - return __trace_array_puts(printk_trace, ip, str, size); -} -EXPORT_SYMBOL_GPL(__trace_puts); - -/** - * __trace_bputs - write the pointer to a constant string into trace buffer - * @ip: The address of the caller - * @str: The constant string to write to the buffer to - */ -int __trace_bputs(unsigned long ip, const char *str) -{ - struct trace_array *tr =3D READ_ONCE(printk_trace); - struct ring_buffer_event *event; - struct trace_buffer *buffer; - struct bputs_entry *entry; - unsigned int trace_ctx; - int size =3D sizeof(struct bputs_entry); - - if (!printk_binsafe(tr)) - return __trace_puts(ip, str, strlen(str)); - - if (!(tr->trace_flags & TRACE_ITER(PRINTK))) - return 0; - - if (unlikely(tracing_selftest_running || tracing_disabled)) - return 0; - - trace_ctx =3D tracing_gen_ctx(); - buffer =3D tr->array_buffer.buffer; - - guard(ring_buffer_nest)(buffer); - event =3D __trace_buffer_lock_reserve(buffer, TRACE_BPUTS, size, - trace_ctx); - if (!event) - return 0; - - entry =3D ring_buffer_event_data(event); - entry->ip =3D ip; - entry->str =3D str; - - __buffer_unlock_commit(buffer, event); - ftrace_trace_stack(tr, buffer, trace_ctx, 4, NULL); - - return 1; -} -EXPORT_SYMBOL_GPL(__trace_bputs); - #ifdef CONFIG_TRACER_SNAPSHOT static void tracing_snapshot_instance_cond(struct trace_array *tr, void *cond_data) @@ -3159,324 +3046,6 @@ void trace_last_func_repeats(struct trace_array *tr, __buffer_unlock_commit(buffer, event); } =20 -/* created for use with alloc_percpu */ -struct trace_buffer_struct { - int nesting; - char buffer[4][TRACE_BUF_SIZE]; -}; - -static struct trace_buffer_struct __percpu *trace_percpu_buffer; - -/* - * This allows for lockless recording. If we're nested too deeply, then - * this returns NULL. - */ -static char *get_trace_buf(void) -{ - struct trace_buffer_struct *buffer =3D this_cpu_ptr(trace_percpu_buffer); - - if (!trace_percpu_buffer || buffer->nesting >=3D 4) - return NULL; - - buffer->nesting++; - - /* Interrupts must see nesting incremented before we use the buffer */ - barrier(); - return &buffer->buffer[buffer->nesting - 1][0]; -} - -static void put_trace_buf(void) -{ - /* Don't let the decrement of nesting leak before this */ - barrier(); - this_cpu_dec(trace_percpu_buffer->nesting); -} - -static int alloc_percpu_trace_buffer(void) -{ - struct trace_buffer_struct __percpu *buffers; - - if (trace_percpu_buffer) - return 0; - - buffers =3D alloc_percpu(struct trace_buffer_struct); - if (MEM_FAIL(!buffers, "Could not allocate percpu trace_printk buffer")) - return -ENOMEM; - - trace_percpu_buffer =3D buffers; - return 0; -} - -static int buffers_allocated; - -void trace_printk_init_buffers(void) -{ - if (buffers_allocated) - return; - - if (alloc_percpu_trace_buffer()) - return; - - /* trace_printk() is for debug use only. Don't use it in production. */ - - pr_warn("\n"); - pr_warn("**********************************************************\n"); - pr_warn("** NOTICE NOTICE NOTICE NOTICE NOTICE NOTICE NOTICE **\n"); - pr_warn("** **\n"); - pr_warn("** trace_printk() being used. Allocating extra memory. **\n"); - pr_warn("** **\n"); - pr_warn("** This means that this is a DEBUG kernel and it is **\n"); - pr_warn("** unsafe for production use. **\n"); - pr_warn("** **\n"); - pr_warn("** If you see this message and you are not debugging **\n"); - pr_warn("** the kernel, report this immediately to your vendor! **\n"); - pr_warn("** **\n"); - pr_warn("** NOTICE NOTICE NOTICE NOTICE NOTICE NOTICE NOTICE **\n"); - pr_warn("**********************************************************\n"); - - /* Expand the buffers to set size */ - if (tracing_update_buffers(NULL) < 0) - pr_err("Failed to expand tracing buffers for trace_printk() calls\n"); - else - buffers_allocated =3D 1; - - /* - * trace_printk_init_buffers() can be called by modules. - * If that happens, then we need to start cmdline recording - * directly here. - */ - if (system_state =3D=3D SYSTEM_RUNNING) - tracing_start_cmdline_record(); -} -EXPORT_SYMBOL_GPL(trace_printk_init_buffers); - -void trace_printk_start_comm(void) -{ - /* Start tracing comms if trace printk is set */ - if (!buffers_allocated) - return; - tracing_start_cmdline_record(); -} - -static void trace_printk_start_stop_comm(int enabled) -{ - if (!buffers_allocated) - return; - - if (enabled) - tracing_start_cmdline_record(); - else - tracing_stop_cmdline_record(); -} - -/** - * trace_vbprintk - write binary msg to tracing buffer - * @ip: The address of the caller - * @fmt: The string format to write to the buffer - * @args: Arguments for @fmt - */ -int trace_vbprintk(unsigned long ip, const char *fmt, va_list args) -{ - struct ring_buffer_event *event; - struct trace_buffer *buffer; - struct trace_array *tr =3D READ_ONCE(printk_trace); - struct bprint_entry *entry; - unsigned int trace_ctx; - char *tbuffer; - int len =3D 0, size; - - if (!printk_binsafe(tr)) - return trace_vprintk(ip, fmt, args); - - if (unlikely(tracing_selftest_running || tracing_disabled)) - return 0; - - /* Don't pollute graph traces with trace_vprintk internals */ - pause_graph_tracing(); - - trace_ctx =3D tracing_gen_ctx(); - guard(preempt_notrace)(); - - tbuffer =3D get_trace_buf(); - if (!tbuffer) { - len =3D 0; - goto out_nobuffer; - } - - len =3D vbin_printf((u32 *)tbuffer, TRACE_BUF_SIZE/sizeof(int), fmt, args= ); - - if (len > TRACE_BUF_SIZE/sizeof(int) || len < 0) - goto out_put; - - size =3D sizeof(*entry) + sizeof(u32) * len; - buffer =3D tr->array_buffer.buffer; - scoped_guard(ring_buffer_nest, buffer) { - event =3D __trace_buffer_lock_reserve(buffer, TRACE_BPRINT, size, - trace_ctx); - if (!event) - goto out_put; - entry =3D ring_buffer_event_data(event); - entry->ip =3D ip; - entry->fmt =3D fmt; - - memcpy(entry->buf, tbuffer, sizeof(u32) * len); - __buffer_unlock_commit(buffer, event); - ftrace_trace_stack(tr, buffer, trace_ctx, 6, NULL); - } -out_put: - put_trace_buf(); - -out_nobuffer: - unpause_graph_tracing(); - - return len; -} -EXPORT_SYMBOL_GPL(trace_vbprintk); - -static __printf(3, 0) -int __trace_array_vprintk(struct trace_buffer *buffer, - unsigned long ip, const char *fmt, va_list args) -{ - struct ring_buffer_event *event; - int len =3D 0, size; - struct print_entry *entry; - unsigned int trace_ctx; - char *tbuffer; - - if (unlikely(tracing_disabled)) - return 0; - - /* Don't pollute graph traces with trace_vprintk internals */ - pause_graph_tracing(); - - trace_ctx =3D tracing_gen_ctx(); - guard(preempt_notrace)(); - - - tbuffer =3D get_trace_buf(); - if (!tbuffer) { - len =3D 0; - goto out_nobuffer; - } - - len =3D vscnprintf(tbuffer, TRACE_BUF_SIZE, fmt, args); - - size =3D sizeof(*entry) + len + 1; - scoped_guard(ring_buffer_nest, buffer) { - event =3D __trace_buffer_lock_reserve(buffer, TRACE_PRINT, size, - trace_ctx); - if (!event) - goto out; - entry =3D ring_buffer_event_data(event); - entry->ip =3D ip; - - memcpy(&entry->buf, tbuffer, len + 1); - __buffer_unlock_commit(buffer, event); - ftrace_trace_stack(printk_trace, buffer, trace_ctx, 6, NULL); - } -out: - put_trace_buf(); - -out_nobuffer: - unpause_graph_tracing(); - - return len; -} - -int trace_array_vprintk(struct trace_array *tr, - unsigned long ip, const char *fmt, va_list args) -{ - if (tracing_selftest_running && (tr->flags & TRACE_ARRAY_FL_GLOBAL)) - return 0; - - return __trace_array_vprintk(tr->array_buffer.buffer, ip, fmt, args); -} - -/** - * trace_array_printk - Print a message to a specific instance - * @tr: The instance trace_array descriptor - * @ip: The instruction pointer that this is called from. - * @fmt: The format to print (printf format) - * - * If a subsystem sets up its own instance, they have the right to - * printk strings into their tracing instance buffer using this - * function. Note, this function will not write into the top level - * buffer (use trace_printk() for that), as writing into the top level - * buffer should only have events that can be individually disabled. - * trace_printk() is only used for debugging a kernel, and should not - * be ever incorporated in normal use. - * - * trace_array_printk() can be used, as it will not add noise to the - * top level tracing buffer. - * - * Note, trace_array_init_printk() must be called on @tr before this - * can be used. - */ -int trace_array_printk(struct trace_array *tr, - unsigned long ip, const char *fmt, ...) -{ - int ret; - va_list ap; - - if (!tr) - return -ENOENT; - - /* This is only allowed for created instances */ - if (tr->flags & TRACE_ARRAY_FL_GLOBAL) - return 0; - - if (!(tr->trace_flags & TRACE_ITER(PRINTK))) - return 0; - - va_start(ap, fmt); - ret =3D trace_array_vprintk(tr, ip, fmt, ap); - va_end(ap); - return ret; -} -EXPORT_SYMBOL_GPL(trace_array_printk); - -/** - * trace_array_init_printk - Initialize buffers for trace_array_printk() - * @tr: The trace array to initialize the buffers for - * - * As trace_array_printk() only writes into instances, they are OK to - * have in the kernel (unlike trace_printk()). This needs to be called - * before trace_array_printk() can be used on a trace_array. - */ -int trace_array_init_printk(struct trace_array *tr) -{ - if (!tr) - return -ENOENT; - - /* This is only allowed for created instances */ - if (tr->flags & TRACE_ARRAY_FL_GLOBAL) - return -EINVAL; - - return alloc_percpu_trace_buffer(); -} -EXPORT_SYMBOL_GPL(trace_array_init_printk); - -int trace_array_printk_buf(struct trace_buffer *buffer, - unsigned long ip, const char *fmt, ...) -{ - int ret; - va_list ap; - - if (!(printk_trace->trace_flags & TRACE_ITER(PRINTK))) - return 0; - - va_start(ap, fmt); - ret =3D __trace_array_vprintk(buffer, ip, fmt, ap); - va_end(ap); - return ret; -} - -int trace_vprintk(unsigned long ip, const char *fmt, va_list args) -{ - return trace_array_vprintk(printk_trace, ip, fmt, args); -} -EXPORT_SYMBOL_GPL(trace_vprintk); - static void trace_iterator_increment(struct trace_iterator *iter) { struct ring_buffer_iter *buf_iter =3D trace_buffer_iter(iter, iter->cpu); diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h index 921e4daa2825..6b0fedf2f532 100644 --- a/kernel/trace/trace.h +++ b/kernel/trace/trace.h @@ -2131,6 +2131,7 @@ extern const char *__stop___tracepoint_str[]; =20 void trace_printk_control(bool enabled); void trace_printk_start_comm(void); +void trace_printk_start_stop_comm(int enabled); int trace_keep_overwrite(struct tracer *tracer, u64 mask, int set); int set_tracer_flag(struct trace_array *tr, u64 mask, int enabled); =20 diff --git a/kernel/trace/trace_printk.c b/kernel/trace/trace_printk.c index 29f6e95439b6..c9cb74a33b3c 100644 --- a/kernel/trace/trace_printk.c +++ b/kernel/trace/trace_printk.c @@ -376,6 +376,437 @@ static const struct file_operations ftrace_formats_fo= ps =3D { .release =3D seq_release, }; =20 +static __always_inline bool printk_binsafe(struct trace_array *tr) +{ + /* + * The binary format of traceprintk can cause a crash if used + * by a buffer from another boot. Force the use of the + * non binary version of trace_printk if the trace_printk + * buffer is a boot mapped ring buffer. + */ + return !(tr->flags & TRACE_ARRAY_FL_BOOT); +} + +int __trace_array_puts(struct trace_array *tr, unsigned long ip, + const char *str, int size) +{ + struct ring_buffer_event *event; + struct trace_buffer *buffer; + struct print_entry *entry; + unsigned int trace_ctx; + int alloc; + + if (!(tr->trace_flags & TRACE_ITER(PRINTK))) + return 0; + + if (unlikely(tracing_selftest_running && + (tr->flags & TRACE_ARRAY_FL_GLOBAL))) + return 0; + + if (unlikely(tracing_disabled)) + return 0; + + alloc =3D sizeof(*entry) + size + 2; /* possible \n added */ + + trace_ctx =3D tracing_gen_ctx(); + buffer =3D tr->array_buffer.buffer; + guard(ring_buffer_nest)(buffer); + event =3D __trace_buffer_lock_reserve(buffer, TRACE_PRINT, alloc, + trace_ctx); + if (!event) + return 0; + + entry =3D ring_buffer_event_data(event); + entry->ip =3D ip; + + memcpy(&entry->buf, str, size); + + /* Add a newline if necessary */ + if (entry->buf[size - 1] !=3D '\n') { + entry->buf[size] =3D '\n'; + entry->buf[size + 1] =3D '\0'; + } else + entry->buf[size] =3D '\0'; + + __buffer_unlock_commit(buffer, event); + ftrace_trace_stack(tr, buffer, trace_ctx, 4, NULL); + return size; +} +EXPORT_SYMBOL_GPL(__trace_array_puts); + +/** + * __trace_puts - write a constant string into the trace buffer. + * @ip: The address of the caller + * @str: The constant string to write + * @size: The size of the string. + */ +int __trace_puts(unsigned long ip, const char *str, int size) +{ + return __trace_array_puts(printk_trace, ip, str, size); +} +EXPORT_SYMBOL_GPL(__trace_puts); + +/** + * __trace_bputs - write the pointer to a constant string into trace buffer + * @ip: The address of the caller + * @str: The constant string to write to the buffer to + */ +int __trace_bputs(unsigned long ip, const char *str) +{ + struct trace_array *tr =3D READ_ONCE(printk_trace); + struct ring_buffer_event *event; + struct trace_buffer *buffer; + struct bputs_entry *entry; + unsigned int trace_ctx; + int size =3D sizeof(struct bputs_entry); + + if (!printk_binsafe(tr)) + return __trace_puts(ip, str, strlen(str)); + + if (!(tr->trace_flags & TRACE_ITER(PRINTK))) + return 0; + + if (unlikely(tracing_selftest_running || tracing_disabled)) + return 0; + + trace_ctx =3D tracing_gen_ctx(); + buffer =3D tr->array_buffer.buffer; + + guard(ring_buffer_nest)(buffer); + event =3D __trace_buffer_lock_reserve(buffer, TRACE_BPUTS, size, + trace_ctx); + if (!event) + return 0; + + entry =3D ring_buffer_event_data(event); + entry->ip =3D ip; + entry->str =3D str; + + __buffer_unlock_commit(buffer, event); + ftrace_trace_stack(tr, buffer, trace_ctx, 4, NULL); + + return 1; +} +EXPORT_SYMBOL_GPL(__trace_bputs); + +/* created for use with alloc_percpu */ +struct trace_buffer_struct { + int nesting; + char buffer[4][TRACE_BUF_SIZE]; +}; + +static struct trace_buffer_struct __percpu *trace_percpu_buffer; + +/* + * This allows for lockless recording. If we're nested too deeply, then + * this returns NULL. + */ +static char *get_trace_buf(void) +{ + struct trace_buffer_struct *buffer =3D this_cpu_ptr(trace_percpu_buffer); + + if (!trace_percpu_buffer || buffer->nesting >=3D 4) + return NULL; + + buffer->nesting++; + + /* Interrupts must see nesting incremented before we use the buffer */ + barrier(); + return &buffer->buffer[buffer->nesting - 1][0]; +} + +static void put_trace_buf(void) +{ + /* Don't let the decrement of nesting leak before this */ + barrier(); + this_cpu_dec(trace_percpu_buffer->nesting); +} + +static int alloc_percpu_trace_buffer(void) +{ + struct trace_buffer_struct __percpu *buffers; + + if (trace_percpu_buffer) + return 0; + + buffers =3D alloc_percpu(struct trace_buffer_struct); + if (MEM_FAIL(!buffers, "Could not allocate percpu trace_printk buffer")) + return -ENOMEM; + + trace_percpu_buffer =3D buffers; + return 0; +} + +static int buffers_allocated; + +void trace_printk_init_buffers(void) +{ + if (buffers_allocated) + return; + + if (alloc_percpu_trace_buffer()) + return; + + /* trace_printk() is for debug use only. Don't use it in production. */ + + pr_warn("\n"); + pr_warn("**********************************************************\n"); + pr_warn("** NOTICE NOTICE NOTICE NOTICE NOTICE NOTICE NOTICE **\n"); + pr_warn("** **\n"); + pr_warn("** trace_printk() being used. Allocating extra memory. **\n"); + pr_warn("** **\n"); + pr_warn("** This means that this is a DEBUG kernel and it is **\n"); + pr_warn("** unsafe for production use. **\n"); + pr_warn("** **\n"); + pr_warn("** If you see this message and you are not debugging **\n"); + pr_warn("** the kernel, report this immediately to your vendor! **\n"); + pr_warn("** **\n"); + pr_warn("** NOTICE NOTICE NOTICE NOTICE NOTICE NOTICE NOTICE **\n"); + pr_warn("**********************************************************\n"); + + /* Expand the buffers to set size */ + if (tracing_update_buffers(NULL) < 0) + pr_err("Failed to expand tracing buffers for trace_printk() calls\n"); + else + buffers_allocated =3D 1; + + /* + * trace_printk_init_buffers() can be called by modules. + * If that happens, then we need to start cmdline recording + * directly here. + */ + if (system_state =3D=3D SYSTEM_RUNNING) + tracing_start_cmdline_record(); +} +EXPORT_SYMBOL_GPL(trace_printk_init_buffers); + +void trace_printk_start_comm(void) +{ + /* Start tracing comms if trace printk is set */ + if (!buffers_allocated) + return; + tracing_start_cmdline_record(); +} + +void trace_printk_start_stop_comm(int enabled) +{ + if (!buffers_allocated) + return; + + if (enabled) + tracing_start_cmdline_record(); + else + tracing_stop_cmdline_record(); +} + +/** + * trace_vbprintk - write binary msg to tracing buffer + * @ip: The address of the caller + * @fmt: The string format to write to the buffer + * @args: Arguments for @fmt + */ +int trace_vbprintk(unsigned long ip, const char *fmt, va_list args) +{ + struct ring_buffer_event *event; + struct trace_buffer *buffer; + struct trace_array *tr =3D READ_ONCE(printk_trace); + struct bprint_entry *entry; + unsigned int trace_ctx; + char *tbuffer; + int len =3D 0, size; + + if (!printk_binsafe(tr)) + return trace_vprintk(ip, fmt, args); + + if (unlikely(tracing_selftest_running || tracing_disabled)) + return 0; + + /* Don't pollute graph traces with trace_vprintk internals */ + pause_graph_tracing(); + + trace_ctx =3D tracing_gen_ctx(); + guard(preempt_notrace)(); + + tbuffer =3D get_trace_buf(); + if (!tbuffer) { + len =3D 0; + goto out_nobuffer; + } + + len =3D vbin_printf((u32 *)tbuffer, TRACE_BUF_SIZE/sizeof(int), fmt, args= ); + + if (len > TRACE_BUF_SIZE/sizeof(int) || len < 0) + goto out_put; + + size =3D sizeof(*entry) + sizeof(u32) * len; + buffer =3D tr->array_buffer.buffer; + scoped_guard(ring_buffer_nest, buffer) { + event =3D __trace_buffer_lock_reserve(buffer, TRACE_BPRINT, size, + trace_ctx); + if (!event) + goto out_put; + entry =3D ring_buffer_event_data(event); + entry->ip =3D ip; + entry->fmt =3D fmt; + + memcpy(entry->buf, tbuffer, sizeof(u32) * len); + __buffer_unlock_commit(buffer, event); + ftrace_trace_stack(tr, buffer, trace_ctx, 6, NULL); + } +out_put: + put_trace_buf(); + +out_nobuffer: + unpause_graph_tracing(); + + return len; +} +EXPORT_SYMBOL_GPL(trace_vbprintk); + +static __printf(3, 0) +int __trace_array_vprintk(struct trace_buffer *buffer, + unsigned long ip, const char *fmt, va_list args) +{ + struct ring_buffer_event *event; + int len =3D 0, size; + struct print_entry *entry; + unsigned int trace_ctx; + char *tbuffer; + + if (unlikely(tracing_disabled)) + return 0; + + /* Don't pollute graph traces with trace_vprintk internals */ + pause_graph_tracing(); + + trace_ctx =3D tracing_gen_ctx(); + guard(preempt_notrace)(); + + + tbuffer =3D get_trace_buf(); + if (!tbuffer) { + len =3D 0; + goto out_nobuffer; + } + + len =3D vscnprintf(tbuffer, TRACE_BUF_SIZE, fmt, args); + + size =3D sizeof(*entry) + len + 1; + scoped_guard(ring_buffer_nest, buffer) { + event =3D __trace_buffer_lock_reserve(buffer, TRACE_PRINT, size, + trace_ctx); + if (!event) + goto out; + entry =3D ring_buffer_event_data(event); + entry->ip =3D ip; + + memcpy(&entry->buf, tbuffer, len + 1); + __buffer_unlock_commit(buffer, event); + ftrace_trace_stack(printk_trace, buffer, trace_ctx, 6, NULL); + } +out: + put_trace_buf(); + +out_nobuffer: + unpause_graph_tracing(); + + return len; +} + +int trace_array_vprintk(struct trace_array *tr, + unsigned long ip, const char *fmt, va_list args) +{ + if (tracing_selftest_running && (tr->flags & TRACE_ARRAY_FL_GLOBAL)) + return 0; + + return __trace_array_vprintk(tr->array_buffer.buffer, ip, fmt, args); +} + +/** + * trace_array_printk - Print a message to a specific instance + * @tr: The instance trace_array descriptor + * @ip: The instruction pointer that this is called from. + * @fmt: The format to print (printf format) + * + * If a subsystem sets up its own instance, they have the right to + * printk strings into their tracing instance buffer using this + * function. Note, this function will not write into the top level + * buffer (use trace_printk() for that), as writing into the top level + * buffer should only have events that can be individually disabled. + * trace_printk() is only used for debugging a kernel, and should not + * be ever incorporated in normal use. + * + * trace_array_printk() can be used, as it will not add noise to the + * top level tracing buffer. + * + * Note, trace_array_init_printk() must be called on @tr before this + * can be used. + */ +int trace_array_printk(struct trace_array *tr, + unsigned long ip, const char *fmt, ...) +{ + int ret; + va_list ap; + + if (!tr) + return -ENOENT; + + /* This is only allowed for created instances */ + if (tr->flags & TRACE_ARRAY_FL_GLOBAL) + return 0; + + if (!(tr->trace_flags & TRACE_ITER(PRINTK))) + return 0; + + va_start(ap, fmt); + ret =3D trace_array_vprintk(tr, ip, fmt, ap); + va_end(ap); + return ret; +} +EXPORT_SYMBOL_GPL(trace_array_printk); + +/** + * trace_array_init_printk - Initialize buffers for trace_array_printk() + * @tr: The trace array to initialize the buffers for + * + * As trace_array_printk() only writes into instances, they are OK to + * have in the kernel (unlike trace_printk()). This needs to be called + * before trace_array_printk() can be used on a trace_array. + */ +int trace_array_init_printk(struct trace_array *tr) +{ + if (!tr) + return -ENOENT; + + /* This is only allowed for created instances */ + if (tr->flags & TRACE_ARRAY_FL_GLOBAL) + return -EINVAL; + + return alloc_percpu_trace_buffer(); +} +EXPORT_SYMBOL_GPL(trace_array_init_printk); + +int trace_array_printk_buf(struct trace_buffer *buffer, + unsigned long ip, const char *fmt, ...) +{ + int ret; + va_list ap; + + if (!(printk_trace->trace_flags & TRACE_ITER(PRINTK))) + return 0; + + va_start(ap, fmt); + ret =3D __trace_array_vprintk(buffer, ip, fmt, ap); + va_end(ap); + return ret; +} + +int trace_vprintk(unsigned long ip, const char *fmt, va_list args) +{ + return trace_array_vprintk(printk_trace, ip, fmt, args); +} +EXPORT_SYMBOL_GPL(trace_vprintk); + static __init int init_trace_printk_function_export(void) { int ret; --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id CE3112E7BCC for ; Mon, 9 Feb 2026 02:20:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603608; cv=none; b=hb10KVZpxe9U6uJeSc2fcZ9UMWXTTzkjzOm+FA5OjwFxazHFXcGNQmNLlTsM7jkglx9TE0ID8HANWhpomUtRXVOPO1j/P/h29bzWpTuVtRLeISlA8LwgSHixgcVtns9F5QWONCwxkZxlqM2MzjKpW8/bg2wauobVKkiYQuzojhw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603608; c=relaxed/simple; bh=Q1kuPYt/DO3cGYWRxtkhf1E30qRaVXz4G9Lrk9qgC90=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=HjGhQ09dBtGgKUdBq+Oqvo1VkEmQ3K2A2p3i2OTM8UBEbOJjYikqD9E3QAZmEyEw9KJSvPpYJY5R7xFwjSl3BZcdUbdITc1Zf0+vlCMtNd+qTjuKLixlzUXpVArCzMqH9HDPpccFJZ4wsv1itJ3TCfjlyyzxfr3oOnu6hTbLcjM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=B7QNk8lO; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="B7QNk8lO" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 57131C16AAE; Mon, 9 Feb 2026 02:20:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603608; bh=Q1kuPYt/DO3cGYWRxtkhf1E30qRaVXz4G9Lrk9qgC90=; h=Date:From:To:Cc:Subject:References:From; b=B7QNk8lOP280624D1jVxt3rQM7xVv/kL/9kmN75EETctbBDxiCfuCX9czbM+CcWoO gHstoTWosH8osd3JJS0mUHNbqcOSxIq2BTSoaJIIJyfp9J9KltdiOS0Qv5hl4FCB/i HGIi1Uq0NRepWCOuzArtJGzmxzYUaOangcMOeMSuJoFlPX3nRsmdAgEeGKlfQdk8x2 jGFEPzYGQdhLLDw1S3Wzla1U3uckiXjeJp0pgMZhQ2XdFqjBJ4XwjK927FXUZiQfZv /VS6gR2BRd81BCHO+HKQIjNXBRHmKLxWWxWOOBxHGKm/Att4qjMWT5dCa++GPnNIIl QZwLEvyB78Ozg== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtS-0000000Aa8N-482l; Sun, 08 Feb 2026 21:20:54 -0500 Message-ID: <20260209022054.840151863@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:43 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 17/20] tracing: Move pid filtering into trace_pid.c References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt The trace.c file was a dumping ground for most tracing code. Start organizing it better by moving various functions out into their own files. Move the PID filtering functions from trace.c into its own trace_pid.c file. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208032450.998330662@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/Makefile | 1 + kernel/trace/trace.c | 242 -------------------------------------- kernel/trace/trace_pid.c | 246 +++++++++++++++++++++++++++++++++++++++ 3 files changed, 247 insertions(+), 242 deletions(-) create mode 100644 kernel/trace/trace_pid.c diff --git a/kernel/trace/Makefile b/kernel/trace/Makefile index fc5dcc888e13..04096c21d06b 100644 --- a/kernel/trace/Makefile +++ b/kernel/trace/Makefile @@ -68,6 +68,7 @@ obj-$(CONFIG_TRACING) +=3D trace_output.o obj-$(CONFIG_TRACING) +=3D trace_seq.o obj-$(CONFIG_TRACING) +=3D trace_stat.o obj-$(CONFIG_TRACING) +=3D trace_printk.o +obj-$(CONFIG_TRACING) +=3D trace_pid.o obj-$(CONFIG_TRACING) +=3D pid_list.o obj-$(CONFIG_TRACING_MAP) +=3D tracing_map.o obj-$(CONFIG_PREEMPTIRQ_DELAY_TEST) +=3D preemptirq_delay_test.o diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 5812b830c1fa..551a452befa0 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -637,248 +637,6 @@ int tracing_check_open_get_tr(struct trace_array *tr) return 0; } =20 -/** - * trace_find_filtered_pid - check if a pid exists in a filtered_pid list - * @filtered_pids: The list of pids to check - * @search_pid: The PID to find in @filtered_pids - * - * Returns true if @search_pid is found in @filtered_pids, and false other= wise. - */ -bool -trace_find_filtered_pid(struct trace_pid_list *filtered_pids, pid_t search= _pid) -{ - return trace_pid_list_is_set(filtered_pids, search_pid); -} - -/** - * trace_ignore_this_task - should a task be ignored for tracing - * @filtered_pids: The list of pids to check - * @filtered_no_pids: The list of pids not to be traced - * @task: The task that should be ignored if not filtered - * - * Checks if @task should be traced or not from @filtered_pids. - * Returns true if @task should *NOT* be traced. - * Returns false if @task should be traced. - */ -bool -trace_ignore_this_task(struct trace_pid_list *filtered_pids, - struct trace_pid_list *filtered_no_pids, - struct task_struct *task) -{ - /* - * If filtered_no_pids is not empty, and the task's pid is listed - * in filtered_no_pids, then return true. - * Otherwise, if filtered_pids is empty, that means we can - * trace all tasks. If it has content, then only trace pids - * within filtered_pids. - */ - - return (filtered_pids && - !trace_find_filtered_pid(filtered_pids, task->pid)) || - (filtered_no_pids && - trace_find_filtered_pid(filtered_no_pids, task->pid)); -} - -/** - * trace_filter_add_remove_task - Add or remove a task from a pid_list - * @pid_list: The list to modify - * @self: The current task for fork or NULL for exit - * @task: The task to add or remove - * - * If adding a task, if @self is defined, the task is only added if @self - * is also included in @pid_list. This happens on fork and tasks should - * only be added when the parent is listed. If @self is NULL, then the - * @task pid will be removed from the list, which would happen on exit - * of a task. - */ -void trace_filter_add_remove_task(struct trace_pid_list *pid_list, - struct task_struct *self, - struct task_struct *task) -{ - if (!pid_list) - return; - - /* For forks, we only add if the forking task is listed */ - if (self) { - if (!trace_find_filtered_pid(pid_list, self->pid)) - return; - } - - /* "self" is set for forks, and NULL for exits */ - if (self) - trace_pid_list_set(pid_list, task->pid); - else - trace_pid_list_clear(pid_list, task->pid); -} - -/** - * trace_pid_next - Used for seq_file to get to the next pid of a pid_list - * @pid_list: The pid list to show - * @v: The last pid that was shown (+1 the actual pid to let zero be displ= ayed) - * @pos: The position of the file - * - * This is used by the seq_file "next" operation to iterate the pids - * listed in a trace_pid_list structure. - * - * Returns the pid+1 as we want to display pid of zero, but NULL would - * stop the iteration. - */ -void *trace_pid_next(struct trace_pid_list *pid_list, void *v, loff_t *pos) -{ - long pid =3D (unsigned long)v; - unsigned int next; - - (*pos)++; - - /* pid already is +1 of the actual previous bit */ - if (trace_pid_list_next(pid_list, pid, &next) < 0) - return NULL; - - pid =3D next; - - /* Return pid + 1 to allow zero to be represented */ - return (void *)(pid + 1); -} - -/** - * trace_pid_start - Used for seq_file to start reading pid lists - * @pid_list: The pid list to show - * @pos: The position of the file - * - * This is used by seq_file "start" operation to start the iteration - * of listing pids. - * - * Returns the pid+1 as we want to display pid of zero, but NULL would - * stop the iteration. - */ -void *trace_pid_start(struct trace_pid_list *pid_list, loff_t *pos) -{ - unsigned long pid; - unsigned int first; - loff_t l =3D 0; - - if (trace_pid_list_first(pid_list, &first) < 0) - return NULL; - - pid =3D first; - - /* Return pid + 1 so that zero can be the exit value */ - for (pid++; pid && l < *pos; - pid =3D (unsigned long)trace_pid_next(pid_list, (void *)pid, &l)) - ; - return (void *)pid; -} - -/** - * trace_pid_show - show the current pid in seq_file processing - * @m: The seq_file structure to write into - * @v: A void pointer of the pid (+1) value to display - * - * Can be directly used by seq_file operations to display the current - * pid value. - */ -int trace_pid_show(struct seq_file *m, void *v) -{ - unsigned long pid =3D (unsigned long)v - 1; - - seq_printf(m, "%lu\n", pid); - return 0; -} - -/* 128 should be much more than enough */ -#define PID_BUF_SIZE 127 - -int trace_pid_write(struct trace_pid_list *filtered_pids, - struct trace_pid_list **new_pid_list, - const char __user *ubuf, size_t cnt) -{ - struct trace_pid_list *pid_list; - struct trace_parser parser; - unsigned long val; - int nr_pids =3D 0; - ssize_t read =3D 0; - ssize_t ret; - loff_t pos; - pid_t pid; - - if (trace_parser_get_init(&parser, PID_BUF_SIZE + 1)) - return -ENOMEM; - - /* - * Always recreate a new array. The write is an all or nothing - * operation. Always create a new array when adding new pids by - * the user. If the operation fails, then the current list is - * not modified. - */ - pid_list =3D trace_pid_list_alloc(); - if (!pid_list) { - trace_parser_put(&parser); - return -ENOMEM; - } - - if (filtered_pids) { - /* copy the current bits to the new max */ - ret =3D trace_pid_list_first(filtered_pids, &pid); - while (!ret) { - ret =3D trace_pid_list_set(pid_list, pid); - if (ret < 0) - goto out; - - ret =3D trace_pid_list_next(filtered_pids, pid + 1, &pid); - nr_pids++; - } - } - - ret =3D 0; - while (cnt > 0) { - - pos =3D 0; - - ret =3D trace_get_user(&parser, ubuf, cnt, &pos); - if (ret < 0) - break; - - read +=3D ret; - ubuf +=3D ret; - cnt -=3D ret; - - if (!trace_parser_loaded(&parser)) - break; - - ret =3D -EINVAL; - if (kstrtoul(parser.buffer, 0, &val)) - break; - - pid =3D (pid_t)val; - - if (trace_pid_list_set(pid_list, pid) < 0) { - ret =3D -1; - break; - } - nr_pids++; - - trace_parser_clear(&parser); - ret =3D 0; - } - out: - trace_parser_put(&parser); - - if (ret < 0) { - trace_pid_list_free(pid_list); - return ret; - } - - if (!nr_pids) { - /* Cleared the list of pids */ - trace_pid_list_free(pid_list); - pid_list =3D NULL; - } - - *new_pid_list =3D pid_list; - - return read; -} - static u64 buffer_ftrace_now(struct array_buffer *buf, int cpu) { u64 ts; diff --git a/kernel/trace/trace_pid.c b/kernel/trace/trace_pid.c new file mode 100644 index 000000000000..7127c8de4174 --- /dev/null +++ b/kernel/trace/trace_pid.c @@ -0,0 +1,246 @@ +// SPDX-License-Identifier: GPL-2.0 + +#include "trace.h" + +/** + * trace_find_filtered_pid - check if a pid exists in a filtered_pid list + * @filtered_pids: The list of pids to check + * @search_pid: The PID to find in @filtered_pids + * + * Returns true if @search_pid is found in @filtered_pids, and false other= wise. + */ +bool +trace_find_filtered_pid(struct trace_pid_list *filtered_pids, pid_t search= _pid) +{ + return trace_pid_list_is_set(filtered_pids, search_pid); +} + +/** + * trace_ignore_this_task - should a task be ignored for tracing + * @filtered_pids: The list of pids to check + * @filtered_no_pids: The list of pids not to be traced + * @task: The task that should be ignored if not filtered + * + * Checks if @task should be traced or not from @filtered_pids. + * Returns true if @task should *NOT* be traced. + * Returns false if @task should be traced. + */ +bool +trace_ignore_this_task(struct trace_pid_list *filtered_pids, + struct trace_pid_list *filtered_no_pids, + struct task_struct *task) +{ + /* + * If filtered_no_pids is not empty, and the task's pid is listed + * in filtered_no_pids, then return true. + * Otherwise, if filtered_pids is empty, that means we can + * trace all tasks. If it has content, then only trace pids + * within filtered_pids. + */ + + return (filtered_pids && + !trace_find_filtered_pid(filtered_pids, task->pid)) || + (filtered_no_pids && + trace_find_filtered_pid(filtered_no_pids, task->pid)); +} + +/** + * trace_filter_add_remove_task - Add or remove a task from a pid_list + * @pid_list: The list to modify + * @self: The current task for fork or NULL for exit + * @task: The task to add or remove + * + * If adding a task, if @self is defined, the task is only added if @self + * is also included in @pid_list. This happens on fork and tasks should + * only be added when the parent is listed. If @self is NULL, then the + * @task pid will be removed from the list, which would happen on exit + * of a task. + */ +void trace_filter_add_remove_task(struct trace_pid_list *pid_list, + struct task_struct *self, + struct task_struct *task) +{ + if (!pid_list) + return; + + /* For forks, we only add if the forking task is listed */ + if (self) { + if (!trace_find_filtered_pid(pid_list, self->pid)) + return; + } + + /* "self" is set for forks, and NULL for exits */ + if (self) + trace_pid_list_set(pid_list, task->pid); + else + trace_pid_list_clear(pid_list, task->pid); +} + +/** + * trace_pid_next - Used for seq_file to get to the next pid of a pid_list + * @pid_list: The pid list to show + * @v: The last pid that was shown (+1 the actual pid to let zero be displ= ayed) + * @pos: The position of the file + * + * This is used by the seq_file "next" operation to iterate the pids + * listed in a trace_pid_list structure. + * + * Returns the pid+1 as we want to display pid of zero, but NULL would + * stop the iteration. + */ +void *trace_pid_next(struct trace_pid_list *pid_list, void *v, loff_t *pos) +{ + long pid =3D (unsigned long)v; + unsigned int next; + + (*pos)++; + + /* pid already is +1 of the actual previous bit */ + if (trace_pid_list_next(pid_list, pid, &next) < 0) + return NULL; + + pid =3D next; + + /* Return pid + 1 to allow zero to be represented */ + return (void *)(pid + 1); +} + +/** + * trace_pid_start - Used for seq_file to start reading pid lists + * @pid_list: The pid list to show + * @pos: The position of the file + * + * This is used by seq_file "start" operation to start the iteration + * of listing pids. + * + * Returns the pid+1 as we want to display pid of zero, but NULL would + * stop the iteration. + */ +void *trace_pid_start(struct trace_pid_list *pid_list, loff_t *pos) +{ + unsigned long pid; + unsigned int first; + loff_t l =3D 0; + + if (trace_pid_list_first(pid_list, &first) < 0) + return NULL; + + pid =3D first; + + /* Return pid + 1 so that zero can be the exit value */ + for (pid++; pid && l < *pos; + pid =3D (unsigned long)trace_pid_next(pid_list, (void *)pid, &l)) + ; + return (void *)pid; +} + +/** + * trace_pid_show - show the current pid in seq_file processing + * @m: The seq_file structure to write into + * @v: A void pointer of the pid (+1) value to display + * + * Can be directly used by seq_file operations to display the current + * pid value. + */ +int trace_pid_show(struct seq_file *m, void *v) +{ + unsigned long pid =3D (unsigned long)v - 1; + + seq_printf(m, "%lu\n", pid); + return 0; +} + +/* 128 should be much more than enough */ +#define PID_BUF_SIZE 127 + +int trace_pid_write(struct trace_pid_list *filtered_pids, + struct trace_pid_list **new_pid_list, + const char __user *ubuf, size_t cnt) +{ + struct trace_pid_list *pid_list; + struct trace_parser parser; + unsigned long val; + int nr_pids =3D 0; + ssize_t read =3D 0; + ssize_t ret; + loff_t pos; + pid_t pid; + + if (trace_parser_get_init(&parser, PID_BUF_SIZE + 1)) + return -ENOMEM; + + /* + * Always recreate a new array. The write is an all or nothing + * operation. Always create a new array when adding new pids by + * the user. If the operation fails, then the current list is + * not modified. + */ + pid_list =3D trace_pid_list_alloc(); + if (!pid_list) { + trace_parser_put(&parser); + return -ENOMEM; + } + + if (filtered_pids) { + /* copy the current bits to the new max */ + ret =3D trace_pid_list_first(filtered_pids, &pid); + while (!ret) { + ret =3D trace_pid_list_set(pid_list, pid); + if (ret < 0) + goto out; + + ret =3D trace_pid_list_next(filtered_pids, pid + 1, &pid); + nr_pids++; + } + } + + ret =3D 0; + while (cnt > 0) { + + pos =3D 0; + + ret =3D trace_get_user(&parser, ubuf, cnt, &pos); + if (ret < 0) + break; + + read +=3D ret; + ubuf +=3D ret; + cnt -=3D ret; + + if (!trace_parser_loaded(&parser)) + break; + + ret =3D -EINVAL; + if (kstrtoul(parser.buffer, 0, &val)) + break; + + pid =3D (pid_t)val; + + if (trace_pid_list_set(pid_list, pid) < 0) { + ret =3D -1; + break; + } + nr_pids++; + + trace_parser_clear(&parser); + ret =3D 0; + } + out: + trace_parser_put(&parser); + + if (ret < 0) { + trace_pid_list_free(pid_list); + return ret; + } + + if (!nr_pids) { + /* Cleared the list of pids */ + trace_pid_list_free(pid_list); + pid_list =3D NULL; + } + + *new_pid_list =3D pid_list; + + return read; +} + --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D1CC72E88BD for ; Mon, 9 Feb 2026 02:20:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603608; cv=none; b=RGbhFa6cNDiXJEB488as3QgrWbNFbFD451OgT9ix7nZmcXID6s8NasOkB8YLjnDSSH/Dwuk0OjqtBKSiO77emeFi9gLlFqUfNZ8xyf5REgjWW5+ejSJ5bC00Zlm4VUgDqmFHoGrY9heKwSYQjuFmaK1lZus/m4CHdGQiKEUHWEU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603608; c=relaxed/simple; bh=DA6bKqd9VZ0QGn0YWb5aLO0EWPZmn/QMNEAC4zw+pRg=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=k4MssKEupHP+xq6YPU5FaqJBIuCdLIegd16tNebzrohy5kliytyaoF4otv4uxnqAzRhjozMy0lPvA+1IgKrAcYasg1pT4ROGZU9XzT+29dBCdOix2Vy8OgareAlz3Awz/ErIN5iyEm2/GyOHUjH4aXglLBJEZQd9dqCxjDTpTBk= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=jx6FYzWV; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="jx6FYzWV" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 93D25C19425; Mon, 9 Feb 2026 02:20:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603608; bh=DA6bKqd9VZ0QGn0YWb5aLO0EWPZmn/QMNEAC4zw+pRg=; h=Date:From:To:Cc:Subject:References:From; b=jx6FYzWVyyTtXSX51R4mBTsovATBOyaBXUlkXpmflN5/6v6T8ude4SXcjgcya5dgl 3NHHGraliVyinP+SZURefcbGbTeNpCswQXu6w9IWD927Zk8JZU1h5XdHl8puFk8Tff Qzn6ol8aKiB/9wr307anQPfc2swKPlE6yZWY2E9vVZQp6aHAqwd0H9VgzcZ25+Q4sT SdNdaj0xVIgwLzLkHgbrm/LS7jXu/LFq0OKr1U2U/mHu7eMO+uXecPVcAsIpmdv07U +7+pJ8lrOGXwFKpKyXNZ9iVrwTwuHsFiBTY/LpZuPZ6Aq5mkuqNHWdQEgUoblWeNyB hiwjzHEG4hMFQ== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtT-0000000Aa8r-0eMb; Sun, 08 Feb 2026 21:20:55 -0500 Message-ID: <20260209022055.006763362@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:44 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 18/20] tracing: Rename trace_array field max_buffer to snapshot_buffer References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt When tracing was first added, there were latency tracers that would take a snapshot of the current trace when a new max latency was hit. This snapshot buffer was called "max_buffer". Since then, a snapshot feature was added that allowed user space or event triggers to trigger a snapshot of the current buffer using the same max_buffer of the trace_array. As this snapshot buffer now has a more generic use case, calling it "max_buffer" is confusing. Rename it to snapshot_buffer. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208183856.428446729@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 72 +++++++++++++++++------------------ kernel/trace/trace.h | 13 ++++--- kernel/trace/trace_selftest.c | 10 ++--- 3 files changed, 48 insertions(+), 47 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 551a452befa0..98524d0656bf 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -934,12 +934,12 @@ int tracing_alloc_snapshot_instance(struct trace_arra= y *tr) =20 /* Make the snapshot buffer have the same order as main buffer */ order =3D ring_buffer_subbuf_order_get(tr->array_buffer.buffer); - ret =3D ring_buffer_subbuf_order_set(tr->max_buffer.buffer, order); + ret =3D ring_buffer_subbuf_order_set(tr->snapshot_buffer.buffer, order); if (ret < 0) return ret; =20 /* allocate spare buffer */ - ret =3D resize_buffer_duplicate_size(&tr->max_buffer, + ret =3D resize_buffer_duplicate_size(&tr->snapshot_buffer, &tr->array_buffer, RING_BUFFER_ALL_CPUS); if (ret < 0) return ret; @@ -957,10 +957,10 @@ static void free_snapshot(struct trace_array *tr) * The max_tr ring buffer has some state (e.g. ring->clock) and * we want preserve it. */ - ring_buffer_subbuf_order_set(tr->max_buffer.buffer, 0); - ring_buffer_resize(tr->max_buffer.buffer, 1, RING_BUFFER_ALL_CPUS); - set_buffer_entries(&tr->max_buffer, 1); - tracing_reset_online_cpus(&tr->max_buffer); + ring_buffer_subbuf_order_set(tr->snapshot_buffer.buffer, 0); + ring_buffer_resize(tr->snapshot_buffer.buffer, 1, RING_BUFFER_ALL_CPUS); + set_buffer_entries(&tr->snapshot_buffer, 1); + tracing_reset_online_cpus(&tr->snapshot_buffer); tr->allocated_snapshot =3D false; } =20 @@ -1556,7 +1556,7 @@ static void __update_max_tr(struct trace_array *tr, struct task_struct *tsk, int cpu) { struct array_buffer *trace_buf =3D &tr->array_buffer; - struct array_buffer *max_buf =3D &tr->max_buffer; + struct array_buffer *max_buf =3D &tr->snapshot_buffer; struct trace_array_cpu *data =3D per_cpu_ptr(trace_buf->data, cpu); struct trace_array_cpu *max_data =3D per_cpu_ptr(max_buf->data, cpu); =20 @@ -1616,9 +1616,9 @@ update_max_tr(struct trace_array *tr, struct task_str= uct *tsk, int cpu, =20 /* Inherit the recordable setting from array_buffer */ if (ring_buffer_record_is_set_on(tr->array_buffer.buffer)) - ring_buffer_record_on(tr->max_buffer.buffer); + ring_buffer_record_on(tr->snapshot_buffer.buffer); else - ring_buffer_record_off(tr->max_buffer.buffer); + ring_buffer_record_off(tr->snapshot_buffer.buffer); =20 #ifdef CONFIG_TRACER_SNAPSHOT if (tr->cond_snapshot && !tr->cond_snapshot->update(tr, cond_data)) { @@ -1626,7 +1626,7 @@ update_max_tr(struct trace_array *tr, struct task_str= uct *tsk, int cpu, return; } #endif - swap(tr->array_buffer.buffer, tr->max_buffer.buffer); + swap(tr->array_buffer.buffer, tr->snapshot_buffer.buffer); =20 __update_max_tr(tr, tsk, cpu); =20 @@ -1661,7 +1661,7 @@ update_max_tr_single(struct trace_array *tr, struct t= ask_struct *tsk, int cpu) =20 arch_spin_lock(&tr->max_lock); =20 - ret =3D ring_buffer_swap_cpu(tr->max_buffer.buffer, tr->array_buffer.buff= er, cpu); + ret =3D ring_buffer_swap_cpu(tr->snapshot_buffer.buffer, tr->array_buffer= .buffer, cpu); =20 if (ret =3D=3D -EBUSY) { /* @@ -1671,7 +1671,7 @@ update_max_tr_single(struct trace_array *tr, struct t= ask_struct *tsk, int cpu) * and flag that it failed. * Another reason is resize is in progress. */ - trace_array_printk_buf(tr->max_buffer.buffer, _THIS_IP_, + trace_array_printk_buf(tr->snapshot_buffer.buffer, _THIS_IP_, "Failed to swap buffers due to commit or resize in progress\n"); } =20 @@ -1722,7 +1722,7 @@ static int wait_on_pipe(struct trace_iterator *iter, = int full) * to happen, this would now be the main buffer. */ if (iter->snapshot) - iter->array_buffer =3D &iter->tr->max_buffer; + iter->array_buffer =3D &iter->tr->snapshot_buffer; #endif return ret; } @@ -1790,7 +1790,7 @@ static int run_tracer_selftest(struct tracer *type) if (type->use_max_tr) { /* If we expanded the buffers, make sure the max is expanded too */ if (tr->ring_buffer_expanded) - ring_buffer_resize(tr->max_buffer.buffer, trace_buf_size, + ring_buffer_resize(tr->snapshot_buffer.buffer, trace_buf_size, RING_BUFFER_ALL_CPUS); tr->allocated_snapshot =3D true; } @@ -1817,7 +1817,7 @@ static int run_tracer_selftest(struct tracer *type) =20 /* Shrink the max buffer again */ if (tr->ring_buffer_expanded) - ring_buffer_resize(tr->max_buffer.buffer, 1, + ring_buffer_resize(tr->snapshot_buffer.buffer, 1, RING_BUFFER_ALL_CPUS); } #endif @@ -2060,7 +2060,7 @@ void tracing_reset_all_online_cpus_unlocked(void) tr->clear_trace =3D false; tracing_reset_online_cpus(&tr->array_buffer); #ifdef CONFIG_TRACER_MAX_TRACE - tracing_reset_online_cpus(&tr->max_buffer); + tracing_reset_online_cpus(&tr->snapshot_buffer); #endif } } @@ -2100,7 +2100,7 @@ static void tracing_start_tr(struct trace_array *tr) ring_buffer_record_enable(buffer); =20 #ifdef CONFIG_TRACER_MAX_TRACE - buffer =3D tr->max_buffer.buffer; + buffer =3D tr->snapshot_buffer.buffer; if (buffer) ring_buffer_record_enable(buffer); #endif @@ -2136,7 +2136,7 @@ static void tracing_stop_tr(struct trace_array *tr) ring_buffer_record_disable(buffer); =20 #ifdef CONFIG_TRACER_MAX_TRACE - buffer =3D tr->max_buffer.buffer; + buffer =3D tr->snapshot_buffer.buffer; if (buffer) ring_buffer_record_disable(buffer); #endif @@ -3943,7 +3943,7 @@ __tracing_open(struct inode *inode, struct file *file= , bool snapshot) #ifdef CONFIG_TRACER_MAX_TRACE /* Currently only the top directory has a snapshot */ if (tr->current_trace->print_max || snapshot) - iter->array_buffer =3D &tr->max_buffer; + iter->array_buffer =3D &tr->snapshot_buffer; else #endif iter->array_buffer =3D &tr->array_buffer; @@ -4146,7 +4146,7 @@ static int tracing_open(struct inode *inode, struct f= ile *file) =20 #ifdef CONFIG_TRACER_MAX_TRACE if (tr->current_trace->print_max) - trace_buf =3D &tr->max_buffer; + trace_buf =3D &tr->snapshot_buffer; #endif =20 if (cpu =3D=3D RING_BUFFER_ALL_CPUS) @@ -4359,14 +4359,14 @@ int tracing_set_cpumask(struct trace_array *tr, !cpumask_test_cpu(cpu, tracing_cpumask_new)) { ring_buffer_record_disable_cpu(tr->array_buffer.buffer, cpu); #ifdef CONFIG_TRACER_MAX_TRACE - ring_buffer_record_disable_cpu(tr->max_buffer.buffer, cpu); + ring_buffer_record_disable_cpu(tr->snapshot_buffer.buffer, cpu); #endif } if (!cpumask_test_cpu(cpu, tr->tracing_cpumask) && cpumask_test_cpu(cpu, tracing_cpumask_new)) { ring_buffer_record_enable_cpu(tr->array_buffer.buffer, cpu); #ifdef CONFIG_TRACER_MAX_TRACE - ring_buffer_record_enable_cpu(tr->max_buffer.buffer, cpu); + ring_buffer_record_enable_cpu(tr->snapshot_buffer.buffer, cpu); #endif } } @@ -4576,7 +4576,7 @@ int set_tracer_flag(struct trace_array *tr, u64 mask,= int enabled) case TRACE_ITER(OVERWRITE): ring_buffer_change_overwrite(tr->array_buffer.buffer, enabled); #ifdef CONFIG_TRACER_MAX_TRACE - ring_buffer_change_overwrite(tr->max_buffer.buffer, enabled); + ring_buffer_change_overwrite(tr->snapshot_buffer.buffer, enabled); #endif break; =20 @@ -5294,7 +5294,7 @@ static int __tracing_resize_ring_buffer(struct trace_= array *tr, if (!tr->allocated_snapshot) goto out; =20 - ret =3D ring_buffer_resize(tr->max_buffer.buffer, size, cpu); + ret =3D ring_buffer_resize(tr->snapshot_buffer.buffer, size, cpu); if (ret < 0) { int r =3D resize_buffer_duplicate_size(&tr->array_buffer, &tr->array_buffer, cpu); @@ -5319,7 +5319,7 @@ static int __tracing_resize_ring_buffer(struct trace_= array *tr, goto out_start; } =20 - update_buffer_entries(&tr->max_buffer, cpu); + update_buffer_entries(&tr->snapshot_buffer, cpu); =20 out: #endif /* CONFIG_TRACER_MAX_TRACE */ @@ -7036,9 +7036,9 @@ int tracing_set_clock(struct trace_array *tr, const c= har *clockstr) tracing_reset_online_cpus(&tr->array_buffer); =20 #ifdef CONFIG_TRACER_MAX_TRACE - if (tr->max_buffer.buffer) - ring_buffer_set_clock(tr->max_buffer.buffer, trace_clocks[i].func); - tracing_reset_online_cpus(&tr->max_buffer); + if (tr->snapshot_buffer.buffer) + ring_buffer_set_clock(tr->snapshot_buffer.buffer, trace_clocks[i].func); + tracing_reset_online_cpus(&tr->snapshot_buffer); #endif =20 if (tr->scratch && !(tr->flags & TRACE_ARRAY_FL_LAST_BOOT)) { @@ -7170,7 +7170,7 @@ static int tracing_snapshot_open(struct inode *inode,= struct file *file) ret =3D 0; =20 iter->tr =3D tr; - iter->array_buffer =3D &tr->max_buffer; + iter->array_buffer =3D &tr->snapshot_buffer; iter->cpu_file =3D tracing_get_cpu(inode); m->private =3D iter; file->private_data =3D m; @@ -7233,7 +7233,7 @@ tracing_snapshot_write(struct file *filp, const char = __user *ubuf, size_t cnt, return -EINVAL; #endif if (tr->allocated_snapshot) - ret =3D resize_buffer_duplicate_size(&tr->max_buffer, + ret =3D resize_buffer_duplicate_size(&tr->snapshot_buffer, &tr->array_buffer, iter->cpu_file); =20 ret =3D tracing_arm_snapshot_locked(tr); @@ -7254,9 +7254,9 @@ tracing_snapshot_write(struct file *filp, const char = __user *ubuf, size_t cnt, default: if (tr->allocated_snapshot) { if (iter->cpu_file =3D=3D RING_BUFFER_ALL_CPUS) - tracing_reset_online_cpus(&tr->max_buffer); + tracing_reset_online_cpus(&tr->snapshot_buffer); else - tracing_reset_cpu(&tr->max_buffer, iter->cpu_file); + tracing_reset_cpu(&tr->snapshot_buffer, iter->cpu_file); } break; } @@ -7312,7 +7312,7 @@ static int snapshot_raw_open(struct inode *inode, str= uct file *filp) } =20 info->iter.snapshot =3D true; - info->iter.array_buffer =3D &info->iter.tr->max_buffer; + info->iter.array_buffer =3D &info->iter.tr->snapshot_buffer; =20 return ret; } @@ -9195,7 +9195,7 @@ buffer_subbuf_size_write(struct file *filp, const cha= r __user *ubuf, if (!tr->allocated_snapshot) goto out_max; =20 - ret =3D ring_buffer_subbuf_order_set(tr->max_buffer.buffer, order); + ret =3D ring_buffer_subbuf_order_set(tr->snapshot_buffer.buffer, order); if (ret) { /* Put back the old order */ cnt =3D ring_buffer_subbuf_order_set(tr->array_buffer.buffer, old_order); @@ -9416,7 +9416,7 @@ static int allocate_trace_buffers(struct trace_array = *tr, int size) if (tr->range_addr_start) return 0; =20 - ret =3D allocate_trace_buffer(tr, &tr->max_buffer, + ret =3D allocate_trace_buffer(tr, &tr->snapshot_buffer, allocate_snapshot ? size : 1); if (MEM_FAIL(ret, "Failed to allocate trace buffer\n")) { free_trace_buffer(&tr->array_buffer); @@ -9439,7 +9439,7 @@ static void free_trace_buffers(struct trace_array *tr) kfree(tr->module_delta); =20 #ifdef CONFIG_TRACER_MAX_TRACE - free_trace_buffer(&tr->max_buffer); + free_trace_buffer(&tr->snapshot_buffer); #endif } =20 diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h index 6b0fedf2f532..b50383aa8e50 100644 --- a/kernel/trace/trace.h +++ b/kernel/trace/trace.h @@ -331,17 +331,18 @@ struct trace_array { struct array_buffer array_buffer; #ifdef CONFIG_TRACER_MAX_TRACE /* - * The max_buffer is used to snapshot the trace when a maximum + * The snapshot_buffer is used to snapshot the trace when a maximum * latency is reached, or when the user initiates a snapshot. * Some tracers will use this to store a maximum trace while * it continues examining live traces. * - * The buffers for the max_buffer are set up the same as the array_buffer - * When a snapshot is taken, the buffer of the max_buffer is swapped - * with the buffer of the array_buffer and the buffers are reset for - * the array_buffer so the tracing can continue. + * The buffers for the snapshot_buffer are set up the same as the + * array_buffer. When a snapshot is taken, the buffer of the + * snapshot_buffer is swapped with the buffer of the array_buffer + * and the buffers are reset for the array_buffer so the tracing can + * continue. */ - struct array_buffer max_buffer; + struct array_buffer snapshot_buffer; bool allocated_snapshot; spinlock_t snapshot_trigger_lock; unsigned int snapshot; diff --git a/kernel/trace/trace_selftest.c b/kernel/trace/trace_selftest.c index d88c44f1dfa5..be53fe6fee6a 100644 --- a/kernel/trace/trace_selftest.c +++ b/kernel/trace/trace_selftest.c @@ -1225,7 +1225,7 @@ trace_selftest_startup_irqsoff(struct tracer *trace, = struct trace_array *tr) /* check both trace buffers */ ret =3D trace_test_buffer(&tr->array_buffer, NULL); if (!ret) - ret =3D trace_test_buffer(&tr->max_buffer, &count); + ret =3D trace_test_buffer(&tr->snapshot_buffer, &count); trace->reset(tr); tracing_start(); =20 @@ -1287,7 +1287,7 @@ trace_selftest_startup_preemptoff(struct tracer *trac= e, struct trace_array *tr) /* check both trace buffers */ ret =3D trace_test_buffer(&tr->array_buffer, NULL); if (!ret) - ret =3D trace_test_buffer(&tr->max_buffer, &count); + ret =3D trace_test_buffer(&tr->snapshot_buffer, &count); trace->reset(tr); tracing_start(); =20 @@ -1355,7 +1355,7 @@ trace_selftest_startup_preemptirqsoff(struct tracer *= trace, struct trace_array * if (ret) goto out; =20 - ret =3D trace_test_buffer(&tr->max_buffer, &count); + ret =3D trace_test_buffer(&tr->snapshot_buffer, &count); if (ret) goto out; =20 @@ -1385,7 +1385,7 @@ trace_selftest_startup_preemptirqsoff(struct tracer *= trace, struct trace_array * if (ret) goto out; =20 - ret =3D trace_test_buffer(&tr->max_buffer, &count); + ret =3D trace_test_buffer(&tr->snapshot_buffer, &count); =20 if (!ret && !count) { printk(KERN_CONT ".. no entries found .."); @@ -1513,7 +1513,7 @@ trace_selftest_startup_wakeup(struct tracer *trace, s= truct trace_array *tr) /* check both trace buffers */ ret =3D trace_test_buffer(&tr->array_buffer, NULL); if (!ret) - ret =3D trace_test_buffer(&tr->max_buffer, &count); + ret =3D trace_test_buffer(&tr->snapshot_buffer, &count); =20 =20 trace->reset(tr); --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D4B262E9730 for ; Mon, 9 Feb 2026 02:20:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603608; cv=none; b=l+/+FTym1bpBKNcI/S+CyUyYDSxYGxkx75e7YO+Qwgzh9exPRIQiBlpA2npGV4j65hDQzOKUP0iC3JbRx1KNhVNXa1qZGiftXMx8UrFcdUiRQZIWRyQeNX4gLCtK30T+EIG34+ECqTdkSkKQUwXjrEFqr/6S7jpKROqMlVrVrHE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603608; c=relaxed/simple; bh=N0t4OUsSsWMvamyZFiMQGzAdYxM+dyMAOXO9lzi8zx0=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=OalovyQHB2tnRg/igXQxap8CaYSKqAhICoBrSUyXVWJBT7OaPV+dbYhpCsAHwui27l26QAUHOuipLWf0b9L0LqKz2iu0cXnuLl4hvHASNQARMDBpv3/GUqPIy0rk1Ew8uQcsXNIL+lK9J+B+LjNBGN2Q3CrL5s9VmI7KZgyWpmg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=AqFI+m4b; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="AqFI+m4b" Received: by smtp.kernel.org (Postfix) with ESMTPSA id ACE01C19424; Mon, 9 Feb 2026 02:20:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603608; bh=N0t4OUsSsWMvamyZFiMQGzAdYxM+dyMAOXO9lzi8zx0=; h=Date:From:To:Cc:Subject:References:From; b=AqFI+m4bJEtEE79F/mECgWFZX38uNp5GVZn2hZcSDadc5UXW9s3MP65WERzza3euj uANYI9Dsf9QnCw/D6I15KZvoy9ob157gKJ+vgDV3fnUXhm8d0P52/UicTqrIFoO0RC 6rhpg6Gn/wJubIb/JNx3wT/etrZzeh8Lz8yOYZV3B2xGdZaEBFtA0o9xpwaeD2sYXv Wbvj600+eIPHZKe1WotLDXIPQRsWQN0NLv5N+SZACQywmviPXnZQRtoM970D1uBji1 fxuMxuzvoNVCoNV6xumpLyAiFpSkdPc+QW32VSLcyZR9sQPUwsAwY5IDF9hB+C6IQ2 /47yLEGRcXuYA== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtT-0000000Aa9M-1M7Z; Sun, 08 Feb 2026 21:20:55 -0500 Message-ID: <20260209022055.175527617@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:45 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 19/20] tracing: Add tracer_uses_snapshot() helper to remove #ifdefs References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt Instead of having #ifdef CONFIG_TRACER_MAX_TRACE around every access to the struct tracer's use_max_tr field, add a helper function for that access and if CONFIG_TRACER_MAX_TRACE is not configured it just returns false. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208183856.599390238@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/trace.c | 51 ++++++++++++++------------------------------ kernel/trace/trace.h | 12 +++++++++++ 2 files changed, 28 insertions(+), 35 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 98524d0656bf..405212166677 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -810,7 +810,6 @@ EXPORT_SYMBOL_GPL(tracing_on); static void tracing_snapshot_instance_cond(struct trace_array *tr, void *cond_data) { - struct tracer *tracer =3D tr->current_trace; unsigned long flags; =20 if (in_nmi()) { @@ -827,7 +826,7 @@ static void tracing_snapshot_instance_cond(struct trace= _array *tr, } =20 /* Note, snapshot can not be used when the tracer uses it */ - if (tracer->use_max_tr) { + if (tracer_uses_snapshot(tr->current_trace)) { trace_array_puts(tr, "*** LATENCY TRACER ACTIVE ***\n"); trace_array_puts(tr, "*** Can not use snapshot (sorry) ***\n"); return; @@ -1076,7 +1075,7 @@ int tracing_snapshot_cond_enable(struct trace_array *= tr, void *cond_data, =20 guard(mutex)(&trace_types_lock); =20 - if (tr->current_trace->use_max_tr) + if (tracer_uses_snapshot(tr->current_trace)) return -EBUSY; =20 /* @@ -1787,7 +1786,7 @@ static int run_tracer_selftest(struct tracer *type) tr->current_trace_flags =3D type->flags ? : type->default_flags; =20 #ifdef CONFIG_TRACER_MAX_TRACE - if (type->use_max_tr) { + if (tracer_uses_snapshot(type)) { /* If we expanded the buffers, make sure the max is expanded too */ if (tr->ring_buffer_expanded) ring_buffer_resize(tr->snapshot_buffer.buffer, trace_buf_size, @@ -1812,7 +1811,7 @@ static int run_tracer_selftest(struct tracer *type) tracing_reset_online_cpus(&tr->array_buffer); =20 #ifdef CONFIG_TRACER_MAX_TRACE - if (type->use_max_tr) { + if (tracer_uses_snapshot(type)) { tr->allocated_snapshot =3D false; =20 /* Shrink the max buffer again */ @@ -3240,10 +3239,8 @@ static void *s_start(struct seq_file *m, loff_t *pos) } mutex_unlock(&trace_types_lock); =20 -#ifdef CONFIG_TRACER_MAX_TRACE - if (iter->snapshot && iter->trace->use_max_tr) + if (iter->snapshot && tracer_uses_snapshot(iter->trace)) return ERR_PTR(-EBUSY); -#endif =20 if (*pos !=3D iter->pos) { iter->ent =3D NULL; @@ -3282,10 +3279,8 @@ static void s_stop(struct seq_file *m, void *p) { struct trace_iterator *iter =3D m->private; =20 -#ifdef CONFIG_TRACER_MAX_TRACE - if (iter->snapshot && iter->trace->use_max_tr) + if (iter->snapshot && tracer_uses_snapshot(iter->trace)) return; -#endif =20 trace_access_unlock(iter->cpu_file); trace_event_read_unlock(); @@ -4177,11 +4172,9 @@ static int tracing_open(struct inode *inode, struct = file *file) static bool trace_ok_for_array(struct tracer *t, struct trace_array *tr) { -#ifdef CONFIG_TRACER_SNAPSHOT /* arrays with mapped buffer range do not have snapshots */ - if (tr->range_addr_start && t->use_max_tr) + if (tr->range_addr_start && tracer_uses_snapshot(t)) return false; -#endif return (tr->flags & TRACE_ARRAY_FL_GLOBAL) || t->allow_instances; } =20 @@ -5550,9 +5543,7 @@ int tracing_set_tracer(struct trace_array *tr, const = char *buf) { struct tracer *trace =3D NULL; struct tracers *t; -#ifdef CONFIG_TRACER_MAX_TRACE bool had_max_tr; -#endif int ret; =20 guard(mutex)(&trace_types_lock); @@ -5580,7 +5571,7 @@ int tracing_set_tracer(struct trace_array *tr, const = char *buf) return 0; =20 #ifdef CONFIG_TRACER_SNAPSHOT - if (trace->use_max_tr) { + if (tracer_uses_snapshot(trace)) { local_irq_disable(); arch_spin_lock(&tr->max_lock); ret =3D tr->cond_snapshot ? -EBUSY : 0; @@ -5612,14 +5603,13 @@ int tracing_set_tracer(struct trace_array *tr, cons= t char *buf) if (tr->current_trace->reset) tr->current_trace->reset(tr); =20 -#ifdef CONFIG_TRACER_MAX_TRACE - had_max_tr =3D tr->current_trace->use_max_tr; + had_max_tr =3D tracer_uses_snapshot(tr->current_trace); =20 /* Current trace needs to be nop_trace before synchronize_rcu */ tr->current_trace =3D &nop_trace; tr->current_trace_flags =3D nop_trace.flags; =20 - if (had_max_tr && !trace->use_max_tr) { + if (had_max_tr && !tracer_uses_snapshot(trace)) { /* * We need to make sure that the update_max_tr sees that * current_trace changed to nop_trace to keep it from @@ -5632,24 +5622,19 @@ int tracing_set_tracer(struct trace_array *tr, cons= t char *buf) tracing_disarm_snapshot(tr); } =20 - if (!had_max_tr && trace->use_max_tr) { + if (!had_max_tr && tracer_uses_snapshot(trace)) { ret =3D tracing_arm_snapshot_locked(tr); if (ret) return ret; } -#else - tr->current_trace =3D &nop_trace; -#endif =20 tr->current_trace_flags =3D t->flags ? : t->tracer->flags; =20 if (trace->init) { ret =3D tracer_init(trace, tr); if (ret) { -#ifdef CONFIG_TRACER_MAX_TRACE - if (trace->use_max_tr) + if (tracer_uses_snapshot(trace)) tracing_disarm_snapshot(tr); -#endif tr->current_trace_flags =3D nop_trace.flags; return ret; } @@ -7207,7 +7192,7 @@ tracing_snapshot_write(struct file *filp, const char = __user *ubuf, size_t cnt, =20 guard(mutex)(&trace_types_lock); =20 - if (tr->current_trace->use_max_tr) + if (tracer_uses_snapshot(tr->current_trace)) return -EBUSY; =20 local_irq_disable(); @@ -7306,7 +7291,7 @@ static int snapshot_raw_open(struct inode *inode, str= uct file *filp) =20 info =3D filp->private_data; =20 - if (info->iter.trace->use_max_tr) { + if (tracer_uses_snapshot(info->iter.trace)) { tracing_buffers_release(inode, filp); return -EBUSY; } @@ -7862,10 +7847,8 @@ tracing_buffers_read(struct file *filp, char __user = *ubuf, if (!count) return 0; =20 -#ifdef CONFIG_TRACER_MAX_TRACE - if (iter->snapshot && iter->tr->current_trace->use_max_tr) + if (iter->snapshot && tracer_uses_snapshot(iter->tr->current_trace)) return -EBUSY; -#endif =20 page_size =3D ring_buffer_subbuf_size_get(iter->array_buffer->buffer); =20 @@ -8049,10 +8032,8 @@ tracing_buffers_splice_read(struct file *file, loff_= t *ppos, int entries, i; ssize_t ret =3D 0; =20 -#ifdef CONFIG_TRACER_MAX_TRACE - if (iter->snapshot && iter->tr->current_trace->use_max_tr) + if (iter->snapshot && tracer_uses_snapshot(iter->tr->current_trace)) return -EBUSY; -#endif =20 page_size =3D ring_buffer_subbuf_size_get(iter->array_buffer->buffer); if (*ppos & (page_size - 1)) diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h index b50383aa8e50..ebb47abc0ee7 100644 --- a/kernel/trace/trace.h +++ b/kernel/trace/trace.h @@ -817,6 +817,18 @@ static inline void __trace_stack(struct trace_array *t= r, unsigned int trace_ctx, } #endif /* CONFIG_STACKTRACE */ =20 +#ifdef CONFIG_TRACER_MAX_TRACE +static inline bool tracer_uses_snapshot(struct tracer *tracer) +{ + return tracer->use_max_tr; +} +#else +static inline bool tracer_uses_snapshot(struct tracer *tracer) +{ + return false; +} +#endif + void trace_last_func_repeats(struct trace_array *tr, struct trace_func_repeats *last_info, unsigned int trace_ctx); --=20 2.51.0 From nobody Wed Feb 11 04:18:09 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1A2D92EB87F for ; Mon, 9 Feb 2026 02:20:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603609; cv=none; b=K7THMXAq6zPF7WB06cEExzHFahmpJtAxmMbqRHOnxjHjGBIxZdERmAYy+epOh8kCYQsgIWGi2AIk8NH1HuGxFAaYqtaNUWUVelxAxVIR4jfFbDbDWzcYdEnYxmppOV14MpTmuPNxz+aXyHGRDwLXiH8723/uH17TOY3mhBbDK88= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770603609; c=relaxed/simple; bh=hackC6XcyM/M+opMUK5WvhB6XXKw3kL10CqCaucSXWA=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=CwpUQrR0mKp83AttZvGRLhIosNHLTHGrAS5Ixu1+lsGc4dP5ltoHISowT3FI8HkvMzJ68R7hKly8aRRLCFWSGgM9QzDiSQkkg7tiflURlOJkVnCPX68I+hOZNWRpy/5jlM5JYIsUttc70x5oUTOBksDYN5fM8ou77z45sjgFaYw= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=rYgQfhCb; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="rYgQfhCb" Received: by smtp.kernel.org (Postfix) with ESMTPSA id D1384C2BCAF; Mon, 9 Feb 2026 02:20:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1770603609; bh=hackC6XcyM/M+opMUK5WvhB6XXKw3kL10CqCaucSXWA=; h=Date:From:To:Cc:Subject:References:From; b=rYgQfhCb5tWVFpawra6ooIrnNxUH6bxgmqh5qeGBwkP+tbd+R1DLpfmyJxCkBnFvI uroXgnl1rXGRsFeLwdqcYSFm3MPfXuxbfWH243TaAuxtDXdEax8/BjK7LZwK2/266/ v7GwECZaK2VZxETm2tXPq8IDS/jTx/ZDteKNgdgWjC/F31EjTFLTB0umsVoKCUOIil 7ctmW4u3E2GgV13CBQ2w8/QMDlFbDCnyh0zwTcdbP4dWY0ukEq6ViXQsurN/LTfUXQ CgyJ7jx+9KYXwXal8xMI3KIOBWX3qf1xC7SOpawAjWurxsRWWsxpe7o9q4fpOED6aM FAfFrMhstVX/g== Received: from rostedt by gandalf with local (Exim 4.99.1) (envelope-from ) id 1vpGtT-0000000Aa9s-22Ve; Sun, 08 Feb 2026 21:20:55 -0500 Message-ID: <20260209022055.341867623@kernel.org> User-Agent: quilt/0.68 Date: Sun, 08 Feb 2026 21:20:46 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton Subject: [for-next][PATCH 20/20] tracing: Better separate SNAPSHOT and MAX_TRACE options References: <20260209022026.627895421@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Steven Rostedt The latency tracers (scheduler, irqsoff, etc) were created when tracing was first added. These tracers required a "snapshot" buffer that was the same size as the ring buffer being written to. When a new max latency was hit, the main ring buffer would swap with the snapshot buffer so that the trace leading up to the latency would be saved in the snapshot buffer (The snapshot buffer is never written to directly and the data within it can be viewed without fear of being overwritten). Later, a new feature was added to allow snapshots to be taken by user space or even event triggers. This created a "snapshot" file that allowed users to trigger a snapshot from user space to save the current trace. The config for this new feature (CONFIG_TRACER_SNAPSHOT) would select the latency tracer config (CONFIG_TRACER_MAX_LATENCY) as it would need all the functionality from it as it already existed. But this was incorrect. As the snapshot feature is really what the latency tracers need and not the other way around. Have CONFIG_TRACER_MAX_TRACE select CONFIG_TRACER_SNAPSHOT where the tracers that needs the max latency buffer selects the TRACE_MAX_TRACE which will then select TRACER_SNAPSHOT. Also, go through trace.c and trace.h and make the code that only needs the TRACER_MAX_TRACE protected by that and the code that always requires the snapshot to be protected by TRACER_SNAPSHOT. Cc: Masami Hiramatsu Cc: Mark Rutland Cc: Mathieu Desnoyers Cc: Andrew Morton Link: https://patch.msgid.link/20260208183856.767870992@kernel.org Signed-off-by: Steven Rostedt (Google) --- kernel/trace/Kconfig | 8 ++--- kernel/trace/trace.c | 73 +++++++++++++++++++++++--------------------- kernel/trace/trace.h | 19 +++++++----- 3 files changed, 53 insertions(+), 47 deletions(-) diff --git a/kernel/trace/Kconfig b/kernel/trace/Kconfig index bfa2ec46e075..bedb2f982823 100644 --- a/kernel/trace/Kconfig +++ b/kernel/trace/Kconfig @@ -133,6 +133,7 @@ config BUILDTIME_MCOUNT_SORT =20 config TRACER_MAX_TRACE bool + select TRACER_SNAPSHOT =20 config TRACE_CLOCK bool @@ -422,7 +423,6 @@ config IRQSOFF_TRACER select GENERIC_TRACER select TRACER_MAX_TRACE select RING_BUFFER_ALLOW_SWAP - select TRACER_SNAPSHOT select TRACER_SNAPSHOT_PER_CPU_SWAP help This option measures the time spent in irqs-off critical @@ -445,7 +445,6 @@ config PREEMPT_TRACER select GENERIC_TRACER select TRACER_MAX_TRACE select RING_BUFFER_ALLOW_SWAP - select TRACER_SNAPSHOT select TRACER_SNAPSHOT_PER_CPU_SWAP select TRACE_PREEMPT_TOGGLE help @@ -467,7 +466,6 @@ config SCHED_TRACER select GENERIC_TRACER select CONTEXT_SWITCH_TRACER select TRACER_MAX_TRACE - select TRACER_SNAPSHOT help This tracer tracks the latency of the highest priority task to be scheduled in, starting from the point it has woken up. @@ -617,7 +615,6 @@ config TRACE_SYSCALL_BUF_SIZE_DEFAULT =20 config TRACER_SNAPSHOT bool "Create a snapshot trace buffer" - select TRACER_MAX_TRACE help Allow tracing users to take snapshot of the current buffer using the ftrace interface, e.g.: @@ -625,6 +622,9 @@ config TRACER_SNAPSHOT echo 1 > /sys/kernel/tracing/snapshot cat snapshot =20 + Note, the latency tracers select this option. To disable it, + all the latency tracers need to be disabled. + config TRACER_SNAPSHOT_PER_CPU_SWAP bool "Allow snapshot to swap per CPU" depends on TRACER_SNAPSHOT diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 405212166677..845b8a165daf 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -825,15 +825,15 @@ static void tracing_snapshot_instance_cond(struct tra= ce_array *tr, return; } =20 - /* Note, snapshot can not be used when the tracer uses it */ - if (tracer_uses_snapshot(tr->current_trace)) { - trace_array_puts(tr, "*** LATENCY TRACER ACTIVE ***\n"); + if (tr->mapped) { + trace_array_puts(tr, "*** BUFFER MEMORY MAPPED ***\n"); trace_array_puts(tr, "*** Can not use snapshot (sorry) ***\n"); return; } =20 - if (tr->mapped) { - trace_array_puts(tr, "*** BUFFER MEMORY MAPPED ***\n"); + /* Note, snapshot can not be used when the tracer uses it */ + if (tracer_uses_snapshot(tr->current_trace)) { + trace_array_puts(tr, "*** LATENCY TRACER ACTIVE ***\n"); trace_array_puts(tr, "*** Can not use snapshot (sorry) ***\n"); return; } @@ -1555,8 +1555,8 @@ static void __update_max_tr(struct trace_array *tr, struct task_struct *tsk, int cpu) { struct array_buffer *trace_buf =3D &tr->array_buffer; - struct array_buffer *max_buf =3D &tr->snapshot_buffer; struct trace_array_cpu *data =3D per_cpu_ptr(trace_buf->data, cpu); + struct array_buffer *max_buf =3D &tr->snapshot_buffer; struct trace_array_cpu *max_data =3D per_cpu_ptr(max_buf->data, cpu); =20 max_buf->cpu =3D cpu; @@ -1585,7 +1585,14 @@ __update_max_tr(struct trace_array *tr, struct task_= struct *tsk, int cpu) tracing_record_cmdline(tsk); latency_fsnotify(tr); } +#else +static inline void trace_create_maxlat_file(struct trace_array *tr, + struct dentry *d_tracer) { } +static inline void __update_max_tr(struct trace_array *tr, + struct task_struct *tsk, int cpu) { } +#endif /* CONFIG_TRACER_MAX_TRACE */ =20 +#ifdef CONFIG_TRACER_SNAPSHOT /** * update_max_tr - snapshot all trace buffers from global_trace to max_tr * @tr: tracer @@ -1619,12 +1626,11 @@ update_max_tr(struct trace_array *tr, struct task_s= truct *tsk, int cpu, else ring_buffer_record_off(tr->snapshot_buffer.buffer); =20 -#ifdef CONFIG_TRACER_SNAPSHOT if (tr->cond_snapshot && !tr->cond_snapshot->update(tr, cond_data)) { arch_spin_unlock(&tr->max_lock); return; } -#endif + swap(tr->array_buffer.buffer, tr->snapshot_buffer.buffer); =20 __update_max_tr(tr, tsk, cpu); @@ -1679,10 +1685,7 @@ update_max_tr_single(struct trace_array *tr, struct = task_struct *tsk, int cpu) __update_max_tr(tr, tsk, cpu); arch_spin_unlock(&tr->max_lock); } -#else /* !CONFIG_TRACER_MAX_TRACE */ -static inline void trace_create_maxlat_file(struct trace_array *tr, - struct dentry *d_tracer) { } -#endif /* CONFIG_TRACER_MAX_TRACE */ +#endif /* CONFIG_TRACER_SNAPSHOT */ =20 struct pipe_wait { struct trace_iterator *iter; @@ -1715,7 +1718,7 @@ static int wait_on_pipe(struct trace_iterator *iter, = int full) ret =3D ring_buffer_wait(iter->array_buffer->buffer, iter->cpu_file, full, wait_pipe_cond, &pwait); =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT /* * Make sure this is still the snapshot buffer, as if a snapshot were * to happen, this would now be the main buffer. @@ -2058,7 +2061,7 @@ void tracing_reset_all_online_cpus_unlocked(void) continue; tr->clear_trace =3D false; tracing_reset_online_cpus(&tr->array_buffer); -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT tracing_reset_online_cpus(&tr->snapshot_buffer); #endif } @@ -2098,7 +2101,7 @@ static void tracing_start_tr(struct trace_array *tr) if (buffer) ring_buffer_record_enable(buffer); =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT buffer =3D tr->snapshot_buffer.buffer; if (buffer) ring_buffer_record_enable(buffer); @@ -2134,7 +2137,7 @@ static void tracing_stop_tr(struct trace_array *tr) if (buffer) ring_buffer_record_disable(buffer); =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT buffer =3D tr->snapshot_buffer.buffer; if (buffer) ring_buffer_record_disable(buffer); @@ -3757,7 +3760,7 @@ static void test_ftrace_alive(struct seq_file *m) "# MAY BE MISSING FUNCTION EVENTS\n"); } =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT static void show_snapshot_main_help(struct seq_file *m) { seq_puts(m, "# echo 0 > snapshot : Clears and frees snapshot buffer\n" @@ -3935,7 +3938,7 @@ __tracing_open(struct inode *inode, struct file *file= , bool snapshot) =20 iter->tr =3D tr; =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT /* Currently only the top directory has a snapshot */ if (tr->current_trace->print_max || snapshot) iter->array_buffer =3D &tr->snapshot_buffer; @@ -4351,14 +4354,14 @@ int tracing_set_cpumask(struct trace_array *tr, if (cpumask_test_cpu(cpu, tr->tracing_cpumask) && !cpumask_test_cpu(cpu, tracing_cpumask_new)) { ring_buffer_record_disable_cpu(tr->array_buffer.buffer, cpu); -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT ring_buffer_record_disable_cpu(tr->snapshot_buffer.buffer, cpu); #endif } if (!cpumask_test_cpu(cpu, tr->tracing_cpumask) && cpumask_test_cpu(cpu, tracing_cpumask_new)) { ring_buffer_record_enable_cpu(tr->array_buffer.buffer, cpu); -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT ring_buffer_record_enable_cpu(tr->snapshot_buffer.buffer, cpu); #endif } @@ -4568,7 +4571,7 @@ int set_tracer_flag(struct trace_array *tr, u64 mask,= int enabled) =20 case TRACE_ITER(OVERWRITE): ring_buffer_change_overwrite(tr->array_buffer.buffer, enabled); -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT ring_buffer_change_overwrite(tr->snapshot_buffer.buffer, enabled); #endif break; @@ -5232,7 +5235,7 @@ static void update_buffer_entries(struct array_buffer= *buf, int cpu) } } =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT /* resize @tr's buffer to the size of @size_tr's entries */ static int resize_buffer_duplicate_size(struct array_buffer *trace_buf, struct array_buffer *size_buf, int cpu_id) @@ -5258,7 +5261,7 @@ static int resize_buffer_duplicate_size(struct array_= buffer *trace_buf, =20 return ret; } -#endif /* CONFIG_TRACER_MAX_TRACE */ +#endif /* CONFIG_TRACER_SNAPSHOT */ =20 static int __tracing_resize_ring_buffer(struct trace_array *tr, unsigned long size, int cpu) @@ -5283,7 +5286,7 @@ static int __tracing_resize_ring_buffer(struct trace_= array *tr, if (ret < 0) goto out_start; =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT if (!tr->allocated_snapshot) goto out; =20 @@ -5315,7 +5318,7 @@ static int __tracing_resize_ring_buffer(struct trace_= array *tr, update_buffer_entries(&tr->snapshot_buffer, cpu); =20 out: -#endif /* CONFIG_TRACER_MAX_TRACE */ +#endif /* CONFIG_TRACER_SNAPSHOT */ =20 update_buffer_entries(&tr->array_buffer, cpu); out_start: @@ -7020,7 +7023,7 @@ int tracing_set_clock(struct trace_array *tr, const c= har *clockstr) */ tracing_reset_online_cpus(&tr->array_buffer); =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT if (tr->snapshot_buffer.buffer) ring_buffer_set_clock(tr->snapshot_buffer.buffer, trace_clocks[i].func); tracing_reset_online_cpus(&tr->snapshot_buffer); @@ -8167,7 +8170,7 @@ static long tracing_buffers_ioctl(struct file *file, = unsigned int cmd, unsigned return 0; } =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT static int get_snapshot_map(struct trace_array *tr) { int err =3D 0; @@ -9171,7 +9174,7 @@ buffer_subbuf_size_write(struct file *filp, const cha= r __user *ubuf, if (ret) goto out; =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT =20 if (!tr->allocated_snapshot) goto out_max; @@ -9392,7 +9395,7 @@ static int allocate_trace_buffers(struct trace_array = *tr, int size) if (ret) return ret; =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT /* Fix mapped buffer trace arrays do not have snapshot buffers */ if (tr->range_addr_start) return 0; @@ -9419,7 +9422,7 @@ static void free_trace_buffers(struct trace_array *tr) free_trace_buffer(&tr->array_buffer); kfree(tr->module_delta); =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT free_trace_buffer(&tr->snapshot_buffer); #endif } @@ -9561,7 +9564,7 @@ trace_array_create_systems(const char *name, const ch= ar *systems, tr->syscall_buf_sz =3D global_trace.syscall_buf_sz; =20 tr->max_lock =3D (arch_spinlock_t)__ARCH_SPIN_LOCK_UNLOCKED; -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT spin_lock_init(&tr->snapshot_trigger_lock); #endif tr->current_trace =3D &nop_trace; @@ -10515,7 +10518,7 @@ ssize_t trace_parse_run_command(struct file *file, = const char __user *buffer, return done; } =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT __init static bool tr_needs_alloc_snapshot(const char *name) { char *test; @@ -10705,7 +10708,7 @@ __init static void enable_instances(void) } } else { /* Only non mapped buffers have snapshot buffers */ - if (IS_ENABLED(CONFIG_TRACER_MAX_TRACE)) + if (IS_ENABLED(CONFIG_TRACER_SNAPSHOT)) do_allocate_snapshot(name); } =20 @@ -10832,7 +10835,7 @@ __init static int tracer_alloc_buffers(void) global_trace.current_trace_flags =3D nop_trace.flags; =20 global_trace.max_lock =3D (arch_spinlock_t)__ARCH_SPIN_LOCK_UNLOCKED; -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT spin_lock_init(&global_trace.snapshot_trigger_lock); #endif ftrace_init_global_array_ops(&global_trace); @@ -10900,7 +10903,7 @@ struct trace_array *trace_get_global_array(void) =20 void __init ftrace_boot_snapshot(void) { -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT struct trace_array *tr; =20 if (!snapshot_at_boot) diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h index ebb47abc0ee7..649fdd20fc91 100644 --- a/kernel/trace/trace.h +++ b/kernel/trace/trace.h @@ -329,7 +329,7 @@ struct trace_array { struct list_head list; char *name; struct array_buffer array_buffer; -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT /* * The snapshot_buffer is used to snapshot the trace when a maximum * latency is reached, or when the user initiates a snapshot. @@ -346,13 +346,16 @@ struct trace_array { bool allocated_snapshot; spinlock_t snapshot_trigger_lock; unsigned int snapshot; +#ifdef CONFIG_TRACER_MAX_TRACE unsigned long max_latency; #ifdef CONFIG_FSNOTIFY struct dentry *d_max_latency; struct work_struct fsnotify_work; struct irq_work fsnotify_irqwork; -#endif -#endif +#endif /* CONFIG_FSNOTIFY */ +#endif /* CONFIG_TRACER_MAX_TRACE */ +#endif /* CONFIG_TRACER_SNAPSHOT */ + /* The below is for memory mapped ring buffer */ unsigned int mapped; unsigned long range_addr_start; @@ -378,7 +381,7 @@ struct trace_array { * * It is also used in other places outside the update_max_tr * so it needs to be defined outside of the - * CONFIG_TRACER_MAX_TRACE. + * CONFIG_TRACER_SNAPSHOT. */ arch_spinlock_t max_lock; #ifdef CONFIG_FTRACE_SYSCALLS @@ -791,22 +794,22 @@ int trace_pid_write(struct trace_pid_list *filtered_p= ids, struct trace_pid_list **new_pid_list, const char __user *ubuf, size_t cnt); =20 -#ifdef CONFIG_TRACER_MAX_TRACE +#ifdef CONFIG_TRACER_SNAPSHOT void update_max_tr(struct trace_array *tr, struct task_struct *tsk, int cp= u, void *cond_data); void update_max_tr_single(struct trace_array *tr, struct task_struct *tsk, int cpu); =20 -#ifdef CONFIG_FSNOTIFY -#define LATENCY_FS_NOTIFY +#if defined(CONFIG_TRACER_MAX_TRACE) && defined(CONFIG_FSNOTIFY) +# define LATENCY_FS_NOTIFY #endif -#endif /* CONFIG_TRACER_MAX_TRACE */ =20 #ifdef LATENCY_FS_NOTIFY void latency_fsnotify(struct trace_array *tr); #else static inline void latency_fsnotify(struct trace_array *tr) { } #endif +#endif /* CONFIG_TRACER_SNAPSHOT */ =20 #ifdef CONFIG_STACKTRACE void __trace_stack(struct trace_array *tr, unsigned int trace_ctx, int ski= p); --=20 2.51.0