diff options
author | Dan Williams <dan.j.williams@intel.com> | 2008-04-30 03:52:32 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-04-30 11:29:33 -0400 |
commit | 11e2ede0228ee0f81ccacd15894908c3bf241f73 (patch) | |
tree | 78bf7c8e2762d5dc07f2b9acb92ef5804ac40f38 /drivers/md/md.c | |
parent | 242b363e2207d14125f52a6701cfda7376a2a2fc (diff) |
md: prevent duplicates in bind_rdev_to_array
Found when trying to reassemble an active externally managed array. Without
this check we hit the more noisy "sysfs duplicate" warning in the later call
to kobject_add.
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
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>
Diffstat (limited to 'drivers/md/md.c')
-rw-r--r-- | drivers/md/md.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index 75a3f483522d..bec00b201a73 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c | |||
@@ -1370,6 +1370,11 @@ static int bind_rdev_to_array(mdk_rdev_t * rdev, mddev_t * mddev) | |||
1370 | MD_BUG(); | 1370 | MD_BUG(); |
1371 | return -EINVAL; | 1371 | return -EINVAL; |
1372 | } | 1372 | } |
1373 | |||
1374 | /* prevent duplicates */ | ||
1375 | if (find_rdev(mddev, rdev->bdev->bd_dev)) | ||
1376 | return -EEXIST; | ||
1377 | |||
1373 | /* make sure rdev->size exceeds mddev->size */ | 1378 | /* make sure rdev->size exceeds mddev->size */ |
1374 | if (rdev->size && (mddev->size == 0 || rdev->size < mddev->size)) { | 1379 | if (rdev->size && (mddev->size == 0 || rdev->size < mddev->size)) { |
1375 | if (mddev->pers) { | 1380 | if (mddev->pers) { |