examples/performance-thread: fix return sign

The patch fixes an inverted return value in the
cond_destroy and cond_init APIs of the pthread shim
example.

These APIs are now demonstrated in the sample app by
having the mutexes and condition variables
explicitly destroyed before the appplication terminates.

Fixes: 433ba6228f9a77a9b5f4 ("add pthread_shim app")
Signed-off-by: Ian Betts <ian.betts@intel.com>

diff --git a/examples/performance-thread/pthread_shim/main.c b/examples/performance-thread/pthread_shim/main.c
index 2f67c1bc4e912f24c572c646df22498a14425a0b..dab379aca38124090e163085de3d458a40761046 100644 (file)
--- a/examples/performance-thread/pthread_shim/main.c
+++ b/examples/performance-thread/pthread_shim/main.c
@@ -211,6 +211,10 @@ static void initial_lthread(void *args __attribute__((unused)))
                        printf("error on thread exit\n");
        }
 
+       pthread_cond_destroy(&exit_cond);
+       pthread_mutex_destroy(&print_lock);
+       pthread_mutex_destroy(&exit_lock);
+
        /* shutdown the lthread scheduler */
        lthread_scheduler_shutdown(rte_lcore_id());
        lthread_detach();

diff --git a/examples/performance-thread/pthread_shim/pthread_shim.c b/examples/performance-thread/pthread_shim/pthread_shim.c
index 30cd68a566e3b6adae9cb1249d8b26a060ca7770..0fd36bad983ee78493c1c2efeab5cc0dca3b5a75 100644 (file)
--- a/examples/performance-thread/pthread_shim/pthread_shim.c
+++ b/examples/performance-thread/pthread_shim/pthread_shim.c
@@ -327,17 +327,24 @@ int pthread_cond_broadcast(pthread_cond_t *cond)
        return _sys_pthread_funcs.f_pthread_cond_broadcast(cond);
 }
 
+int pthread_mutex_destroy(pthread_mutex_t *mutex)
+{
+       if (override)
+               return lthread_mutex_destroy(*(struct lthread_mutex **)mutex);
+       return _sys_pthread_funcs.f_pthread_mutex_destroy(mutex);
+}
+
 int pthread_cond_destroy(pthread_cond_t *cond)
 {
        if (override)
-               return -lthread_cond_destroy(*(struct lthread_cond **)cond);
+               return lthread_cond_destroy(*(struct lthread_cond **)cond);
        return _sys_pthread_funcs.f_pthread_cond_destroy(cond);
 }
 
 int pthread_cond_init(pthread_cond_t *cond, const pthread_condattr_t *attr)
 {
        if (override)
-               return -lthread_cond_init(NULL,
+               return lthread_cond_init(NULL,
                                (struct lthread_cond **)cond,
                                (const struct lthread_condattr *) attr);
        return _sys_pthread_funcs.f_pthread_cond_init(cond, attr);