[RFC v2 3/5] md: rename mdstat action "recovery" to "recover"
Zheng Qixing <[email protected]>
| Newsgroups | gmane.linux.raid,gmane.linux.kernel |
|---|---|
| Message-ID | <[email protected]> |
From: Zheng Qixing <[email protected]> Simplify the code in status_resync() that prints the progress of sync actions. Also rename the /proc/mdstat action from "recovery" to "recover" to match the naming used by action_store() and action_show(). Note: The md-raid-utilities/mdadm test suite will need to be updated to expect "recover" instead of "recovery" in /proc/mdstat. Signed-off-by: Zheng Qixing <[email protected]> --- drivers/md/md.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/md/md.c b/drivers/md/md.c index 7fe02ee21d3e..f319621c6832 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -8669,6 +8669,7 @@ static int status_resync(struct seq_file *seq, struct mddev *mddev) sector_t rt, curr_mark_cnt, resync_mark_cnt; int scale, recovery_active; unsigned int per_milli; + enum sync_action action; if (test_bit(MD_RECOVERY_SYNC, &mddev->recovery) || test_bit(MD_RECOVERY_RESHAPE, &mddev->recovery)) @@ -8750,13 +8751,10 @@ static int status_resync(struct seq_file *seq, struct mddev *mddev) seq_printf(seq, "."); seq_printf(seq, "] "); } + + action = md_sync_action(mddev); seq_printf(seq, " %s =%3u.%u%% (%llu/%llu)", - (test_bit(MD_RECOVERY_RESHAPE, &mddev->recovery)? - "reshape" : - (test_bit(MD_RECOVERY_CHECK, &mddev->recovery)? - "check" : - (test_bit(MD_RECOVERY_SYNC, &mddev->recovery) ? - "resync" : "recovery"))), + md_sync_action_name(action), per_milli/10, per_milli % 10, (unsigned long long) resync/2, (unsigned long long) max_sectors/2); -- 2.39.2