diff options
| author | Neil Brown <neilb@cse.unsw.edu.au> | 2005-03-09 16:27:26 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@ppc970.osdl.org> | 2005-03-09 16:27:26 -0800 |
| commit | 6ce890047f4a81022389fd74ad47fd7b0598cbe9 (patch) | |
| tree | 00e238d005cbbab4f40eddc81c48769064b8c787 | |
| parent | 6c5b4f37fab326654ab34152f5e084a85dd3ccf7 (diff) | |
[PATCH] md: fix typo in super_1_sync
Instead of setting one value lots of times, let's set lots of values once
each, as we should..
This fixes possible inconsistencies that might arise in a version-1 superblock
when devices fail and are removed.
Usage of version-1 superblocks is not yet widespread and no actual problems
have been reported.
From: Paul Clements <paul.clements@steeleye.com>
Signed-off-by: Neil Brown <neilb@cse.unsw.edu.au>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.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 68669ce10268..78c7514de427 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -940,7 +940,7 @@ static void super_1_sync(mddev_t *mddev, mdk_rdev_t *rdev) sb->max_dev = cpu_to_le32(max_dev); for (i=0; i<max_dev;i++) - sb->dev_roles[max_dev] = cpu_to_le16(0xfffe); + sb->dev_roles[i] = cpu_to_le16(0xfffe); ITERATE_RDEV(mddev,rdev2,tmp) { i = rdev2->desc_nr; |