diff options
| author | Daniel Bristot de Oliveira <bristot@kernel.org> | 2023-08-04 17:52:11 +0200 |
|---|---|---|
| committer | Daniel Bristot de Oliveira <bristot@kernel.org> | 2023-09-12 15:43:17 +0200 |
| commit | 02d89917ef68acbe65c7cc2323f1db4429879878 (patch) | |
| tree | f27d7959d7043c55eb700fd171ef2722abff58f6 | |
| parent | 0bb80ecc33a8fb5a682236443c1e740d5c917d1d (diff) | |
rtla/timerlat_aa: Zero thread sum after every sample analysis
The thread thread_thread_sum accounts for thread interference
during a single activation. It was not being zeroed, so it was
accumulating thread interference over all activations.
It was not that visible when timerlat was the highest priority.
Link: https://lore.kernel.org/lkml/97bff55b0141f2d01b47d9450a5672fde147b89a.1691162043.git.bristot@kernel.org
Fixes: 27e348b221f6 ("rtla/timerlat: Add auto-analysis core")
Signed-off-by: Daniel Bristot de Oliveira <bristot@kernel.org>
| -rw-r--r-- | tools/tracing/rtla/src/timerlat_aa.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tools/tracing/rtla/src/timerlat_aa.c b/tools/tracing/rtla/src/timerlat_aa.c index e0ffe69c271c..dec5b4c4511e 100644 --- a/tools/tracing/rtla/src/timerlat_aa.c +++ b/tools/tracing/rtla/src/timerlat_aa.c @@ -159,6 +159,7 @@ static int timerlat_aa_irq_latency(struct timerlat_aa_data *taa_data, taa_data->thread_nmi_sum = 0; taa_data->thread_irq_sum = 0; taa_data->thread_softirq_sum = 0; + taa_data->thread_thread_sum = 0; taa_data->thread_blocking_duration = 0; taa_data->timer_irq_start_time = 0; taa_data->timer_irq_duration = 0; |