dm: add two lockdep_assert_held() statements
authorBart Van Assche <bart.vanassche@sandisk.com>
Wed, 31 Aug 2016 22:17:04 +0000 (15:17 -0700)
committerMike Snitzer <snitzer@redhat.com>
Wed, 14 Sep 2016 17:56:38 +0000 (13:56 -0400)
Document the locking assumptions for the __bind() and __dm_suspend()
functions.

Signed-off-by: Bart Van Assche <bart.vanassche@sandisk.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm.c

index 0f2928b..0708c62 100644 (file)
@@ -1648,6 +1648,8 @@ static struct dm_table *__bind(struct mapped_device *md, struct dm_table *t,
        struct request_queue *q = md->queue;
        sector_t size;
 
+       lockdep_assert_held(&md->suspend_lock);
+
        size = dm_table_get_size(t);
 
        /*
@@ -2094,6 +2096,8 @@ static int __dm_suspend(struct mapped_device *md, struct dm_table *map,
        bool noflush = suspend_flags & DM_SUSPEND_NOFLUSH_FLAG;
        int r;
 
+       lockdep_assert_held(&md->suspend_lock);
+
        /*
         * DMF_NOFLUSH_SUSPENDING must be set before presuspend.
         * This flag is cleared before dm_suspend returns.