From: Patrick Donnelly <pdonnell@redhat.com>
Date: Tue, 25 Apr 2017 15:13:44 +0000 (-0400)
Subject: mds: break ancestor walk if node is export_pinned
X-Git-Tag: v12.0.3~38^2~20
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=91952aaf582174cd167d3a8967a121cdffab27ff;p=ceph.git

mds: break ancestor walk if node is export_pinned

i.e. don't keep walking the parents after adding the ancestor (or the node
itself) to the export_pin_queue.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
---

diff --git a/src/mds/CInode.cc b/src/mds/CInode.cc
index d551f1837e1e..73054cb8375f 100644
--- a/src/mds/CInode.cc
+++ b/src/mds/CInode.cc
@@ -4414,9 +4414,8 @@ void CInode::maybe_export_pin()
           if (pin != mdcache->mds->get_nodeid()) {
             dout(20) << "adding ancestor to export_pin_queue " << *auth_last << dendl;
             mdcache->export_pin_queue.insert(auth_last);
-          } else {
-            break; /* it is correctly pinned here! */
           }
+          break;
         }
       }
     } else {