dm: use RCU_INIT_POINTER instead of rcu_assign_pointer in __unbind
authorMonam Agarwal <monamagarwal123@gmail.com>
Sun, 23 Mar 2014 18:28:27 +0000 (23:58 +0530)
committerMike Snitzer <snitzer@redhat.com>
Thu, 27 Mar 2014 20:56:24 +0000 (16:56 -0400)
Replace rcu_assign_pointer(p, NULL) with RCU_INIT_POINTER(p, NULL).

The rcu_assign_pointer() ensures that the initialization of a structure
is carried out before storing a pointer to that structure.  And in the
case of the NULL pointer, there is no structure to initialize.  So,
rcu_assign_pointer(p, NULL) can be safely converted to
RCU_INIT_POINTER(p, NULL).

Signed-off-by: Monam Agarwal <monamagarwal123@gmail.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm.c

index 0d52f6f..6382213 100644 (file)
@@ -2163,7 +2163,7 @@ static struct dm_table *__unbind(struct mapped_device *md)
                return NULL;
 
        dm_table_event_callback(map, NULL, NULL);
-       rcu_assign_pointer(md->map, NULL);
+       RCU_INIT_POINTER(md->map, NULL);
        dm_sync_table(md);
 
        return map;