1 files changed, 9 insertions, 11 deletions

diff --git a/drivers/md/dm-table.c b/drivers/md/dm-table.c
index 47818d8249cb..f16062982383 100644
--- a/drivers/md/dm-table.c
+++ b/drivers/md/dm-table.c
@@ -287,9 +287,8 @@ static void free_devices(struct list_head *devices)
 {
         struct list_head *tmp, *next;
 
-        for (tmp = devices->next; tmp != devices; tmp = next) {
+        list_for_each_safe(tmp, next, devices) {
                 struct dm_dev *dd = list_entry(tmp, struct dm_dev, list);
-                next = tmp->next;
                 kfree(dd);
         }
 }
@@ -476,7 +475,7 @@ static int __table_get_device(struct dm_table *t, struct dm_target *ti,
                               int mode, struct dm_dev **result)
 {
         int r;
-        dev_t dev;
+        dev_t uninitialized_var(dev);
         struct dm_dev *dd;
         unsigned int major, minor;
 
@@ -805,7 +804,7 @@ static int setup_indexes(struct dm_table *t)
                 return -ENOMEM;
 
         /* set up internal nodes, bottom-up */
-        for (i = t->depth - 2, total = 0; i >= 0; i--) {
+        for (i = t->depth - 2; i >= 0; i--) {
                 t->index[i] = indexes;
                 indexes += (KEYS_PER_NODE * t->counts[i]);
                 setup_btree_index(i, t);
@@ -993,12 +992,11 @@ int dm_table_resume_targets(struct dm_table *t)
 
 int dm_table_any_congested(struct dm_table *t, int bdi_bits)
 {
-        struct list_head *d, *devices;
+        struct dm_dev *dd;
+        struct list_head *devices = dm_table_get_devices(t);
         int r = 0;
 
-        devices = dm_table_get_devices(t);
-        for (d = devices->next; d != devices; d = d->next) {
-                struct dm_dev *dd = list_entry(d, struct dm_dev, list);
+        list_for_each_entry(dd, devices, list) {
                 struct request_queue *q = bdev_get_queue(dd->bdev);
                 r |= bdi_congested(&q->backing_dev_info, bdi_bits);
         }
@@ -1008,10 +1006,10 @@ int dm_table_any_congested(struct dm_table *t, int bdi_bits)
 
 void dm_table_unplug_all(struct dm_table *t)
 {
-        struct list_head *d, *devices = dm_table_get_devices(t);
+        struct dm_dev *dd;
+        struct list_head *devices = dm_table_get_devices(t);
 
-        for (d = devices->next; d != devices; d = d->next) {
-                struct dm_dev *dd = list_entry(d, struct dm_dev, list);
+        list_for_each_entry(dd, devices, list) {
                 struct request_queue *q = bdev_get_queue(dd->bdev);
 
                 blk_unplug(q);