The progress indicator as it is now is quite confusing, it tends to not go to 100% or it resets to 0%, probably when doing multiple passes. This should be fixed properly, maybe also showing which pass it is doing. But for now this patch just rips out the percentage display. No information is better than confusing information.
Signed-off-by: Malte Schröder <[email protected]> --- fs/bcachefs/backpointers.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/fs/bcachefs/backpointers.c b/fs/bcachefs/backpointers.c index ebeb6a5ff9d2..c5cc3aa04868 100644 --- a/fs/bcachefs/backpointers.c +++ b/fs/bcachefs/backpointers.c @@ -767,12 +767,9 @@ static void progress_update_iter(struct btree_trans *trans, if (progress_update_p(s)) { struct printbuf buf = PRINTBUF; - unsigned percent = s->nodes_total - ? div64_u64(s->nodes_seen * 100, s->nodes_total) - : 0; - prt_printf(&buf, "%s: %d%%, done %llu/%llu nodes, at ", - msg, percent, s->nodes_seen, s->nodes_total); + prt_printf(&buf, "%s: done %llu/%llu nodes, at ", + msg, s->nodes_seen, s->nodes_total); bch2_bbpos_to_text(&buf, BBPOS(iter->btree_id, iter->pos)); bch_info(c, "%s", buf.buf); -- 2.47.1
