]> git.sesse.net Git - bcachefs-tools-debian/blobdiff - libbcachefs/btree_locking.h
New upstream snapshot
[bcachefs-tools-debian] / libbcachefs / btree_locking.h
index 7532bcdef96732b44bafac02aeb77ae92a82ebbd..b4434eca0746c7635c10b291534f526661dc4cf6 100644 (file)
@@ -21,7 +21,7 @@ enum btree_node_locked_type {
        BTREE_NODE_INTENT_LOCKED        = SIX_LOCK_intent,
 };
 
-static inline int btree_node_locked_type(struct btree_iter *iter,
+static inline int btree_node_locked_type(struct btree_path *path,
                                         unsigned level)
 {
        /*
@@ -30,35 +30,35 @@ static inline int btree_node_locked_type(struct btree_iter *iter,
         * branches:
         */
        return BTREE_NODE_UNLOCKED +
-               ((iter->nodes_locked >> level) & 1) +
-               ((iter->nodes_intent_locked >> level) & 1);
+               ((path->nodes_locked >> level) & 1) +
+               ((path->nodes_intent_locked >> level) & 1);
 }
 
-static inline bool btree_node_intent_locked(struct btree_iter *iter,
+static inline bool btree_node_intent_locked(struct btree_path *path,
                                            unsigned level)
 {
-       return btree_node_locked_type(iter, level) == BTREE_NODE_INTENT_LOCKED;
+       return btree_node_locked_type(path, level) == BTREE_NODE_INTENT_LOCKED;
 }
 
-static inline bool btree_node_read_locked(struct btree_iter *iter,
+static inline bool btree_node_read_locked(struct btree_path *path,
                                          unsigned level)
 {
-       return btree_node_locked_type(iter, level) == BTREE_NODE_READ_LOCKED;
+       return btree_node_locked_type(path, level) == BTREE_NODE_READ_LOCKED;
 }
 
-static inline bool btree_node_locked(struct btree_iter *iter, unsigned level)
+static inline bool btree_node_locked(struct btree_path *path, unsigned level)
 {
-       return iter->nodes_locked & (1 << level);
+       return path->nodes_locked & (1 << level);
 }
 
-static inline void mark_btree_node_unlocked(struct btree_iter *iter,
+static inline void mark_btree_node_unlocked(struct btree_path *path,
                                            unsigned level)
 {
-       iter->nodes_locked &= ~(1 << level);
-       iter->nodes_intent_locked &= ~(1 << level);
+       path->nodes_locked &= ~(1 << level);
+       path->nodes_intent_locked &= ~(1 << level);
 }
 
-static inline void mark_btree_node_locked(struct btree_iter *iter,
+static inline void mark_btree_node_locked(struct btree_path *path,
                                          unsigned level,
                                          enum six_lock_type type)
 {
@@ -66,52 +66,52 @@ static inline void mark_btree_node_locked(struct btree_iter *iter,
        BUILD_BUG_ON(SIX_LOCK_read   != 0);
        BUILD_BUG_ON(SIX_LOCK_intent != 1);
 
-       iter->nodes_locked |= 1 << level;
-       iter->nodes_intent_locked |= type << level;
+       path->nodes_locked |= 1 << level;
+       path->nodes_intent_locked |= type << level;
 }
 
-static inline void mark_btree_node_intent_locked(struct btree_iter *iter,
+static inline void mark_btree_node_intent_locked(struct btree_path *path,
                                                 unsigned level)
 {
-       mark_btree_node_locked(iter, level, SIX_LOCK_intent);
+       mark_btree_node_locked(path, level, SIX_LOCK_intent);
 }
 
-static inline enum six_lock_type __btree_lock_want(struct btree_iter *iter, int level)
+static inline enum six_lock_type __btree_lock_want(struct btree_path *path, int level)
 {
-       return level < iter->locks_want
+       return level < path->locks_want
                ? SIX_LOCK_intent
                : SIX_LOCK_read;
 }
 
 static inline enum btree_node_locked_type
-btree_lock_want(struct btree_iter *iter, int level)
+btree_lock_want(struct btree_path *path, int level)
 {
-       if (level < iter->level)
+       if (level < path->level)
                return BTREE_NODE_UNLOCKED;
-       if (level < iter->locks_want)
+       if (level < path->locks_want)
                return BTREE_NODE_INTENT_LOCKED;
-       if (level == iter->level)
+       if (level == path->level)
                return BTREE_NODE_READ_LOCKED;
        return BTREE_NODE_UNLOCKED;
 }
 
-static inline void btree_node_unlock(struct btree_iter *iter, unsigned level)
+static inline void btree_node_unlock(struct btree_path *path, unsigned level)
 {
-       int lock_type = btree_node_locked_type(iter, level);
+       int lock_type = btree_node_locked_type(path, level);
 
        EBUG_ON(level >= BTREE_MAX_DEPTH);
 
        if (lock_type != BTREE_NODE_UNLOCKED)
-               six_unlock_type(&iter->l[level].b->c.lock, lock_type);
-       mark_btree_node_unlocked(iter, level);
+               six_unlock_type(&path->l[level].b->c.lock, lock_type);
+       mark_btree_node_unlocked(path, level);
 }
 
-static inline void __bch2_btree_iter_unlock(struct btree_iter *iter)
+static inline void __bch2_btree_path_unlock(struct btree_path *path)
 {
-       btree_iter_set_dirty(iter, BTREE_ITER_NEED_RELOCK);
+       btree_path_set_dirty(path, BTREE_ITER_NEED_RELOCK);
 
-       while (iter->nodes_locked)
-               btree_node_unlock(iter, __ffs(iter->nodes_locked));
+       while (path->nodes_locked)
+               btree_node_unlock(path, __ffs(path->nodes_locked));
 }
 
 static inline enum bch_time_stats lock_to_time_stat(enum six_lock_type type)
@@ -128,23 +128,35 @@ static inline enum bch_time_stats lock_to_time_stat(enum six_lock_type type)
        }
 }
 
-/*
- * wrapper around six locks that just traces lock contended time
- */
-static inline void __btree_node_lock_type(struct bch_fs *c, struct btree *b,
-                                         enum six_lock_type type)
+static inline bool btree_node_lock_type(struct btree_trans *trans,
+                                      struct btree_path *path,
+                                      struct btree *b,
+                                      struct bpos pos, unsigned level,
+                                      enum six_lock_type type,
+                                      six_lock_should_sleep_fn should_sleep_fn, void *p)
 {
-       u64 start_time = local_clock();
+       struct bch_fs *c = trans->c;
+       u64 start_time;
+       bool ret;
 
-       six_lock_type(&b->c.lock, type, NULL, NULL);
-       bch2_time_stats_update(&c->times[lock_to_time_stat(type)], start_time);
-}
+       if (six_trylock_type(&b->c.lock, type))
+               return true;
 
-static inline void btree_node_lock_type(struct bch_fs *c, struct btree *b,
-                                       enum six_lock_type type)
-{
-       if (!six_trylock_type(&b->c.lock, type))
-               __btree_node_lock_type(c, b, type);
+       start_time = local_clock();
+
+       trans->locking_path_idx = path->idx;
+       trans->locking_pos      = pos;
+       trans->locking_btree_id = path->btree_id;
+       trans->locking_level    = level;
+       trans->locking_lock_type = type;
+       trans->locking          = b;
+       ret = six_lock_type(&b->c.lock, type, should_sleep_fn, p) == 0;
+       trans->locking = NULL;
+
+       if (ret)
+               bch2_time_stats_update(&c->times[lock_to_time_stat(type)], start_time);
+
+       return ret;
 }
 
 /*
@@ -155,11 +167,11 @@ static inline bool btree_node_lock_increment(struct btree_trans *trans,
                                             struct btree *b, unsigned level,
                                             enum btree_node_locked_type want)
 {
-       struct btree_iter *iter;
+       struct btree_path *path;
 
-       trans_for_each_iter(trans, iter)
-               if (iter->l[level].b == b &&
-                   btree_node_locked_type(iter, level) >= want) {
+       trans_for_each_path(trans, path)
+               if (path->l[level].b == b &&
+                   btree_node_locked_type(path, level) >= want) {
                        six_lock_increment(&b->c.lock, want);
                        return true;
                }
@@ -167,40 +179,39 @@ static inline bool btree_node_lock_increment(struct btree_trans *trans,
        return false;
 }
 
-bool __bch2_btree_node_lock(struct btree *, struct bpos, unsigned,
-                           struct btree_iter *, enum six_lock_type,
+bool __bch2_btree_node_lock(struct btree_trans *, struct btree_path *,
+                           struct btree *, struct bpos, unsigned,
+                           enum six_lock_type,
                            six_lock_should_sleep_fn, void *,
                            unsigned long);
 
-static inline bool btree_node_lock(struct btree *b,
-                       struct bpos pos, unsigned level,
-                       struct btree_iter *iter,
+static inline bool btree_node_lock(struct btree_trans *trans,
+                       struct btree_path *path,
+                       struct btree *b, struct bpos pos, unsigned level,
                        enum six_lock_type type,
                        six_lock_should_sleep_fn should_sleep_fn, void *p,
                        unsigned long ip)
 {
-       struct btree_trans *trans = iter->trans;
-
        EBUG_ON(level >= BTREE_MAX_DEPTH);
-       EBUG_ON(!(trans->iters_linked & (1ULL << iter->idx)));
+       EBUG_ON(!(trans->paths_allocated & (1ULL << path->idx)));
 
        return likely(six_trylock_type(&b->c.lock, type)) ||
                btree_node_lock_increment(trans, b, level, type) ||
-               __bch2_btree_node_lock(b, pos, level, iter, type,
+               __bch2_btree_node_lock(trans, path, b, pos, level, type,
                                       should_sleep_fn, p, ip);
 }
 
-bool __bch2_btree_node_relock(struct btree_iter *, unsigned);
+bool __bch2_btree_node_relock(struct btree_trans *, struct btree_path *, unsigned);
 
-static inline bool bch2_btree_node_relock(struct btree_iter *iter,
-                                         unsigned level)
+static inline bool bch2_btree_node_relock(struct btree_trans *trans,
+                                         struct btree_path *path, unsigned level)
 {
-       EBUG_ON(btree_node_locked(iter, level) &&
-               btree_node_locked_type(iter, level) !=
-               __btree_lock_want(iter, level));
+       EBUG_ON(btree_node_locked(path, level) &&
+               btree_node_locked_type(path, level) !=
+               __btree_lock_want(path, level));
 
-       return likely(btree_node_locked(iter, level)) ||
-               __bch2_btree_node_relock(iter, level);
+       return likely(btree_node_locked(path, level)) ||
+               __bch2_btree_node_relock(trans, path, level);
 }
 
 /*
@@ -208,30 +219,35 @@ static inline bool bch2_btree_node_relock(struct btree_iter *iter,
  * succeed:
  */
 static inline void
-bch2_btree_node_unlock_write_inlined(struct btree *b, struct btree_iter *iter)
+bch2_btree_node_unlock_write_inlined(struct btree_trans *trans, struct btree_path *path,
+                                    struct btree *b)
 {
-       struct btree_iter *linked;
+       struct btree_path *linked;
 
-       EBUG_ON(iter->l[b->c.level].b != b);
-       EBUG_ON(iter->l[b->c.level].lock_seq + 1 != b->c.lock.state.seq);
+       EBUG_ON(path->l[b->c.level].b != b);
+       EBUG_ON(path->l[b->c.level].lock_seq + 1 != b->c.lock.state.seq);
 
-       trans_for_each_iter_with_node(iter->trans, b, linked)
+       trans_for_each_path_with_node(trans, b, linked)
                linked->l[b->c.level].lock_seq += 2;
 
        six_unlock_write(&b->c.lock);
 }
 
-void bch2_btree_node_unlock_write(struct btree *, struct btree_iter *);
+void bch2_btree_node_unlock_write(struct btree_trans *,
+                       struct btree_path *, struct btree *);
 
-void __bch2_btree_node_lock_write(struct btree *, struct btree_iter *);
+void __bch2_btree_node_lock_write(struct btree_trans *, struct btree *);
 
-static inline void bch2_btree_node_lock_write(struct btree *b, struct btree_iter *iter)
+static inline void bch2_btree_node_lock_write(struct btree_trans *trans,
+                                             struct btree_path *path,
+                                             struct btree *b)
 {
-       EBUG_ON(iter->l[b->c.level].b != b);
-       EBUG_ON(iter->l[b->c.level].lock_seq != b->c.lock.state.seq);
+       EBUG_ON(path->l[b->c.level].b != b);
+       EBUG_ON(path->l[b->c.level].lock_seq != b->c.lock.state.seq);
+       EBUG_ON(!btree_node_intent_locked(path, b->c.level));
 
        if (unlikely(!six_trylock_write(&b->c.lock)))
-               __bch2_btree_node_lock_write(b, iter);
+               __bch2_btree_node_lock_write(trans, b);
 }
 
 #endif /* _BCACHEFS_BTREE_LOCKING_H */