diff options
| author | Kent Overstreet <kent.overstreet@gmail.com> | 2020-04-07 13:49:14 -0400 |
|---|---|---|
| committer | Kent Overstreet <kent.overstreet@linux.dev> | 2023-10-22 17:08:38 -0400 |
| commit | e77e4efce31f3739fa85a84c5197b7cd18747a64 (patch) | |
| tree | 4f4b101741bc94473ba8e29b0a5f8245f63c3939 /fs/bcachefs/rebalance.c | |
| parent | 0f9dda478fb50bb7c55e4760158a6ef7cca7cb04 (diff) | |
| download | linux-e77e4efce31f3739fa85a84c5197b7cd18747a64.tar.gz linux-e77e4efce31f3739fa85a84c5197b7cd18747a64.tar.bz2 linux-e77e4efce31f3739fa85a84c5197b7cd18747a64.zip | |
bcachefs: Account for ioclock slop when throttling rebalance thread
This should fix an issue where the rebalance thread was spinning
Signed-off-by: Kent Overstreet <kent.overstreet@gmail.com>
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
Diffstat (limited to 'fs/bcachefs/rebalance.c')
| -rw-r--r-- | fs/bcachefs/rebalance.c | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/fs/bcachefs/rebalance.c b/fs/bcachefs/rebalance.c index a0a75cfa41cb..eb3f7d02c791 100644 --- a/fs/bcachefs/rebalance.c +++ b/fs/bcachefs/rebalance.c @@ -204,17 +204,21 @@ static int bch2_rebalance_thread(void *arg) prev_run_time; if (w.dev_most_full_percent < 20 && throttle > 0) { - r->state = REBALANCE_THROTTLED; r->throttled_until_iotime = io_start + div_u64(w.dev_most_full_capacity * (20 - w.dev_most_full_percent), 50); - r->throttled_until_cputime = start + throttle; - bch2_kthread_io_clock_wait(clock, - r->throttled_until_iotime, - throttle); - continue; + if (atomic_long_read(&clock->now) + clock->max_slop < + r->throttled_until_iotime) { + r->throttled_until_cputime = start + throttle; + r->state = REBALANCE_THROTTLED; + + bch2_kthread_io_clock_wait(clock, + r->throttled_until_iotime, + throttle); + continue; + } } /* minimum 1 mb/sec: */ |