alex/bcachefs-tools
1
0
Fork 0
mirror of https://github.com/koverstreet/bcachefs-tools.git synced 2025-02-23 00:00:02 +03:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

cmd_dump: Dump btree nodes correctly

Browse Source 
We were using sectors_written from the parent node - whoops

Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
This commit is contained in:
Kent Overstreet 2024-05-24 18:06:33 -04:00
parent 3f531df2b3
commit 9cd7440649
2 changed files with 16 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
c_src/cmd_dump.c
View File

@ -30,6 +30,16 @@ static void dump_usage(void)
"Report bugs to <linux-bcachefs@vger.kernel.org>"); "Report bugs to <linux-bcachefs@vger.kernel.org>");
} }
static void dump_node(struct bch_fs *c, struct bch_dev *ca, struct bkey_s_c k, ranges *data)
{
struct bkey_ptrs_c ptrs = bch2_bkey_ptrs_c(k);
bkey_for_each_ptr(ptrs, ptr)
if (ptr->dev == ca->dev_idx)
range_add(data, ptr->offset << 9,
(btree_ptr_sectors_written(k) << 9) ?: c->opts.btree_node_size);
}
static void dump_one_device(struct bch_fs *c, struct bch_dev *ca, int fd, static void dump_one_device(struct bch_fs *c, struct bch_dev *ca, int fd,
bool entire_journal) bool entire_journal)
{ {
@ -60,7 +70,6 @@ static void dump_one_device(struct bch_fs *c, struct bch_dev *ca, int fd,
/* Btree: */ /* Btree: */
for (i = 0; i < BTREE_ID_NR; i++) { for (i = 0; i < BTREE_ID_NR; i++) {
struct bkey_ptrs_c ptrs;
struct btree_trans *trans = bch2_trans_get(c); struct btree_trans *trans = bch2_trans_get(c);
struct btree_iter iter; struct btree_iter iter;
struct btree *b; struct btree *b;
@ -70,37 +79,22 @@ static void dump_one_device(struct bch_fs *c, struct bch_dev *ca, int fd,
struct bkey u; struct bkey u;
struct bkey_s_c k; struct bkey_s_c k;
for_each_btree_node_key_unpack(b, k, &iter, &u) { for_each_btree_node_key_unpack(b, k, &iter, &u)
ptrs = bch2_bkey_ptrs_c(k); dump_node(c, ca, k, &data);
bkey_for_each_ptr(ptrs, ptr)
if (ptr->dev == ca->dev_idx)
range_add(&data,
ptr->offset << 9,
btree_ptr_sectors_written(&b->key) << 9);
}
} }
if (ret) if (ret)
die("error %s walking btree nodes", bch2_err_str(ret)); die("error %s walking btree nodes", bch2_err_str(ret));
b = bch2_btree_id_root(c, i)->b; b = bch2_btree_id_root(c, i)->b;
if (!btree_node_fake(b)) { if (!btree_node_fake(b))
ptrs = bch2_bkey_ptrs_c(bkey_i_to_s_c(&b->key)); dump_node(c, ca, bkey_i_to_s_c(&b->key), &data);
bkey_for_each_ptr(ptrs, ptr)
if (ptr->dev == ca->dev_idx)
range_add(&data,
ptr->offset << 9,
btree_ptr_sectors_written(&b->key) << 9);
}
bch2_trans_iter_exit(trans, &iter); bch2_trans_iter_exit(trans, &iter);
bch2_trans_put(trans); bch2_trans_put(trans);
} }
qcow2_write_image(ca->disk_sb.bdev->bd_fd, fd, &data, qcow2_write_image(ca->disk_sb.bdev->bd_fd, fd, &data, block_bytes(c));
max_t(unsigned, c->opts.btree_node_size / 8, block_bytes(c)));
darray_exit(&data); darray_exit(&data);
} }