diff mbox series

[1/2] btrfs: Remove logic for !crc_check case

Message ID 20201203080949.3759006-1-nborisov@suse.com (mailing list archive)
State New, archived
Headers show
Series [1/2] btrfs: Remove logic for !crc_check case | expand

Commit Message

Nikolay Borisov Dec. 3, 2020, 8:09 a.m. UTC 
Following removal of the ino cache io_ctl_init will be called only on
behalf of the freespace inode. In this case we always want to check
crcs so conditional code that dependen on io_ctl::check_crc can be
removed.

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
---
David,

Here are 2 patches that followup based on your feedback of the ino removal
feature, I have run a full xfstest run to validate them.

 fs/btrfs/free-space-cache.c | 38 ++++++-------------------------------
 fs/btrfs/free-space-cache.h |  1 -
 2 files changed, 6 insertions(+), 33 deletions(-)

--
2.25.1

Comments

David Sterba Dec. 3, 2020, 4:33 p.m. UTC | #1 
On Thu, Dec 03, 2020 at 10:09:48AM +0200, Nikolay Borisov wrote:
> Following removal of the ino cache io_ctl_init will be called only on
> behalf of the freespace inode. In this case we always want to check
> crcs so conditional code that dependen on io_ctl::check_crc can be
> removed.
> 
> Signed-off-by: Nikolay Borisov <nborisov@suse.com>
> ---
> David,
> 
> Here are 2 patches that followup based on your feedback of the ino removal
> feature, I have run a full xfstest run to validate them.

Thanks, added after the ino removal in misc-next.
diff mbox series

Patch

diff --git a/fs/btrfs/free-space-cache.c b/fs/btrfs/free-space-cache.c
index cd5996350cf0..e83b3bdc4e46 100644
--- a/fs/btrfs/free-space-cache.c
+++ b/fs/btrfs/free-space-cache.c
@@ -364,16 +364,11 @@  static int io_ctl_init(struct btrfs_io_ctl *io_ctl, struct inode *inode,
 		       int write)
 {
 	int num_pages;
-	int check_crcs = 0;

 	num_pages = DIV_ROUND_UP(i_size_read(inode), PAGE_SIZE);

-	if (btrfs_ino(BTRFS_I(inode)) != BTRFS_FREE_INO_OBJECTID)
-		check_crcs = 1;
-
 	/* Make sure we can fit our crcs and generation into the first page */
-	if (write && check_crcs &&
-	    (num_pages * sizeof(u32) + sizeof(u64)) > PAGE_SIZE)
+	if (write && (num_pages * sizeof(u32) + sizeof(u64)) > PAGE_SIZE)
 		return -ENOSPC;

 	memset(io_ctl, 0, sizeof(struct btrfs_io_ctl));
@@ -384,7 +379,6 @@  static int io_ctl_init(struct btrfs_io_ctl *io_ctl, struct inode *inode,

 	io_ctl->num_pages = num_pages;
 	io_ctl->fs_info = btrfs_sb(inode->i_sb);
-	io_ctl->check_crcs = check_crcs;
 	io_ctl->inode = inode;

 	return 0;
@@ -479,13 +473,8 @@  static void io_ctl_set_generation(struct btrfs_io_ctl *io_ctl, u64 generation)
 	 * Skip the csum areas.  If we don't check crcs then we just have a
 	 * 64bit chunk at the front of the first page.
 	 */
-	if (io_ctl->check_crcs) {
-		io_ctl->cur += (sizeof(u32) * io_ctl->num_pages);
-		io_ctl->size -= sizeof(u64) + (sizeof(u32) * io_ctl->num_pages);
-	} else {
-		io_ctl->cur += sizeof(u64);
-		io_ctl->size -= sizeof(u64) * 2;
-	}
+	io_ctl->cur += (sizeof(u32) * io_ctl->num_pages);
+	io_ctl->size -= sizeof(u64) + (sizeof(u32) * io_ctl->num_pages);

 	put_unaligned_le64(generation, io_ctl->cur);
 	io_ctl->cur += sizeof(u64);
@@ -499,14 +488,9 @@  static int io_ctl_check_generation(struct btrfs_io_ctl *io_ctl, u64 generation)
 	 * Skip the crc area.  If we don't check crcs then we just have a 64bit
 	 * chunk at the front of the first page.
 	 */
-	if (io_ctl->check_crcs) {
-		io_ctl->cur += sizeof(u32) * io_ctl->num_pages;
-		io_ctl->size -= sizeof(u64) +
-			(sizeof(u32) * io_ctl->num_pages);
-	} else {
-		io_ctl->cur += sizeof(u64);
-		io_ctl->size -= sizeof(u64) * 2;
-	}
+	io_ctl->cur += sizeof(u32) * io_ctl->num_pages;
+	io_ctl->size -= sizeof(u64) +
+		(sizeof(u32) * io_ctl->num_pages);

 	cache_gen = get_unaligned_le64(io_ctl->cur);
 	if (cache_gen != generation) {
@@ -526,11 +510,6 @@  static void io_ctl_set_crc(struct btrfs_io_ctl *io_ctl, int index)
 	u32 crc = ~(u32)0;
 	unsigned offset = 0;

-	if (!io_ctl->check_crcs) {
-		io_ctl_unmap_page(io_ctl);
-		return;
-	}
-
 	if (index == 0)
 		offset = sizeof(u32) * io_ctl->num_pages;

@@ -548,11 +527,6 @@  static int io_ctl_check_crc(struct btrfs_io_ctl *io_ctl, int index)
 	u32 crc = ~(u32)0;
 	unsigned offset = 0;

-	if (!io_ctl->check_crcs) {
-		io_ctl_map_page(io_ctl, 0);
-		return 0;
-	}
-
 	if (index == 0)
 		offset = sizeof(u32) * io_ctl->num_pages;

diff --git a/fs/btrfs/free-space-cache.h b/fs/btrfs/free-space-cache.h
index a65ed1967d5d..33687b0b8e7a 100644
--- a/fs/btrfs/free-space-cache.h
+++ b/fs/btrfs/free-space-cache.h
@@ -76,7 +76,6 @@  struct btrfs_io_ctl {
 	int num_pages;
 	int entries;
 	int bitmaps;
-	unsigned check_crcs:1;
 };

 struct inode *lookup_free_space_inode(struct btrfs_block_group *block_group,