]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
helgrind: annotate false-positive race conditions
authorJason Dillaman <dillaman@redhat.com>
Mon, 18 Jan 2016 22:52:51 +0000 (17:52 -0500)
committerJason Dillaman <dillaman@redhat.com>
Mon, 18 Jan 2016 22:52:51 +0000 (17:52 -0500)
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
src/librados/IoCtxImpl.cc
src/msg/simple/Pipe.cc

index 3e6821240c873658ac5194042f4fcd24d3a01cc7..b159be63d4eaa3d2890edffa7e71fe4ca901f310 100644 (file)
@@ -20,6 +20,7 @@
 #include "librados/PoolAsyncCompletionImpl.h"
 #include "librados/RadosClient.h"
 #include "include/assert.h"
+#include "common/valgrind.h"
 
 #define dout_subsys ceph_subsys_rados
 #undef dout_prefix
@@ -1159,6 +1160,8 @@ int librados::IoCtxImpl::getxattrs(const object_t& oid,
 
 void librados::IoCtxImpl::set_sync_op_version(version_t ver)
 {
+  ANNOTATE_BENIGN_RACE_SIZED(&last_objver, sizeof(last_objver),
+                             "IoCtxImpl last_objver");
   last_objver = ver;
 }
 
index d91e6bb9e83b3cf05d9e48b4253438d7dde46cd8..aa6e8fb76e7d51d76f76b489233d4113472bb780 100644 (file)
@@ -84,6 +84,7 @@ Pipe::Pipe(SimpleMessenger *r, int st, PipeConnection *con)
     send_keepalive_ack(false),
     connect_seq(0), peer_global_seq(0),
     out_seq(0), in_seq(0), in_seq_acked(0) {
+  ANNOTATE_BENIGN_RACE_SIZED(&sd, sizeof(sd), "Pipe socket");
   ANNOTATE_BENIGN_RACE_SIZED(&state, sizeof(state), "Pipe state");
   ANNOTATE_BENIGN_RACE_SIZED(&recv_len, sizeof(recv_len), "Pipe recv_len");
   ANNOTATE_BENIGN_RACE_SIZED(&recv_ofs, sizeof(recv_ofs), "Pipe recv_ofs");