才发现pthread_cond_wait原来这么烂

太平绅士

不能unlock 类型为PTHREAD_MUTEX_RECURSIVE的mutex.

太平绅士

只有被仅 lock 过一次的 recursive mutex, 可以由pthread_cond_wait来unlock。.

alexhappy

原帖由 太平绅士 于 2009-2-5 14:36 发表
不能unlock 类型为PTHREAD_MUTEX_RECURSIVE的mutex.

you sure?

alexhappy

原帖由 太平绅士 于 2009-2-5 14:39 发表
只有被仅 lock 过一次的 recursive mutex, 可以由pthread_cond_wait来unlock。.

那是当然啦，对recursive mutex加了几次锁就应当unlock几次，很符合常理嘛

太平绅士

原帖由 alexhappy 于 2009-2-5 14:42 发表

you sure?

sure. 我有测试代码
或者pthread_condattr有什么属性可以设？

1. 
   
2. #include <unistd.h>
   
3. #include <stdio.h>
   
4. #include <pthread.h>
   
5. 
   
6. pthread_mutex_t mutex;
   
7. pthread_cond_t cond;
   
8. 
   
9. 
   
10. void *thread( void *arg )
    
11. {
    
12.         printf( "Entering thread\n" );
    
13. 
    
14.         pthread_mutex_lock( &mutex );
    
15.         pthread_mutex_lock( &mutex );
    
16. 
    
17.         printf( "Before condition wait\n" );
    
18.        
    
19.         pthread_cond_wait( &cond, &mutex );
    
20. 
    
21.         printf( "After condition wait\n" );
    
22. 
    
23.         pthread_mutex_unlock( &mutex );
    
24.         pthread_mutex_unlock( &mutex );
    
25. 
    
26.         return NULL;       
    
27. }
    
28. 
    
29. 
    
30. int main( )
    
31. {
    
32.         pthread_t thread_id;
    
33. 
    
34.         /* Create recursive mutex */
    
35.         pthread_mutexattr_t attr;
    
36.         pthread_mutexattr_init( &attr );
    
37.         pthread_mutexattr_settype( &attr, PTHREAD_MUTEX_RECURSIVE );
    
38.         pthread_mutex_init( &mutex, &attr );
    
39.         pthread_mutexattr_destroy( &attr );
    
40. 
    
41.         pthread_cond_init( &cond, NULL );
    
42. 
    
43.         /* Create a thread to wait on the condition */
    
44.         pthread_create( &thread_id, NULL, &thread, NULL );
    
45. 
    
46.         sleep( 1 );
    
47.         printf( "Before mutex Lock\n" );
    
48.         pthread_mutex_lock( &mutex );
    
49. 
    
50.         printf("After mutex Lock\n");
    
51.         pthread_cond_signal( &cond );
    
52. 
    
53.         pthread_mutex_unlock( &mutex );
    
54. 
    
55.         pthread_join( thread_id, NULL );
    
56. 
    
57.         pthread_cond_destroy( &cond );
    
58.         pthread_mutex_destroy( &mutex );
    
59. 
    
60.         return 0;
    
61. }
    
62. 
    

复制代码

太平绅士

原帖由 alexhappy 于 2009-2-5 14:43 发表

那是当然啦，对recursive mutex加了几次锁就应当unlock几次，很符合常理嘛

为符合condition_wait的目的，condition_wait应该将mutex完全unlock，以便其他线程可以lock.
我认为pthread_cond_wait这样做完全是设计错误。

alexhappy

原帖由 太平绅士 于 2009-2-5 14:48 发表


为符合condition_wait的目的，condition_wait应该将mutex完全unlock，以便其他线程可以lock.
我认为pthread_cond_wait这样做完全是设计错误。

那么你也没有考虑其它加锁的地方呢？
这个recursive lock加了锁的地方现在全被你用一个pthread_cond_wait给解了，那么它们并不知道啊，等于是锁了也白锁

JohnBull

原帖由 太平绅士 于 2009-2-5 14:48 发表


为符合condition_wait的目的，condition_wait应该将mutex完全unlock，以便其他线程可以lock.
我认为pthread_cond_wait这样做完全是设计错误。

无理

samon_fu

设计理念问题，LZ觉得不对可以反馈给nptl开发组的。

不过这种问题，我觉得差下posix关于线程方面的标准可能好些。