b=20373 don't do rep-ack if not created anything

i=johann
i=Z

mds_open currently always put a lock into a rep-ack regardless if something
was created or not. This is pointless and only creates needless contention.
In fact the entire idea was to do this for real creates as a recovery protection.

diff --git a/lustre/mds/mds_open.c b/lustre/mds/mds_open.c
index 31e1768..5460b10 100644 (file)
--- a/lustre/mds/mds_open.c
+++ b/lustre/mds/mds_open.c
@@ -1423,7 +1423,7 @@ found_child:
                         break;
 
                 l_dput(dparent);
-                if (rc)
+                if (rc || !created)
                         ldlm_lock_decref(&parent_lockh, parent_mode);
                 else
                         ptlrpc_save_lock(req, &parent_lockh, parent_mode);