Aleric Inglewood: SNOW-196 Special case for TextureFetch race condition check.

Where the current UUID was just removed from mRequestMap, thus avoiding to the
need for the problematic lock completely. The case that we add something to
mNetworkQueue while it was just removed from mRequestMap is corrected by
LLTextureFetch::sendRequestListToSimulators anyway.

2 files changed, 8 insertions, 35 deletions

diff --git a/linden/indra/llcommon/llthread.cpp b/linden/indra/llcommon/llthread.cpp
index 60ccd5c..692d6c4 100644
--- a/linden/indra/llcommon/llthread.cpp
+++ b/linden/indra/llcommon/llthread.cpp
@@ -283,7 +283,6 @@ LLMutex::LLMutex(apr_pool_t *poolp) :
         apr_thread_mutex_create(&mAPRMutexp, APR_THREAD_MUTEX_UNNESTED, mAPRPoolp);
 }
 
-
 LLMutex::~LLMutex()
 {
 #if _DEBUG
@@ -297,40 +296,14 @@ LLMutex::~LLMutex()
         }
 }
 
-
-void LLMutex::lock()
-{
-        apr_thread_mutex_lock(mAPRMutexp);
-}
-
-void LLMutex::unlock()
-{
-        apr_thread_mutex_unlock(mAPRMutexp);
-}
-
 bool LLMutex::isLocked()
 {
-        apr_status_t status = apr_thread_mutex_trylock(mAPRMutexp);
-        if (APR_STATUS_IS_EBUSY(status))
+        if (!tryLock())
         {
                 return true;
         }
-        else
-        {
-                apr_thread_mutex_unlock(mAPRMutexp);
-                return false;
-        }
-}
-
-//trys to grab a lock and if sucessful returns true
-bool LLMutex::tryLock()
-{
-        apr_status_t status = apr_thread_mutex_trylock(mAPRMutexp);
-        if (APR_STATUS_IS_EBUSY(status))
-        {       
-                return false;
-        }
-        return true;
+        apr_thread_mutex_unlock(mAPRMutexp);
+        return false;
 }
 
 //============================================================================
@@ -343,14 +316,12 @@ LLCondition::LLCondition(apr_pool_t *poolp) :
         apr_thread_cond_create(&mAPRCondp, mAPRPoolp);
 }
 
-
 LLCondition::~LLCondition()
 {
         apr_thread_cond_destroy(mAPRCondp);
         mAPRCondp = NULL;
 }
 
-
 void LLCondition::wait()
 {
         apr_thread_cond_wait(mAPRCondp, mAPRMutexp);
diff --git a/linden/indra/llcommon/llthread.h b/linden/indra/llcommon/llthread.h
index d773fa4..ed76a3b 100644
--- a/linden/indra/llcommon/llthread.h
+++ b/linden/indra/llcommon/llthread.h
@@ -131,10 +131,12 @@ public:
         LLMutex(apr_pool_t *apr_poolp); // NULL pool constructs a new pool for the mutex
         ~LLMutex();
         
-        void lock();            // blocks
-        void unlock();
+        void lock() { apr_thread_mutex_lock(mAPRMutexp); }
+        void unlock() { apr_thread_mutex_unlock(mAPRMutexp); }
+        // Returns true if lock was obtained successfully.
+        bool tryLock() { return !APR_STATUS_IS_EBUSY(apr_thread_mutex_trylock(mAPRMutexp)); }
+
         bool isLocked();        // non-blocking, but does do a lock/unlock so not free
-        bool tryLock();         //non-blocking, but not free, returns true if grabed lock
 
 protected:
         apr_thread_mutex_t *mAPRMutexp;