diff options
| author | Kent Overstreet <kent.overstreet@linux.dev> | 2023-11-26 23:11:18 -0500 |
|---|---|---|
| committer | Kent Overstreet <kent.overstreet@linux.dev> | 2023-11-28 22:58:22 -0500 |
| commit | 1b1bd0fd41577400434d5948edc2679092c57b08 (patch) | |
| tree | cdbfd603830c20311f5c8ff5d69272f3d0e11f4a | |
| parent | ae4d612cc1f23a6137d0978b1620a2af2e201c1a (diff) | |
| download | linux-xfs-1b1bd0fd41577400434d5948edc2679092c57b08.tar.gz | |
bcachefs: -EROFS doesn't count as move_extent_start_fail
The automated tests check if we've hit too many slowpath/error path
events and fail the test - if we're just shutting down, that naturally
shouldn't count.
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
| -rw-r--r-- | fs/bcachefs/move.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/bcachefs/move.c b/fs/bcachefs/move.c index 0a04adff114416..67ac68f9dd3f1c 100644 --- a/fs/bcachefs/move.c +++ b/fs/bcachefs/move.c @@ -338,6 +338,10 @@ err: if (ret == -BCH_ERR_data_update_done) return 0; + if (bch2_err_matches(ret, EROFS) || + bch2_err_matches(ret, BCH_ERR_transaction_restart)) + return ret; + this_cpu_inc(c->counters[BCH_COUNTER_move_extent_start_fail]); if (trace_move_extent_start_fail_enabled()) { struct printbuf buf = PRINTBUF; |