diff mbox series

btrfs: subpage: remove the unused error bitmap dumpping

Message ID b965322f7a4825bc5f4a094f3138e1369534f8bb.1717966923.git.wqu@suse.com (mailing list archive)
State New
Headers show
Series btrfs: subpage: remove the unused error bitmap dumpping | expand

Commit Message

Qu Wenruo June 9, 2024, 9:02 p.m. UTC 
Since commit 2b2553f12355 ("btrfs: stop setting PageError in the data I/O
path") btrfs no longer utilize subpage error bitmaps anymore, but the
commit forgot to remove the error bitmap in btrfs_subpage_dump_bitmap(),
resulting possible meaningless result for the error bitmap.

Fix it by just removing the error bitmap dumping.

Signed-off-by: Qu Wenruo <wqu@suse.com>
---
 fs/btrfs/subpage.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

David Sterba June 12, 2024, 7:52 p.m. UTC | #1 
On Mon, Jun 10, 2024 at 06:32:07AM +0930, Qu Wenruo wrote:
> Since commit 2b2553f12355 ("btrfs: stop setting PageError in the data I/O
> path") btrfs no longer utilize subpage error bitmaps anymore, but the
> commit forgot to remove the error bitmap in btrfs_subpage_dump_bitmap(),
> resulting possible meaningless result for the error bitmap.
> 
> Fix it by just removing the error bitmap dumping.
> 
> Signed-off-by: Qu Wenruo <wqu@suse.com>

Reviewed-by: David Sterba <dsterba@suse.com>
diff mbox series

Patch

diff --git a/fs/btrfs/subpage.c b/fs/btrfs/subpage.c
index fc7db52e8f58..1a4717bcce23 100644
--- a/fs/btrfs/subpage.c
+++ b/fs/btrfs/subpage.c
@@ -909,7 +909,6 @@  void __cold btrfs_subpage_dump_bitmap(const struct btrfs_fs_info *fs_info,
 	struct btrfs_subpage_info *subpage_info = fs_info->subpage_info;
 	struct btrfs_subpage *subpage;
 	unsigned long uptodate_bitmap;
-	unsigned long error_bitmap;
 	unsigned long dirty_bitmap;
 	unsigned long writeback_bitmap;
 	unsigned long ordered_bitmap;
@@ -931,10 +930,9 @@  void __cold btrfs_subpage_dump_bitmap(const struct btrfs_fs_info *fs_info,
 
 	dump_page(folio_page(folio, 0), "btrfs subpage dump");
 	btrfs_warn(fs_info,
-"start=%llu len=%u page=%llu, bitmaps uptodate=%*pbl error=%*pbl dirty=%*pbl writeback=%*pbl ordered=%*pbl checked=%*pbl",
+"start=%llu len=%u page=%llu, bitmaps uptodate=%*pbl dirty=%*pbl writeback=%*pbl ordered=%*pbl checked=%*pbl",
 		    start, len, folio_pos(folio),
 		    subpage_info->bitmap_nr_bits, &uptodate_bitmap,
-		    subpage_info->bitmap_nr_bits, &error_bitmap,
 		    subpage_info->bitmap_nr_bits, &dirty_bitmap,
 		    subpage_info->bitmap_nr_bits, &writeback_bitmap,
 		    subpage_info->bitmap_nr_bits, &ordered_bitmap,