* John Kacur | 2015-07-10 14:25:27 [+0200]:
>diff --git a/src/pi_tests/pi_stress.c b/src/pi_tests/pi_stress.c
>index aaa36c362445..1d1cc58fae54 100644
>--- a/src/pi_tests/pi_stress.c
>+++ b/src/pi_tests/pi_stress.c
>@@ -727,17 +727,24 @@ void *low_priority(void *arg)
> status = pthread_barrier_wait(&p->locked_barrier);
> if (status && status != PTHREAD_BARRIER_SERIAL_THREAD) {
> pi_error
>- ("low_priority[%d]: pthread_barrier_wait(locked): %x\n",
>- p->id, status);
>+ ("low_priority[%d]: pthread_barrier_wait(locked): %x\n",
>+ p->id, status);
>+ /* release the mutex */
>+ pi_debug("low_priority[%d]: unlocking mutex\n", p->id);
>+ pthread_mutex_unlock(&p->mutex);
> return NULL;
> }
>+
> /* wait for priority boost */
> pi_debug("low_priority[%d]: entering elevated wait\n", p->id);
> status = pthread_barrier_wait(&p->elevate_barrier);
> if (status && status != PTHREAD_BARRIER_SERIAL_THREAD) {
> pi_error
>- ("low_priority[%d]: pthread_barrier_wait(elevate): %x\n",
>- p->id, status);
>+ ("low_priority[%d]: pthread_barrier_wait(elevate): %x\n",
>+ p->id, status);
>+ /* release the mutex */
>+ pi_debug("low_priority[%d]: unlocking mutex\n", p->id);
>+ pthread_mutex_unlock(&p->mutex);
> return NULL;
> }
>
What about having an out: label which does all the clean up at one spot
instead of doing copy/paste here and there?
Sebastian
--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
