From: Sage Weil <sage@newdream.net>
Date: Wed, 14 Jan 2009 17:53:09 +0000 (-0800)
Subject: mds: some cleanup
X-Git-Tag: v0.6~1^2~72
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=cbbe5438dbbf4d61d2e485bfeb633a6e74cd4fae;p=ceph.git

mds: some cleanup
---

diff --git a/src/mds/Server.cc b/src/mds/Server.cc
index 97ca4eab3dcf..e02670f7f9e1 100644
--- a/src/mds/Server.cc
+++ b/src/mds/Server.cc
@@ -2157,13 +2157,6 @@ void Server::handle_client_readdir(MDRequest *mdr)
   dout(10) << "handle_client_readdir on " << *dir << dendl;
   assert(dir->is_auth());
 
-  // check perm
-  /*
-  if (!mds->locker->inode_hard_rdlock_start(diri, mdr))
-    return;
-  mds->locker->inode_hard_rdlock_finish(diri, mdr);
-  */
-
   if (!dir->is_complete()) {
     // fetch
     dout(10) << " incomplete dir contents for readdir on " << *dir << ", fetching" << dendl;
@@ -2406,8 +2399,6 @@ void Server::handle_client_mkdir(MDRequest *mdr)
   newdir->mark_complete();
   newdir->pre_dirty();
 
-  //if (mds->logger) mds->logger->inc("mkdir");
-
   // prepare finisher
   mdr->ls = mdlog->get_current_segment();
   EUpdate *le = new EUpdate(mdlog, "mkdir");
@@ -4774,8 +4765,6 @@ void Server::handle_client_truncate(MDRequest *mdr)
   }
     
 
-  // check permissions?  
-
   // xlock inode
   set<SimpleLock*> rdlocks = mdr->rdlocks;
   set<SimpleLock*> wrlocks = mdr->wrlocks;
@@ -4861,8 +4850,6 @@ void Server::handle_client_open(MDRequest *mdr)
     return;
   }
 
-  // hmm, check permissions or something.
-
   // O_TRUNC
   if ((flags & O_TRUNC) &&
       !(req->get_retry_attempt() &&