This isn't really needed at the moment, but it is more consistant with the interface and may be needed later. Signed-off-by: Neil Brown <neilb@xxxxxxxxxxxxxxx> ----------- Diffstat output ------------ ./drivers/md/md.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff ./drivers/md/md.c~current~ ./drivers/md/md.c --- ./drivers/md/md.c~current~ 2004-05-28 15:48:47.000000000 +1000 +++ ./drivers/md/md.c 2004-05-28 15:48:47.000000000 +1000 @@ -3478,8 +3478,8 @@ void md_check_recovery(mddev_t *mddev) if (rdev->raid_disk >= 0 && rdev->faulty && atomic_read(&rdev->nr_pending)==0) { - mddev->pers->hot_remove_disk(mddev, rdev->raid_disk); - rdev->raid_disk = -1; + if (mddev->pers->hot_remove_disk(mddev, rdev->raid_disk)==0) + rdev->raid_disk = -1; } if (!rdev->faulty && rdev->raid_disk >= 0 && !rdev->in_sync) spares++; - 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