diff options
| author |   Mike Snitzer <snitzer@redhat.com> | 2013-08-27 18:57:03 -0400 |
|---|---|---|
| committer | Mike Snitzer <snitzer@redhat.com> | 2013-09-05 20:46:06 -0400 |
| commit | 00c4fc3b1f590288cb3c42f36da50f49a513cfcf (patch) | |
| tree | e185bc97fc6529d1299e5e13987fe5b86cf21868 /drivers/md/dm.c | |
| parent | dm ioctl: prevent rename to empty name or uuid (diff) | |
| download | linux-dev-00c4fc3b1f590288cb3c42f36da50f49a513cfcf.tar.xz linux-dev-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.c')
| -rw-r--r-- | drivers/md/dm.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/md/dm.c b/drivers/md/dm.c index ef095a96d039..7faeaa3d4835 100644 --- a/drivers/md/dm.c +++ b/drivers/md/dm.c @@ -2235,11 +2235,13 @@ void dm_unlock_md_type(struct mapped_device *md) void dm_set_md_type(struct mapped_device *md, unsigned type) { + BUG_ON(!mutex_is_locked(&md->type_lock)); md->type = type; } unsigned dm_get_md_type(struct mapped_device *md) { + BUG_ON(!mutex_is_locked(&md->type_lock)); return md->type; } |