Boost C++ Libraries: Ticket #13212: Windows: condition_variable::notify_one consumes semaphore handles

attachment set

henri.hakonen@… — Thu, 14 Sep 2017 16:37:35 GMT

attachment → condition_variable_test.cpp

Test program

attachment set

henri.hakonen@… — Thu, 14 Sep 2017 16:38:20 GMT

attachment → handles.png

Handles

owner, status, summary changed

viboes — Sat, 16 Sep 2017 15:23:19 GMT

owner changed from Anthony Williams to viboes
status new → assigned
summary condition_variable::notify_one consumes semaphore handles → Windows: condition_variable::notify_one consumes semaphore handles

I've no access to a Windows machine. Please, could you count the number of notify_one per second?

viboes — Sat, 16 Sep 2017 16:41:55 GMT

See #11457

harris.pc@… — Fri, 18 May 2018 06:15:11 GMT

A note, perhaps not useful, but regardless:

I thought I had a handle leak in my program on windows 7/10, so I used this test code to try out this particular possibility.

I tested the attached code test with Boost 1.60.0 + MSVC2010, and it worked as expected - ie no handle leak.

And now I can't seem to reproduce the problem in my program. So I can't be sure if it was just a windows quirk that has magically gone away.

Sorry I don't have something more useful to share

henri.hakonen@… — Fri, 18 May 2018 14:30:29 GMT

It seems that problem is gone now when I test with boost 1.67.0. There is no leak anymore.

I think that this problem was specific to boost 1.65.0. My quick fix was to comment one line in condition_variable.hpp. (here X: (upper) is my version of 1.65.0 and E: (lower) is original boost.

X:\boost\thread\win32>fc condition_variable.hpp e:\prj\boost_git\libs\thread\include\boost\thread\win32\condition_variable.hpp Comparing files condition_variable.hpp and E:\PRJ\BOOST_GIT\LIBS\THREAD\INCLUDE\BOOST\THREAD\WIN32\CONDITION_VARIABLE.HPP * condition_variable.hpp

              // do it here to avoid throwing on the destructor
              //entry->remove_waiter();
              locker.lock();

* E:\PRJ\BOOST_GIT\LIBS\THREAD\INCLUDE\BOOST\THREAD\WIN32\CONDITION_VARIABLE.HPP

              // do it here to avoid throwing on the destructor
              entry->remove_waiter();
              locker.lock();

status changed; resolution set; milestone deleted

viboes — Tue, 07 Aug 2018 12:24:31 GMT

status assigned → closed
resolution → fixed
milestone To Be Determined

Thanks for the information.

Already fixed.