Tinderbox owners: deadlock, beware

Stephan Bergmann <sbergman -AT- redhat.com>

Fri, 14 Aug 2015 13:45:48 +0200

In the window of time between

commit 54f10a9654b617c4c993044e52e7bd40d0151c53
Author: Stephan Bergmann <sbergman@redhat.com>
Date:   Thu Aug 13 12:04:55 2015 +0200

    tdf#93404: Forgot to launch WakeUpThread

    ...in 017f250764ec7b4ecb82ac19f5b3f68cadf1bf56 "Ensure WakeUpThread is joined
    before exit"

    Change-Id: Iaa5a5772f099b11229bd40c3cc10d863ef0ad5b3

and

commit ded58e062c9e9b8b5f9dc9a18777a1be68359a91
Author: Stephan Bergmann <sbergman@redhat.com>
Date:   Thu Aug 13 15:16:38 2015 +0200

     Related tdf#93404: Fix deadlock when joining WakeUpThread

     ...after 54f10a9654b617c4c993044e52e7bd40d0151c53 "tdf#93404: Forgot to launch
     WakeUpThread," when the WakeUpThread blocks on locking
     StatusIndicatorFactory::m_mutex in StatusIndicatorFactory::update() while the
     joining thread blocks on StatusIndicatorFactory::impl_stopWakeUpThread() ->
     WakeUpThread::stop() -> Thread::join().

     Change-Id: I3480014b1f522901064ea9b71ffa2ebf5d74fef5

master builds could easily deadlock in certain unit tests. (And

similarly for libreoffice-5-0.)

There are a handful of tinderboxes that are suspiciously silent since

about that time...

Context

Tinderbox owners: deadlock, beware · Stephan Bergmann

Re: Tinderbox owners: deadlock, beware · Norbert Thiebaud

Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.