diff options
author | NeilBrown <neilb@suse.de> | 2009-03-31 00:24:32 -0400 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2009-03-31 00:24:32 -0400 |
commit | d1a7c50369835f9ecbd7752016cd9302ecfae678 (patch) | |
tree | 2c6bcc8c5cb4d045d555e0d030321cb887a4e05a | |
parent | 88ce4930e2b80378d45506ce2c3bb5820e156e85 (diff) |
md: don't display meaningless values in sysfs files resync_start and sync_speed
When no resync if happening, both of these files currently have
meaningless values (is slightly different ways).
Change them to "none" in that case.
Signed-off-by: NeilBrown <neilb@suse.de>
-rw-r--r-- | drivers/md/md.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index c50931352b23..2be574c0a27a 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c | |||
@@ -2888,6 +2888,8 @@ __ATTR(chunk_size, S_IRUGO|S_IWUSR, chunk_size_show, chunk_size_store); | |||
2888 | static ssize_t | 2888 | static ssize_t |
2889 | resync_start_show(mddev_t *mddev, char *page) | 2889 | resync_start_show(mddev_t *mddev, char *page) |
2890 | { | 2890 | { |
2891 | if (mddev->recovery_cp == MaxSector) | ||
2892 | return sprintf(page, "none\n"); | ||
2891 | return sprintf(page, "%llu\n", (unsigned long long)mddev->recovery_cp); | 2893 | return sprintf(page, "%llu\n", (unsigned long long)mddev->recovery_cp); |
2892 | } | 2894 | } |
2893 | 2895 | ||
@@ -3469,6 +3471,8 @@ static ssize_t | |||
3469 | sync_speed_show(mddev_t *mddev, char *page) | 3471 | sync_speed_show(mddev_t *mddev, char *page) |
3470 | { | 3472 | { |
3471 | unsigned long resync, dt, db; | 3473 | unsigned long resync, dt, db; |
3474 | if (mddev->curr_resync == 0) | ||
3475 | return sprintf(page, "none\n"); | ||
3472 | resync = mddev->curr_mark_cnt - atomic_read(&mddev->recovery_active); | 3476 | resync = mddev->curr_mark_cnt - atomic_read(&mddev->recovery_active); |
3473 | dt = (jiffies - mddev->resync_mark) / HZ; | 3477 | dt = (jiffies - mddev->resync_mark) / HZ; |
3474 | if (!dt) dt++; | 3478 | if (!dt) dt++; |