Boost C++ Libraries: Ticket #8136: [windows] boost::this_thread::sleep_for() sleeps longer than it should in Windows

owner, status changed

viboes — Thu, 21 Feb 2013 18:23:46 GMT

owner changed from Anthony Williams to viboes
status new → assigned

I have no access to a windows machine now. Could you do the test for 5 seconds? and 100ms?

Thu, 21 Feb 2013 21:18:32 GMT

Same behaviour - for 5 seconds sleeps 10 secs, for 100ms sleeps 200ms.

milestone changed

viboes — Sun, 03 Mar 2013 14:08:37 GMT

milestone To Be Determined → Boost 1.54.0

Please, could you check if the following update works for you?

Committed in trunk revision [83282].

Mon, 04 Mar 2013 15:28:02 GMT

I cannot link without BOOST_USE_WINDOWS_H defined.

With BOOST_USE_WINDOWS_H defined, it works ok.

Shouldn't be the test in thread/win32/thread_primitives.hpp:21

#if _WIN32_WINNT >= 0x0600

instead of

#if WINNT >= 0x0600

MS Visual Studio (IDE) doesn't know WINNT, but is happy with _WIN32_WINNT

viboes — Mon, 04 Mar 2013 17:48:53 GMT

Replying to Ondrej Siler <ondrej.siler@…>:

I cannot link without BOOST_USE_WINDOWS_H defined.

With BOOST_USE_WINDOWS_H defined, it works ok.

Shouldn't be the test in thread/win32/thread_primitives.hpp:21
#if _WIN32_WINNT >= 0x0600
instead of
#if WINNT >= 0x0600
MS Visual Studio (IDE) doesn't know WINNT, but is happy with _WIN32_WINNT

I suspect that you are talking of #8070, isn't it?

Mon, 04 Mar 2013 19:17:55 GMT

Yes, you are right. According to MSDN: "To compile an application that uses this function, define _WIN32_WINNT as 0x0600 or later."

viboes — Sat, 09 Mar 2013 16:05:48 GMT

Committed revision [83294][83318].

status changed; resolution set

viboes — Sat, 23 Mar 2013 01:54:12 GMT

status assigned → closed
resolution → fixed

Committed revision [83525].

anonymous — Sun, 14 Sep 2014 10:07:53 GMT

on boost 1.56 this bug is back. sleeping for 1000 microseconds sleeps for about 2,000 microseconds. sleeping for 1 millisecond sleeps for about 2 milliseconds. sleeping for 5 seconds sleeps for 5 seconds.

Micro and milliseconds are doubling, but seconds are not doubling.

status changed; resolution deleted

viboes — Sun, 14 Sep 2014 13:57:48 GMT

status closed → reopened
resolution fixed

Niall, do you think this could be related to you improvement with GetTickCount64 or the PR related to WinRT?

summary changed

viboes — Sun, 14 Sep 2014 20:23:31 GMT

summary boost::this_thread::sleep_for() sleeps longer than it should in Windows → [windows] boost::this_thread::sleep_for() sleeps longer than it should in Windows

milestone changed

viboes — Tue, 23 Sep 2014 00:18:12 GMT

milestone Boost 1.54.0 → Boost 1.57.0

owner, status changed

viboes — Tue, 23 Sep 2014 00:20:45 GMT

owner changed from viboes to Niall Douglas
status reopened → new

status changed; resolution set

viboes — Fri, 26 Sep 2014 06:06:23 GMT

status new → closed
resolution → fixed

status changed; resolution deleted

viboes — Fri, 26 Sep 2014 06:09:52 GMT

status closed → reopened
resolution fixed

Niall Douglas — Sat, 04 Oct 2014 19:35:35 GMT

I ran this test on Win8.1 x86 using VS2013 release x86 build on Boost trunk (1.57):

1015707

I ran it a few more times, all were minor variations around 1015xxx.

I'd say this bug is stale.

Niall

status changed; resolution set; milestone deleted

viboes — Sat, 11 Oct 2014 05:27:02 GMT

status reopened → closed
resolution → worksforme
milestone Boost 1.57.0