]> git.sesse.net Git - bcachefs-tools-debian/blobdiff - libbcachefs/bkey_sort.c
Move c_src dirs back to toplevel
[bcachefs-tools-debian] / libbcachefs / bkey_sort.c
index 18f842012f05ab1afe57a630c5c2c898c281669a..bcca9e76a0b4bf40f20903c856e7559e60b87da1 100644 (file)
@@ -1,6 +1,7 @@
 // SPDX-License-Identifier: GPL-2.0
 #include "bcachefs.h"
-#include "bkey_on_stack.h"
+#include "bkey_buf.h"
+#include "bkey_cmp.h"
 #include "bkey_sort.h"
 #include "bset.h"
 #include "extents.h"
@@ -14,9 +15,8 @@ static inline bool sort_iter_end(struct sort_iter *iter)
        return !iter->used;
 }
 
-static inline void __sort_iter_sift(struct sort_iter *iter,
-                                   unsigned from,
-                                   sort_cmp_fn cmp)
+static inline void sort_iter_sift(struct sort_iter *iter, unsigned from,
+                                 sort_cmp_fn cmp)
 {
        unsigned i;
 
@@ -27,18 +27,12 @@ static inline void __sort_iter_sift(struct sort_iter *iter,
                swap(iter->data[i], iter->data[i + 1]);
 }
 
-static inline void sort_iter_sift(struct sort_iter *iter, sort_cmp_fn cmp)
-{
-
-       __sort_iter_sift(iter, 0, cmp);
-}
-
 static inline void sort_iter_sort(struct sort_iter *iter, sort_cmp_fn cmp)
 {
        unsigned i = iter->used;
 
        while (i--)
-               __sort_iter_sift(iter, i, cmp);
+               sort_iter_sift(iter, i, cmp);
 }
 
 static inline struct bkey_packed *sort_iter_peek(struct sort_iter *iter)
@@ -46,26 +40,20 @@ static inline struct bkey_packed *sort_iter_peek(struct sort_iter *iter)
        return !sort_iter_end(iter) ? iter->data->k : NULL;
 }
 
-static inline void __sort_iter_advance(struct sort_iter *iter,
-                                      unsigned idx, sort_cmp_fn cmp)
+static inline void sort_iter_advance(struct sort_iter *iter, sort_cmp_fn cmp)
 {
-       struct sort_iter_set *i = iter->data + idx;
+       struct sort_iter_set *i = iter->data;
 
-       BUG_ON(idx >= iter->used);
+       BUG_ON(!iter->used);
 
-       i->k = bkey_next_skip_noops(i->k, i->end);
+       i->k = bkey_p_next(i->k);
 
        BUG_ON(i->k > i->end);
 
        if (i->k == i->end)
-               array_remove_item(iter->data, iter->used, idx);
+               array_remove_item(iter->data, iter->used, 0);
        else
-               __sort_iter_sift(iter, idx, cmp);
-}
-
-static inline void sort_iter_advance(struct sort_iter *iter, sort_cmp_fn cmp)
-{
-       __sort_iter_advance(iter, 0, cmp);
+               sort_iter_sift(iter, 0, cmp);
 }
 
 static inline struct bkey_packed *sort_iter_next(struct sort_iter *iter,
@@ -86,7 +74,7 @@ static inline int key_sort_fix_overlapping_cmp(struct btree *b,
                                               struct bkey_packed *l,
                                               struct bkey_packed *r)
 {
-       return bkey_cmp_packed(b, l, r) ?:
+       return bch2_bkey_cmp_packed(b, l, r) ?:
                cmp_int((unsigned long) l, (unsigned long) r);
 }
 
@@ -98,7 +86,7 @@ static inline bool should_drop_next_key(struct sort_iter *iter)
         * and should be dropped.
         */
        return iter->used >= 2 &&
-               !bkey_cmp_packed(iter->b,
+               !bch2_bkey_cmp_packed(iter->b,
                                 iter->data[0].k,
                                 iter->data[1].k);
 }
@@ -116,11 +104,11 @@ bch2_key_sort_fix_overlapping(struct bch_fs *c, struct bset *dst,
        sort_iter_sort(iter, key_sort_fix_overlapping_cmp);
 
        while ((k = sort_iter_peek(iter))) {
-               if (!bkey_whiteout(k) &&
+               if (!bkey_deleted(k) &&
                    !should_drop_next_key(iter)) {
-                       bkey_copy(out, k);
+                       bkey_p_copy(out, k);
                        btree_keys_account_key_add(&nr, 0, out);
-                       out = bkey_next(out);
+                       out = bkey_p_next(out);
                }
 
                sort_iter_advance(iter, key_sort_fix_overlapping_cmp);
@@ -130,46 +118,6 @@ bch2_key_sort_fix_overlapping(struct bch_fs *c, struct bset *dst,
        return nr;
 }
 
-static void extent_sort_advance_prev(struct bkey_format *f,
-                                    struct btree_nr_keys *nr,
-                                    struct bkey_packed *start,
-                                    struct bkey_packed **prev)
-{
-       if (*prev) {
-               bch2_bkey_pack(*prev, (void *) *prev, f);
-
-               btree_keys_account_key_add(nr, 0, *prev);
-               *prev = bkey_next(*prev);
-       } else {
-               *prev = start;
-       }
-}
-
-static void extent_sort_append(struct bch_fs *c,
-                              struct bkey_format *f,
-                              struct btree_nr_keys *nr,
-                              struct bkey_packed *start,
-                              struct bkey_packed **prev,
-                              struct bkey_s k)
-{
-       if (bkey_whiteout(k.k))
-               return;
-
-       /*
-        * prev is always unpacked, for key merging - until right before we
-        * advance it:
-        */
-
-       if (*prev &&
-           bch2_bkey_merge(c, bkey_i_to_s((void *) *prev), k) ==
-           BCH_MERGE_MERGE)
-               return;
-
-       extent_sort_advance_prev(f, nr, start, prev);
-
-       bkey_reassemble((void *) *prev, k.s_c);
-}
-
 /* Sort + repack in a new format: */
 struct btree_nr_keys
 bch2_sort_repack(struct bset *dst, struct btree *src,
@@ -180,66 +128,37 @@ bch2_sort_repack(struct bset *dst, struct btree *src,
        struct bkey_format *in_f = &src->format;
        struct bkey_packed *in, *out = vstruct_last(dst);
        struct btree_nr_keys nr;
+       bool transform = memcmp(out_f, &src->format, sizeof(*out_f));
 
        memset(&nr, 0, sizeof(nr));
 
        while ((in = bch2_btree_node_iter_next_all(src_iter, src))) {
-               if (filter_whiteouts && bkey_whiteout(in))
+               if (filter_whiteouts && bkey_deleted(in))
                        continue;
 
-               if (bch2_bkey_transform(out_f, out, bkey_packed(in)
-                                      ? in_f : &bch2_bkey_format_current, in))
+               if (!transform)
+                       bkey_p_copy(out, in);
+               else if (bch2_bkey_transform(out_f, out, bkey_packed(in)
+                                            ? in_f : &bch2_bkey_format_current, in))
                        out->format = KEY_FORMAT_LOCAL_BTREE;
                else
                        bch2_bkey_unpack(src, (void *) out, in);
 
+               out->needs_whiteout = false;
+
                btree_keys_account_key_add(&nr, 0, out);
-               out = bkey_next(out);
+               out = bkey_p_next(out);
        }
 
        dst->u64s = cpu_to_le16((u64 *) out - dst->_data);
        return nr;
 }
 
-/* Sort, repack, and merge: */
-struct btree_nr_keys
-bch2_sort_repack_merge(struct bch_fs *c,
-                      struct bset *dst, struct btree *src,
-                      struct btree_node_iter *iter,
-                      struct bkey_format *out_f,
-                      bool filter_whiteouts)
-{
-       struct bkey_packed *prev = NULL, *k_packed;
-       struct bkey_s k;
-       struct btree_nr_keys nr;
-       struct bkey unpacked;
-
-       memset(&nr, 0, sizeof(nr));
-
-       while ((k_packed = bch2_btree_node_iter_next_all(iter, src))) {
-               if (filter_whiteouts && bkey_whiteout(k_packed))
-                       continue;
-
-               k = __bkey_disassemble(src, k_packed, &unpacked);
-
-               if (filter_whiteouts &&
-                   bch2_bkey_normalize(c, k))
-                       continue;
-
-               extent_sort_append(c, out_f, &nr, vstruct_last(dst), &prev, k);
-       }
-
-       extent_sort_advance_prev(out_f, &nr, vstruct_last(dst), &prev);
-
-       dst->u64s = cpu_to_le16((u64 *) prev - dst->_data);
-       return nr;
-}
-
 static inline int sort_keys_cmp(struct btree *b,
                                struct bkey_packed *l,
                                struct bkey_packed *r)
 {
-       return bkey_cmp_packed(b, l, r) ?:
+       return bch2_bkey_cmp_packed_inlined(b, l, r) ?:
                (int) bkey_deleted(r) - (int) bkey_deleted(l) ?:
                (int) l->needs_whiteout - (int) r->needs_whiteout;
 }
@@ -254,260 +173,28 @@ unsigned bch2_sort_keys(struct bkey_packed *dst,
        sort_iter_sort(iter, sort_keys_cmp);
 
        while ((in = sort_iter_next(iter, sort_keys_cmp))) {
-               if (bkey_whiteout(in) &&
+               bool needs_whiteout = false;
+
+               if (bkey_deleted(in) &&
                    (filter_whiteouts || !in->needs_whiteout))
                        continue;
 
-               if (bkey_whiteout(in) &&
-                   (next = sort_iter_peek(iter)) &&
-                   !bkey_cmp_packed(iter->b, in, next)) {
+               while ((next = sort_iter_peek(iter)) &&
+                      !bch2_bkey_cmp_packed_inlined(iter->b, in, next)) {
                        BUG_ON(in->needs_whiteout &&
                               next->needs_whiteout);
-                       /*
-                        * XXX racy, called with read lock from write path
-                        *
-                        * leads to spurious BUG_ON() in bkey_unpack_key() in
-                        * debug mode
-                        */
-                       next->needs_whiteout |= in->needs_whiteout;
-                       continue;
+                       needs_whiteout |= in->needs_whiteout;
+                       in = sort_iter_next(iter, sort_keys_cmp);
                }
 
-               if (bkey_whiteout(in)) {
-                       memcpy_u64s(out, in, bkeyp_key_u64s(f, in));
+               if (bkey_deleted(in)) {
+                       memcpy_u64s_small(out, in, bkeyp_key_u64s(f, in));
                        set_bkeyp_val_u64s(f, out, 0);
                } else {
-                       bkey_copy(out, in);
-               }
-               out = bkey_next(out);
-       }
-
-       return (u64 *) out - (u64 *) dst;
-}
-
-/* Compat code for btree_node_old_extent_overwrite: */
-
-/*
- * If keys compare equal, compare by pointer order:
- *
- * Necessary for sort_fix_overlapping() - if there are multiple keys that
- * compare equal in different sets, we have to process them newest to oldest.
- */
-static inline int extent_sort_fix_overlapping_cmp(struct btree *b,
-                                                 struct bkey_packed *l,
-                                                 struct bkey_packed *r)
-{
-       struct bkey ul = bkey_unpack_key(b, l);
-       struct bkey ur = bkey_unpack_key(b, r);
-
-       return bkey_cmp(bkey_start_pos(&ul),
-                       bkey_start_pos(&ur)) ?:
-               cmp_int((unsigned long) r, (unsigned long) l);
-}
-
-struct btree_nr_keys
-bch2_extent_sort_fix_overlapping(struct bch_fs *c, struct bset *dst,
-                                struct sort_iter *iter)
-{
-       struct btree *b = iter->b;
-       struct bkey_format *f = &b->format;
-       struct sort_iter_set *_l = iter->data, *_r = iter->data + 1;
-       struct bkey_packed *prev = NULL;
-       struct bkey l_unpacked, r_unpacked;
-       struct bkey_s l, r;
-       struct btree_nr_keys nr;
-       struct bkey_on_stack split;
-
-       memset(&nr, 0, sizeof(nr));
-       bkey_on_stack_init(&split);
-
-       sort_iter_sort(iter, extent_sort_fix_overlapping_cmp);
-
-       while (!sort_iter_end(iter)) {
-               l = __bkey_disassemble(b, _l->k, &l_unpacked);
-
-               if (iter->used == 1) {
-                       extent_sort_append(c, f, &nr, dst->start, &prev, l);
-                       sort_iter_advance(iter,
-                                         extent_sort_fix_overlapping_cmp);
-                       continue;
-               }
-
-               r = __bkey_disassemble(b, _r->k, &r_unpacked);
-
-               /* If current key and next key don't overlap, just append */
-               if (bkey_cmp(l.k->p, bkey_start_pos(r.k)) <= 0) {
-                       extent_sort_append(c, f, &nr, dst->start, &prev, l);
-                       sort_iter_advance(iter,
-                                         extent_sort_fix_overlapping_cmp);
-                       continue;
-               }
-
-               /* Skip 0 size keys */
-               if (!r.k->size) {
-                       __sort_iter_advance(iter, 1,
-                                           extent_sort_fix_overlapping_cmp);
-                       continue;
-               }
-
-               /*
-                * overlap: keep the newer key and trim the older key so they
-                * don't overlap. comparing pointers tells us which one is
-                * newer, since the bsets are appended one after the other.
-                */
-
-               /* can't happen because of comparison func */
-               BUG_ON(_l->k < _r->k &&
-                      !bkey_cmp(bkey_start_pos(l.k), bkey_start_pos(r.k)));
-
-               if (_l->k > _r->k) {
-                       /* l wins, trim r */
-                       if (bkey_cmp(l.k->p, r.k->p) >= 0) {
-                               __sort_iter_advance(iter, 1,
-                                        extent_sort_fix_overlapping_cmp);
-                       } else {
-                               bch2_cut_front_s(l.k->p, r);
-                               extent_save(b, _r->k, r.k);
-                               __sort_iter_sift(iter, 1,
-                                        extent_sort_fix_overlapping_cmp);
-                       }
-               } else if (bkey_cmp(l.k->p, r.k->p) > 0) {
-
-                       /*
-                        * r wins, but it overlaps in the middle of l - split l:
-                        */
-                       bkey_on_stack_reassemble(&split, c, l.s_c);
-                       bch2_cut_back(bkey_start_pos(r.k), split.k);
-
-                       bch2_cut_front_s(r.k->p, l);
-                       extent_save(b, _l->k, l.k);
-
-                       __sort_iter_sift(iter, 0,
-                                        extent_sort_fix_overlapping_cmp);
-
-                       extent_sort_append(c, f, &nr, dst->start,
-                                          &prev, bkey_i_to_s(split.k));
-               } else {
-                       bch2_cut_back_s(bkey_start_pos(r.k), l);
-                       extent_save(b, _l->k, l.k);
-               }
-       }
-
-       extent_sort_advance_prev(f, &nr, dst->start, &prev);
-
-       dst->u64s = cpu_to_le16((u64 *) prev - dst->_data);
-
-       bkey_on_stack_exit(&split, c);
-       return nr;
-}
-
-static inline int sort_extents_cmp(struct btree *b,
-                                  struct bkey_packed *l,
-                                  struct bkey_packed *r)
-{
-       return bkey_cmp_packed(b, l, r) ?:
-               (int) bkey_deleted(l) - (int) bkey_deleted(r);
-}
-
-unsigned bch2_sort_extents(struct bkey_packed *dst,
-                          struct sort_iter *iter,
-                          bool filter_whiteouts)
-{
-       struct bkey_packed *in, *out = dst;
-
-       sort_iter_sort(iter, sort_extents_cmp);
-
-       while ((in = sort_iter_next(iter, sort_extents_cmp))) {
-               if (bkey_deleted(in))
-                       continue;
-
-               if (bkey_whiteout(in) &&
-                   (filter_whiteouts || !in->needs_whiteout))
-                       continue;
-
-               bkey_copy(out, in);
-               out = bkey_next(out);
-       }
-
-       return (u64 *) out - (u64 *) dst;
-}
-
-static inline int sort_extent_whiteouts_cmp(struct btree *b,
-                                           struct bkey_packed *l,
-                                           struct bkey_packed *r)
-{
-       struct bkey ul = bkey_unpack_key(b, l);
-       struct bkey ur = bkey_unpack_key(b, r);
-
-       return bkey_cmp(bkey_start_pos(&ul), bkey_start_pos(&ur));
-}
-
-unsigned bch2_sort_extent_whiteouts(struct bkey_packed *dst,
-                                   struct sort_iter *iter)
-{
-       const struct bkey_format *f = &iter->b->format;
-       struct bkey_packed *in, *out = dst;
-       struct bkey_i l, r;
-       bool prev = false, l_packed = false;
-       u64 max_packed_size     = bkey_field_max(f, BKEY_FIELD_SIZE);
-       u64 max_packed_offset   = bkey_field_max(f, BKEY_FIELD_OFFSET);
-       u64 new_size;
-
-       max_packed_size = min_t(u64, max_packed_size, KEY_SIZE_MAX);
-
-       sort_iter_sort(iter, sort_extent_whiteouts_cmp);
-
-       while ((in = sort_iter_next(iter, sort_extent_whiteouts_cmp))) {
-               if (bkey_deleted(in))
-                       continue;
-
-               EBUG_ON(bkeyp_val_u64s(f, in));
-               EBUG_ON(in->type != KEY_TYPE_discard);
-
-               r.k = bkey_unpack_key(iter->b, in);
-
-               if (prev &&
-                   bkey_cmp(l.k.p, bkey_start_pos(&r.k)) >= 0) {
-                       if (bkey_cmp(l.k.p, r.k.p) >= 0)
-                               continue;
-
-                       new_size = l_packed
-                               ? min(max_packed_size, max_packed_offset -
-                                     bkey_start_offset(&l.k))
-                               : KEY_SIZE_MAX;
-
-                       new_size = min(new_size, r.k.p.offset -
-                                      bkey_start_offset(&l.k));
-
-                       BUG_ON(new_size < l.k.size);
-
-                       bch2_key_resize(&l.k, new_size);
-
-                       if (bkey_cmp(l.k.p, r.k.p) >= 0)
-                               continue;
-
-                       bch2_cut_front(l.k.p, &r);
-               }
-
-               if (prev) {
-                       if (!bch2_bkey_pack(out, &l, f)) {
-                               BUG_ON(l_packed);
-                               bkey_copy(out, &l);
-                       }
-                       out = bkey_next(out);
-               }
-
-               l = r;
-               prev = true;
-               l_packed = bkey_packed(in);
-       }
-
-       if (prev) {
-               if (!bch2_bkey_pack(out, &l, f)) {
-                       BUG_ON(l_packed);
-                       bkey_copy(out, &l);
+                       bkey_p_copy(out, in);
                }
-               out = bkey_next(out);
+               out->needs_whiteout |= needs_whiteout;
+               out = bkey_p_next(out);
        }
 
        return (u64 *) out - (u64 *) dst;