Use a rw_semaphore in dm_target.c rather than a rwlock_t, just to keep in line with dm.c --- diff/drivers/md/dm-target.c 2002-11-29 09:44:59.000000000 +0000 +++ source/drivers/md/dm-target.c 2003-01-10 10:10:42.000000000 +0000 @@ -18,7 +18,7 @@ }; static LIST_HEAD(_targets); -static rwlock_t _lock = RW_LOCK_UNLOCKED; +static DECLARE_RWSEM(_lock); #define DM_MOD_NAME_SIZE 32 @@ -41,7 +41,7 @@ { struct tt_internal *ti; - read_lock(&_lock); + down_read(&_lock); ti = __find_target_type(name); if (ti) { @@ -49,7 +49,7 @@ __MOD_INC_USE_COUNT(ti->tt.module); ti->use++; } - read_unlock(&_lock); + up_read(&_lock); return ti; } @@ -84,13 +84,13 @@ { struct tt_internal *ti = (struct tt_internal *) t; - read_lock(&_lock); + down_read(&_lock); if (--ti->use == 0 && ti->tt.module) __MOD_DEC_USE_COUNT(ti->tt.module); if (ti->use < 0) BUG(); - read_unlock(&_lock); + up_read(&_lock); return; } @@ -115,13 +115,13 @@ if (!ti) return -ENOMEM; - write_lock(&_lock); + down_write(&_lock); if (__find_target_type(t->name)) rv = -EEXIST; else list_add(&ti->list, &_targets); - write_unlock(&_lock); + up_write(&_lock); return rv; } @@ -129,21 +129,21 @@ { struct tt_internal *ti; - write_lock(&_lock); + down_write(&_lock); if (!(ti = __find_target_type(t->name))) { - write_unlock(&_lock); + up_write(&_lock); return -EINVAL; } if (ti->use) { - write_unlock(&_lock); + up_write(&_lock); return -ETXTBSY; } list_del(&ti->list); kfree(ti); - write_unlock(&_lock); + up_write(&_lock); return 0; }