From: Yan, Zheng <zyan@redhat.com>
Date: Wed, 27 Jan 2016 07:17:29 +0000 (+0800)
Subject: mds: remove obsoleted FIXME from scrubstack code
X-Git-Tag: v10.1.0~176^2~1^2~11
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=f2d01c18636e0a9a47c3a36d26eb19115d6d80f8;p=ceph.git

mds: remove obsoleted FIXME from scrubstack code

Signed-off-by: Yan, Zheng <zyan@redhat.com>
---

diff --git a/src/mds/ScrubStack.cc b/src/mds/ScrubStack.cc
index f3e5ce887be3..5ef567443d4a 100644
--- a/src/mds/ScrubStack.cc
+++ b/src/mds/ScrubStack.cc
@@ -185,8 +185,6 @@ void ScrubStack::scrub_dir_inode(CInode *in,
       scrub_dirfrag(cur_dir, header,
 		    &frag_added_children, &frag_terminal, &frag_done);
       if (frag_done) {
-	// FIXME is this right?  Can we end up hitting this more than
-	// once and is that a problem?
 	cur_dir->inode->scrub_dirfrag_finished(cur_dir->frag);
       }
       *added_children |= frag_added_children;
@@ -317,8 +315,6 @@ void ScrubStack::scrub_dirfrag(CDir *dir,
     scrubs_in_progress++;
     r = dir->scrub_dentry_next(&scrub_kick, &dn);
     if (r != EAGAIN) {
-      // ctx only used by scrub_dentry_next in EAGAIN case
-      // FIXME It's kind of annoying to keep allocating and deleting a ctx here
       scrubs_in_progress--;
     }
 
@@ -360,8 +356,6 @@ void ScrubStack::scrub_dirfrag(CDir *dir,
     // never get random IO errors here.
     assert(r == 0);
 
-    // FIXME: Do I *really* need to construct a kick context for every
-    // single dentry I'm going to scrub?
     _enqueue_inode(dn->get_projected_inode(), dn, header, NULL, true);
 
     *added_children = true;