Instead of setting one value lots of times, let's set lots of values once each, as we should.. From: Paul Clements <paul.clements@xxxxxxxxxxxx> Signed-off-by: Neil Brown <neilb@xxxxxxxxxxxxxxx> ### Diffstat output ./drivers/md/md.c | 2 +- 1 files changed, 1 insertion(+), 1 deletion(-) diff ./drivers/md/md.c~current~ ./drivers/md/md.c --- ./drivers/md/md.c~current~ 2005-03-07 11:32:41.000000000 +1100 +++ ./drivers/md/md.c 2005-03-07 11:32:41.000000000 +1100 @@ -1010,7 +1010,7 @@ static void super_1_sync(mddev_t *mddev, 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; - To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html