back-port windows / yield fix ?
Michael Stahl
mstahl at redhat.com
Thu Oct 22 06:34:55 PDT 2015
On 16.10.2015 20:50, Michael Meeks wrote:
> Hi Michael,
>
> I just hit a nasty windows deadlock; and noticed that it's fixed
> on master with:
>
> commit 482c52e91fe41a52e68827e9bf64a9736427d517
> Author: Michael Stahl <mstahl at redhat.com>
> Date: Fri Jun 26 13:01:51 2015 +0200
>
> vcl: fix Win32 deadlocks from SolarMutexReleaser
>
> Turned out to be very reproducible ;-) and your fix above (which
> looks sensible) fixes it.
you can reproduce that with the UI? that's amazing...
> Any reason not to have that in -5-0 ?
well it's a little scary, let me try to remember the follow-up fixes...
- the assumption that the thread that uses SolarMutexReleaser actually
holds the SolarMutex turned out not to hold
commit c18f11587d37f285a95447dd8996c8b605732e00 "tdf#92460: fpicker: fix
VistaFilePicker using SolarMutexReleaser ... on a thread that doesn't
ever lock SolarMutex."
commit 4231274abaec8e45538788bd67261044d7ef4bea "tdf#76845: vcl: main
thread does not necessarily have SolarMutex... at least not if
VCLXWindowImpl::OnProcessCallbacks drops it and calls arbitrary event
handlers. "
- the LazyDelete related data race still needs fixing, as documented in
commit 815fe9110897d5d8c122a5743e3765b67013dd8b
More information about the LibreOffice
mailing list