diff mbox

[2/3] btrfs-progs: fix backref after init-csum-tree

Message ID 1385964715-23355-2-git-send-email-anand.jain@oracle.com (mailing list archive)
State Accepted, archived
Headers show

Commit Message

Anand Jain Dec. 2, 2013, 6:11 a.m. UTC
btrfsck reports backref error after running init-csum-tree

btrfsck --init-csum-tree /dev/sdc
btrfsck /dev/sdc
::
ref mismatch on [29474816 16384] extent item 1, found 0
Backref 29474816 root 7 not referenced back 0x1101d30
Incorrect global backref count on 29474816 found 1 wanted 0
backpointer mismatch on [29474816 16384]
owner ref check failed [29474816 16384]
Errors found in extent allocation tree or chunk allocation
::

Signed-off-by: Anand Jain <anand.jain@oracle.com>
---
 cmds-check.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

Comments

Anand Jain Dec. 2, 2013, 6:12 a.m. UTC | #1
Further this needs your review. How should be a complete use case
  of this init-csum-tree feature. mainly how do we expect this work
  in the real scenario. what problem does this solve.

Thanks, Anand


On 12/02/13 02:11 PM, Anand Jain wrote:
> btrfsck reports backref error after running init-csum-tree
>
> btrfsck --init-csum-tree /dev/sdc
> btrfsck /dev/sdc
> ::
> ref mismatch on [29474816 16384] extent item 1, found 0
> Backref 29474816 root 7 not referenced back 0x1101d30
> Incorrect global backref count on 29474816 found 1 wanted 0
> backpointer mismatch on [29474816 16384]
> owner ref check failed [29474816 16384]
> Errors found in extent allocation tree or chunk allocation
> ::
>
> Signed-off-by: Anand Jain <anand.jain@oracle.com>
> ---
>   cmds-check.c |    6 ++++++
>   1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/cmds-check.c b/cmds-check.c
> index afad8a1..6a8d351 100644
> --- a/cmds-check.c
> +++ b/cmds-check.c
> @@ -6161,6 +6161,7 @@ int cmd_check(int argc, char **argv)
>   		} else if (option_index == 2) {
>   			printf("Creating a new CRC tree\n");
>   			init_csum_tree = 1;
> +			repair = 1;
>   			ctree_flags |= OPEN_CTREE_WRITES;
>   		} else if (option_index == 3) {
>   			init_extent_tree = 1;
> @@ -6234,6 +6235,11 @@ int cmd_check(int argc, char **argv)
>   		ret = btrfs_commit_transaction(trans, info->csum_root);
>   		if (ret)
>   			exit(1);
> +
> +		ret = check_chunks_and_extents(root);
> +		if (ret)
> +			fprintf(stderr,
> +				"Errors found in extent allocation tree or chunk allocation\n");
>   		goto out;
>   	}
>   	ret = check_chunks_and_extents(root);
>
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/cmds-check.c b/cmds-check.c
index afad8a1..6a8d351 100644
--- a/cmds-check.c
+++ b/cmds-check.c
@@ -6161,6 +6161,7 @@  int cmd_check(int argc, char **argv)
 		} else if (option_index == 2) {
 			printf("Creating a new CRC tree\n");
 			init_csum_tree = 1;
+			repair = 1;
 			ctree_flags |= OPEN_CTREE_WRITES;
 		} else if (option_index == 3) {
 			init_extent_tree = 1;
@@ -6234,6 +6235,11 @@  int cmd_check(int argc, char **argv)
 		ret = btrfs_commit_transaction(trans, info->csum_root);
 		if (ret)
 			exit(1);
+
+		ret = check_chunks_and_extents(root);
+		if (ret)
+			fprintf(stderr,
+				"Errors found in extent allocation tree or chunk allocation\n");
 		goto out;
 	}
 	ret = check_chunks_and_extents(root);