From: Markus Elfring <elfr...@users.sourceforge.net>
Date: Sat, 1 Oct 2016 07:10:43 +0200

Move the definition for the variable "template_disk" into an if branch
so that an extra initialisation can be avoided at the beginning
by this refactoring.

Signed-off-by: Markus Elfring <elfr...@users.sourceforge.net>
---
 drivers/md/dm-table.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/md/dm-table.c b/drivers/md/dm-table.c
index f6b817c..73d38d0 100644
--- a/drivers/md/dm-table.c
+++ b/drivers/md/dm-table.c
@@ -1391,14 +1391,13 @@ int dm_calculate_queue_limits(struct dm_table *table,
  */
 static void dm_table_verify_integrity(struct dm_table *t)
 {
-       struct gendisk *template_disk = NULL;
-
        if (t->integrity_supported) {
                /*
                 * Verify that the original integrity profile
                 * matches all the devices in this table.
                 */
-               template_disk = dm_table_get_integrity_disk(t);
+               struct gendisk *template_disk = dm_table_get_integrity_disk(t);
+
                if (template_disk &&
                    blk_integrity_compare(dm_disk(t->md), template_disk) >= 0)
                        return;
-- 
2.10.0

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel

Reply via email to