Re: [Boost-bugs] [Boost C++ Libraries] #6200: patch to have condition_variable and mutex error better handle EINTR

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #6200: patch to have condition_variable and mutex error better handle EINTR
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2011-12-02 20:19:59

#6200: patch to have condition_variable and mutex error better handle EINTR
--------------------------------------+-------------------------------------
  Reporter:  blentz@…                 |       Owner:  viboes                                   
      Type:  Patches                  |      Status:  assigned                                 
 Milestone:  To Be Determined         |   Component:  thread                                   
   Version:  Boost Development Trunk  |    Severity:  Problem                                  
Resolution:                           |    Keywords:  1.49 thread mutex eintr phusion passenger
--------------------------------------+-------------------------------------
Changes (by viboes):

 * cc: viboes (added)
  * keywords:  thread mutex eintr phusion passenger => 1.49 thread mutex
               eintr phusion passenger
  * status:  new => assigned
  * owner:  anthonyw => viboes

Comment:

 This part is not correct


 {{{
         55      res=pthread_cond_wait(&cond,&internal_mutex);

         55      int ret;
         56      do {
         57        ret =
 pthread_cond_wait(&cond,m.mutex()->native_handle());
         58      } while (ret == EINTR);

 }}}

 Note that before the mutex used was internal_mutex. Are you sure that this
 patch is working for your applications?

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/6200#comment:1>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.