]> 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 cdf0a554954b8d03825d42b5e48ec40d6af27d40..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:
+               return BCH_STR_HASH_CRC32C;
+       case BCH_STR_HASH_OPT_crc64:
+               return BCH_STR_HASH_CRC64;
+       case BCH_STR_HASH_OPT_siphash:
+               return c->sb.features & (1ULL << BCH_FEATURE_new_siphash)
+                       ? BCH_STR_HASH_SIPHASH
+                       : BCH_STR_HASH_SIPHASH_OLD;
+       default:
+            BUG();
+       }
+}
 
 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
-bch2_hash_info_init(struct bch_fs *c,
-                  const struct bch_inode_unpacked *bi)
+bch2_hash_info_init(struct bch_fs *c, const struct bch_inode_unpacked *bi)
 {
        /* XXX ick */
        struct bch_hash_info info = {
                .type = (bi->bi_flags >> INODE_STR_HASH_OFFSET) &
-                       ~(~0U << INODE_STR_HASH_BITS)
+                       ~(~0U << INODE_STR_HASH_BITS),
+               .siphash_key = { .k0 = bi->bi_hash_seed }
        };
 
-       switch (info.type) {
-       case BCH_STR_HASH_CRC32C:
-       case BCH_STR_HASH_CRC64:
-               info.crc_key = bi->bi_hash_seed;
-               break;
-       case BCH_STR_HASH_SIPHASH: {
+       if (unlikely(info.type == BCH_STR_HASH_SIPHASH_OLD)) {
                SHASH_DESC_ON_STACK(desc, c->sha256);
                u8 digest[SHA256_DIGEST_SIZE];
 
@@ -46,10 +59,6 @@ bch2_hash_info_init(struct bch_fs *c,
                crypto_shash_digest(desc, (void *) &bi->bi_hash_seed,
                                    sizeof(bi->bi_hash_seed), digest);
                memcpy(&info.siphash_key, digest, sizeof(info.siphash_key));
-               break;
-       }
-       default:
-               BUG();
        }
 
        return info;
@@ -68,11 +77,14 @@ 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:
                SipHash24_Init(&ctx->siphash, &info->siphash_key);
                break;
@@ -92,6 +104,7 @@ static inline void bch2_str_hash_update(struct bch_str_hash_ctx *ctx,
        case BCH_STR_HASH_CRC64:
                ctx->crc64 = crc64_be(ctx->crc64, data, len);
                break;
+       case BCH_STR_HASH_SIPHASH_OLD:
        case BCH_STR_HASH_SIPHASH:
                SipHash24_Update(&ctx->siphash, data, len);
                break;
@@ -108,6 +121,7 @@ static inline u64 bch2_str_hash_end(struct bch_str_hash_ctx *ctx,
                return ctx->crc32c;
        case BCH_STR_HASH_CRC64:
                return ctx->crc64 >> 1;
+       case BCH_STR_HASH_SIPHASH_OLD:
        case BCH_STR_HASH_SIPHASH:
                return SipHash24_End(&ctx->siphash) >> 1;
        default:
@@ -125,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)
@@ -144,39 +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_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);
 
-       return ERR_PTR(ret ?: -ENOSPC);
+       return ret ?: -ENOSPC;
 }
 
 static __always_inline
@@ -185,30 +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);
-       if (IS_ERR(iter))
-               return PTR_ERR(iter);
+       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;
 }
 
@@ -218,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;
@@ -226,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) {
@@ -237,23 +250,19 @@ int bch2_hash_set(struct btree_trans *trans,
                        continue;
                }
 
-               if (!slot &&
-                   !(flags & BCH_HASH_SET_MUST_REPLACE)) {
-                       slot = bch2_trans_copy_iter(trans, iter);
-                       if (IS_ERR(slot))
-                               return PTR_ERR(slot);
-               }
+               if (!slot.path &&
+                   !(flags & BCH_HASH_SET_MUST_REPLACE))
+                       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:
-       if (slot)
-               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:
@@ -265,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);
+               insert->k.p = iter.pos;
+               ret = bch2_trans_update(trans, &iter, insert, 0);
        }
 
        goto out;
@@ -284,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);
-       return 0;
+       return bch2_trans_update(trans, iter, delete, 0);
 }
 
 static __always_inline
@@ -306,14 +315,17 @@ 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;
 
-       return bch2_hash_delete_at(trans, desc, info, iter);
+       ret = bch2_hash_delete_at(trans, desc, info, &iter);
+       bch2_trans_iter_exit(trans, &iter);
+       return ret;
 }
 
 #endif /* _BCACHEFS_STR_HASH_H */