X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=include%2Fvlc_threads_funcs.h;h=2823aa8385214c3458ab2b312324ec6406b8535a;hb=220fb0d5e0e280fab999907d8cab87bda89e49e8;hp=ad28a8ea90e17643f6c4cf0a2030277d4d67adf8;hpb=955b873df37a3f0beb70c7bd0da1a0471ba206f0;p=vlc diff --git a/include/vlc_threads_funcs.h b/include/vlc_threads_funcs.h index ad28a8ea90..2823aa8385 100644 --- a/include/vlc_threads_funcs.h +++ b/include/vlc_threads_funcs.h @@ -2,7 +2,7 @@ * vlc_threads_funcs.h : threads implementation for the VideoLAN client * This header provides a portable threads implementation. ***************************************************************************** - * Copyright (C) 1999, 2002 the VideoLAN team + * Copyright (C) 1999-2007 the VideoLAN team * $Id$ * * Authors: Jean-Marc Dressler @@ -25,20 +25,33 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA. *****************************************************************************/ +#if !defined( __LIBVLC__ ) + #error You are not libvlc or one of its plugins. You cannot include this file +#endif + +#ifndef _VLC_THREADFUNCS_H_ +#define _VLC_THREADFUNCS_H_ + /***************************************************************************** * Function definitions *****************************************************************************/ -VLC_EXPORT( int, __vlc_threads_init, ( vlc_object_t * ) ); -VLC_EXPORT( int, __vlc_threads_end, ( vlc_object_t * ) ); +VLC_EXPORT( void, __vlc_threads_error, ( vlc_object_t *) ); VLC_EXPORT( int, __vlc_mutex_init, ( vlc_object_t *, vlc_mutex_t * ) ); +VLC_EXPORT( int, __vlc_mutex_init_recursive, ( vlc_object_t *, vlc_mutex_t * ) ); VLC_EXPORT( int, __vlc_mutex_destroy, ( const char *, int, vlc_mutex_t * ) ); VLC_EXPORT( int, __vlc_cond_init, ( vlc_object_t *, vlc_cond_t * ) ); VLC_EXPORT( int, __vlc_cond_destroy, ( const char *, int, vlc_cond_t * ) ); +VLC_EXPORT( int, __vlc_threadvar_create, (vlc_object_t *, vlc_threadvar_t * ) ); VLC_EXPORT( int, __vlc_thread_create, ( vlc_object_t *, const char *, int, const char *, void * ( * ) ( void * ), int, vlc_bool_t ) ); VLC_EXPORT( int, __vlc_thread_set_priority, ( vlc_object_t *, const char *, int, int ) ); VLC_EXPORT( void, __vlc_thread_ready, ( vlc_object_t * ) ); VLC_EXPORT( void, __vlc_thread_join, ( vlc_object_t *, const char *, int ) ); +/***************************************************************************** + * vlc_threads_error: Signalize an error in the threading system + *****************************************************************************/ +#define vlc_threads_error( P_THIS ) \ + __vlc_threads_error( VLC_OBJECT(P_THIS) ) /***************************************************************************** * vlc_threads_init: initialize threads system @@ -58,19 +71,33 @@ VLC_EXPORT( void, __vlc_thread_join, ( vlc_object_t *, const char *, int ) ); #define vlc_mutex_init( P_THIS, P_MUTEX ) \ __vlc_mutex_init( VLC_OBJECT(P_THIS), P_MUTEX ) +/***************************************************************************** + * vlc_mutex_init: initialize a recursive mutex (Don't use it) + *****************************************************************************/ +#define vlc_mutex_init_recursive( P_THIS, P_MUTEX ) \ + __vlc_mutex_init_recursive( VLC_OBJECT(P_THIS), P_MUTEX ) + /***************************************************************************** * vlc_mutex_lock: lock a mutex *****************************************************************************/ #define vlc_mutex_lock( P_MUTEX ) \ __vlc_mutex_lock( __FILE__, __LINE__, P_MUTEX ) +#if defined( PTHREAD_COND_T_IN_PTHREAD_H ) +static inline unsigned long int CAST_PTHREAD_TO_INT (pthread_t th) +{ + union { pthread_t th; unsigned long int i; } v = { }; + v.th = th; + return v.i; +} +#endif + static inline int __vlc_mutex_lock( const char * psz_file, int i_line, vlc_mutex_t * p_mutex ) { int i_result; /* In case of error : */ - int i_thread = -1; - const char * psz_error = ""; + unsigned long int i_thread = 0; #if defined( PTH_INIT_IN_PTH_H ) i_result = ( pth_mutex_acquire( &p_mutex->mutex, FALSE, NULL ) == FALSE ); @@ -108,11 +135,14 @@ static inline int __vlc_mutex_lock( const char * psz_file, int i_line, } #elif defined( PTHREAD_COND_T_IN_PTHREAD_H ) +# define vlc_assert_locked( m ) \ + assert (pthread_mutex_lock (&((m)->mutex)) == EDEADLK) + i_result = pthread_mutex_lock( &p_mutex->mutex ); if ( i_result ) { - i_thread = (int)pthread_self(); - psz_error = strerror(i_result); + i_thread = CAST_PTHREAD_TO_INT(pthread_self()); + errno = i_result; } #elif defined( HAVE_CTHREADS_H ) @@ -124,12 +154,17 @@ static inline int __vlc_mutex_lock( const char * psz_file, int i_line, if( i_result ) { msg_Err( p_mutex->p_this, - "thread %u: mutex_lock failed at %s:%d (%d:%s)", - i_thread, psz_file, i_line, i_result, psz_error ); + "thread %li: mutex_lock failed at %s:%d (%d:%m)", + i_thread, psz_file, i_line, i_result ); + vlc_threads_error( p_mutex->p_this ); } return i_result; } +#ifndef vlc_assert_locked +# define vlc_assert_locked( m ) (void)0 +#endif + /***************************************************************************** * vlc_mutex_unlock: unlock a mutex *****************************************************************************/ @@ -141,8 +176,7 @@ static inline int __vlc_mutex_unlock( const char * psz_file, int i_line, { int i_result; /* In case of error : */ - int i_thread = -1; - const char * psz_error = ""; + unsigned long int i_thread = 0; #if defined( PTH_INIT_IN_PTH_H ) i_result = ( pth_mutex_release( &p_mutex->mutex ) == FALSE ); @@ -184,8 +218,8 @@ static inline int __vlc_mutex_unlock( const char * psz_file, int i_line, i_result = pthread_mutex_unlock( &p_mutex->mutex ); if ( i_result ) { - i_thread = (int)pthread_self(); - psz_error = strerror(i_result); + i_thread = CAST_PTHREAD_TO_INT(pthread_self()); + errno = i_result; } #elif defined( HAVE_CTHREADS_H ) @@ -197,8 +231,9 @@ static inline int __vlc_mutex_unlock( const char * psz_file, int i_line, if( i_result ) { msg_Err( p_mutex->p_this, - "thread %u: mutex_unlock failed at %s:%d (%d:%s)", - i_thread, psz_file, i_line, i_result, psz_error ); + "thread %li: mutex_unlock failed at %s:%d (%d:%m)", + i_thread, psz_file, i_line, i_result ); + vlc_threads_error( p_mutex->p_this ); } return i_result; @@ -227,8 +262,7 @@ static inline int __vlc_cond_signal( const char * psz_file, int i_line, { int i_result; /* In case of error : */ - int i_thread = -1; - const char * psz_error = ""; + unsigned long int i_thread = 0; #if defined( PTH_INIT_IN_PTH_H ) i_result = ( pth_cond_notify( &p_condvar->cond, FALSE ) == FALSE ); @@ -246,6 +280,11 @@ static inline int __vlc_cond_signal( const char * psz_file, int i_line, * by a mutex. This will prevent another thread from stealing the signal */ if( !p_condvar->semaphore ) { + /* + ** PulseEvent() only works if none of the waiting threads is suspended. + ** this is particularily problematic under a debug session. + ** as documented in http://support.microsoft.com/kb/q173260/ + */ PulseEvent( p_condvar->event ); } else if( p_condvar->i_win9x_cv == 1 ) @@ -322,8 +361,8 @@ static inline int __vlc_cond_signal( const char * psz_file, int i_line, i_result = pthread_cond_signal( &p_condvar->cond ); if ( i_result ) { - i_thread = (int)pthread_self(); - psz_error = strerror(i_result); + i_thread = CAST_PTHREAD_TO_INT(pthread_self()); + errno = i_result; } #elif defined( HAVE_CTHREADS_H ) @@ -339,8 +378,9 @@ static inline int __vlc_cond_signal( const char * psz_file, int i_line, if( i_result ) { msg_Err( p_condvar->p_this, - "thread %u: cond_signal failed at %s:%d (%d:%s)", - i_thread, psz_file, i_line, i_result, psz_error ); + "thread %li: cond_signal failed at %s:%d (%d:%m)", + i_thread, psz_file, i_line, i_result ); + vlc_threads_error( p_condvar->p_this ); } return i_result; @@ -357,8 +397,7 @@ static inline int __vlc_cond_wait( const char * psz_file, int i_line, { int i_result; /* In case of error : */ - int i_thread = -1; - const char * psz_error = ""; + unsigned long int i_thread = 0; #if defined( PTH_INIT_IN_PTH_H ) i_result = ( pth_cond_await( &p_condvar->cond, &p_mutex->mutex, NULL ) @@ -485,23 +524,21 @@ static inline int __vlc_cond_wait( const char * psz_file, int i_line, # ifdef DEBUG /* In debug mode, timeout */ - struct timeval now; - struct timespec timeout; - - gettimeofday( &now, NULL ); - timeout.tv_sec = now.tv_sec + THREAD_COND_TIMEOUT; - timeout.tv_nsec = now.tv_usec * 1000; + struct timespec timeout = { + (mdate() / 1000000) + THREAD_COND_TIMEOUT, + 0 /* 1Hz precision is sufficient here :-) */ + }; i_result = pthread_cond_timedwait( &p_condvar->cond, &p_mutex->mutex, &timeout ); if( i_result == ETIMEDOUT ) { - /* People keep pissing me off with this. --Meuuh */ + errno = ETIMEDOUT; msg_Dbg( p_condvar->p_this, - "thread %u: secret message triggered " - "at %s:%d (%s)", (int)pthread_self(), - psz_file, i_line, strerror(i_result) ); + "thread %li: possible condition deadlock " + "at %s:%d (%m)", CAST_PTHREAD_TO_INT(pthread_self()), + psz_file, i_line ); i_result = pthread_cond_wait( &p_condvar->cond, &p_mutex->mutex ); } @@ -512,8 +549,8 @@ static inline int __vlc_cond_wait( const char * psz_file, int i_line, if ( i_result ) { - i_thread = (int)pthread_self(); - psz_error = strerror(i_result); + i_thread = CAST_PTHREAD_TO_INT(pthread_self()); + errno = i_result; } #elif defined( HAVE_CTHREADS_H ) @@ -525,19 +562,241 @@ static inline int __vlc_cond_wait( const char * psz_file, int i_line, if( i_result ) { msg_Err( p_condvar->p_this, - "thread %u: cond_wait failed at %s:%d (%d:%s)", - i_thread, psz_file, i_line, i_result, psz_error ); + "thread %li: cond_wait failed at %s:%d (%d:%m)", + i_thread, psz_file, i_line, i_result ); + vlc_threads_error( p_condvar->p_this ); } return i_result; } + +/***************************************************************************** + * vlc_cond_timedwait: wait until condition completion or expiration + ***************************************************************************** + * Returns 0 if object signaled, an error code in case of timeout or error. + *****************************************************************************/ +#define vlc_cond_timedwait( P_COND, P_MUTEX, DEADLINE ) \ + __vlc_cond_timedwait( __FILE__, __LINE__, P_COND, P_MUTEX, DEADLINE ) + +static inline int __vlc_cond_timedwait( const char * psz_file, int i_line, + vlc_cond_t *p_condvar, + vlc_mutex_t *p_mutex, + mtime_t deadline ) +{ + int i_res; + unsigned long int i_thread = 0; + +#if defined( PTH_INIT_IN_PTH_H ) +# error Unimplemented +#elif defined( ST_INIT_IN_ST_H ) +# error Unimplemented +#elif defined( UNDER_CE ) + mtime_t delay_ms = (deadline - mdate())/1000; + + DWORD result; + if( delay_ms < 0 ) + delay_ms = 0; + + p_condvar->i_waiting_threads++; + LeaveCriticalSection( &p_mutex->csection ); + result = WaitForSingleObject( p_condvar->event, delay_ms ); + p_condvar->i_waiting_threads--; + + /* Reacquire the mutex before returning. */ + vlc_mutex_lock( p_mutex ); + + i_res = (int)result; + +#elif defined( WIN32 ) + DWORD result; + + mtime_t delay_ms = (deadline - mdate())/1000; + if( delay_ms < 0 ) + delay_ms = 0; + + if( !p_condvar->semaphore ) + { + /* Increase our wait count */ + p_condvar->i_waiting_threads++; + + if( p_mutex->mutex ) + { + /* It is only possible to atomically release the mutex and + * initiate the waiting on WinNT/2K/XP. Win9x doesn't have + * SignalObjectAndWait(). */ + result = p_condvar->SignalObjectAndWait( p_mutex->mutex, + p_condvar->event, + delay_ms, FALSE ); + } + else + { + LeaveCriticalSection( &p_mutex->csection ); + result = WaitForSingleObject( p_condvar->event, delay_ms ); + } + + p_condvar->i_waiting_threads--; + } + else if( p_condvar->i_win9x_cv == 1 ) + { + int i_waiting_threads; + + /* Wait for the gate to be open */ + result = WaitForSingleObject( p_condvar->event, delay_ms ); + + /* Increase our wait count */ + p_condvar->i_waiting_threads++; + + LeaveCriticalSection( &p_mutex->csection ); + if( !result ) + { + /* recaculate remaining delay */ + delay_ms = (deadline - mdate())/1000; + if( delay_ms < 0 ) + delay_ms = 0; + + result = WaitForSingleObject( p_condvar->semaphore, delay_ms ); + } + + /* Decrement and test must be atomic */ + EnterCriticalSection( &p_condvar->csection ); + + /* Decrease our wait count */ + i_waiting_threads = --p_condvar->i_waiting_threads; + + LeaveCriticalSection( &p_condvar->csection ); + + /* Reopen the gate if we were the last waiting thread */ + if( !i_waiting_threads ) + SetEvent( p_condvar->event ); + } + else + { + int i_waiting_threads; + + /* Increase our wait count */ + p_condvar->i_waiting_threads++; + + LeaveCriticalSection( &p_mutex->csection ); + result = WaitForSingleObject( p_condvar->semaphore, delay_ms ); + + /* Decrement and test must be atomic */ + EnterCriticalSection( &p_condvar->csection ); + + /* Decrease our wait count */ + i_waiting_threads = --p_condvar->i_waiting_threads; + + LeaveCriticalSection( &p_condvar->csection ); + + /* Signal that the last waiting thread just went through */ + if( !i_waiting_threads ) + SetEvent( p_condvar->event ); + } + + /* Reacquire the mutex before returning. */ + vlc_mutex_lock( p_mutex ); + + i_res = (int)result; + +#elif defined( HAVE_KERNEL_SCHEDULER_H ) +# error Unimplemented +#elif defined( PTHREAD_COND_T_IN_PTHREAD_H ) + lldiv_t d = lldiv( deadline, 1000000 ); + struct timespec ts = { d.quot, d.rem * 1000 }; + + i_res = pthread_cond_timedwait( &p_condvar->cond, &p_mutex->mutex, &ts ); + if( i_res == ETIMEDOUT ) + return ETIMEDOUT; /* this error is perfectly normal */ + else + if ( i_res ) /* other errors = bug */ + { + i_thread = CAST_PTHREAD_TO_INT(pthread_self()); + errno = i_res; + } + +#elif defined( HAVE_CTHREADS_H ) +# error Unimplemented +#endif + + if( i_res ) + { + msg_Err( p_condvar->p_this, + "thread %li: cond_wait failed at %s:%d (%d:%m)", + i_thread, psz_file, i_line, i_res ); + vlc_threads_error( p_condvar->p_this ); + } + + return i_res; +} + /***************************************************************************** * vlc_cond_destroy: destroy a condition *****************************************************************************/ #define vlc_cond_destroy( P_COND ) \ __vlc_cond_destroy( __FILE__, __LINE__, P_COND ) +/***************************************************************************** + * vlc_threadvar_create: create a thread-local variable + *****************************************************************************/ +#define vlc_threadvar_create( PTHIS, P_TLS ) \ + __vlc_threadvar_create( PTHIS, P_TLS ) + +/***************************************************************************** + * vlc_threadvar_set: create: set the value of a thread-local variable + *****************************************************************************/ +static inline int vlc_threadvar_set( vlc_threadvar_t * p_tls, void *p_value ) +{ + int i_ret; + +#if defined( PTH_INIT_IN_PTH_H ) + return pth_key_setdata( p_tls->handle, p_value ); +#elif defined( ST_INIT_IN_ST_H ) + return st_thread_setspecific( p_tls->handle, p_value ); +#elif defined( HAVE_KERNEL_SCHEDULER_H ) + return -1; + +#elif defined( UNDER_CE ) || defined( WIN32 ) + i_ret = ( TlsSetValue( p_tls->handle, p_value ) != 0 ); + +#elif defined( PTHREAD_COND_T_IN_PTHREAD_H ) + i_ret = pthread_setspecific( p_tls->handle, p_value ); + +#elif defined( HAVE_CTHREADS_H ) + i_ret = cthread_setspecific( p_tls->handle, p_value ); +#endif + + return i_ret; +} + +/***************************************************************************** + * vlc_threadvar_get: create: get the value of a thread-local variable + *****************************************************************************/ +static inline void* vlc_threadvar_get( vlc_threadvar_t * p_tls ) +{ + void* p_ret; + +#if defined( PTH_INIT_IN_PTH_H ) + p_ret = pth_key_getdata( p_handle->key ); +#elif defined( ST_INIT_IN_ST_H ) + p_ret = st_thread_getspecific( p_handle->key ); +#elif defined( HAVE_KERNEL_SCHEDULER_H ) + p_ret = NULL; +#elif defined( UNDER_CE ) || defined( WIN32 ) + p_ret = TlsGetValue( p_tls->handle ); + +#elif defined( PTHREAD_COND_T_IN_PTHREAD_H ) + p_ret = pthread_getspecific( p_tls->handle ); + +#elif defined( HAVE_CTHREADS_H ) + if ( !cthread_getspecific( p_tls->handle, &p_ret ) ) + { + p_ret = NULL; + } +#endif + + return p_ret; +} + /***************************************************************************** * vlc_thread_create: create a thread *****************************************************************************/ @@ -561,3 +820,5 @@ static inline int __vlc_cond_wait( const char * psz_file, int i_line, *****************************************************************************/ #define vlc_thread_join( P_THIS ) \ __vlc_thread_join( VLC_OBJECT(P_THIS), __FILE__, __LINE__ ) + +#endif