[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH v2 upstream 07/22] add assertions on the owner o
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] Re: [PATCH v2 upstream 07/22] add assertions on the owner of a QemuMutex |
Date: |
Sun, 27 Feb 2011 10:33:50 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2011-02-26 16:40, Paolo Bonzini wrote:
> These are already present in the Win32 implementation, add them to
> the pthread wrappers as well. Use PTHREAD_MUTEX_ERRORCHECK for mutex
> operations, and track the owner separately for cond_signal/broadcast.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> qemu-thread-posix.c | 23 +++++++++++++++++++++--
> qemu-thread-posix.h | 1 +
> 2 files changed, 22 insertions(+), 2 deletions(-)
>
> diff --git a/qemu-thread-posix.c b/qemu-thread-posix.c
> index e307773..a4c6e25 100644
> --- a/qemu-thread-posix.c
> +++ b/qemu-thread-posix.c
> @@ -16,9 +16,12 @@
> #include <time.h>
> #include <signal.h>
> #include <stdint.h>
> +#include <assert.h>
> #include <string.h>
> #include "qemu-thread.h"
>
> +static pthread_t pthread_null;
> +
> static void error_exit(int err, const char *msg)
> {
> fprintf(stderr, "qemu: %s: %s\n", msg, strerror(err));
> @@ -28,8 +31,13 @@ static void error_exit(int err, const char *msg)
> void qemu_mutex_init(QemuMutex *mutex)
> {
> int err;
> + pthread_mutexattr_t mutexattr;
>
> - err = pthread_mutex_init(&mutex->lock, NULL);
> + mutex->owner = pthread_null;
> + pthread_mutexattr_init(&mutexattr);
> + pthread_mutexattr_settype(&mutexattr, PTHREAD_MUTEX_ERRORCHECK);
> + err = pthread_mutex_init(&mutex->lock, &mutexattr);
> + pthread_mutexattr_destroy(&mutexattr);
> if (err)
> error_exit(err, __func__);
> }
> @@ -48,13 +56,20 @@ void qemu_mutex_lock(QemuMutex *mutex)
> int err;
>
> err = pthread_mutex_lock(&mutex->lock);
> + mutex->owner = pthread_self();
> if (err)
> error_exit(err, __func__);
> }
>
> int qemu_mutex_trylock(QemuMutex *mutex)
> {
> - return pthread_mutex_trylock(&mutex->lock);
> + int err;
> + err = pthread_mutex_trylock(&mutex->lock);
> + if (err == 0) {
> + mutex->owner = pthread_self();
> + }
> +
> + return !!err;
> }
>
> static void timespec_add_ms(struct timespec *ts, uint64_t msecs)
> @@ -85,6 +100,7 @@ void qemu_mutex_unlock(QemuMutex *mutex)
> {
> int err;
>
> + mutex->owner = pthread_null;
> err = pthread_mutex_unlock(&mutex->lock);
> if (err)
> error_exit(err, __func__);
> @@ -130,7 +146,10 @@ void qemu_cond_wait(QemuCond *cond, QemuMutex *mutex)
> {
> int err;
>
> + assert(pthread_equal(mutex->owner, pthread_self()));
> + mutex->owner = pthread_null;
> err = pthread_cond_wait(&cond->cond, &mutex->lock);
Though POSIX is not 100% explicit on this, every sane pthread_cond_wait
implementation will apply the same error checking as on
pthread_mutex_unlock when the given mutex is of
PTHREAD_MUTEX_ERRORCHECK. So, this assert is actually redundant as well.
Now that we are left without any assertions, I start wondering about one
of the original missions: enforce qemu_cond_signal/broadcast to be
called under a mutex. What about extending those services with a mutex
argument and applying the assert there? Could become a static-inline
wrapper so that the argument is optimized away if assert() is inactive.
Jan
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v2 uq/master 00/22] Win32 iothread support, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 01/22] unlock iothread during WaitForMultipleObjects, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 02/22] implement win32 dynticks timer, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 03/22] use win32 timer queues, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 04/22] Refactor thread retrieval and check, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 05/22] add win32 qemu-thread implementation, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 06/22] include qemu-thread.h early, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 07/22] add assertions on the owner of a QemuMutex, Paolo Bonzini, 2011/02/26
- [Qemu-devel] Re: [PATCH v2 upstream 07/22] add assertions on the owner of a QemuMutex,
Jan Kiszka <=
- [Qemu-devel] [PATCH v2 upstream 08/22] remove CONFIG_THREAD, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 09/22] target-sh4: move intr_at_halt out of cpu_halted(), Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 10/22] inline cpu_halted into sole caller, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 11/22] always qemu_cpu_kick after unhalting a cpu, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 12/22] exit round-robin vcpu loop if cpu->stopped is true, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 13/22] always signal pause_cond after stopping a VCPU, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 14/22] do not use timedwait on qemu_halt_cond, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 16/22] do not use timedwait on qemu_pause_cond, Paolo Bonzini, 2011/02/26
- [Qemu-devel] [PATCH v2 upstream 15/22] do not use timedwait on qemu_system_cond, Paolo Bonzini, 2011/02/26