From: haoyixing <haoyixing@kuaishou.com>
Date: Thu, 17 Dec 2020 08:07:47 +0000 (+0800)
Subject: MDSLogContext: fix typo in MDSLogContextBase and add more comments
X-Git-Tag: v16.1.0~179^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F38631%2Fhead;p=ceph.git

MDSLogContext: fix typo in MDSLogContextBase and add more comments

Fix typos in MDSLogContextBase's definition and function '::complete',
also add more comments about setting safe_pos.

Signed-off-by: haoyixing <haoyixing@kuaishou.com>
---

diff --git a/src/mds/MDSContext.cc b/src/mds/MDSContext.cc
index c702d0253f74..a17da0a01f0a 100644
--- a/src/mds/MDSContext.cc
+++ b/src/mds/MDSContext.cc
@@ -119,8 +119,9 @@ void MDSLogContextBase::complete(int r) {
   MDLog *mdlog = get_mds()->mdlog;
   uint64_t safe_pos = write_pos;
   pre_finish(r);
-  // MDSContextBase::complete() free this
+  // MDSIOContext::complete() free this
   MDSIOContextBase::complete(r);
+  // safe_pos must be updated after MDSIOContext::complete() call
   mdlog->set_safe_pos(safe_pos);
 }
 
diff --git a/src/mds/MDSContext.h b/src/mds/MDSContext.h
index d544115b9b51..319af50d3542 100644
--- a/src/mds/MDSContext.h
+++ b/src/mds/MDSContext.h
@@ -118,7 +118,7 @@ private:
 /**
  * Completion for an log operation, takes big MDSRank lock
  * before executing finish function. Update log's safe pos
- * after finish functuon return.
+ * after finish function return.
  */
 class MDSLogContextBase : public MDSIOContextBase
 {