diff mbox series

[PATCHv2,1/2] trace/hwlat: Do not wipe the contents of per-cpu thread data

Message ID 20230310100451.3948583-2-tero.kristo@linux.intel.com (mailing list archive)
State Accepted
Commit 4c42f5f0d1dd20bddd9f940beb1e6ccad60c4498
Headers show
Series trace/hwlat: Prevent startup of multiple per-cpu threads | expand

Commit Message

Tero Kristo March 10, 2023, 10:04 a.m. UTC
Do not wipe the contents of the per-cpu kthread data when starting the
tracer, as this will completely forget about already running instances
and can later start new additional per-cpu threads.

Signed-off-by: Tero Kristo <tero.kristo@linux.intel.com>
---
v2:
  * split into separate patch

 kernel/trace/trace_hwlat.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

Daniel Bristot de Oliveira March 10, 2023, 1:52 p.m. UTC | #1
On 3/10/23 11:04, Tero Kristo wrote:
> Do not wipe the contents of the per-cpu kthread data when starting the
> tracer, as this will completely forget about already running instances
> and can later start new additional per-cpu threads.
> 
> Signed-off-by: Tero Kristo <tero.kristo@linux.intel.com>

Acked-by: Daniel Bristot de Oliveira <bristot@kernel.org>
-- Daniel
diff mbox series

Patch

diff --git a/kernel/trace/trace_hwlat.c b/kernel/trace/trace_hwlat.c
index d440ddd5fd8b..edc26dc22c3f 100644
--- a/kernel/trace/trace_hwlat.c
+++ b/kernel/trace/trace_hwlat.c
@@ -584,9 +584,6 @@  static int start_per_cpu_kthreads(struct trace_array *tr)
 	 */
 	cpumask_and(current_mask, cpu_online_mask, tr->tracing_cpumask);
 
-	for_each_online_cpu(cpu)
-		per_cpu(hwlat_per_cpu_data, cpu).kthread = NULL;
-
 	for_each_cpu(cpu, current_mask) {
 		retval = start_cpu_kthread(cpu);
 		if (retval)