if (!mdev)
return;
- idr_remove(&mdev->tconn->volumes, minor);
+ idr_remove(&mdev->tconn->volumes, mdev->vnr);
+ idr_remove(&minors, minor);
+ synchronize_rcu();
/* paranoia asserts */
D_ASSERT(mdev->open_cnt == 0);
* allocated from drbd_new_device
* and actually free the mdev itself */
drbd_free_mdev(mdev);
- idr_remove(&minors, minor);
}
static void drbd_cleanup(void)
return ERR_NOMEM;
mdev->tconn = tconn;
- if (!idr_pre_get(&tconn->volumes, GFP_KERNEL))
- goto out_no_idr;
- if (idr_get_new(&tconn->volumes, mdev, &vnr_got))
- goto out_no_idr;
- if (vnr_got != vnr) {
- dev_err(DEV, "vnr_got (%d) != vnr (%d)\n", vnr_got, vnr);
- goto out_no_q;
- }
-
mdev->minor = minor;
drbd_init_set_defaults(mdev);
INIT_LIST_HEAD(&mdev->current_epoch->list);
mdev->epochs = 1;
+ if (!idr_pre_get(&tconn->volumes, GFP_KERNEL))
+ goto out_no_vol_idr;
+ if (idr_get_new(&tconn->volumes, mdev, &vnr_got))
+ goto out_no_vol_idr;
+ if (vnr_got != vnr) {
+ dev_err(DEV, "vnr_got (%d) != vnr (%d)\n", vnr_got, vnr);
+ goto out_idr_remove_vol;
+ }
+
if (!idr_pre_get(&minors, GFP_KERNEL))
- goto out_no_minor_idr;
+ goto out_idr_remove_vol;
if (idr_get_new(&minors, mdev, &minor_got))
- goto out_no_minor_idr;
+ goto out_idr_remove_vol;
if (minor_got != minor) {
- idr_remove(&minors, minor_got);
- goto out_no_minor_idr;
+ /* minor exists, or other idr strangeness? */
+ dev_err(DEV, "available minor (%d) != requested minor (%d)\n",
+ minor_got, minor);
+ goto out_idr_remove_minor;
}
add_disk(disk);
return NO_ERROR;
-out_no_minor_idr:
+out_idr_remove_minor:
+ idr_remove(&minors, minor_got);
+out_idr_remove_vol:
+ idr_remove(&tconn->volumes, vnr_got);
+ synchronize_rcu();
+out_no_vol_idr:
kfree(mdev->current_epoch);
out_no_epoch:
drbd_bm_cleanup(mdev);
out_no_disk:
blk_cleanup_queue(q);
out_no_q:
- idr_remove(&tconn->volumes, vnr_got);
-out_no_idr:
kfree(mdev);
return ERR_NOMEM;
}