[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] win32: replace custom mutex and condition varia
From: |
Andrew Baumann |
Subject: |
Re: [Qemu-devel] [PATCH] win32: replace custom mutex and condition variable with native primitives |
Date: |
Fri, 24 Mar 2017 23:14:20 +0000 |
> From: Paolo Bonzini [mailto:address@hidden
> Sent: Friday, 24 March 2017 15:29
> > From: Andrey Shedel <address@hidden>
> >
> > The multithreaded TCG implementation exposed deadlocks in the win32
> > condition variables: as implemented, qemu_cond_broadcast waited on
> > receivers, whereas the pthreads API it was intended to emulate does
> > not. This was causing a deadlock because broadcast was called while
> > holding the IO lock, as well as all possible waiters blocked on the
> > same lock.
> >
> > This patch replaces all the custom synchronisation code for mutexes
> > and condition variables with native Windows primitives (SRWlocks and
> > condition variables) with the same semantics as their POSIX
> > equivalents. To enable that, it requires a Windows Vista or newer host
> > OS.
> >
> > [AB: edited commit message]
> > Signed-off-by: Andrew Baumann <address@hidden>
>
> Oops, just a nit but an important one: there should be a
> Signed-off-by for Andrey as well.
Oops, my fault, since I took his code and prepared the patch submission. We can
resend with the signoff, but perhaps I should wait for a review?
Andrew