diff mbox

BUG: use metadata_profile instead of data_profile

Message ID 1418472351-5113-2-git-send-email-kreijack@inwind.it (mailing list archive)
State New, archived
Headers show

Commit Message

Goffredo Baroncelli Dec. 13, 2014, 12:05 p.m. UTC
The function test_num_disk_vs_raid() show an error message if
the raid profile is incompatible with the number of devices.

Unfortunately when the error is related to data profile,
the message print the metadata profile.

This patch correct the bug.

Signed-off-by: Goffredo Baroncelli <kreijack@inwind.it>
---
 utils.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

David Sterba Dec. 17, 2014, 2:32 p.m. UTC | #1
On Sat, Dec 13, 2014 at 01:05:51PM +0100, Goffredo Baroncelli wrote:
> The function test_num_disk_vs_raid() show an error message if
> the raid profile is incompatible with the number of devices.
> 
> Unfortunately when the error is related to data profile,
> the message print the metadata profile.

Already fixed by 373110d84f3a1c5bdd305d2e6dcaeb98b5e6dbe7.
--
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/utils.c b/utils.c
index fc69e9b..e1b56ac 100644
--- a/utils.c
+++ b/utils.c
@@ -2116,7 +2116,7 @@  int test_num_disk_vs_raid(u64 metadata_profile, u64 data_profile,
 	if (data_profile & ~allowed) {
 		snprintf(estr, sz, "unable to create FS with data "
 			"profile %llu (have %llu devices)\n",
-			metadata_profile, dev_cnt);
+			data_profile, dev_cnt);
 		return 1;
 	}