[v3] zonefs: move super block reading from page to folio

Commit Message

Johannes Thumshirn May 23, 2024, 3:02 p.m. UTC

From: Johannes Thumshirn <[e-mail]>

Move reading of the on-disk superblock from page to folios.

Cc: Matthew Wilcox <[e-mail]>
Signed-off-by: Johannes Thumshirn <[e-mail]>
---

Changes to v2:
- Rebase onto Linus' master
- Remove now unused varuabls
- Return read_mapping_folio()s error in case it fails

 fs/zonefs/super.c | 37 ++++++++++++++-----------------------
 1 file changed, 14 insertions(+), 23 deletions(-)

Comments

Matthew Wilcox May 23, 2024, 4:19 p.m. UTC | #1

On Thu, May 23, 2024 at 05:02:53PM +0200, Johannes Thumshirn wrote:
> From: Johannes Thumshirn <[e-mail]>
> 
> Move reading of the on-disk superblock from page to folios.
> 
> Cc: Matthew Wilcox <[e-mail]>
> Signed-off-by: Johannes Thumshirn <[e-mail]>

Reviewed-by: Matthew Wilcox (Oracle) <[e-mail]>

Damien Le Moal May 27, 2024, 4:35 a.m. UTC | #2

On 5/24/24 12:02 AM, Johannes Thumshirn wrote:
> From: Johannes Thumshirn <[e-mail]>
> 
> Move reading of the on-disk superblock from page to folios.
> 
> Cc: Matthew Wilcox <[e-mail]>
> Signed-off-by: Johannes Thumshirn <[e-mail]>

Applied to for-6.11. Thanks !

Patch

diff --git a/fs/zonefs/super.c b/fs/zonefs/super.c
index faf1eb87895d..e2bb7556a364 100644
--- a/fs/zonefs/super.c
+++ b/fs/zonefs/super.c
@@ -1109,30 +1109,21 @@  static int zonefs_init_zgroups(struct super_block *sb)
 static int zonefs_read_super(struct super_block *sb)
 {
 	struct zonefs_sb_info *sbi = ZONEFS_SB(sb);
+	struct address_space *address_space = sb->s_bdev->bd_mapping;
 	struct zonefs_super *super;
 	u32 crc, stored_crc;
-	struct page *page;
-	struct bio_vec bio_vec;
-	struct bio bio;
+	struct folio *folio;
 	int ret;
 
-	page = alloc_page(GFP_KERNEL);
-	if (!page)
-		return -ENOMEM;
-
-	bio_init(&bio, sb->s_bdev, &bio_vec, 1, REQ_OP_READ);
-	bio.bi_iter.bi_sector = 0;
-	__bio_add_page(&bio, page, PAGE_SIZE, 0);
-
-	ret = submit_bio_wait(&bio);
-	if (ret)
-		goto free_page;
+	folio = read_mapping_folio(address_space, 0, NULL);
+	if (IS_ERR(folio))
+		return PTR_ERR(folio);
 
-	super = page_address(page);
+	super = folio_address(folio);
 
 	ret = -EINVAL;
 	if (le32_to_cpu(super->s_magic) != ZONEFS_MAGIC)
-		goto free_page;
+		goto put_folio;
 
 	stored_crc = le32_to_cpu(super->s_crc);
 	super->s_crc = 0;
@@ -1140,14 +1131,14 @@  static int zonefs_read_super(struct super_block *sb)
 	if (crc != stored_crc) {
 		zonefs_err(sb, "Invalid checksum (Expected 0x%08x, got 0x%08x)",
 			   crc, stored_crc);
-		goto free_page;
+		goto put_folio;
 	}
 
 	sbi->s_features = le64_to_cpu(super->s_features);
 	if (sbi->s_features & ~ZONEFS_F_DEFINED_FEATURES) {
 		zonefs_err(sb, "Unknown features set 0x%llx\n",
 			   sbi->s_features);
-		goto free_page;
+		goto put_folio;
 	}
 
 	if (sbi->s_features & ZONEFS_F_UID) {
@@ -1155,7 +1146,7 @@  static int zonefs_read_super(struct super_block *sb)
 				       le32_to_cpu(super->s_uid));
 		if (!uid_valid(sbi->s_uid)) {
 			zonefs_err(sb, "Invalid UID feature\n");
-			goto free_page;
+			goto put_folio;
 		}
 	}
 
@@ -1164,7 +1155,7 @@  static int zonefs_read_super(struct super_block *sb)
 				       le32_to_cpu(super->s_gid));
 		if (!gid_valid(sbi->s_gid)) {
 			zonefs_err(sb, "Invalid GID feature\n");
-			goto free_page;
+			goto put_folio;
 		}
 	}
 
@@ -1173,14 +1164,14 @@  static int zonefs_read_super(struct super_block *sb)
 
 	if (memchr_inv(super->s_reserved, 0, sizeof(super->s_reserved))) {
 		zonefs_err(sb, "Reserved area is being used\n");
-		goto free_page;
+		goto put_folio;
 	}
 
 	import_uuid(&sbi->s_uuid, super->s_uuid);
 	ret = 0;
 
-free_page:
-	__free_page(page);
+put_folio:
+	folio_put(folio);
 
 	return ret;
 }