File btrfs-progs-convert-do-not-clear-header-rev.patch of Package btrfsprogs

Overview Repositories Revisions Requests Users Attributes Meta

File btrfs-progs-convert-do-not-clear-header-rev.patch of Package btrfsprogs (Revision 284f19b03a5b80740585ce2295b7a003)

Currently displaying revision 284f19b03a5b80740585ce2295b7a003 , Show latest

From: Liu Bo <bo.li.liu@oracle.com>
Subject: Btrfs-progs: convert: do not clear header rev
Git-commit: 1d46d6ba2c888b55c064d5c3c4cb49902e201279
Patch-mainline: v4.11.1
References: bsc#1042369

So btrfs_set_header_flags() vs btrfs_set_header_flag, the difference is
sort of similar to "=" vs "|=", when creating and initialising a new
extent buffer, convert uses the former one which clears header_rev by
accident.

Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
Reviewed-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Acked-by: Jeff Mahoney <jeffm@suse.com>
---
 utils.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/utils.c b/utils.c
index 40bf32cf..f0dd2cfc 100644
--- a/utils.c
+++ b/utils.c
@@ -167,7 +167,7 @@ static int setup_temp_extent_buffer(struct extent_buffer *buf,
 	btrfs_set_header_generation(buf, 1);
 	btrfs_set_header_backref_rev(buf, BTRFS_MIXED_BACKREF_REV);
 	btrfs_set_header_owner(buf, owner);
-	btrfs_set_header_flags(buf, BTRFS_HEADER_FLAG_WRITTEN);
+	btrfs_set_header_flag(buf, BTRFS_HEADER_FLAG_WRITTEN);
 	write_extent_buffer(buf, chunk_uuid, btrfs_header_chunk_tree_uuid(buf),
 			    BTRFS_UUID_SIZE);
 	write_extent_buffer(buf, fsid, btrfs_header_fsid(), BTRFS_FSID_SIZE);

Places

File btrfs-progs-convert-do-not-clear-header-rev.patch of Package btrfsprogs (Revision 284f19b03a5b80740585ce2295b7a003)

Places