diff options
author | eroman@chromium.org <eroman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-29 00:46:39 +0000 |
---|---|---|
committer | eroman@chromium.org <eroman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-29 00:46:39 +0000 |
commit | e3b140eae1c14a6b76dedf0d5b7d9dd3c40af895 (patch) | |
tree | 73559388f92b8f998dcd2c2e3c34acd91df61d05 /base/synchronization/lock_impl_posix.cc | |
parent | 5875dd0fca116e8205cf2b050e58ed0390f6d360 (diff) | |
download | chromium_src-e3b140eae1c14a6b76dedf0d5b7d9dd3c40af895.zip chromium_src-e3b140eae1c14a6b76dedf0d5b7d9dd3c40af895.tar.gz chromium_src-e3b140eae1c14a6b76dedf0d5b7d9dd3c40af895.tar.bz2 |
Temporary instrumentation to help understand lock errors on mac debug bots.
BUG=102161
Review URL: http://codereview.chromium.org/8414031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107835 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/synchronization/lock_impl_posix.cc')
-rw-r--r-- | base/synchronization/lock_impl_posix.cc | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/base/synchronization/lock_impl_posix.cc b/base/synchronization/lock_impl_posix.cc index f638fcd..8db953a 100644 --- a/base/synchronization/lock_impl_posix.cc +++ b/base/synchronization/lock_impl_posix.cc @@ -12,6 +12,10 @@ namespace base { namespace internal { LockImpl::LockImpl() { +#ifdef LOCK_IMPL_CHECK_LIVENESS + liveness_token_ = LT_ALIVE; +#endif + #ifndef NDEBUG // In debug, setup attributes for lock error checking. pthread_mutexattr_t mta; @@ -30,25 +34,45 @@ LockImpl::LockImpl() { } LockImpl::~LockImpl() { +#ifdef LOCK_IMPL_CHECK_LIVENESS + CheckIsAlive(); + liveness_token_ = LT_DELETED; +#endif int rv = pthread_mutex_destroy(&os_lock_); DCHECK_EQ(rv, 0); } bool LockImpl::Try() { +#ifdef LOCK_IMPL_CHECK_LIVENESS + CheckIsAlive(); +#endif int rv = pthread_mutex_trylock(&os_lock_); DCHECK(rv == 0 || rv == EBUSY); return rv == 0; } void LockImpl::Lock() { +#ifdef LOCK_IMPL_CHECK_LIVENESS + CheckIsAlive(); +#endif int rv = pthread_mutex_lock(&os_lock_); DCHECK_EQ(rv, 0); } void LockImpl::Unlock() { +#ifdef LOCK_IMPL_CHECK_LIVENESS + CheckIsAlive(); +#endif int rv = pthread_mutex_unlock(&os_lock_); DCHECK_EQ(rv, 0); } +#ifdef LOCK_IMPL_CHECK_LIVENESS +void LockImpl::CheckIsAlive() { + CHECK_EQ(LT_ALIVE, liveness_token_) + << "Lock was invalid. Please see: http://crbug.com/102161"; +} +#endif + } // namespace internal } // namespace base |