From: Sage Weil <sage.weil@dreamhost.com>
Date: Mon, 29 Aug 2011 22:53:57 +0000 (-0700)
Subject: client: debug link/unlink and dn_set parent relationship
X-Git-Tag: v0.35~186
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=b140ff223edabd5850f275aea7392b4d75486cbc;p=ceph.git

client: debug link/unlink and dn_set parent relationship

Signed-off-by: Sage Weil <sage.weil@dreamhost.com>
---

diff --git a/src/client/Client.cc b/src/client/Client.cc
index 856df3fbf8a7..41e56025e2d4 100644
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -1788,24 +1788,26 @@ void Client::close_dir(Dir *dir)
    */
 Dentry* Client::link(Dir *dir, const string& name, Inode *in, Dentry *dn)
 {
-  if (!dn) { //create a new Dentry
+  if (!dn) {
+    // create a new Dentry
     dn = new Dentry;
     dn->name = name;
     
     // link to dir
     dn->dir = dir;
-    //cout << "link dir " << dir->parent_inode->ino << " '" << name
-    //<< "' -> inode " << in->ino << endl;
     dir->dentries[dn->name] = dn;
     dir->dentry_map[dn->name] = dn;
     lru.lru_insert_mid(dn);    // mid or top?
+
+    ldout(cct, 15) << "link " << dir->parent_inode << " '" << name << "' to " << in
+		   << " dn " << dn << " (new dn)" << dendl;
+  } else {
+    ldout(cct, 15) << "link " << dir->parent_inode << " '" << name << "' to " << in
+		   << " dn " << dn << " (old dn)" << dendl;
   }
 
   if (in) {    // link to inode
     dn->inode = in;
-    if (!in->dn_set.empty())
-      ldout(cct, 5) << "adding new hard link to " << in->vino()
-		    << " from " << dn << dendl;
     in->get();
     if (in->dir)
       dn->get();  // dir -> dn pin
@@ -1816,6 +1818,8 @@ Dentry* Client::link(Dir *dir, const string& name, Inode *in, Dentry *dn)
     }
 
     in->dn_set.insert(dn);
+
+    ldout(cct, 20) << "link  inode " << in << " parents now " << in->dn_set << dendl; 
   }
   
   return dn;
@@ -1824,6 +1828,8 @@ Dentry* Client::link(Dir *dir, const string& name, Inode *in, Dentry *dn)
 void Client::unlink(Dentry *dn, bool keepdir)
 {
   Inode *in = dn->inode;
+  ldout(cct, 15) << "unlink " << dn->dir->parent_inode << " '" << dn->name << "' dn " << dn
+		 << " inode " << dn->inode << dendl;
 
   // unlink from inode
   if (in) {
@@ -1831,6 +1837,7 @@ void Client::unlink(Dentry *dn, bool keepdir)
       dn->put();        // dir -> dn pin
     dn->inode = 0;
     in->dn_set.erase(dn);
+    ldout(cct, 20) << "unlink  inode " << in << " parents now " << in->dn_set << dendl; 
     put_inode(in);
   }