]> git.sesse.net Git - bcachefs-tools-debian/blobdiff - libbcachefs/journal.c
Update bcachefs sources to 3856459b1b bcachefs: bch2_btree_iter_peek_node_and_restart()
[bcachefs-tools-debian] / libbcachefs / journal.c
index 340f0bed7391f11cbca3e850d471a37ccf88ae84..e0c4f51a1fde4862165dab8b53348e32fd2de26c 100644 (file)
 
 #include <trace/events/bcachefs.h>
 
+#define x(n)   #n,
+static const char * const bch2_journal_watermarks[] = {
+       JOURNAL_WATERMARKS()
+       NULL
+};
+
+static const char * const bch2_journal_errors[] = {
+       JOURNAL_ERRORS()
+       NULL
+};
+#undef x
+
 static inline bool journal_seq_unwritten(struct journal *j, u64 seq)
 {
        return seq > j->seq_ondisk;
@@ -187,12 +199,6 @@ static bool journal_entry_close(struct journal *j)
 /*
  * should _only_ called from journal_res_get() - when we actually want a
  * journal reservation - journal entry is open means journal is dirty:
- *
- * returns:
- * 0:          success
- * -ENOSPC:    journal currently full, must invoke reclaim
- * -EAGAIN:    journal blocked, must wait
- * -EROFS:     insufficient rw devices or journal error
  */
 static int journal_entry_open(struct journal *j)
 {
@@ -208,19 +214,19 @@ static int journal_entry_open(struct journal *j)
        BUG_ON(BCH_SB_CLEAN(c->disk_sb.sb));
 
        if (j->blocked)
-               return cur_entry_blocked;
+               return JOURNAL_ERR_blocked;
 
        if (j->cur_entry_error)
                return j->cur_entry_error;
 
        if (bch2_journal_error(j))
-               return cur_entry_insufficient_devices; /* -EROFS */
+               return JOURNAL_ERR_insufficient_devices; /* -EROFS */
 
        if (!fifo_free(&j->pin))
-               return cur_entry_journal_pin_full;
+               return JOURNAL_ERR_journal_pin_full;
 
-       if (nr_unwritten_journal_entries(j) == ARRAY_SIZE(j->buf) - 1)
-               return cur_entry_max_in_flight;
+       if (nr_unwritten_journal_entries(j) == ARRAY_SIZE(j->buf))
+               return JOURNAL_ERR_max_in_flight;
 
        BUG_ON(!j->cur_entry_sectors);
 
@@ -238,8 +244,8 @@ static int journal_entry_open(struct journal *j)
                journal_entry_overhead(j);
        u64s = clamp_t(int, u64s, 0, JOURNAL_ENTRY_CLOSED_VAL - 1);
 
-       if (u64s <= 0)
-               return cur_entry_journal_full;
+       if (u64s <= (ssize_t) j->early_journal_entries.nr)
+               return JOURNAL_ERR_journal_full;
 
        if (fifo_empty(&j->pin) && j->reclaim_thread)
                wake_up_process(j->reclaim_thread);
@@ -263,6 +269,12 @@ static int journal_entry_open(struct journal *j)
        buf->data->seq  = cpu_to_le64(journal_cur_seq(j));
        buf->data->u64s = 0;
 
+       if (j->early_journal_entries.nr) {
+               memcpy(buf->data->_data, j->early_journal_entries.data,
+                      j->early_journal_entries.nr * sizeof(u64));
+               le32_add_cpu(&buf->data->u64s, j->early_journal_entries.nr);
+       }
+
        /*
         * Must be set before marking the journal entry as open:
         */
@@ -279,7 +291,9 @@ static int journal_entry_open(struct journal *j)
                BUG_ON(new.idx != (journal_cur_seq(j) & JOURNAL_BUF_MASK));
 
                journal_state_inc(&new);
-               new.cur_entry_offset = 0;
+
+               /* Handle any already added entries */
+               new.cur_entry_offset = le32_to_cpu(buf->data->u64s);
        } while ((v = atomic64_cmpxchg(&j->reservations.counter,
                                       old.v, new.v)) != old.v);
 
@@ -292,6 +306,9 @@ static int journal_entry_open(struct journal *j)
                         &j->write_work,
                         msecs_to_jiffies(c->opts.journal_flush_delay));
        journal_wake(j);
+
+       if (j->early_journal_entries.nr)
+               darray_exit(&j->early_journal_entries);
        return 0;
 }
 
@@ -341,7 +358,7 @@ retry:
                return 0;
 
        if (bch2_journal_error(j))
-               return -EROFS;
+               return -BCH_ERR_erofs_journal_err;
 
        spin_lock(&j->lock);
 
@@ -355,13 +372,12 @@ retry:
                return 0;
        }
 
-       if (!(flags & JOURNAL_RES_GET_RESERVED) &&
-           !test_bit(JOURNAL_MAY_GET_UNRESERVED, &j->flags)) {
+       if ((flags & JOURNAL_WATERMARK_MASK) < j->watermark) {
                /*
                 * Don't want to close current journal entry, just need to
                 * invoke reclaim:
                 */
-               ret = cur_entry_journal_full;
+               ret = JOURNAL_ERR_journal_full;
                goto unlock;
        }
 
@@ -379,13 +395,13 @@ retry:
        __journal_entry_close(j, JOURNAL_ENTRY_CLOSED_VAL);
        ret = journal_entry_open(j);
 
-       if (ret == cur_entry_max_in_flight)
-               trace_journal_entry_full(c);
+       if (ret == JOURNAL_ERR_max_in_flight)
+               trace_and_count(c, journal_entry_full, c);
 unlock:
-       if ((ret && ret != cur_entry_insufficient_devices) &&
+       if ((ret && ret != JOURNAL_ERR_insufficient_devices) &&
            !j->res_get_blocked_start) {
                j->res_get_blocked_start = local_clock() ?: 1;
-               trace_journal_full(c);
+               trace_and_count(c, journal_full, c);
        }
 
        can_discard = j->can_discard;
@@ -394,14 +410,15 @@ unlock:
        if (!ret)
                goto retry;
 
-       if ((ret == cur_entry_journal_full ||
-            ret == cur_entry_journal_pin_full) &&
+       if ((ret == JOURNAL_ERR_journal_full ||
+            ret == JOURNAL_ERR_journal_pin_full) &&
            !can_discard &&
            !nr_unwritten_journal_entries(j) &&
-           (flags & JOURNAL_RES_GET_RESERVED)) {
+           (flags & JOURNAL_WATERMARK_MASK) == JOURNAL_WATERMARK_reserved) {
                struct printbuf buf = PRINTBUF;
 
-               bch_err(c, "Journal stuck! Hava a pre-reservation but journal full");
+               bch_err(c, "Journal stuck! Hava a pre-reservation but journal full (ret %s)",
+                       bch2_journal_errors[ret]);
 
                bch2_journal_debug_to_text(&buf, j);
                bch_err(c, "%s", buf.buf);
@@ -419,8 +436,8 @@ unlock:
         * Journal is full - can't rely on reclaim from work item due to
         * freezing:
         */
-       if ((ret == cur_entry_journal_full ||
-            ret == cur_entry_journal_pin_full) &&
+       if ((ret == JOURNAL_ERR_journal_full ||
+            ret == JOURNAL_ERR_journal_pin_full) &&
            !(flags & JOURNAL_RES_GET_NONBLOCK)) {
                if (can_discard) {
                        bch2_journal_do_discards(j);
@@ -433,7 +450,9 @@ unlock:
                }
        }
 
-       return ret == cur_entry_insufficient_devices ? -EROFS : -EAGAIN;
+       return ret == JOURNAL_ERR_insufficient_devices
+               ? -EROFS
+               : -BCH_ERR_journal_res_get_blocked;
 }
 
 /*
@@ -452,7 +471,8 @@ int bch2_journal_res_get_slowpath(struct journal *j, struct journal_res *res,
        int ret;
 
        closure_wait_event(&j->async_wait,
-                  (ret = __journal_res_get(j, res, flags)) != -EAGAIN ||
+                  (ret = __journal_res_get(j, res, flags)) !=
+                  -BCH_ERR_journal_res_get_blocked||
                   (flags & JOURNAL_RES_GET_NONBLOCK));
        return ret;
 }
@@ -708,39 +728,6 @@ int bch2_journal_meta(struct journal *j)
        return bch2_journal_flush_seq(j, res.seq);
 }
 
-int bch2_journal_log_msg(struct journal *j, const char *fmt, ...)
-{
-       struct jset_entry_log *entry;
-       struct journal_res res = { 0 };
-       unsigned msglen, u64s;
-       va_list args;
-       int ret;
-
-       va_start(args, fmt);
-       msglen = vsnprintf(NULL, 0, fmt, args) + 1;
-       va_end(args);
-
-       u64s = jset_u64s(DIV_ROUND_UP(msglen, sizeof(u64)));
-
-       ret = bch2_journal_res_get(j, &res, u64s, 0);
-       if (ret)
-               return ret;
-
-       entry = container_of(journal_res_entry(j, &res),
-                            struct jset_entry_log, entry);;
-       memset(entry, 0, u64s * sizeof(u64));
-       entry->entry.type = BCH_JSET_ENTRY_log;
-       entry->entry.u64s = u64s - 1;
-
-       va_start(args, fmt);
-       vsnprintf(entry->d, INT_MAX, fmt, args);
-       va_end(args);
-
-       bch2_journal_res_put(j, &res);
-
-       return bch2_journal_flush_seq(j, res.seq);
-}
-
 /* block/unlock the journal: */
 
 void bch2_journal_unblock(struct journal *j)
@@ -767,7 +754,6 @@ static int __bch2_set_nr_journal_buckets(struct bch_dev *ca, unsigned nr,
                                         bool new_fs, struct closure *cl)
 {
        struct bch_fs *c = ca->fs;
-       struct journal *j = &c->journal;
        struct journal_device *ja = &ca->journal;
        u64 *new_bucket_seq = NULL, *new_buckets = NULL;
        struct open_bucket **ob = NULL;
@@ -780,13 +766,16 @@ static int __bch2_set_nr_journal_buckets(struct bch_dev *ca, unsigned nr,
        unsigned old_cur_idx            = ja->cur_idx;
        int ret = 0;
 
-       bch2_journal_block(j);
-       bch2_journal_flush_all_pins(j);
+       if (c) {
+               bch2_journal_flush_all_pins(&c->journal);
+               bch2_journal_block(&c->journal);
+               mutex_lock(&c->sb_lock);
+       }
 
-       bu              = kzalloc(nr_want * sizeof(*bu), GFP_KERNEL);
-       ob              = kzalloc(nr_want * sizeof(*ob), GFP_KERNEL);
-       new_buckets     = kzalloc(nr * sizeof(u64), GFP_KERNEL);
-       new_bucket_seq  = kzalloc(nr * sizeof(u64), GFP_KERNEL);
+       bu              = kcalloc(nr_want, sizeof(*bu), GFP_KERNEL);
+       ob              = kcalloc(nr_want, sizeof(*ob), GFP_KERNEL);
+       new_buckets     = kcalloc(nr, sizeof(u64), GFP_KERNEL);
+       new_bucket_seq  = kcalloc(nr, sizeof(u64), GFP_KERNEL);
        if (!bu || !ob || !new_buckets || !new_bucket_seq) {
                ret = -ENOMEM;
                goto err_unblock;
@@ -796,16 +785,14 @@ static int __bch2_set_nr_journal_buckets(struct bch_dev *ca, unsigned nr,
                if (new_fs) {
                        bu[nr_got] = bch2_bucket_alloc_new_fs(ca);
                        if (bu[nr_got] < 0) {
-                               ret = -ENOSPC;
+                               ret = -BCH_ERR_ENOSPC_bucket_alloc;
                                break;
                        }
                } else {
-                       ob[nr_got] = bch2_bucket_alloc(c, ca, RESERVE_NONE,
-                                              false, cl);
-                       if (IS_ERR(ob[nr_got])) {
-                               ret = cl ? -EAGAIN : -ENOSPC;
+                       ob[nr_got] = bch2_bucket_alloc(c, ca, RESERVE_none, cl);
+                       ret = PTR_ERR_OR_ZERO(ob[nr_got]);
+                       if (ret)
                                break;
-                       }
 
                        bu[nr_got] = ob[nr_got]->bucket;
                }
@@ -819,7 +806,7 @@ static int __bch2_set_nr_journal_buckets(struct bch_dev *ca, unsigned nr,
         * actually been added to the running filesystem:
         */
        if (!new_fs)
-               spin_lock(&j->lock);
+               spin_lock(&c->journal.lock);
 
        memcpy(new_buckets,     ja->buckets,    ja->nr * sizeof(u64));
        memcpy(new_bucket_seq,  ja->bucket_seq, ja->nr * sizeof(u64));
@@ -860,16 +847,20 @@ static int __bch2_set_nr_journal_buckets(struct bch_dev *ca, unsigned nr,
        }
 
        if (!new_fs)
-               spin_unlock(&j->lock);
+               spin_unlock(&c->journal.lock);
 
-       bch2_journal_unblock(j);
+       if (ja->nr != old_nr && !new_fs)
+               bch2_write_super(c);
+
+       if (c)
+               bch2_journal_unblock(&c->journal);
 
        if (ret)
                goto err;
 
        if (!new_fs) {
                for (i = 0; i < nr_got; i++) {
-                       ret = bch2_trans_do(c, NULL, NULL, BTREE_INSERT_NOFAIL,
+                       ret = bch2_trans_run(c,
                                bch2_trans_mark_metadata_bucket(&trans, ca,
                                                bu[i], BCH_DATA_journal,
                                                ca->mi.bucket_size));
@@ -880,6 +871,9 @@ static int __bch2_set_nr_journal_buckets(struct bch_dev *ca, unsigned nr,
                }
        }
 err:
+       if (c)
+               mutex_unlock(&c->sb_lock);
+
        if (ob && !new_fs)
                for (i = 0; i < nr_got; i++)
                        bch2_open_bucket_put(c, ob[i]);
@@ -891,7 +885,8 @@ err:
 
        return ret;
 err_unblock:
-       bch2_journal_unblock(j);
+       if (c)
+               bch2_journal_unblock(&c->journal);
        goto err;
 }
 
@@ -904,7 +899,6 @@ int bch2_set_nr_journal_buckets(struct bch_fs *c, struct bch_dev *ca,
 {
        struct journal_device *ja = &ca->journal;
        struct closure cl;
-       unsigned current_nr;
        int ret = 0;
 
        /* don't handle reducing nr of buckets yet: */
@@ -913,36 +907,38 @@ int bch2_set_nr_journal_buckets(struct bch_fs *c, struct bch_dev *ca,
 
        closure_init_stack(&cl);
 
-       while (ja->nr != nr && (ret == 0 || ret == -EAGAIN)) {
+       while (ja->nr != nr) {
                struct disk_reservation disk_res = { 0, 0 };
 
-               closure_sync(&cl);
-
-               mutex_lock(&c->sb_lock);
-               current_nr = ja->nr;
-
                /*
                 * note: journal buckets aren't really counted as _sectors_ used yet, so
                 * we don't need the disk reservation to avoid the BUG_ON() in buckets.c
                 * when space used goes up without a reservation - but we do need the
                 * reservation to ensure we'll actually be able to allocate:
+                *
+                * XXX: that's not right, disk reservations only ensure a
+                * filesystem-wide allocation will succeed, this is a device
+                * specific allocation - we can hang here:
                 */
 
-               if (bch2_disk_reservation_get(c, &disk_res,
-                                             bucket_to_sector(ca, nr - ja->nr), 1, 0)) {
-                       mutex_unlock(&c->sb_lock);
-                       return -ENOSPC;
-               }
+               ret = bch2_disk_reservation_get(c, &disk_res,
+                                               bucket_to_sector(ca, nr - ja->nr), 1, 0);
+               if (ret)
+                       break;
 
                ret = __bch2_set_nr_journal_buckets(ca, nr, false, &cl);
 
                bch2_disk_reservation_put(c, &disk_res);
 
-               if (ja->nr != current_nr)
-                       bch2_write_super(c);
-               mutex_unlock(&c->sb_lock);
+               closure_sync(&cl);
+
+               if (ret && ret != -BCH_ERR_bucket_alloc_blocked)
+                       break;
        }
 
+       if (ret)
+               bch_err(c, "%s: err %s", __func__, bch2_err_str(ret));
+
        return ret;
 }
 
@@ -1016,17 +1012,25 @@ void bch2_fs_journal_stop(struct journal *j)
        cancel_delayed_work_sync(&j->write_work);
 }
 
-int bch2_fs_journal_start(struct journal *j, u64 cur_seq,
-                         struct list_head *journal_entries)
+int bch2_fs_journal_start(struct journal *j, u64 cur_seq)
 {
        struct bch_fs *c = container_of(j, struct bch_fs, journal);
        struct journal_entry_pin_list *p;
-       struct journal_replay *i;
+       struct journal_replay *i, **_i;
+       struct genradix_iter iter;
+       bool had_entries = false;
+       unsigned ptr;
        u64 last_seq = cur_seq, nr, seq;
 
-       if (!list_empty(journal_entries))
-               last_seq = le64_to_cpu(list_last_entry(journal_entries,
-                               struct journal_replay, list)->j.last_seq);
+       genradix_for_each_reverse(&c->journal_entries, iter, _i) {
+               i = *_i;
+
+               if (!i || i->ignore)
+                       continue;
+
+               last_seq = le64_to_cpu(i->j.last_seq);
+               break;
+       }
 
        nr = cur_seq - last_seq;
 
@@ -1048,14 +1052,14 @@ int bch2_fs_journal_start(struct journal *j, u64 cur_seq,
        j->pin.back             = cur_seq;
        atomic64_set(&j->seq, cur_seq - 1);
 
-       if (list_empty(journal_entries))
-               j->last_empty_seq = cur_seq - 1;
-
        fifo_for_each_entry_ptr(p, &j->pin, seq)
                journal_pin_list_init(p, 1);
 
-       list_for_each_entry(i, journal_entries, list) {
-               unsigned ptr;
+       genradix_for_each(&c->journal_entries, iter, _i) {
+               i = *_i;
+
+               if (!i || i->ignore)
+                       continue;
 
                seq = le64_to_cpu(i->j.seq);
                BUG_ON(seq >= cur_seq);
@@ -1071,9 +1075,11 @@ int bch2_fs_journal_start(struct journal *j, u64 cur_seq,
                p->devs.nr = 0;
                for (ptr = 0; ptr < i->nr_ptrs; ptr++)
                        bch2_dev_list_add_dev(&p->devs, i->ptrs[ptr].dev);
+
+               had_entries = true;
        }
 
-       if (list_empty(journal_entries))
+       if (!had_entries)
                j->last_empty_seq = cur_seq;
 
        spin_lock(&j->lock);
@@ -1112,7 +1118,7 @@ int bch2_dev_journal_init(struct bch_dev *ca, struct bch_sb *sb)
                bch2_sb_get_journal(sb);
        struct bch_sb_field_journal_v2 *journal_buckets_v2 =
                bch2_sb_get_journal_v2(sb);
-       unsigned i;
+       unsigned i, nr_bvecs;
 
        ja->nr = 0;
 
@@ -1129,11 +1135,14 @@ int bch2_dev_journal_init(struct bch_dev *ca, struct bch_sb *sb)
        if (!ja->bucket_seq)
                return -ENOMEM;
 
-       ca->journal.bio = bio_kmalloc(GFP_KERNEL,
-                       DIV_ROUND_UP(JOURNAL_ENTRY_SIZE_MAX, PAGE_SIZE));
+       nr_bvecs = DIV_ROUND_UP(JOURNAL_ENTRY_SIZE_MAX, PAGE_SIZE);
+
+       ca->journal.bio = bio_kmalloc(nr_bvecs, GFP_KERNEL);
        if (!ca->journal.bio)
                return -ENOMEM;
 
+       bio_init(ca->journal.bio, NULL, ca->journal.bio->bi_inline_vecs, nr_bvecs, 0);
+
        ja->buckets = kcalloc(ja->nr, sizeof(u64), GFP_KERNEL);
        if (!ja->buckets)
                return -ENOMEM;
@@ -1158,6 +1167,8 @@ void bch2_fs_journal_exit(struct journal *j)
 {
        unsigned i;
 
+       darray_exit(&j->early_journal_entries);
+
        for (i = 0; i < ARRAY_SIZE(j->buf); i++)
                kvpfree(j->buf[i].data, j->buf[i].buf_size);
        free_fifo(&j->pin);
@@ -1218,89 +1229,91 @@ void __bch2_journal_debug_to_text(struct printbuf *out, struct journal *j)
        u64 seq;
        unsigned i;
 
+       if (!out->nr_tabstops)
+               printbuf_tabstop_push(out, 24);
        out->atomic++;
-       out->tabstops[0] = 24;
 
        rcu_read_lock();
        s = READ_ONCE(j->reservations);
 
-       pr_buf(out, "dirty journal entries:\t%llu\n",   fifo_used(&j->pin));
-       pr_buf(out, "seq:\t\t\t%llu\n",                 journal_cur_seq(j));
-       pr_buf(out, "seq_ondisk:\t\t%llu\n",            j->seq_ondisk);
-       pr_buf(out, "last_seq:\t\t%llu\n",              journal_last_seq(j));
-       pr_buf(out, "last_seq_ondisk:\t%llu\n",         j->last_seq_ondisk);
-       pr_buf(out, "flushed_seq_ondisk:\t%llu\n",      j->flushed_seq_ondisk);
-       pr_buf(out, "prereserved:\t\t%u/%u\n",          j->prereserved.reserved, j->prereserved.remaining);
-       pr_buf(out, "each entry reserved:\t%u\n",       j->entry_u64s_reserved);
-       pr_buf(out, "nr flush writes:\t%llu\n",         j->nr_flush_writes);
-       pr_buf(out, "nr noflush writes:\t%llu\n",       j->nr_noflush_writes);
-       pr_buf(out, "nr direct reclaim:\t%llu\n",       j->nr_direct_reclaim);
-       pr_buf(out, "nr background reclaim:\t%llu\n",   j->nr_background_reclaim);
-       pr_buf(out, "reclaim kicked:\t\t%u\n",          j->reclaim_kicked);
-       pr_buf(out, "reclaim runs in:\t%u ms\n",        time_after(j->next_reclaim, now)
+       prt_printf(out, "dirty journal entries:\t%llu/%llu\n",  fifo_used(&j->pin), j->pin.size);
+       prt_printf(out, "seq:\t\t\t%llu\n",                     journal_cur_seq(j));
+       prt_printf(out, "seq_ondisk:\t\t%llu\n",                j->seq_ondisk);
+       prt_printf(out, "last_seq:\t\t%llu\n",          journal_last_seq(j));
+       prt_printf(out, "last_seq_ondisk:\t%llu\n",             j->last_seq_ondisk);
+       prt_printf(out, "flushed_seq_ondisk:\t%llu\n",  j->flushed_seq_ondisk);
+       prt_printf(out, "prereserved:\t\t%u/%u\n",              j->prereserved.reserved, j->prereserved.remaining);
+       prt_printf(out, "watermark:\t\t%s\n",           bch2_journal_watermarks[j->watermark]);
+       prt_printf(out, "each entry reserved:\t%u\n",   j->entry_u64s_reserved);
+       prt_printf(out, "nr flush writes:\t%llu\n",             j->nr_flush_writes);
+       prt_printf(out, "nr noflush writes:\t%llu\n",   j->nr_noflush_writes);
+       prt_printf(out, "nr direct reclaim:\t%llu\n",   j->nr_direct_reclaim);
+       prt_printf(out, "nr background reclaim:\t%llu\n",       j->nr_background_reclaim);
+       prt_printf(out, "reclaim kicked:\t\t%u\n",              j->reclaim_kicked);
+       prt_printf(out, "reclaim runs in:\t%u ms\n",    time_after(j->next_reclaim, now)
               ? jiffies_to_msecs(j->next_reclaim - jiffies) : 0);
-       pr_buf(out, "current entry sectors:\t%u\n",     j->cur_entry_sectors);
-       pr_buf(out, "current entry error:\t%u\n",       j->cur_entry_error);
-       pr_buf(out, "current entry:\t\t");
+       prt_printf(out, "current entry sectors:\t%u\n", j->cur_entry_sectors);
+       prt_printf(out, "current entry error:\t%s\n",   bch2_journal_errors[j->cur_entry_error]);
+       prt_printf(out, "current entry:\t\t");
 
        switch (s.cur_entry_offset) {
        case JOURNAL_ENTRY_ERROR_VAL:
-               pr_buf(out, "error");
+               prt_printf(out, "error");
                break;
        case JOURNAL_ENTRY_CLOSED_VAL:
-               pr_buf(out, "closed");
+               prt_printf(out, "closed");
                break;
        default:
-               pr_buf(out, "%u/%u", s.cur_entry_offset, j->cur_entry_u64s);
+               prt_printf(out, "%u/%u", s.cur_entry_offset, j->cur_entry_u64s);
                break;
        }
 
-       pr_newline(out);
+       prt_newline(out);
 
        for (seq = journal_cur_seq(j);
             seq >= journal_last_unwritten_seq(j);
             --seq) {
                i = seq & JOURNAL_BUF_MASK;
 
-               pr_buf(out, "unwritten entry:");
-               pr_tab(out);
-               pr_buf(out, "%llu", seq);
-               pr_newline(out);
-               pr_indent_push(out, 2);
+               prt_printf(out, "unwritten entry:");
+               prt_tab(out);
+               prt_printf(out, "%llu", seq);
+               prt_newline(out);
+               printbuf_indent_add(out, 2);
 
-               pr_buf(out, "refcount:");
-               pr_tab(out);
-               pr_buf(out, "%u", journal_state_count(s, i));
-               pr_newline(out);
+               prt_printf(out, "refcount:");
+               prt_tab(out);
+               prt_printf(out, "%u", journal_state_count(s, i));
+               prt_newline(out);
 
-               pr_buf(out, "sectors:");
-               pr_tab(out);
-               pr_buf(out, "%u", j->buf[i].sectors);
-               pr_newline(out);
+               prt_printf(out, "sectors:");
+               prt_tab(out);
+               prt_printf(out, "%u", j->buf[i].sectors);
+               prt_newline(out);
 
-               pr_buf(out, "expires");
-               pr_tab(out);
-               pr_buf(out, "%li jiffies", j->buf[i].expires - jiffies);
-               pr_newline(out);
+               prt_printf(out, "expires");
+               prt_tab(out);
+               prt_printf(out, "%li jiffies", j->buf[i].expires - jiffies);
+               prt_newline(out);
 
-               pr_indent_pop(out, 2);
+               printbuf_indent_sub(out, 2);
        }
 
-       pr_buf(out,
+       prt_printf(out,
               "replay done:\t\t%i\n",
               test_bit(JOURNAL_REPLAY_DONE,    &j->flags));
 
-       pr_buf(out, "space:\n");
-       pr_buf(out, "\tdiscarded\t%u:%u\n",
+       prt_printf(out, "space:\n");
+       prt_printf(out, "\tdiscarded\t%u:%u\n",
               j->space[journal_space_discarded].next_entry,
               j->space[journal_space_discarded].total);
-       pr_buf(out, "\tclean ondisk\t%u:%u\n",
+       prt_printf(out, "\tclean ondisk\t%u:%u\n",
               j->space[journal_space_clean_ondisk].next_entry,
               j->space[journal_space_clean_ondisk].total);
-       pr_buf(out, "\tclean\t\t%u:%u\n",
+       prt_printf(out, "\tclean\t\t%u:%u\n",
               j->space[journal_space_clean].next_entry,
               j->space[journal_space_clean].total);
-       pr_buf(out, "\ttotal\t\t%u:%u\n",
+       prt_printf(out, "\ttotal\t\t%u:%u\n",
               j->space[journal_space_total].next_entry,
               j->space[journal_space_total].total);
 
@@ -1314,14 +1327,14 @@ void __bch2_journal_debug_to_text(struct printbuf *out, struct journal *j)
                if (!ja->nr)
                        continue;
 
-               pr_buf(out, "dev %u:\n",                i);
-               pr_buf(out, "\tnr\t\t%u\n",             ja->nr);
-               pr_buf(out, "\tbucket size\t%u\n",      ca->mi.bucket_size);
-               pr_buf(out, "\tavailable\t%u:%u\n",     bch2_journal_dev_buckets_available(j, ja, journal_space_discarded), ja->sectors_free);
-               pr_buf(out, "\tdiscard_idx\t%u\n",      ja->discard_idx);
-               pr_buf(out, "\tdirty_ondisk\t%u (seq %llu)\n", ja->dirty_idx_ondisk,    ja->bucket_seq[ja->dirty_idx_ondisk]);
-               pr_buf(out, "\tdirty_idx\t%u (seq %llu)\n", ja->dirty_idx,              ja->bucket_seq[ja->dirty_idx]);
-               pr_buf(out, "\tcur_idx\t\t%u (seq %llu)\n", ja->cur_idx,                ja->bucket_seq[ja->cur_idx]);
+               prt_printf(out, "dev %u:\n",            i);
+               prt_printf(out, "\tnr\t\t%u\n",         ja->nr);
+               prt_printf(out, "\tbucket size\t%u\n",  ca->mi.bucket_size);
+               prt_printf(out, "\tavailable\t%u:%u\n", bch2_journal_dev_buckets_available(j, ja, journal_space_discarded), ja->sectors_free);
+               prt_printf(out, "\tdiscard_idx\t%u\n",  ja->discard_idx);
+               prt_printf(out, "\tdirty_ondisk\t%u (seq %llu)\n", ja->dirty_idx_ondisk,        ja->bucket_seq[ja->dirty_idx_ondisk]);
+               prt_printf(out, "\tdirty_idx\t%u (seq %llu)\n", ja->dirty_idx,          ja->bucket_seq[ja->dirty_idx]);
+               prt_printf(out, "\tcur_idx\t\t%u (seq %llu)\n", ja->cur_idx,            ja->bucket_seq[ja->cur_idx]);
        }
 
        rcu_read_unlock();
@@ -1353,31 +1366,31 @@ bool bch2_journal_seq_pins_to_text(struct printbuf *out, struct journal *j, u64
 
        pin_list = journal_seq_pin(j, *seq);
 
-       pr_buf(out, "%llu: count %u", *seq, atomic_read(&pin_list->count));
-       pr_newline(out);
-       pr_indent_push(out, 2);
+       prt_printf(out, "%llu: count %u", *seq, atomic_read(&pin_list->count));
+       prt_newline(out);
+       printbuf_indent_add(out, 2);
 
        list_for_each_entry(pin, &pin_list->list, list) {
-               pr_buf(out, "\t%px %ps", pin, pin->flush);
-               pr_newline(out);
+               prt_printf(out, "\t%px %ps", pin, pin->flush);
+               prt_newline(out);
        }
 
        list_for_each_entry(pin, &pin_list->key_cache_list, list) {
-               pr_buf(out, "\t%px %ps", pin, pin->flush);
-               pr_newline(out);
+               prt_printf(out, "\t%px %ps", pin, pin->flush);
+               prt_newline(out);
        }
 
        if (!list_empty(&pin_list->flushed)) {
-               pr_buf(out, "flushed:");
-               pr_newline(out);
+               prt_printf(out, "flushed:");
+               prt_newline(out);
        }
 
        list_for_each_entry(pin, &pin_list->flushed, list) {
-               pr_buf(out, "\t%px %ps", pin, pin->flush);
-               pr_newline(out);
+               prt_printf(out, "\t%px %ps", pin, pin->flush);
+               prt_newline(out);
        }
 
-       pr_indent_pop(out, 2);
+       printbuf_indent_sub(out, 2);
 
        --out->atomic;
        spin_unlock(&j->lock);