Boost C++ Libraries: Ticket #6200: patch to have condition_variable and mutex error better handle EINTR

attachment set

blentz@… — Fri, 02 Dec 2011 15:49:54 GMT

attachment → thread_eintr.patch

patch to correct for receiving EINTR in condition_variable and mutex.

keywords, status, owner changed; cc set

viboes — Fri, 02 Dec 2011 20:19:59 GMT

cc viboes added
keywords 1.49 added
status new → assigned
owner changed from Anthony Williams to viboes

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?

milestone changed

viboes — Sat, 03 Dec 2011 10:10:46 GMT

milestone To Be Determined → Boost 1.49.0

Committed in trunk revision [75781] condition variable part.

blentz@… — Mon, 05 Dec 2011 15:11:28 GMT

It looks like only the condition_variable and condition_variable_fwd portion of the patch was committed.

The changes to mutex are missing in revision 75781.

Was this done by choice or is it an oversight?

viboes — Wed, 07 Dec 2011 23:16:15 GMT

Committed in trunk revision [75882] mutex part

status changed; resolution set

viboes — Sat, 07 Jan 2012 22:56:22 GMT

status assigned → closed
resolution → fixed

Committed in release at [76291]