summaryrefslogtreecommitdiffstats
path: root/drivers/md/dm-table.c
diff options
context:
space:
mode:
authorMike Snitzer <snitzer@redhat.com>2013-08-28 00:57:03 +0200
committerMike Snitzer <snitzer@redhat.com>2013-09-06 02:46:06 +0200
commit00c4fc3b1f590288cb3c42f36da50f49a513cfcf (patch)
treee185bc97fc6529d1299e5e13987fe5b86cf21868 /drivers/md/dm-table.c
parentdm ioctl: prevent rename to empty name or uuid (diff)
downloadlinux-00c4fc3b1f590288cb3c42f36da50f49a513cfcf.tar.xz
linux-00c4fc3b1f590288cb3c42f36da50f49a513cfcf.zip
dm ioctl: increase granularity of type_lock when loading table
Hold the mapped device's type_lock before calling populate_table() since it is where the table's type is determined based on the specified targets. There is no need to allow concurrent table loads to race to establish the table's targets or type. This eliminates the need to grab the lock in dm_table_set_type(). Also verify that the type_lock is held in both dm_set_md_type() and dm_get_md_type(). Signed-off-by: Mike Snitzer <snitzer@redhat.com> Signed-off-by: Alasdair G Kergon <agk@redhat.com>
Diffstat (limited to 'drivers/md/dm-table.c')
-rw-r--r--drivers/md/dm-table.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/md/dm-table.c b/drivers/md/dm-table.c
index f309477d7efe..8f8783533ac7 100644
--- a/drivers/md/dm-table.c
+++ b/drivers/md/dm-table.c
@@ -888,9 +888,7 @@ static int dm_table_set_type(struct dm_table *t)
* Determine the type from the live device.
* Default to bio-based if device is new.
*/
- dm_lock_md_type(t->md);
live_md_type = dm_get_md_type(t->md);
- dm_unlock_md_type(t->md);
if (live_md_type == DM_TYPE_REQUEST_BASED)
request_based = 1;
else