本页说明bthread下使用pthread-local可能会导致的问题。bthread-local的使用方法见这里

thread-local问题

调用阻塞的bthread函数后,所在的pthread很可能改变,这使pthread_getspecificgcc __thread和c++11thread_local变量,pthread_self()等的值变化了,如下代码的行为是不可预计的:

  1. thread_local SomeObject obj;
  2. ...
  3. SomeObject* p = &obj;
  4. p->bar();
  5. bthread_usleep(1000);
  6. p->bar();

bthread_usleep之后,该bthread很可能身处不同的pthread,这时p指向了之前pthread的thread_local变量,继续访问p的结果无法预计。这种使用模式往往发生在用户使用线程级变量传递业务变量的情况。为了防止这种情况,应该谨记:

  • 不使用线程级变量传递业务数据。这是一种槽糕的设计模式,依赖线程级数据的函数也难以单测。判断是否滥用:如果不使用线程级变量,业务逻辑是否还能正常运行?线程级变量应只用作优化手段,使用过程中不应直接或间接调用任何可能阻塞的bthread函数。比如使用线程级变量的tcmalloc就不会和bthread有任何冲突。
  • 如果一定要(在业务中)使用线程级变量,使用bthread_key_create和bthread_getspecific。

gcc4下的errno问题

gcc4会优化标记为attribute((const))的函数,这个标记大致指只要参数不变,输出就不会变。所以当一个函数中以相同参数出现多次时,gcc4会合并为一次。比如在我们的系统中errno是内容为*__errno_location()的宏,这个函数的签名是:

  1. /* Function to get address of global `errno' variable. */
  2. extern int *__errno_location (void) __THROW __attribute__ ((__const__));

由于此函数被标记为const,且没有参数,当你在一个函数中调用多次errno时,可能只有第一次才调用__errno_location(),而之后只是访问其返回的int。在pthread中这没有问题,因为返回的int是thread-local的,一个给定的pthread中是不会变化的。但是在bthread中,这是不成立的,因为一个bthread很可能在调用一些函数后跑到另一个pthread去,如果gcc4做了类似的优化,即一个函数内所有的errno都替换为第一次调用返回的int*,这中间bthread又切换了pthread,那么可能会访问之前pthread的errno,从而造成未定义行为。

比如下文是一种errno的使用场景:

  1. Use errno ... (original pthread)
  2. bthread functions that may switch to another pthread.
  3. Use errno ... (another pthread)

我们期望看到的行为:

  1. Use *__errno_location() ... - the thread-local errno of original pthread
  2. bthread may switch another pthread ...
  3. Use *__errno_location() ... - the thread-local errno of another pthread

使用gcc4时的实际行为:

  1. int* p= __errno_location();
  2. Use *p ... - the thread-local errno of original pthread
  3. bthread context switches ...
  4. Use *p ... - still the errno of original pthread, undefined behavior!!

严格地说这个问题不是gcc4导致的,而是glibc给errno_location的签名不够准确,一个返回thread-local指针的函数依赖于段寄存器(TLS的一般实现方式),这怎么能算const呢?由于我们还未找到覆盖errno_location的方法,所以这个问题目前实际的解决方法是:

务必在直接或间接使用bthread的项目的gcc编译选项中添加-Dconst=,即把const定义为空,避免gcc4做相关优化。

const定义为空对程序其他部分的影响几乎为0。另外如果你没有直接使用errno(即你的项目中没有出现errno),或使用的是gcc3.4,即使没有定义-Dconst=,程序的正确性也不会受影响,但为了防止未来可能的问题,我们强烈建议加上。

需要说明的是,和errno类似,pthreadself也有类似的问题,不过一般pthreadself除了打日志没有其他用途,影响面较小,在-D__const=后pthread_self也会正常。