From a20cb9d0275eb7f0618b4ef2448544056e1035df Mon Sep 17 00:00:00 2001 From: Chengwen Feng Date: Fri, 16 Apr 2021 16:18:26 +0800 Subject: [PATCH] test: check thread creation There was a call for thread create function without result check. Add result check and message print out after failure. Fixes: 086eb64db39e ("test/pdump: add unit test for pdump library") Cc: stable@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Min Hu (Connor) Acked-by: Reshma Pattan --- app/test/process.h | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/app/test/process.h b/app/test/process.h index 27f1b1c0e6..a09a088477 100644 --- a/app/test/process.h +++ b/app/test/process.h @@ -48,6 +48,7 @@ process_dup(const char *const argv[], int numargs, const char *env_value) #ifdef RTE_LIB_PDUMP #ifdef RTE_NET_RING pthread_t thread; + int rc; #endif #endif @@ -126,8 +127,13 @@ process_dup(const char *const argv[], int numargs, const char *env_value) /* parent process does a wait */ #ifdef RTE_LIB_PDUMP #ifdef RTE_NET_RING - if ((strcmp(env_value, "run_pdump_server_tests") == 0)) - pthread_create(&thread, NULL, &send_pkts, NULL); + if ((strcmp(env_value, "run_pdump_server_tests") == 0)) { + rc = pthread_create(&thread, NULL, &send_pkts, NULL); + if (rc != 0) { + rte_panic("Cannot start send pkts thread: %s\n", + strerror(rc)); + } + } #endif #endif -- 2.20.1