common/LogClient: make last_log non-atomic

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit c36a98be97e15294f57c6a640fe6a1c277dce8a3)

diff --git a/src/common/LogClient.cc b/src/common/LogClient.cc
index 84ceec8705216cc6acbc0169226a93d4f16484cf..aeb2f5bfcec82946ac7d1d600fb6668f90e30b67 100644 (file)
--- a/src/common/LogClient.cc
+++ b/src/common/LogClient.cc
@@ -337,6 +337,7 @@ version_t LogClient::queue(LogEntry &entry)
 
 uint64_t LogClient::get_next_seq()
 {
+  Mutex::Locker l(log_lock);
   return ++last_log;
 }
 

diff --git a/src/common/LogClient.h b/src/common/LogClient.h
index 56bb91ba9934c41aa5b958cb92a24f36a9d2dd58..be70e4512837b8c1975631c76514f95ef35739d1 100644 (file)
--- a/src/common/LogClient.h
+++ b/src/common/LogClient.h
@@ -245,7 +245,7 @@ private:
   bool is_mon;
   Mutex log_lock;
   version_t last_log_sent;
-  std::atomic<uint64_t> last_log;
+  version_t last_log;
   std::deque<LogEntry> log_queue;
 
   std::map<std::string, LogChannelRef> channels;