Commit 7d9f107a authored by Yu Kuai's avatar Yu Kuai Committed by Song Liu

md: use new helpers in md_do_sync()

Make code cleaner. and also use the action_name directly in kernel log:
 - "check" instead of "data-check"
 - "repair" instead of "requested-resync"
Signed-off-by: default avatarYu Kuai <yukuai3@huawei.com>
Signed-off-by: default avatarSong Liu <song@kernel.org>
Link: https://lore.kernel.org/r/20240611132251.1967786-9-yukuai1@huaweicloud.com
parent 5ce10a38
...@@ -8929,7 +8929,8 @@ void md_do_sync(struct md_thread *thread) ...@@ -8929,7 +8929,8 @@ void md_do_sync(struct md_thread *thread)
sector_t last_check; sector_t last_check;
int skipped = 0; int skipped = 0;
struct md_rdev *rdev; struct md_rdev *rdev;
char *desc, *action = NULL; enum sync_action action;
const char *desc;
struct blk_plug plug; struct blk_plug plug;
int ret; int ret;
...@@ -8960,21 +8961,9 @@ void md_do_sync(struct md_thread *thread) ...@@ -8960,21 +8961,9 @@ void md_do_sync(struct md_thread *thread)
goto skip; goto skip;
} }
if (test_bit(MD_RECOVERY_SYNC, &mddev->recovery)) { action = md_sync_action(mddev);
if (test_bit(MD_RECOVERY_CHECK, &mddev->recovery)) { desc = md_sync_action_name(action);
desc = "data-check"; mddev->last_sync_action = desc;
action = "check";
} else if (test_bit(MD_RECOVERY_REQUESTED, &mddev->recovery)) {
desc = "requested-resync";
action = "repair";
} else
desc = "resync";
} else if (test_bit(MD_RECOVERY_RESHAPE, &mddev->recovery))
desc = "reshape";
else
desc = "recovery";
mddev->last_sync_action = action ?: desc;
/* /*
* Before starting a resync we must have set curr_resync to * Before starting a resync we must have set curr_resync to
......
...@@ -432,7 +432,7 @@ struct mddev { ...@@ -432,7 +432,7 @@ struct mddev {
* when the sync thread is "frozen" (interrupted) or "idle" (stopped * when the sync thread is "frozen" (interrupted) or "idle" (stopped
* or finished). It is overwritten when a new sync operation is begun. * or finished). It is overwritten when a new sync operation is begun.
*/ */
char *last_sync_action; const char *last_sync_action;
sector_t curr_resync; /* last block scheduled */ sector_t curr_resync; /* last block scheduled */
/* As resync requests can complete out of order, we cannot easily track /* As resync requests can complete out of order, we cannot easily track
* how much resync has been completed. So we occasionally pause until * how much resync has been completed. So we occasionally pause until
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment