]> git.sesse.net Git - bcachefs-tools-debian/blobdiff - libbcachefs/str_hash.h
Update bcachefs sources to 3f3f969859 bcachefs: Fix some compiler warnings
[bcachefs-tools-debian] / libbcachefs / str_hash.h
index 1ecf72c9487cdedb6c2fb94e891dde8dd12f9753..c6a132b3c5bb2eb24cf112f11fd23d254e01b499 100644 (file)
 
 #include <linux/crc32c.h>
 #include <crypto/hash.h>
-#include <crypto/sha.h>
+#include <crypto/sha2.h>
 
 static inline enum bch_str_hash_type
 bch2_str_hash_opt_to_type(struct bch_fs *c, enum bch_str_hash_opts opt)
 {
        switch (opt) {
-       case BCH_STR_HASH_OPT_CRC32C:
+       case BCH_STR_HASH_OPT_crc32c:
                return BCH_STR_HASH_CRC32C;
-       case BCH_STR_HASH_OPT_CRC64:
+       case BCH_STR_HASH_OPT_crc64:
                return BCH_STR_HASH_CRC64;
-       case BCH_STR_HASH_OPT_SIPHASH:
+       case BCH_STR_HASH_OPT_siphash:
                return c->sb.features & (1ULL << BCH_FEATURE_new_siphash)
                        ? BCH_STR_HASH_SIPHASH
                        : BCH_STR_HASH_SIPHASH_OLD;
@@ -33,10 +33,11 @@ bch2_str_hash_opt_to_type(struct bch_fs *c, enum bch_str_hash_opts opt)
 
 struct bch_hash_info {
        u8                      type;
-       union {
-               __le64          crc_key;
-               SIPHASH_KEY     siphash_key;
-       };
+       /*
+        * For crc32 or crc64 string hashes the first key value of
+        * the siphash_key (k0) is used as the key.
+        */
+       SIPHASH_KEY     siphash_key;
 };
 
 static inline struct bch_hash_info
@@ -46,7 +47,7 @@ bch2_hash_info_init(struct bch_fs *c, const struct bch_inode_unpacked *bi)
        struct bch_hash_info info = {
                .type = (bi->bi_flags >> INODE_STR_HASH_OFFSET) &
                        ~(~0U << INODE_STR_HASH_BITS),
-               .crc_key = bi->bi_hash_seed,
+               .siphash_key = { .k0 = bi->bi_hash_seed }
        };
 
        if (unlikely(info.type == BCH_STR_HASH_SIPHASH_OLD)) {
@@ -76,10 +77,12 @@ static inline void bch2_str_hash_init(struct bch_str_hash_ctx *ctx,
 {
        switch (info->type) {
        case BCH_STR_HASH_CRC32C:
-               ctx->crc32c = crc32c(~0, &info->crc_key, sizeof(info->crc_key));
+               ctx->crc32c = crc32c(~0, &info->siphash_key.k0,
+                                    sizeof(info->siphash_key.k0));
                break;
        case BCH_STR_HASH_CRC64:
-               ctx->crc64 = crc64_be(~0, &info->crc_key, sizeof(info->crc_key));
+               ctx->crc64 = crc64_be(~0, &info->siphash_key.k0,
+                                     sizeof(info->siphash_key.k0));
                break;
        case BCH_STR_HASH_SIPHASH_OLD:
        case BCH_STR_HASH_SIPHASH:
@@ -136,18 +139,18 @@ struct bch_hash_desc {
        bool            (*cmp_bkey)(struct bkey_s_c, struct bkey_s_c);
 };
 
-static __always_inline struct btree_iter *
+static __always_inline int
 bch2_hash_lookup(struct btree_trans *trans,
+                struct btree_iter *iter,
                 const struct bch_hash_desc desc,
                 const struct bch_hash_info *info,
                 u64 inode, const void *key,
                 unsigned flags)
 {
-       struct btree_iter *iter;
        struct bkey_s_c k;
        int ret;
 
-       for_each_btree_key(trans, iter, desc.btree_id,
+       for_each_btree_key(trans, *iter, desc.btree_id,
                           POS(inode, desc.hash_key(info, key)),
                           BTREE_ITER_SLOTS|flags, k, ret) {
                if (iter->pos.inode != inode)
@@ -155,43 +158,41 @@ bch2_hash_lookup(struct btree_trans *trans,
 
                if (k.k->type == desc.key_type) {
                        if (!desc.cmp_key(k, key))
-                               return iter;
-               } else if (k.k->type == KEY_TYPE_whiteout) {
+                               return 0;
+               } else if (k.k->type == KEY_TYPE_hash_whiteout) {
                        ;
                } else {
                        /* hole, not found */
                        break;
                }
        }
-       bch2_trans_iter_put(trans, iter);
+       bch2_trans_iter_exit(trans, iter);
 
-       return ERR_PTR(ret ?: -ENOENT);
+       return ret ?: -ENOENT;
 }
 
-static __always_inline struct btree_iter *
+static __always_inline int
 bch2_hash_hole(struct btree_trans *trans,
+              struct btree_iter *iter,
               const struct bch_hash_desc desc,
               const struct bch_hash_info *info,
               u64 inode, const void *key)
 {
-       struct btree_iter *iter;
        struct bkey_s_c k;
        int ret;
 
-       for_each_btree_key(trans, iter, desc.btree_id,
+       for_each_btree_key(trans, *iter, desc.btree_id,
                           POS(inode, desc.hash_key(info, key)),
                           BTREE_ITER_SLOTS|BTREE_ITER_INTENT, k, ret) {
                if (iter->pos.inode != inode)
                        break;
 
                if (k.k->type != desc.key_type)
-                       return iter;
+                       return 0;
        }
+       bch2_trans_iter_exit(trans, iter);
 
-       iter->flags |= BTREE_ITER_KEEP_UNTIL_COMMIT;
-       bch2_trans_iter_put(trans, iter);
-
-       return ERR_PTR(ret ?: -ENOSPC);
+       return ret ?: -ENOSPC;
 }
 
 static __always_inline
@@ -200,28 +201,27 @@ int bch2_hash_needs_whiteout(struct btree_trans *trans,
                             const struct bch_hash_info *info,
                             struct btree_iter *start)
 {
-       struct btree_iter *iter;
+       struct btree_iter iter;
        struct bkey_s_c k;
        int ret;
 
-       iter = bch2_trans_copy_iter(trans, start);
+       bch2_trans_copy_iter(&iter, start);
 
-       bch2_btree_iter_next_slot(iter);
+       bch2_btree_iter_advance(&iter);
 
        for_each_btree_key_continue(iter, BTREE_ITER_SLOTS, k, ret) {
                if (k.k->type != desc.key_type &&
-                   k.k->type != KEY_TYPE_whiteout)
+                   k.k->type != KEY_TYPE_hash_whiteout)
                        break;
 
                if (k.k->type == desc.key_type &&
                    desc.hash_bkey(info, k) <= start->pos.offset) {
-                       iter->flags |= BTREE_ITER_KEEP_UNTIL_COMMIT;
                        ret = 1;
                        break;
                }
        }
 
-       bch2_trans_iter_put(trans, iter);
+       bch2_trans_iter_exit(trans, &iter);
        return ret;
 }
 
@@ -231,7 +231,7 @@ int bch2_hash_set(struct btree_trans *trans,
                  const struct bch_hash_info *info,
                  u64 inode, struct bkey_i *insert, int flags)
 {
-       struct btree_iter *iter, *slot = NULL;
+       struct btree_iter iter, slot = { NULL };
        struct bkey_s_c k;
        bool found = false;
        int ret;
@@ -239,7 +239,7 @@ int bch2_hash_set(struct btree_trans *trans,
        for_each_btree_key(trans, iter, desc.btree_id,
                           POS(inode, desc.hash_bkey(info, bkey_i_to_s_c(insert))),
                           BTREE_ITER_SLOTS|BTREE_ITER_INTENT, k, ret) {
-               if (iter->pos.inode != inode)
+               if (iter.pos.inode != inode)
                        break;
 
                if (k.k->type == desc.key_type) {
@@ -250,19 +250,19 @@ int bch2_hash_set(struct btree_trans *trans,
                        continue;
                }
 
-               if (!slot &&
+               if (!slot.path &&
                    !(flags & BCH_HASH_SET_MUST_REPLACE))
-                       slot = bch2_trans_copy_iter(trans, iter);
+                       bch2_trans_copy_iter(&slot, &iter);
 
-               if (k.k->type != KEY_TYPE_whiteout)
+               if (k.k->type != KEY_TYPE_hash_whiteout)
                        goto not_found;
        }
 
        if (!ret)
                ret = -ENOSPC;
 out:
-       bch2_trans_iter_put(trans, slot);
-       bch2_trans_iter_put(trans, iter);
+       bch2_trans_iter_exit(trans, &slot);
+       bch2_trans_iter_exit(trans, &iter);
 
        return ret;
 found:
@@ -274,11 +274,11 @@ not_found:
        } else if (found && (flags & BCH_HASH_SET_MUST_CREATE)) {
                ret = -EEXIST;
        } else {
-               if (!found && slot)
+               if (!found && slot.path)
                        swap(iter, slot);
 
-               insert->k.p = iter->pos;
-               bch2_trans_update(trans, iter, insert, 0);
+               insert->k.p = iter.pos;
+               ret = bch2_trans_update(trans, &iter, insert, 0);
        }
 
        goto out;
@@ -293,20 +293,20 @@ int bch2_hash_delete_at(struct btree_trans *trans,
        struct bkey_i *delete;
        int ret;
 
+       delete = bch2_trans_kmalloc(trans, sizeof(*delete));
+       ret = PTR_ERR_OR_ZERO(delete);
+       if (ret)
+               return ret;
+
        ret = bch2_hash_needs_whiteout(trans, desc, info, iter);
        if (ret < 0)
                return ret;
 
-       delete = bch2_trans_kmalloc(trans, sizeof(*delete));
-       if (IS_ERR(delete))
-               return PTR_ERR(delete);
-
        bkey_init(&delete->k);
        delete->k.p = iter->pos;
-       delete->k.type = ret ? KEY_TYPE_whiteout : KEY_TYPE_deleted;
+       delete->k.type = ret ? KEY_TYPE_hash_whiteout : KEY_TYPE_deleted;
 
-       bch2_trans_update(trans, iter, delete, 0);
-       return 0;
+       return bch2_trans_update(trans, iter, delete, 0);
 }
 
 static __always_inline
@@ -315,16 +315,16 @@ int bch2_hash_delete(struct btree_trans *trans,
                     const struct bch_hash_info *info,
                     u64 inode, const void *key)
 {
-       struct btree_iter *iter;
+       struct btree_iter iter;
        int ret;
 
-       iter = bch2_hash_lookup(trans, desc, info, inode, key,
+       ret = bch2_hash_lookup(trans, &iter, desc, info, inode, key,
                                BTREE_ITER_INTENT);
-       if (IS_ERR(iter))
-               return PTR_ERR(iter);
+       if (ret)
+               return ret;
 
-       ret = bch2_hash_delete_at(trans, desc, info, iter);
-       bch2_trans_iter_put(trans, iter);
+       ret = bch2_hash_delete_at(trans, desc, info, &iter);
+       bch2_trans_iter_exit(trans, &iter);
        return ret;
 }