]> 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 02052165f74d1b04df6722700cdf05f25c8253c8..c6a132b3c5bb2eb24cf112f11fd23d254e01b499 100644 (file)
@@ -1,5 +1,6 @@
-#ifndef _BCACHE_STR_HASH_H
-#define _BCACHE_STR_HASH_H
+/* SPDX-License-Identifier: GPL-2.0 */
+#ifndef _BCACHEFS_STR_HASH_H
+#define _BCACHEFS_STR_HASH_H
 
 #include "btree_iter.h"
 #include "btree_update.h"
 
 #include <linux/crc32c.h>
 #include <crypto/hash.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->i_flags >> INODE_STR_HASH_OFFSET) &
-                       ~(~0U << INODE_STR_HASH_BITS)
+               .type = (bi->bi_flags >> INODE_STR_HASH_OFFSET) &
+                       ~(~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->i_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[crypto_shash_digestsize(c->sha256)];
+               u8 digest[SHA256_DIGEST_SIZE];
 
                desc->tfm = c->sha256;
-               desc->flags = 0;
 
-               crypto_shash_digest(desc, (void *) &bi->i_hash_seed,
-                                   sizeof(bi->i_hash_seed), digest);
+               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;
@@ -67,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 = bch2_crc64_update(~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;
@@ -89,8 +102,9 @@ static inline void bch2_str_hash_update(struct bch_str_hash_ctx *ctx,
                ctx->crc32c = crc32c(ctx->crc32c, data, len);
                break;
        case BCH_STR_HASH_CRC64:
-               ctx->crc64 = bch2_crc64_update(ctx->crc64, data, len);
+               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;
@@ -107,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:
@@ -117,7 +132,6 @@ static inline u64 bch2_str_hash_end(struct bch_str_hash_ctx *ctx,
 struct bch_hash_desc {
        enum btree_id   btree_id;
        u8              key_type;
-       u8              whiteout_type;
 
        u64             (*hash_key)(const struct bch_hash_info *, const void *);
        u64             (*hash_bkey)(const struct bch_hash_info *, struct bkey_s_c);
@@ -125,281 +139,193 @@ struct bch_hash_desc {
        bool            (*cmp_bkey)(struct bkey_s_c, struct bkey_s_c);
 };
 
-static inline struct bkey_s_c
-bch2_hash_lookup_at(const struct bch_hash_desc desc,
-                  const struct bch_hash_info *info,
-                  struct btree_iter *iter, const void *search)
+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)
 {
-       u64 inode = iter->pos.inode;
-
-       do {
-               struct bkey_s_c k = bch2_btree_iter_peek_with_holes(iter);
-
-               if (btree_iter_err(k))
-                       return k;
+       struct bkey_s_c k;
+       int ret;
 
-               if (k.k->type == desc.key_type) {
-                       if (!desc.cmp_key(k, search))
-                               return k;
-               } else if (k.k->type == desc.whiteout_type) {
-                       ;
-               } else {
-                       /* hole, not found */
+       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)
                        break;
-               }
-
-               bch2_btree_iter_advance_pos(iter);
-       } while (iter->pos.inode == inode);
-
-       return bkey_s_c_err(-ENOENT);
-}
-
-static inline struct bkey_s_c
-bch2_hash_lookup_bkey_at(const struct bch_hash_desc desc,
-                       const struct bch_hash_info *info,
-                       struct btree_iter *iter, struct bkey_s_c search)
-{
-       u64 inode = iter->pos.inode;
-
-       do {
-               struct bkey_s_c k = bch2_btree_iter_peek_with_holes(iter);
-
-               if (btree_iter_err(k))
-                       return k;
 
                if (k.k->type == desc.key_type) {
-                       if (!desc.cmp_bkey(k, search))
-                               return k;
-               } else if (k.k->type == desc.whiteout_type) {
+                       if (!desc.cmp_key(k, key))
+                               return 0;
+               } else if (k.k->type == KEY_TYPE_hash_whiteout) {
                        ;
                } else {
                        /* hole, not found */
                        break;
                }
+       }
+       bch2_trans_iter_exit(trans, iter);
 
-               bch2_btree_iter_advance_pos(iter);
-       } while (iter->pos.inode == inode);
-
-       return bkey_s_c_err(-ENOENT);
-}
-
-static inline struct bkey_s_c
-bch2_hash_lookup(const struct bch_hash_desc desc,
-               const struct bch_hash_info *info,
-               struct bch_fs *c, u64 inode,
-               struct btree_iter *iter, const void *key)
-{
-       bch2_btree_iter_init(iter, c, desc.btree_id,
-                           POS(inode, desc.hash_key(info, key)));
-
-       return bch2_hash_lookup_at(desc, info, iter, key);
-}
-
-static inline struct bkey_s_c
-bch2_hash_lookup_intent(const struct bch_hash_desc desc,
-                      const struct bch_hash_info *info,
-                      struct bch_fs *c, u64 inode,
-                      struct btree_iter *iter, const void *key)
-{
-       bch2_btree_iter_init_intent(iter, c, desc.btree_id,
-                           POS(inode, desc.hash_key(info, key)));
-
-       return bch2_hash_lookup_at(desc, info, iter, key);
+       return ret ?: -ENOENT;
 }
 
-static inline struct bkey_s_c
-bch2_hash_hole_at(const struct bch_hash_desc desc, struct btree_iter *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)
 {
-       while (1) {
-               struct bkey_s_c k = bch2_btree_iter_peek_with_holes(iter);
+       struct bkey_s_c k;
+       int ret;
 
-               if (btree_iter_err(k))
-                       return k;
+       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 k;
-
-               /* hash collision, keep going */
-               bch2_btree_iter_advance_pos(iter);
-               if (iter->pos.inode != k.k->p.inode)
-                       return bkey_s_c_err(-ENOENT);
+                       return 0;
        }
-}
-
-static inline struct bkey_s_c bch2_hash_hole(const struct bch_hash_desc desc,
-                                           const struct bch_hash_info *info,
-                                           struct bch_fs *c, u64 inode,
-                                           struct btree_iter *iter,
-                                           const void *key)
-{
-       bch2_btree_iter_init_intent(iter, c, desc.btree_id,
-                           POS(inode, desc.hash_key(info, key)));
+       bch2_trans_iter_exit(trans, iter);
 
-       return bch2_hash_hole_at(desc, iter);
+       return ret ?: -ENOSPC;
 }
 
-static inline int bch2_hash_needs_whiteout(const struct bch_hash_desc desc,
-                                          const struct bch_hash_info *info,
-                                          struct btree_iter *iter,
-                                          struct btree_iter *start)
+static __always_inline
+int bch2_hash_needs_whiteout(struct btree_trans *trans,
+                            const struct bch_hash_desc desc,
+                            const struct bch_hash_info *info,
+                            struct btree_iter *start)
 {
-       bch2_btree_iter_set_pos(iter,
-                       btree_type_successor(start->btree_id, start->pos));
+       struct btree_iter iter;
+       struct bkey_s_c k;
+       int ret;
 
-       while (1) {
-               struct bkey_s_c k = bch2_btree_iter_peek_with_holes(iter);
-               int ret = btree_iter_err(k);
+       bch2_trans_copy_iter(&iter, start);
 
-               if (ret)
-                       return ret;
+       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 != desc.whiteout_type)
-                       return false;
+                   k.k->type != KEY_TYPE_hash_whiteout)
+                       break;
 
                if (k.k->type == desc.key_type &&
-                   desc.hash_bkey(info, k) <= start->pos.offset)
-                       return true;
-
-               bch2_btree_iter_advance_pos(iter);
+                   desc.hash_bkey(info, k) <= start->pos.offset) {
+                       ret = 1;
+                       break;
+               }
        }
-}
 
-#define BCH_HASH_SET_MUST_CREATE       (1 << 4)
-#define BCH_HASH_SET_MUST_REPLACE      (1 << 5)
+       bch2_trans_iter_exit(trans, &iter);
+       return ret;
+}
 
-static inline int bch2_hash_set(const struct bch_hash_desc desc,
-                              const struct bch_hash_info *info,
-                              struct bch_fs *c, u64 inode,
-                              u64 *journal_seq,
-                              struct bkey_i *insert, int flags)
+static __always_inline
+int bch2_hash_set(struct btree_trans *trans,
+                 const struct bch_hash_desc desc,
+                 const struct bch_hash_info *info,
+                 u64 inode, struct bkey_i *insert, int flags)
 {
-       struct btree_iter iter, hashed_slot;
+       struct btree_iter iter, slot = { NULL };
        struct bkey_s_c k;
+       bool found = false;
        int ret;
 
-       bch2_btree_iter_init_intent(&hashed_slot, c, desc.btree_id,
-               POS(inode, desc.hash_bkey(info, bkey_i_to_s_c(insert))));
-       bch2_btree_iter_init_intent(&iter, c, desc.btree_id, hashed_slot.pos);
-       bch2_btree_iter_link(&hashed_slot, &iter);
-retry:
-       /*
-        * On hash collision, we have to keep the slot we hashed to locked while
-        * we do the insert - to avoid racing with another thread deleting
-        * whatever's in the slot we hashed to:
-        */
-       ret = bch2_btree_iter_traverse(&hashed_slot);
-       if (ret)
-               goto err;
-
-       /*
-        * On -EINTR/retry, we dropped locks - always restart from the slot we
-        * hashed to:
-        */
-       bch2_btree_iter_copy(&iter, &hashed_slot);
+       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)
+                       break;
 
-       k = bch2_hash_lookup_bkey_at(desc, info, &iter, bkey_i_to_s_c(insert));
+               if (k.k->type == desc.key_type) {
+                       if (!desc.cmp_bkey(k, bkey_i_to_s_c(insert)))
+                               goto found;
 
-       ret = btree_iter_err(k);
-       if (ret == -ENOENT) {
-               if (flags & BCH_HASH_SET_MUST_REPLACE) {
-                       ret = -ENOENT;
-                       goto err;
+                       /* hash collision: */
+                       continue;
                }
 
-               /*
-                * Not found, so we're now looking for any open
-                * slot - we might have skipped over a whiteout
-                * that we could have used, so restart from the
-                * slot we hashed to:
-                */
-               bch2_btree_iter_copy(&iter, &hashed_slot);
-               k = bch2_hash_hole_at(desc, &iter);
-               if ((ret = btree_iter_err(k)))
-                       goto err;
-       } else if (!ret) {
-               if (flags & BCH_HASH_SET_MUST_CREATE) {
-                       ret = -EEXIST;
-                       goto err;
-               }
-       } else {
-               goto err;
+               if (!slot.path &&
+                   !(flags & BCH_HASH_SET_MUST_REPLACE))
+                       bch2_trans_copy_iter(&slot, &iter);
+
+               if (k.k->type != KEY_TYPE_hash_whiteout)
+                       goto not_found;
        }
 
-       insert->k.p = iter.pos;
-       ret = bch2_btree_insert_at(c, NULL, NULL, journal_seq,
-                                 BTREE_INSERT_ATOMIC|flags,
-                                 BTREE_INSERT_ENTRY(&iter, insert));
-err:
-       if (ret == -EINTR)
-               goto retry;
+       if (!ret)
+               ret = -ENOSPC;
+out:
+       bch2_trans_iter_exit(trans, &slot);
+       bch2_trans_iter_exit(trans, &iter);
 
-       /*
-        * On successful insert, we don't want to clobber ret with error from
-        * iter:
-        */
-       bch2_btree_iter_unlock(&iter);
-       bch2_btree_iter_unlock(&hashed_slot);
        return ret;
+found:
+       found = true;
+not_found:
+
+       if (!found && (flags & BCH_HASH_SET_MUST_REPLACE)) {
+               ret = -ENOENT;
+       } else if (found && (flags & BCH_HASH_SET_MUST_CREATE)) {
+               ret = -EEXIST;
+       } else {
+               if (!found && slot.path)
+                       swap(iter, slot);
+
+               insert->k.p = iter.pos;
+               ret = bch2_trans_update(trans, &iter, insert, 0);
+       }
+
+       goto out;
 }
 
-static inline int bch2_hash_delete_at(const struct bch_hash_desc desc,
-                                     const struct bch_hash_info *info,
-                                     struct btree_iter *iter,
-                                     u64 *journal_seq)
+static __always_inline
+int bch2_hash_delete_at(struct btree_trans *trans,
+                       const struct bch_hash_desc desc,
+                       const struct bch_hash_info *info,
+                       struct btree_iter *iter)
 {
-       struct btree_iter whiteout_iter;
-       struct bkey_i delete;
-       int ret = -ENOENT;
+       struct bkey_i *delete;
+       int ret;
 
-       bch2_btree_iter_init(&whiteout_iter, iter->c, desc.btree_id,
-                            iter->pos);
-       bch2_btree_iter_link(iter, &whiteout_iter);
+       delete = bch2_trans_kmalloc(trans, sizeof(*delete));
+       ret = PTR_ERR_OR_ZERO(delete);
+       if (ret)
+               return ret;
 
-       ret = bch2_hash_needs_whiteout(desc, info, &whiteout_iter, iter);
+       ret = bch2_hash_needs_whiteout(trans, desc, info, iter);
        if (ret < 0)
-               goto err;
-
-       bkey_init(&delete.k);
-       delete.k.p = iter->pos;
-       delete.k.type = ret ? desc.whiteout_type : KEY_TYPE_DELETED;
-
-       ret = bch2_btree_insert_at(iter->c, NULL, NULL, journal_seq,
-                                 BTREE_INSERT_NOFAIL|
-                                 BTREE_INSERT_ATOMIC,
-                                 BTREE_INSERT_ENTRY(iter, &delete));
-err:
-       bch2_btree_iter_unlink(&whiteout_iter);
-       return ret;
+               return ret;
+
+       bkey_init(&delete->k);
+       delete->k.p = iter->pos;
+       delete->k.type = ret ? KEY_TYPE_hash_whiteout : KEY_TYPE_deleted;
+
+       return bch2_trans_update(trans, iter, delete, 0);
 }
 
-static inline int bch2_hash_delete(const struct bch_hash_desc desc,
-                                 const struct bch_hash_info *info,
-                                 struct bch_fs *c, u64 inode,
-                                 u64 *journal_seq, const void *key)
+static __always_inline
+int bch2_hash_delete(struct btree_trans *trans,
+                    const struct bch_hash_desc desc,
+                    const struct bch_hash_info *info,
+                    u64 inode, const void *key)
 {
-       struct btree_iter iter, whiteout_iter;
-       struct bkey_s_c k;
-       int ret = -ENOENT;
-
-       bch2_btree_iter_init_intent(&iter, c, desc.btree_id,
-                           POS(inode, desc.hash_key(info, key)));
-       bch2_btree_iter_init(&whiteout_iter, c, desc.btree_id,
-                           POS(inode, desc.hash_key(info, key)));
-       bch2_btree_iter_link(&iter, &whiteout_iter);
-retry:
-       k = bch2_hash_lookup_at(desc, info, &iter, key);
-       if ((ret = btree_iter_err(k)))
-               goto err;
-
-       ret = bch2_hash_delete_at(desc, info, &iter, journal_seq);
-err:
-       if (ret == -EINTR)
-               goto retry;
-
-       bch2_btree_iter_unlock(&whiteout_iter);
-       bch2_btree_iter_unlock(&iter);
+       struct btree_iter iter;
+       int ret;
+
+       ret = bch2_hash_lookup(trans, &iter, desc, info, inode, key,
+                               BTREE_ITER_INTENT);
+       if (ret)
+               return ret;
+
+       ret = bch2_hash_delete_at(trans, desc, info, &iter);
+       bch2_trans_iter_exit(trans, &iter);
        return ret;
 }
 
-#endif /* _BCACHE_STR_HASH_H */
+#endif /* _BCACHEFS_STR_HASH_H */