From: Kefu Chai <kchai@redhat.com>
Date: Mon, 24 Aug 2020 10:35:22 +0000 (+0800)
Subject: test/osd/RadosModel: do not set done
X-Git-Tag: v16.1.0~1279^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=79345046d11048ec3e03c8c58c13c43c5c371870;p=ceph.git

test/osd/RadosModel: do not set done

we don't reuse TestOp, and `done` should have been reset in the
constructor.

Signed-off-by: Kefu Chai <kchai@redhat.com>
---

diff --git a/src/test/osd/RadosModel.h b/src/test/osd/RadosModel.h
index 809cf6548817..ae0efb373cdf 100644
--- a/src/test/osd/RadosModel.h
+++ b/src/test/osd/RadosModel.h
@@ -779,9 +779,9 @@ public:
 		
   void _begin() override
   {
-    std::lock_guard state_locker{context->state_lock};
-    done = 0;
+    assert(!done);
     stringstream acc;
+    std::lock_guard state_locker{context->state_lock};
     acc << context->prefix << "OID: " << oid << " snap " << context->current_snap << std::endl;
     string prefix = acc.str();