MDS: remove a few other unnecessary is_base() checks

We should let users remove xattrs as well as set them. ;) And
the check in handle_client_setlayout was totally useless -- perhaps
intended for setdirlayout?

This is a follow-on to 9f82ae60fac30391dfa9d17d2fc014bf9e21f387 and
should be taken wherever it goes.

Signed-off-by: Greg Farnum <greg@inktank.com>

diff --git a/src/mds/Server.cc b/src/mds/Server.cc
index 4f21601ebe818000a05f0d8a533f9bca5990d180..38c7642bd7caaa33b006c8e386954eec621f8612 100644 (file)
--- a/src/mds/Server.cc
+++ b/src/mds/Server.cc
@@ -3363,10 +3363,6 @@ void Server::handle_client_setlayout(MDRequest *mdr)
     reply_request(mdr, -EROFS);
     return;
   }
-  if(cur->is_base()) {
-    reply_request(mdr, -EINVAL);   // for now
-    return;
-  }
   if (cur->is_dir()) {
     reply_request(mdr, -EISDIR);
     return;
@@ -3855,10 +3851,6 @@ void Server::handle_client_removexattr(MDRequest *mdr)
     reply_request(mdr, -EROFS);
     return;
   }
-  if (cur->is_base()) {
-    reply_request(mdr, -EINVAL);   // for now
-    return;
-  }
 
   if (name.find("ceph.") == 0) {
     handle_remove_vxattr(mdr, cur, rdlocks, wrlocks, xlocks);