From 534a591e4cf98d036e478b93de4a95ff126fb018 Mon Sep 17 00:00:00 2001 From: Kent Overstreet Date: Sat, 27 Aug 2022 17:47:27 -0400 Subject: bcachefs: Delete time_stats for lock contended times Since we've now got time_stats for lock hold times (per btree transaction), we don't need this anymore. Signed-off-by: Kent Overstreet --- fs/bcachefs/btree_locking.h | 25 +------------------------ 1 file changed, 1 insertion(+), 24 deletions(-) (limited to 'fs/bcachefs/btree_locking.h') diff --git a/fs/bcachefs/btree_locking.h b/fs/bcachefs/btree_locking.h index 3bc5df4263f8..1e4c81d8084b 100644 --- a/fs/bcachefs/btree_locking.h +++ b/fs/bcachefs/btree_locking.h @@ -123,20 +123,6 @@ static void btree_trans_lock_hold_time_update(struct btree_trans *trans, #endif } -static inline enum bch_time_stats lock_to_time_stat(enum six_lock_type type) -{ - switch (type) { - case SIX_LOCK_read: - return BCH_TIME_btree_lock_contended_read; - case SIX_LOCK_intent: - return BCH_TIME_btree_lock_contended_intent; - case SIX_LOCK_write: - return BCH_TIME_btree_lock_contended_write; - default: - BUG(); - } -} - /* unlock: */ static inline void btree_node_unlock(struct btree_trans *trans, @@ -206,15 +192,11 @@ static inline int btree_node_lock_type(struct btree_trans *trans, enum six_lock_type type, six_lock_should_sleep_fn should_sleep_fn, void *p) { - struct bch_fs *c = trans->c; - u64 start_time; int ret; if (six_trylock_type(&b->lock, type)) return 0; - start_time = local_clock(); - trans->locking_path_idx = path->idx; trans->locking_pos = pos; trans->locking_btree_id = path->btree_id; @@ -223,12 +205,7 @@ static inline int btree_node_lock_type(struct btree_trans *trans, trans->locking = b; ret = six_lock_type(&b->lock, type, should_sleep_fn, p); trans->locking = NULL; - - if (ret) - return ret; - - bch2_time_stats_update(&c->times[lock_to_time_stat(type)], start_time); - return 0; + return ret; } /* -- cgit v1.2.3