From: Xiubo Li <xiubli@redhat.com>
Date: Tue, 5 Jul 2022 02:40:23 +0000 (+0800)
Subject: ceph: remove useless check for the folio
X-Git-Tag: v6.1-rc5~621^2~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c460f4e4bba2d3f8dc0b5bfa8995d6e8d2d527a1;p=platform%2Fkernel%2Flinux-starfive.git

ceph: remove useless check for the folio

The netfs_write_begin() won't set the folio if the return value
is non-zero.

Signed-off-by: Xiubo Li <xiubli@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
---

diff --git a/fs/ceph/addr.c b/fs/ceph/addr.c
index 2f886ec..de12715 100644
--- a/fs/ceph/addr.c
+++ b/fs/ceph/addr.c
@@ -1326,16 +1326,13 @@ static int ceph_write_begin(struct file *file, struct address_space *mapping,
 	int r;
 
 	r = netfs_write_begin(&ci->netfs, file, inode->i_mapping, pos, len, &folio, NULL);
-	if (r == 0)
-		folio_wait_fscache(folio);
-	if (r < 0) {
-		if (folio)
-			folio_put(folio);
-	} else {
-		WARN_ON_ONCE(!folio_test_locked(folio));
-		*pagep = &folio->page;
-	}
-	return r;
+	if (r < 0)
+		return r;
+
+	folio_wait_fscache(folio);
+	WARN_ON_ONCE(!folio_test_locked(folio));
+	*pagep = &folio->page;
+	return 0;
 }
 
 /*