如果我理解C编译器如何处理局部变量,那么IsShutdownInProgress()不需要任何锁定,因为shutdownInProgress静态变量将被放置在堆栈上.我对么?
- class MyClass
- {
- private:
- // Irrelevant code commented away
- static pthread_mutex_t mutex;
- static bool shutdownInProgress;
- public:
- static void ShutdownIsInProgress()
- {
- pthread_mutex_lock(mutex);
- shutdownInProgress = true;
- pthread_mutex_unlock(mutex);
- }
- static bool IsShutdownInProgress()
- {
- // pthread_mutex_lock(mutex);
- // pthread_mutex_unlock(mutex);
- return shutdownInProgress;
- }
- }
解决方法
Am I correct?
不会.这会将其复制回来;但读取它以使该副本不同步将导致数据竞争,具有未定义的行为.您需要在锁定互斥锁的情况下制作本地副本:
- static bool IsShutdownInProgress()
- {
- pthread_mutex_lock(mutex);
- bool result = shutdownInProgress;
- pthread_mutex_unlock(mutex);
- return result;
- }
或者,使用不易出错的RAII锁定类型:
- static bool IsShutdownInProgress()
- {
- lock_guard lock(mutex);
- return shutdownInProgress;
- }
在C 11中,您可以考虑使用std :: atomic< bool>为了更方便,也许更有效,从多个线程访问简单类型.