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 c35a53521e8130bf4dc09fe0ddc2e233b15734d5..43218e7b153fd6409edbcf70490bb82ffc4b1a25 100644 (file)
--- 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 3a417b02ab9b03bdc37ab88cc4f1a6cc39b74667..52357d260ee4785d1569a80ff53c3e6c22ad1c2b 100644 (file)
--- 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 1b94e66f8aef4aeaa3e4dd326b9fceafceb002fa..ba2d43f96e0312d2eaf438ef36fd7049ae47873d 100644 (file)
--- 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);