From 123f6211d3ff930231d47f3e9bad3ee38c402337 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Thu, 9 Apr 2009 13:26:17 -0700 Subject: [PATCH] mds: don't rdlock_try authlock for path_traverse permission check We don't actually check anything anyway, just ping the lock. If the client is trusted with AUTH_EXCL, this is pointless anyway. It is only really useful with untrusted (e.g., fuse) clients, but they'll need some sort of special support for that later. --- src/mds/MDCache.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/mds/MDCache.cc b/src/mds/MDCache.cc index 57d5fd3da7479..23c911e596681 100644 --- a/src/mds/MDCache.cc +++ b/src/mds/MDCache.cc @@ -5622,12 +5622,14 @@ int MDCache::path_traverse(MDRequest *mdr, Message *req, // who */ // must read directory hard data (permissions, x bit) to traverse +#if 0 if (!noperm && !mds->locker->rdlock_try(&cur->authlock, client, 0)) { dout(7) << "traverse: waiting on authlock rdlock on " << *cur << dendl; cur->authlock.add_waiter(SimpleLock::WAIT_RD, _get_waiter(mdr, req)); return 1; } +#endif // check permissions? // XXX -- 2.39.5