diff options
author | Alex Elder <elder@dreamhost.com> | 2012-01-11 22:42:15 -0500 |
---|---|---|
committer | Sage Weil <sage@newdream.net> | 2012-01-12 14:00:50 -0500 |
commit | 0e805a1d857799352e51e8697c0b1a30aec16913 (patch) | |
tree | bc6ad83e25af597f40b03170026c3e5d331c2814 /drivers/block/rbd.c | |
parent | a40dc6cc2e121abcbd1b22583ef5447763df510c (diff) |
rbd: initialize snap_rwsem in rbd_add()
New rbd device structures get initialized in rbd_add(). Many of
the fields rely on being initially zero-filled. However we lockdep
was noticing that the rw_semaphore embedded in the header field
was not getting properly initialized. Fix that.
Signed-off-by: Alex Elder <elder@dreamhost.com>
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'drivers/block/rbd.c')
-rw-r--r-- | drivers/block/rbd.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c index 148ab944378d..3fd31dec8c9c 100644 --- a/drivers/block/rbd.c +++ b/drivers/block/rbd.c | |||
@@ -2184,6 +2184,8 @@ static ssize_t rbd_add(struct bus_type *bus, | |||
2184 | INIT_LIST_HEAD(&rbd_dev->node); | 2184 | INIT_LIST_HEAD(&rbd_dev->node); |
2185 | INIT_LIST_HEAD(&rbd_dev->snaps); | 2185 | INIT_LIST_HEAD(&rbd_dev->snaps); |
2186 | 2186 | ||
2187 | init_rwsem(&rbd_dev->header.snap_rwsem); | ||
2188 | |||
2187 | /* generate unique id: find highest unique id, add one */ | 2189 | /* generate unique id: find highest unique id, add one */ |
2188 | mutex_lock_nested(&ctl_mutex, SINGLE_DEPTH_NESTING); | 2190 | mutex_lock_nested(&ctl_mutex, SINGLE_DEPTH_NESTING); |
2189 | 2191 | ||