[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20251117184409.42831-1-wander@redhat.com>
Date: Mon, 17 Nov 2025 15:41:07 -0300
From: Wander Lairson Costa <wander@...hat.com>
To: Steven Rostedt <rostedt@...dmis.org>,
Wander Lairson Costa <wander@...hat.com>,
Tomas Glozar <tglozar@...hat.com>,
Ivan Pravdin <ipravdin.official@...il.com>,
Crystal Wood <crwood@...hat.com>,
John Kacur <jkacur@...hat.com>,
Costa Shulyupin <costa.shul@...hat.com>,
Tiezhu Yang <yangtiezhu@...ngson.cn>,
linux-trace-kernel@...r.kernel.org (open list:Real-time Linux Analysis (RTLA) tools),
linux-kernel@...r.kernel.org (open list),
bpf@...r.kernel.org (open list:BPF [MISC]:Keyword:(?:\b|_)bpf(?:\b|_))
Subject: [PATCH 0/13] rtla: Code robustness and maintainability improvements
This patch series enhances the robustness and maintainability of the
RTLA (Real-Time Linux Analysis) tool through systematic improvements
to error handling, code clarity, and consistency.
The changes strengthen defensive programming practices throughout the
codebase by improving input validation and memory management. Several
new helper functions and macros are introduced to reduce code duplication
and provide safer, more readable alternatives to common operations. The
series also consolidates duplicate logic across modules and adopts
standard conventions where appropriate.
These improvements make the code more resilient and easier to maintain
while preserving existing functionality and behavior.
Wander Lairson Costa (13):
rtla: Check for memory allocation failures
rtla: Use strdup() to simplify code
rtla: Introduce for_each_action() helper
rtla: Replace atoi() with a robust strtoi()
rtla: Simplify argument parsing
rtla: Use strncmp_static() in more places
rtla: Introduce timerlat_restart() helper
rtla: Use standard exit codes for result enum
rtla: Exit if trace output action fails
rtla: Remove redundant memset after calloc
rtla: Replace magic number with MAX_PATH
rtla: Remove unused headers
rtla: Fix inconsistent state in actions_add_* functions
tools/tracing/rtla/src/actions.c | 91 ++++++++++++++++++--------
tools/tracing/rtla/src/actions.h | 7 +-
tools/tracing/rtla/src/osnoise.c | 6 +-
tools/tracing/rtla/src/osnoise_hist.c | 1 -
tools/tracing/rtla/src/timerlat.c | 36 +++++++++-
tools/tracing/rtla/src/timerlat.h | 9 +++
tools/tracing/rtla/src/timerlat_hist.c | 32 +++++----
tools/tracing/rtla/src/timerlat_top.c | 27 ++++----
tools/tracing/rtla/src/timerlat_u.c | 4 +-
tools/tracing/rtla/src/trace.c | 23 ++++---
tools/tracing/rtla/src/utils.c | 48 +++++++++++---
tools/tracing/rtla/src/utils.h | 23 +++++--
12 files changed, 220 insertions(+), 87 deletions(-)
--
2.51.1
Powered by blists - more mailing lists