From: dparmar18 <dparmar@redhat.com>
Date: Thu, 28 Jul 2022 11:09:07 +0000 (+0530)
Subject: mds/Server: improve reclaim_session code
X-Git-Tag: v16.2.11~81^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=dec9bb35312a09fcb77721397ac653ce2461da67;p=ceph.git

mds/Server: improve reclaim_session code

Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
(cherry picked from commit c62ac888ccffb0799590174d4e353a64e12e4303)
---

diff --git a/src/mds/Server.cc b/src/mds/Server.cc
index a04b441ed9a0..400ff5266fb2 100644
--- a/src/mds/Server.cc
+++ b/src/mds/Server.cc
@@ -435,7 +435,7 @@ void Server::reclaim_session(Session *session, const cref_t<MClientReclaim> &m)
   unsigned flags = m->get_flags();
   if (flags != CEPH_RECLAIM_RESET) { // currently only support reset
     dout(10) << __func__ << " unsupported flags" << dendl;
-    reply->set_result(-CEPHFS_EOPNOTSUPP);
+    reply->set_result(-CEPHFS_EINVAL);
     mds->send_message_client(reply, session);
     return;
   }
@@ -457,10 +457,7 @@ void Server::reclaim_session(Session *session, const cref_t<MClientReclaim> &m)
 
   if (flags & CEPH_RECLAIM_RESET) {
     finish_reclaim_session(session, reply);
-    return;
-  }
-
-  ceph_abort();
+  } else ceph_assert(0); /* no other flags are handled at this time */
 }
 
 void Server::finish_reclaim_session(Session *session, const ref_t<MClientReclaimReply> &reply)