LU-8371 llite: Trust creates in revalidate too.

[fs/lustre-release.git] / lustre / llite / dcache.c

diff --git a/lustre/llite/dcache.c b/lustre/llite/dcache.c
index 8c3db0a..c42e89d 100644 (file)
--- a/lustre/llite/dcache.c
+++ b/lustre/llite/dcache.c
@@ -150,17 +150,6 @@ static int ll_ddelete(HAVE_D_DELETE_CONST struct dentry *de)
        LASSERT(ll_d_count(de) == 1);
 #endif
 
-       /* Disable this piece of code temproarily because this is called
-        * inside dcache_lock so it's not appropriate to do lots of work
-        * here. ATTENTION: Before this piece of code enabling, LU-2487 must be
-        * resolved. */
-#if 0
-       /* if not ldlm lock for this inode, set i_nlink to 0 so that
-        * this inode can be recycled later b=20433 */
-       if (de->d_inode && !find_cbdata(de->d_inode))
-               clear_nlink(de->d_inode);
-#endif
-
        if (d_lustre_invalid((struct dentry *)de))
                RETURN(1);
        RETURN(0);
@@ -208,8 +197,8 @@ void ll_intent_drop_lock(struct lookup_intent *it)
 
                handle.cookie = it->it_lock_handle;
 
-               CDEBUG(D_DLMTRACE, "releasing lock with cookie "LPX64
-                      " from it %p\n", handle.cookie, it);
+               CDEBUG(D_DLMTRACE, "releasing lock with cookie %#llx from it %p\n",
+                      handle.cookie, it);
                ldlm_lock_decref(&handle, it->it_lock_mode);
 
                /* bug 494: intent_release may be called multiple times, from
@@ -219,7 +208,7 @@ void ll_intent_drop_lock(struct lookup_intent *it)
                        handle.cookie = it->it_remote_lock_handle;
 
                        CDEBUG(D_DLMTRACE, "releasing remote lock with cookie"
-                              LPX64" from it %p\n", handle.cookie, it);
+                              "%#llx from it %p\n", handle.cookie, it);
                        ldlm_lock_decref(&handle,
                                         it->it_remote_lock_mode);
                        it->it_remote_lock_mode = 0;
@@ -331,16 +320,20 @@ static int ll_revalidate_dentry(struct dentry *dentry,
                return 1;
 
        /* Symlink - always valid as long as the dentry was found */
+#ifdef HAVE_IOP_GET_LINK
+       if (dentry->d_inode && dentry->d_inode->i_op->get_link)
+#else
        if (dentry->d_inode && dentry->d_inode->i_op->follow_link)
+#endif
                return 1;
 
-       /* Last path component lookup for open or create - we always
-        * return 0 here to go through re-lookup and properly signal
-        * MDS whenever we do or do not want an open-cache to be engaged.
-        * For create we also ensure the entry is really created no matter
-        * what races might have happened.
-        * LU-4367 */
-       if (lookup_flags & (LOOKUP_OPEN | LOOKUP_CREATE))
+       /*
+        * VFS warns us that this is the second go around and previous
+        * operation failed (most likely open|creat), so this time
+        * we better talk to the server via the lookup path by name,
+        * not by fid.
+        */
+       if (lookup_flags & LOOKUP_REVAL)
                return 0;
 
        if (!dentry_may_statahead(dir, dentry))