Re: [PATCH] selftests: timers: improve timer_create failure message

[Shuah Khan]

On 8/29/24 09:37, Gianfranco Trad wrote:

improve timer_create failure message with strerror() function
to give more information to the user.

Signed-off-by: Gianfranco Trad <gianf.t...@gmail.com>
---
  tools/testing/selftests/timers/alarmtimer-suspend.c | 5 +++--
  1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/timers/alarmtimer-suspend.c 
b/tools/testing/selftests/timers/alarmtimer-suspend.c
index ad52e608b88e..74acdb87d4f4 100644
--- a/tools/testing/selftests/timers/alarmtimer-suspend.c
+++ b/tools/testing/selftests/timers/alarmtimer-suspend.c
@@ -28,6 +28,7 @@
  #include <signal.h>
  #include <stdlib.h>
  #include <pthread.h>
+#include <errno.h>
  #include "../kselftest.h"

#define CLOCK_REALTIME 0

@@ -142,8 +143,8 @@ int main(void)

alarmcount = 0;

                if (timer_create(alarm_clock_id, &se, &tm1) == -1) {
-                       printf("timer_create failed, %s unsupported?\n",
-                                       clockstring(alarm_clock_id));
+                       printf("timer_create failed, %s unsupported?: %s\n",
+                                       clockstring(alarm_clock_id), 
strerror(errno));
                        break;
                }

Jon, Does this look good to you? I can pick this up if you
okay with this change.

thanks,
-- Shuah