Commit c8908959 authored by Kent Overstreet's avatar Kent Overstreet
Browse files

bcachefs: kill bch2_bkey_needs_rebalance() 



Dead code

Signed-off-by: default avatarKent Overstreet <kent.overstreet@linux.dev>
parent 015fafc4

fs/bcachefs/extents.c

+0 −17
Original line number Diff line number Diff line
@@ -1442,23 +1442,6 @@ unsigned bch2_bkey_ptrs_need_rebalance(struct bch_fs *c, struct bkey_s_c k, 
	return rewrite_ptrs;

}



bool bch2_bkey_needs_rebalance(struct bch_fs *c, struct bkey_s_c k)

{

	const struct bch_extent_rebalance *r = bch2_bkey_rebalance_opts(k);



	/*

	 * If it's an indirect extent, we don't delete the rebalance entry when

	 * done so that we know what options were applied - check if it still

	 * needs work done:

	 */

	if (r &&

	    k.k->type == KEY_TYPE_reflink_v &&

	    !bch2_bkey_ptrs_need_rebalance(c, k, r->target, r->compression))

		r = NULL;



	return r != NULL;

}



static u64 __bch2_bkey_sectors_need_rebalance(struct bch_fs *c, struct bkey_s_c k,

				       unsigned target, unsigned compression)

{

fs/bcachefs/extents.h

+0 −1
Original line number Diff line number Diff line
@@ -713,7 +713,6 @@ void bch2_ptr_swab(struct bkey_s); 
const struct bch_extent_rebalance *bch2_bkey_rebalance_opts(struct bkey_s_c);

unsigned bch2_bkey_ptrs_need_rebalance(struct bch_fs *, struct bkey_s_c,

				       unsigned, unsigned);

bool bch2_bkey_needs_rebalance(struct bch_fs *, struct bkey_s_c);

u64 bch2_bkey_sectors_need_rebalance(struct bch_fs *, struct bkey_s_c);



int bch2_bkey_set_needs_rebalance(struct bch_fs *, struct bkey_i *,