]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
btrfs: tests: print all values as decimal in messages for extent map tests
authorFilipe Manana <fdmanana@suse.com>
Mon, 4 Dec 2023 16:20:26 +0000 (16:20 +0000)
committerDavid Sterba <dsterba@suse.com>
Fri, 15 Dec 2023 21:59:01 +0000 (22:59 +0100)
Some error messages of the extent map tests print decimal values of start
offsets and lengths, while other are oddly printing in hexadecimal, which
is far less human friendly, specially taking into consideration that all
the values are small and multiples of 4K, so it's a lot easier to read
them as decimal values. Change the format specifiers to print as decimal
instead.

Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/tests/extent-map-tests.c

index 024588d025513385c046a672306a308574972028..1eb442ea89a529c609eceaa1d75183066c1a2381 100644 (file)
@@ -25,7 +25,7 @@ static void free_extent_map_tree(struct extent_map_tree *em_tree)
 #ifdef CONFIG_BTRFS_DEBUG
                if (refcount_read(&em->refs) != 1) {
                        test_err(
-"em leak: em (start 0x%llx len 0x%llx block_start 0x%llx block_len 0x%llx) refs %d",
+"em leak: em (start %llu len %llu block_start %llu block_len %llu) refs %d",
                                 em->start, em->len, em->block_start,
                                 em->block_len, refcount_read(&em->refs));
 
@@ -277,12 +277,12 @@ static int __test_case_3(struct btrfs_fs_info *fs_info,
        ret = btrfs_add_extent_mapping(fs_info, em_tree, &em, start, len);
        write_unlock(&em_tree->lock);
        if (ret) {
-               test_err("case3 [0x%llx 0x%llx): ret %d",
+               test_err("case3 [%llu %llu): ret %d",
                         start, start + len, ret);
                goto out;
        }
        if (!em) {
-               test_err("case3 [0x%llx 0x%llx): no extent map returned",
+               test_err("case3 [%llu %llu): no extent map returned",
                         start, start + len);
                ret = -ENOENT;
                goto out;
@@ -294,7 +294,7 @@ static int __test_case_3(struct btrfs_fs_info *fs_info,
        if (start < em->start || start + len > extent_map_end(em) ||
            em->start != em->block_start || em->len != em->block_len) {
                test_err(
-"case3 [0x%llx 0x%llx): ret %d em (start 0x%llx len 0x%llx block_start 0x%llx block_len 0x%llx)",
+"case3 [%llu %llu): ret %d em (start %llu len %llu block_start %llu block_len %llu)",
                         start, start + len, ret, em->start, em->len,
                         em->block_start, em->block_len);
                ret = -EINVAL;
@@ -401,19 +401,19 @@ static int __test_case_4(struct btrfs_fs_info *fs_info,
        ret = btrfs_add_extent_mapping(fs_info, em_tree, &em, start, len);
        write_unlock(&em_tree->lock);
        if (ret) {
-               test_err("case4 [0x%llx 0x%llx): ret %d",
+               test_err("case4 [%llu %llu): ret %d",
                         start, start + len, ret);
                goto out;
        }
        if (!em) {
-               test_err("case4 [0x%llx 0x%llx): no extent map returned",
+               test_err("case4 [%llu %llu): no extent map returned",
                         start, start + len);
                ret = -ENOENT;
                goto out;
        }
        if (start < em->start || start + len > extent_map_end(em)) {
                test_err(
-"case4 [0x%llx 0x%llx): ret %d, added wrong em (start 0x%llx len 0x%llx block_start 0x%llx block_len 0x%llx)",
+"case4 [%llu %llu): ret %d, added wrong em (start %llu len %llu block_start %llu block_len %llu)",
                         start, start + len, ret, em->start, em->len, em->block_start,
                         em->block_len);
                ret = -EINVAL;