Boost C++ Libraries: Ticket #7238: this_thread::sleep_for() does not respond to interrupt()

Nick F — Thu, 16 Aug 2012 21:07:14 GMT

I am also observing the issue on Ubuntu 12.04. See the test reproducer below.

#include <iostream>
#include <boost/thread.hpp>
using namespace boost;
using namespace boost::chrono;
void f()
{
    try
    {
        std::cout << "Starting sleep in thread" << std::endl;
        while(true)
        {
            this_thread::sleep_for(seconds(60));
        }
    }
    catch(const thread_interrupted&)
    {
        std::cout << "Thread interrupted." << std::endl;
    }
}
int main(int argc, char** argv)
{
    thread t(f);
    t.interrupt();
    t.join();
    std::cout << "Joined with thread." << std::endl;
    return 0;
}

The problem seems to be due to the following. On platforms where BOOST_HAS_NANOSLEEP is defined, a nanosleep-based implementation is used for sleep_for (see thread/src/pthread/thread.cpp), which presumably does not allow the interruption mechanisms to work. Commenting out the BOOST_HAS_NANOSLEEP section in the above file and allowing the condition_variable based implementation to be used causes the above example to run as expected.

severity changed

viboes — Fri, 17 Aug 2012 14:37:51 GMT

severity Showstopper → Problem

Thanks for catching this.

I don't think this is a Showstopper as you can add a call to this_thread::interruption_point() together with the call to sleep_for, isn't it?

viboes — Sat, 18 Aug 2012 08:17:03 GMT

Replying to viboes:

Thanks for catching this.

I don't think this is a Showstopper as you can add a call to this_thread::interruption_point() together with the call to sleep_for, isn't it?

Forget the last comment.

attachment set

viboes — Sat, 18 Aug 2012 09:27:26 GMT

attachment → 7238.patch

Please, could you try this patch?

milestone changed

viboes — Sat, 18 Aug 2012 11:27:33 GMT

milestone To Be Determined → Boost 1.52.0

Committed in trunk revision 80076.

owner, status changed

viboes — Sat, 18 Aug 2012 11:30:56 GMT

owner changed from Anthony Williams to viboes
status new → assigned

status changed; resolution set

viboes — Sun, 09 Sep 2012 18:09:52 GMT

status assigned → closed
resolution → fixed

Merged to release [80450].