From: Alex Zhuravlev <bzzz@whamcloud.com>
Date: Tue, 10 May 2022 07:48:55 +0000 (+0300)
Subject: LU-15800 ofd: take a read lock for fallocate
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=64e269c800533c4ce15235b21597038c29f0040a;p=fs%2Flustre-release.git

LU-15800 ofd: take a read lock for fallocate

there is no need to take an write (exclusive) object's
lock for fallocate - we just need to serialize fallocate
vs destroy, all internal structures should be protected
by OSD and disk filesystem like the write path does.

Fixes: cdaaa87f6b ("LU-14214 ofd: fix locking in ofd_object_fallocate()")
Signed-off-by: Alex Zhuravlev <bzzz@whamcloud.com>
Change-Id: I65986745865ee329c5257a7efca5e79403830608
---

diff --git a/lustre/ofd/ofd_objects.c b/lustre/ofd/ofd_objects.c
index 41fe74b..16f9fba 100644
--- a/lustre/ofd/ofd_objects.c
+++ b/lustre/ofd/ofd_objects.c
@@ -801,7 +801,7 @@ int ofd_object_fallocate(const struct lu_env *env, struct ofd_object *fo,
 	if (rc)
 		GOTO(stop, rc);
 
-	ofd_write_lock(env, fo);
+	ofd_read_lock(env, fo);
 	if (!ofd_object_exists(fo))
 		GOTO(unlock, rc = -ENOENT);
 
@@ -824,7 +824,7 @@ int ofd_object_fallocate(const struct lu_env *env, struct ofd_object *fo,
 			filter_fid_le_to_cpu(&fo->ofo_ff, ff, sizeof(*ff));
 	}
 unlock:
-	ofd_write_unlock(env, fo);
+	ofd_read_unlock(env, fo);
 stop:
 	ofd_trans_stop(env, ofd, th, rc);
 	RETURN(rc);