bcache: Remove unnecessary check in should_split()

Checking i->seq was redundant, because since ages ago we always
initialize the new bset when advancing b->written

Signed-off-by: Kent Overstreet <kmo@daterainc.com>

diff --git a/drivers/md/bcache/btree.h b/drivers/md/bcache/btree.h
index 3333d37..8a1c7e6 100644 (file)
--- a/drivers/md/bcache/btree.h
+++ b/drivers/md/bcache/btree.h
@@ -353,7 +353,7 @@ static inline void rw_unlock(bool w, struct btree *b)
                    _w == insert_lock(op, _b))                          \
                        _r = bch_btree_ ## fn(_b, op, ##__VA_ARGS__);   \
                rw_unlock(_w, _b);                                      \
-               bch_cannibalize_unlock(c, &(op)->cl);           \
+               bch_cannibalize_unlock(c, &(op)->cl);                   \
        } while (_r == -EINTR);                                         \
                                                                        \
        _r;                                                             \
@@ -363,8 +363,7 @@ static inline bool should_split(struct btree *b)
 {
        struct bset *i = write_block(b);
        return b->written >= btree_blocks(b) ||
-               (i->seq == b->sets[0].data->seq &&
-                b->written + __set_blocks(i, i->keys + 15, b->c)
+               (b->written + __set_blocks(i, i->keys + 15, b->c)
                 > btree_blocks(b));
 }