From fe4c658bd33c56e85baf1a0fb4fe83e96d7124d2 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Tue, 17 Jul 2012 16:01:11 -0700 Subject: [PATCH] librados: simplify locking slightly No reason to hold mylock_all here. Signed-off-by: Sage Weil --- src/librados/IoCtxImpl.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/librados/IoCtxImpl.cc b/src/librados/IoCtxImpl.cc index 9519c85eb38..cf295cb139c 100644 --- a/src/librados/IoCtxImpl.cc +++ b/src/librados/IoCtxImpl.cc @@ -1450,17 +1450,16 @@ int librados::IoCtxImpl::notify(const object_t& oid, uint64_t ver, bufferlist& b 0, onack, NULL, &objver); lock->Unlock(); - mylock_all.Lock(); mylock.Lock(); while (!done) cond.Wait(mylock); mylock.Unlock(); + mylock_all.Lock(); if (r == 0) { while (!done_all) cond_all.Wait(mylock_all); } - mylock_all.Unlock(); lock->Lock();