X-Git-Url: https://git.cworth.org/git?a=blobdiff_plain;f=common%2Fos_thread.hpp;h=f3ae210053da2b08ef711f516a92a3be433860b1;hb=dfd413a5f54bd450850b5e84886949bcdf85b1e7;hp=6c0b488c0152474a031a82839b0c767777714fdd;hpb=bcb5850afb381e4814d247917311494d7edd2e91;p=apitrace diff --git a/common/os_thread.hpp b/common/os_thread.hpp index 6c0b488..f3ae210 100644 --- a/common/os_thread.hpp +++ b/common/os_thread.hpp @@ -24,9 +24,9 @@ **************************************************************************/ /* - * Simple OS abstraction. + * OS native thread abstraction. * - * Mimics C++11 / boost threads. + * Mimics C++11 threads. */ #ifndef _OS_THREAD_HPP_ @@ -40,6 +40,12 @@ #endif +/* + * This feature is not supported on Windows XP + */ +#define USE_WIN32_CONDITION_VARIABLES 0 + + /** * Compiler TLS. * @@ -47,13 +53,10 @@ * - http://gcc.gnu.org/onlinedocs/gcc-4.6.3/gcc/Thread_002dLocal.html * - http://msdn.microsoft.com/en-us/library/9w1sdazb.aspx */ -#if defined(_MSC_VER) -# define thread_specific __declspec(thread) -#elif defined(__GNUC__) -# define thread_specific __thread +#if defined(HAVE_COMPILER_TLS) +# define OS_THREAD_SPECIFIC_PTR(_type) HAVE_COMPILER_TLS _type * #else -# define thread_specific -# error "Unsupported compiler" +# define OS_THREAD_SPECIFIC_PTR(_type) os::thread_specific_ptr< _type > #endif @@ -203,16 +206,31 @@ namespace os { */ class condition_variable { - public: + private: #ifdef _WIN32 +# if USE_WIN32_CONDITION_VARIABLES + // XXX: Only supported on Vista an higher. Not yet supported by WINE. typedef CONDITION_VARIABLE native_handle_type; + native_handle_type _native_handle; +#else + // http://www.cs.wustl.edu/~schmidt/win32-cv-1.html + LONG cWaiters; + HANDLE hEvent; +#endif #else typedef pthread_cond_t native_handle_type; + native_handle_type _native_handle; #endif + public: condition_variable() { #ifdef _WIN32 +# if USE_WIN32_CONDITION_VARIABLES InitializeConditionVariable(&_native_handle); +# else + cWaiters = 0; + hEvent = CreateEvent(NULL, FALSE, FALSE, NULL); +# endif #else pthread_cond_init(&_native_handle, NULL); #endif @@ -220,7 +238,11 @@ namespace os { ~condition_variable() { #ifdef _WIN32 +# if USE_WIN32_CONDITION_VARIABLES /* No-op */ +# else + CloseHandle(hEvent); +# endif #else pthread_cond_destroy(&_native_handle); #endif @@ -229,7 +251,13 @@ namespace os { inline void signal(void) { #ifdef _WIN32 +# if USE_WIN32_CONDITION_VARIABLES WakeConditionVariable(&_native_handle); +# else + if (cWaiters) { + SetEvent(hEvent); + } +# endif #else pthread_cond_signal(&_native_handle); #endif @@ -239,21 +267,22 @@ namespace os { wait(unique_lock & lock) { mutex::native_handle_type & mutex_native_handle = lock.mutex()->native_handle(); #ifdef _WIN32 - /* FIXME */ +# if USE_WIN32_CONDITION_VARIABLES SleepConditionVariableCS(&_native_handle, &mutex_native_handle, INFINITE); +# else + InterlockedIncrement(&cWaiters); + LeaveCriticalSection(&mutex_native_handle); + WaitForSingleObject(hEvent, INFINITE); + EnterCriticalSection(&mutex_native_handle); + InterlockedDecrement(&cWaiters); +# endif #else pthread_cond_wait(&_native_handle, &mutex_native_handle); #endif } - - protected: - native_handle_type _native_handle; }; - /** - * Same interface as boost::thread_specific_ptr. - */ template class thread_specific_ptr { @@ -262,10 +291,6 @@ namespace os { DWORD dwTlsIndex; #else pthread_key_t key; - - static void destructor(void *ptr) { - delete static_cast(ptr); - } #endif public: @@ -273,7 +298,7 @@ namespace os { #ifdef _WIN32 dwTlsIndex = TlsAlloc(); #else - pthread_key_create(&key, &destructor); + pthread_key_create(&key, NULL); #endif } @@ -285,7 +310,8 @@ namespace os { #endif } - T* get(void) const { + inline T * + get(void) const { void *ptr; #ifdef _WIN32 ptr = TlsGetValue(dwTlsIndex); @@ -295,32 +321,27 @@ namespace os { return static_cast(ptr); } - T* operator -> (void) const + inline + operator T * (void) const { return get(); } - T& operator * (void) const + inline T * + operator -> (void) const { - return *get(); + return get(); } - void reset(T* new_value=0) { - T * old_value = get(); + inline T * + operator = (T * new_value) + { set(new_value); - if (old_value) { - delete old_value; - } + return new_value; } - T* release (void) { - T * old_value = get(); - set(0); - return old_value; - } - -private: - void set(T* new_value) { + inline void + set(T* new_value) { #ifdef _WIN32 TlsSetValue(dwTlsIndex, new_value); #else @@ -341,17 +362,43 @@ private: typedef pthread_t native_handle_type; #endif + inline + thread() : + _native_handle(0) + { + } + + inline + thread(const thread &other) : + _native_handle(other._native_handle) + { + } + + inline + ~thread() { + } + template< class Function, class Arg > explicit thread( Function& f, Arg arg ) { #ifdef _WIN32 - /* FIXME */ DWORD id = 0; _native_handle = CreateThread(NULL, 0, (LPTHREAD_START_ROUTINE)f, (LPVOID)arg, 0, &id); #else - pthread_create(&_native_handle, NULL, f, arg); + pthread_create(&_native_handle, NULL, (void *(*) (void *))f, (void *)arg); #endif } + inline thread & + operator =(const thread &other) { + _native_handle = other._native_handle; + return *this; + } + + inline bool + joinable(void) const { + return _native_handle != 0; + } + inline void join() { #ifdef _WIN32