Commit fb8b8183 authored Jan 06, 2026 by Wander Lairson Costa Committed by Tomas Glozar Jan 07, 2026

rtla: Fix parse_cpu_set() return value documentation

Correct the return value documentation for parse_cpu_set() function
in utils.c. The comment incorrectly stated that the function returns
1 on success and 0 on failure, but the actual implementation returns
0 on success and 1 on failure, following the common error-on-nonzero
convention used throughout the codebase.

This documentation fix ensures that developers reading the code
understand the correct return value semantics and prevents potential
misuse of the function's return value in conditional checks.

Signed-off-by: Wander Lairson Costa <wander@redhat.com>
Link: https://lore.kernel.org/r/20260106133655.249887-18-wander@redhat.com

Signed-off-by: Tomas Glozar <tglozar@redhat.com>

parent 33e3c807

tools/tracing/rtla/src/utils.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -113,7 +113,7 @@ void get_duration(time_t start_time, char *output, int output_size)
		* Receives a cpu list, like 1-3,5 (cpus 1, 2, 3, 5), and then set
		* filling cpu_set_t argument.
		*
		* Returns 1 on success, 0 otherwise.
		* Returns 0 on success, 1 otherwise.
		*/
		int parse_cpu_set(char cpu_list, cpu_set_t set)
		{