diff options
author | NeilBrown <neilb@suse.de> | 2008-03-10 14:43:47 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-03-10 21:01:19 -0400 |
commit | 52720ae77d392d3f4c12281c37304edbc8cb51f1 (patch) | |
tree | 2a04d72705155fbdf2fe5181da414bc0c5261525 | |
parent | 69682d852f5c94ee94e21174b3e8b719626c98db (diff) |
md: fix formatting error in /proc/mdstat
If an md array is "auto-read-only", then this appears in /proc/mdstat as
/dev/md0: active(auto-read-only)
whereas if it is truely readonly, it appears as
/dev/md0: active (read-only)
The difference being a space.
One program known to parse this file expects the space and gets badly
confused. It will be fixed, but it would be best if what the kernel generates
is more consistent too.
Signed-off-by: Neil Brown <neilb@suse.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | drivers/md/md.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index 827824a9f3e9..ccbbf63727cc 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c | |||
@@ -5149,7 +5149,7 @@ static int md_seq_show(struct seq_file *seq, void *v) | |||
5149 | if (mddev->ro==1) | 5149 | if (mddev->ro==1) |
5150 | seq_printf(seq, " (read-only)"); | 5150 | seq_printf(seq, " (read-only)"); |
5151 | if (mddev->ro==2) | 5151 | if (mddev->ro==2) |
5152 | seq_printf(seq, "(auto-read-only)"); | 5152 | seq_printf(seq, " (auto-read-only)"); |
5153 | seq_printf(seq, " %s", mddev->pers->name); | 5153 | seq_printf(seq, " %s", mddev->pers->name); |
5154 | } | 5154 | } |
5155 | 5155 | ||