> The first chunk of yor patch is in bind_rdev_to_array. > This is called from > autorun_devices which has just allocated the mddev so the > semaphore is still locked (it is allocated locked, and > then unlocked). > add_new_disk and hot_add_disk which are only called from > md_ioctl *after* the mddev has been locked with > lock_mddev_interruptible(mddev) > > So in that case the extra locking is not needed and would deadlock. Of course you're right. Too much reading, not enough understanding ::-). Back to the code. Ross Vandegrift ross@willow.seitz.com - To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html