From: Xiubo Li <xiubli@redhat.com>
Date: Thu, 26 May 2022 06:13:24 +0000 (+0800)
Subject: mds, client: remove useless feature required code
X-Git-Tag: v18.0.0~753^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=c6cb986a2eea10b8ef15d1c6539873a88c5a69aa;p=ceph.git

mds, client: remove useless feature required code

This code was added but never ever used.

Signed-off-by: Xiubo Li <xiubli@redhat.com>
---

diff --git a/src/client/Client.cc b/src/client/Client.cc
index c35a53521e813..43218e7b153fd 100644
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -2330,15 +2330,6 @@ void Client::handle_client_session(const MConstRef<MClientSession>& m)
       if (!session->seq && m->get_seq())
         session->seq = m->get_seq();
 
-      feature_bitset_t missing_features(CEPHFS_FEATURES_CLIENT_REQUIRED);
-      missing_features -= m->supported_features;
-      if (!missing_features.empty()) {
-	lderr(cct) << "mds." << from << " lacks required features '"
-		   << missing_features << "', closing session " << dendl;
-	_close_mds_session(session.get());
-	_closed_mds_session(session.get(), -CEPHFS_EPERM, true);
-	break;
-      }
       session->mds_features = std::move(m->supported_features);
       session->mds_metric_flags = std::move(m->metric_spec.metric_flags);
 
diff --git a/src/mds/MDSMap.cc b/src/mds/MDSMap.cc
index 3a417b02ab9b0..52357d260ee47 100644
--- a/src/mds/MDSMap.cc
+++ b/src/mds/MDSMap.cc
@@ -1135,7 +1135,7 @@ bool MDSMap::is_degraded() const {
 
 void MDSMap::set_min_compat_client(ceph_release_t version)
 {
-  vector<size_t> bits = CEPHFS_FEATURES_MDS_REQUIRED;
+  vector<size_t> bits;
 
   if (version >= ceph_release_t::octopus)
     bits.push_back(CEPHFS_FEATURE_OCTOPUS);
diff --git a/src/mds/cephfs_features.h b/src/mds/cephfs_features.h
index 1b94e66f8aef4..ba2d43f96e031 100644
--- a/src/mds/cephfs_features.h
+++ b/src/mds/cephfs_features.h
@@ -84,10 +84,7 @@ namespace ceph {
 }
 
 #define CEPHFS_FEATURES_MDS_SUPPORTED CEPHFS_FEATURES_ALL
-#define CEPHFS_FEATURES_MDS_REQUIRED {}
-
 #define CEPHFS_FEATURES_CLIENT_SUPPORTED CEPHFS_FEATURES_ALL
-#define CEPHFS_FEATURES_CLIENT_REQUIRED {}
 
 extern std::string_view cephfs_feature_name(size_t id);
 extern int cephfs_feature_from_name(std::string_view name);