diff options
author | Kent Overstreet <kent.overstreet@linux.dev> | 2025-01-25 21:29:45 -0500 |
---|---|---|
committer | Kent Overstreet <kent.overstreet@linux.dev> | 2025-02-06 22:35:11 -0500 |
commit | 4be214c26936813b636eed2fac906f585ddbf0f9 (patch) | |
tree | baa68ff28776dab78b221c751914796bc0605199 /.clang-format | |
parent | 3539880ef1a5f8c970d0f69a6fdcfeffc000e63d (diff) |
bcachefs: bch2_bkey_sectors_need_rebalance() now only depends on bch_extent_rebalance
Previously, bch2_bkey_sectors_need_rebalance() called
bch2_target_accepts_data(), checking whether the target is writable.
However, this means that adding or removing devices from a target would
change the value of bch2_bkey_sectors_need_rebalance() for an existing
extent; this needs to be invariant so that the extent trigger can
correctly maintain rebalance_work accounting.
Instead, check target_accepts_data() in io_opts_to_rebalance_opts(),
before creating the bch_extent_rebalance entry.
This fixes (one?) cause of rebalance_work accounting being off.
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
Diffstat (limited to '.clang-format')
0 files changed, 0 insertions, 0 deletions