On 6/11/14, 9:25 PM, Gui Hecheng wrote: > When run chunk-recover on a health btrfs(data profile raid0, with > plenty of data), the program has a chance to abort on the number > of mirrors of an extent. > > According to the kernel code, the max mirror number of an extent > is 3 not 2: > ctree.h: BTRFS_MAX_MIRRORS 3 > chunk-recover.c : BTRFS_NUM_MIRRORS 2 > just change BTRFS_NUM_MIRRORS to 3, and everything goes well.
Wouldn't it make a lot more sense, then, to change the userspace macro to be called BTRFS_MAX_MIRRORS as well? -Eric > Signed-off-by: Gui Hecheng <guihc.f...@cn.fujitsu.com> > --- > chunk-recover.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/chunk-recover.c b/chunk-recover.c > index 9b46b0b..d5a688e 100644 > --- a/chunk-recover.c > +++ b/chunk-recover.c > @@ -42,7 +42,7 @@ > #include "btrfsck.h" > #include "commands.h" > > -#define BTRFS_NUM_MIRRORS 2 > +#define BTRFS_NUM_MIRRORS 3 > > struct recover_control { > int verbose; > -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html