diff options
author | Kent Overstreet <kent.overstreet@linux.dev> | 2023-02-04 14:31:26 -0500 |
---|---|---|
committer | Kent Overstreet <kent.overstreet@linux.dev> | 2023-02-05 16:42:02 -0500 |
commit | 0e3adb08a37c07ffa211f3f42307c9385fc18532 (patch) | |
tree | c608d0f3c4157f7564569805f3485d7792fb0bcb | |
parent | 7a04f5360cfddf6d96e0bff957774d7b63b420d7 (diff) |
fixup! bcachefs: Rename lock_held_stats -> btree_transaction_stats
-rw-r--r-- | fs/bcachefs/bcachefs.h | 2 | ||||
-rw-r--r-- | fs/bcachefs/btree_iter.c | 15 |
2 files changed, 12 insertions, 5 deletions
diff --git a/fs/bcachefs/bcachefs.h b/fs/bcachefs/bcachefs.h index 3be6a884f994..5a7f68039933 100644 --- a/fs/bcachefs/bcachefs.h +++ b/fs/bcachefs/bcachefs.h @@ -567,7 +567,7 @@ struct btree_debug { struct btree_transaction_stats { struct mutex lock; - struct bch2_time_stats lock_hold_times; + struct bch2_time_stats lock_hold_times; unsigned nr_max_paths; unsigned max_mem; char *max_paths_text; diff --git a/fs/bcachefs/btree_iter.c b/fs/bcachefs/btree_iter.c index a7173ca7674c..4f8799e30982 100644 --- a/fs/bcachefs/btree_iter.c +++ b/fs/bcachefs/btree_iter.c @@ -3121,8 +3121,10 @@ void bch2_fs_btree_iter_exit(struct bch_fs *c) for (s = c->btree_transaction_stats; s < c->btree_transaction_stats + ARRAY_SIZE(c->btree_transaction_stats); - s++) + s++) { kfree(s->max_paths_text); + bch2_time_stats_exit(&s->lock_hold_times); + } if (c->btree_trans_barrier_initialized) cleanup_srcu_struct(&c->btree_trans_barrier); @@ -3132,11 +3134,16 @@ void bch2_fs_btree_iter_exit(struct bch_fs *c) int bch2_fs_btree_iter_init(struct bch_fs *c) { - unsigned i, nr = BTREE_ITER_MAX; + struct btree_transaction_stats *s; + unsigned nr = BTREE_ITER_MAX; int ret; - for (i = 0; i < ARRAY_SIZE(c->btree_transaction_stats); i++) - mutex_init(&c->btree_transaction_stats[i].lock); + for (s = c->btree_transaction_stats; + s < c->btree_transaction_stats + ARRAY_SIZE(c->btree_transaction_stats); + s++) { + mutex_init(&s->lock); + bch2_time_stats_init(&s->lock_hold_times); + } INIT_LIST_HEAD(&c->btree_trans_list); mutex_init(&c->btree_trans_lock); |