bcache: kill index()
authorKent Overstreet <kmo@daterainc.com>
Wed, 18 Dec 2013 05:46:35 +0000 (21:46 -0800)
committerKent Overstreet <kmo@daterainc.com>
Wed, 8 Jan 2014 21:05:10 +0000 (13:05 -0800)
That was a terrible name for a macro, add some better helpers to replace it.

Signed-off-by: Kent Overstreet <kmo@daterainc.com>
drivers/md/bcache/bcache.h
drivers/md/bcache/btree.c
drivers/md/bcache/btree.h
drivers/md/bcache/debug.c

index 94d346e2ea17372b49a1ed9f729adc5902fa9201..d955a49346163d141e18488b342a4dc129e59822 100644 (file)
@@ -736,10 +736,6 @@ static inline unsigned local_clock_us(void)
 #define node(i, j)             ((struct bkey *) ((i)->d + (j)))
 #define end(i)                 node(i, (i)->keys)
 
-#define index(i, b)                                                    \
-       ((size_t) (((void *) i - (void *) (b)->sets[0].data) /          \
-                  block_bytes(b->c)))
-
 #define btree_data_space(b)    (PAGE_SIZE << (b)->page_order)
 
 #define prios_per_bucket(c)                            \
index 5a6b7522eb20cb695a01ff921fa2f73b7c41c44d..8e2573a009f9fc09fee9f294cd6052ae3f413bea 100644 (file)
@@ -258,7 +258,7 @@ static void bch_btree_node_read_done(struct btree *b)
 
        err = "corrupted btree";
        for (i = write_block(b);
-            index(i, b) < btree_blocks(b);
+            bset_sector_offset(b, i) < KEY_SIZE(&b->key);
             i = ((void *) i) + block_bytes(b->c))
                if (i->seq == b->sets[0].data->seq)
                        goto err;
@@ -278,9 +278,9 @@ out:
        return;
 err:
        set_btree_node_io_error(b);
-       bch_cache_set_error(b->c, "%s at bucket %zu, block %zu, %u keys",
+       bch_cache_set_error(b->c, "%s at bucket %zu, block %u, %u keys",
                            err, PTR_BUCKET_NR(b->c, &b->key, 0),
-                           index(i, b), i->keys);
+                           bset_block_offset(b, i), i->keys);
        goto out;
 }
 
index 4f0378ac1f7b57dbe59b0fc31aaa9f81f2e762ea..12c99b1a764d13094e081870c90fe1e68c22649b 100644 (file)
@@ -185,6 +185,26 @@ static inline unsigned bset_offset(struct btree *b, struct bset *i)
        return (((size_t) i) - ((size_t) b->sets->data)) >> 9;
 }
 
+static inline struct bset *btree_bset_first(struct btree *b)
+{
+       return b->sets->data;
+}
+
+static inline unsigned bset_byte_offset(struct btree *b, struct bset *i)
+{
+       return ((size_t) i) - ((size_t) b->sets->data);
+}
+
+static inline unsigned bset_sector_offset(struct btree *b, struct bset *i)
+{
+       return (((void *) i) - ((void *) btree_bset_first(b))) >> 9;
+}
+
+static inline unsigned bset_block_offset(struct btree *b, struct bset *i)
+{
+       return bset_sector_offset(b, i) >> b->c->block_bits;
+}
+
 static inline struct bset *write_block(struct btree *b)
 {
        return ((void *) b->sets[0].data) + b->written * block_bytes(b->c);
index fab3767d6d284a5a330254cb685f1cd1266d0401..473e8d5a7fe10978b610055f774e2d5976b4fea7 100644 (file)
@@ -88,7 +88,7 @@ static void dump_bset(struct btree *b, struct bset *i)
                next = bkey_next(k);
 
                bch_bkey_to_text(buf, sizeof(buf), k);
-               printk(KERN_ERR "block %zu key %zi/%u: %s", index(i, b),
+               printk(KERN_ERR "block %u key %zi/%u: %s", bset_block_offset(b, i),
                       (uint64_t *) k - i->d, i->keys, buf);
 
                for (j = 0; j < KEY_PTRS(k); j++) {