mds: slient MDCache::trim_non_auth()

No need to output the function's debug message to console.

Signed-off-by: Yan, Zheng <zheng.z.yan@intel.com>

diff --git a/src/mds/MDCache.cc b/src/mds/MDCache.cc
index 5c2732bf0f352de74e34fd437c53083294c33c31..a67f605efd9e6bab413e05327fbe22f3f99c2327 100644 (file)
--- a/src/mds/MDCache.cc
+++ b/src/mds/MDCache.cc
@@ -6130,7 +6130,6 @@ void MDCache::trim_inode(CDentry *dn, CInode *in, CDir *con, map<int, MCacheExpi
 void MDCache::trim_non_auth()
 {
   dout(7) << "trim_non_auth" << dendl;
-  stringstream warn_str_dirs;
   
   // temporarily pin all subtree roots
   for (map<CDir*, set<CDir*> >::iterator p = subtrees.begin();
@@ -6164,22 +6163,18 @@ void MDCache::trim_non_auth()
       assert(dir);
 
       // unlink the dentry
-      dout(15) << "trim_non_auth removing " << *dn << dendl;
+      dout(10) << " removing " << *dn << dendl;
       if (dnl->is_remote()) {
        dir->unlink_inode(dn);
       } 
       else if (dnl->is_primary()) {
        CInode *in = dnl->get_inode();
+       dout(10) << " removing " << *in << dendl;
        list<CDir*> ls;
-        warn_str_dirs << in->get_parent_dn()->get_name() << "\n";
        in->get_dirfrags(ls);
        for (list<CDir*>::iterator p = ls.begin(); p != ls.end(); ++p) {
          CDir *subdir = *p;
-         filepath fp;
-         subdir->get_inode()->make_path(fp);
-         warn_str_dirs << fp << "\n";
-         if (subdir->is_subtree_root()) 
-           remove_subtree(subdir);
+         assert(!subdir->is_subtree_root());
          in->close_dirfrag(subdir->dirfrag().frag);
        }
        dir->unlink_inode(dn);
@@ -6218,18 +6213,13 @@ void MDCache::trim_non_auth()
        for (list<CDir*>::iterator p = ls.begin();
             p != ls.end();
             ++p) {
-         dout(0) << " ... " << **p << dendl;
-         CInode *diri = (*p)->get_inode();
-         filepath fp;
-         diri->make_path(fp);
-         warn_str_dirs << fp << "\n";
+         dout(10) << " removing " << **p << dendl;
          assert((*p)->get_num_ref() == 1);  // SUBTREE
          remove_subtree((*p));
          in->close_dirfrag((*p)->dirfrag().frag);
        }
-       dout(0) << " ... " << *in << dendl;
-       if (in->get_parent_dn())
-         warn_str_dirs << in->get_parent_dn()->get_name() << "\n";
+       dout(10) << " removing " << *in << dendl;
+       assert(!in->get_parent_dn());
        assert(in->get_num_ref() == 0);
        remove_inode(in);
       }
@@ -6238,10 +6228,6 @@ void MDCache::trim_non_auth()
   }
 
   show_subtrees();
-  if (warn_str_dirs.peek() != EOF) {
-    mds->clog.info() << "trim_non_auth has deleted paths: " << "\n";
-    mds->clog.info(warn_str_dirs);
-  }
 }
 
 /**