Re: [Qemu-devel] [PATCH 2/5] semaphore: implement fallback counting semaphores with mutex+condvar

[Brad Smith]

On 11/02/12 09:14, Paolo Bonzini wrote:
> OpenBSD and Darwin do not have sem_timedwait.  Implement a fallback
> for them.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
>   qemu-thread-posix.c | 74 +++++++++++++++++++++++++++++++++++++++++++++++++++++
>   qemu-thread-posix.h |  6 +++++
>   2 file modificati, 80 inserzioni(+)
>
> diff --git a/qemu-thread-posix.c b/qemu-thread-posix.c
> index 6a3d3a1..048db8f 100644
> --- a/qemu-thread-posix.c
> +++ b/qemu-thread-posix.c
> @@ -122,36 +122,100 @@ void qemu_sem_init(QemuSemaphore *sem, int init)
>   {
>       int rc;
>
> +#if defined(__OpenBSD__) || defined(__APPLE__) || defined(__NetBSD__)

OpenBSD 5.2 & -current (libpthread) / NetBSD -current (librt) have 
supported sem_timedwait() for roughly 8 months now. Please change this 
to properly test for the presence of sem_timedwait() within the 
configure script.

--
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.