From: John Spray <john.spray@redhat.com>
Date: Thu, 23 Feb 2017 23:23:11 +0000 (+0000)
Subject: client: fix unmounting while blacklisted
X-Git-Tag: v12.1.0~10^2~27^2~10
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=4a9386e4e74f87c66ba221856b1bc46212c4a442;p=ceph.git

client: fix unmounting while blacklisted

Previously, if you ever made it as far as ~Client
with any inodes in your cache, you'd hit the ObjectCacher
is_locked_by_me assertion during release_set on those inode's osets.

Signed-off-by: John Spray <john.spray@redhat.com>
---

diff --git a/src/client/Client.cc b/src/client/Client.cc
index 17ed10c8b88d..56ea53d07999 100644
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -301,7 +301,12 @@ Client::~Client()
 {
   assert(!client_lock.is_locked());
 
+  // It is necessary to hold client_lock, because any inode destruction
+  // may call into ObjectCacher, which asserts that it's lock (which is
+  // client_lock) is held.
+  client_lock.Lock();
   tear_down_cache();
+  client_lock.Unlock();
 }
 
 void Client::tear_down_cache()
@@ -2438,6 +2443,11 @@ void Client::handle_osd_map(MOSDMap *m)
       _closed_mds_session(session);
     }
 
+    // Since we know all our OSD ops will fail, cancel them all preemtively,
+    // so that on an unhealthy cluster we can umount promptly even if e.g.
+    // some PGs were inaccessible.
+    objecter->op_cancel_writes(-EBLACKLISTED);
+
   } else if (blacklisted) {
     // Handle case where we were blacklisted but no longer are
     blacklisted = objecter->with_osdmap([myaddr](const OSDMap &o){
@@ -5812,6 +5822,17 @@ void Client::unmount()
 
   if (blacklisted) {
     ldout(cct, 0) << " skipping clean shutdown, we are blacklisted" << dendl;
+
+    if (cct->_conf->client_oc) {
+      // Purge all cached data so that ObjectCacher doesn't get hung up
+      // trying to flush it.  ObjectCacher's behaviour on EBLACKLISTED
+      // is to just leave things marked dirty
+      // (http://tracker.ceph.com/issues/9105)
+      for (const auto &i : inode_map) {
+        objectcacher->purge_set(&(i.second->oset));
+      }
+    }
+
     mounted = false;
     return;
   }