md/bitmap: change a 'goto' to a normal 'if' construct.
authorNeilBrown <neilb@suse.de>
Mon, 19 Mar 2012 01:46:40 +0000 (12:46 +1100)
committerNeilBrown <neilb@suse.de>
Mon, 19 Mar 2012 01:46:40 +0000 (12:46 +1100)
The use of a goto makes the control flow more obscure here.

So make it a normal:
  if (x) {
     Y;
  }

No functional change.

Signed-off-by: NeilBrown <neilb@suse.de>
drivers/md/bitmap.c

index 04df18e8885fc129a1c9ec13f76665737ea71faa..fcf3c9465fd894f924a9510dcf59640e70c4c905 100644 (file)
@@ -633,26 +633,28 @@ static int bitmap_read_sb(struct bitmap *bitmap)
        /* keep the array size field of the bitmap superblock up to date */
        sb->sync_size = cpu_to_le64(bitmap->mddev->resync_max_sectors);
 
-       if (!bitmap->mddev->persistent)
-               goto success;
-
-       /*
-        * if we have a persistent array superblock, compare the
-        * bitmap's UUID and event counter to the mddev's
-        */
-       if (memcmp(sb->uuid, bitmap->mddev->uuid, 16)) {
-               printk(KERN_INFO "%s: bitmap superblock UUID mismatch\n",
-                       bmname(bitmap));
-               goto out;
-       }
-       events = le64_to_cpu(sb->events);
-       if (events < bitmap->mddev->events) {
-               printk(KERN_INFO "%s: bitmap file is out of date (%llu < %llu) "
-                       "-- forcing full recovery\n", bmname(bitmap), events,
-                       (unsigned long long) bitmap->mddev->events);
-               sb->state |= cpu_to_le32(BITMAP_STALE);
+       if (bitmap->mddev->persistent) {
+               /*
+                * We have a persistent array superblock, so compare the
+                * bitmap's UUID and event counter to the mddev's
+                */
+               if (memcmp(sb->uuid, bitmap->mddev->uuid, 16)) {
+                       printk(KERN_INFO
+                              "%s: bitmap superblock UUID mismatch\n",
+                              bmname(bitmap));
+                       goto out;
+               }
+               events = le64_to_cpu(sb->events);
+               if (events < bitmap->mddev->events) {
+                       printk(KERN_INFO
+                              "%s: bitmap file is out of date (%llu < %llu) "
+                              "-- forcing full recovery\n",
+                              bmname(bitmap), events,
+                              (unsigned long long) bitmap->mddev->events);
+                       sb->state |= cpu_to_le32(BITMAP_STALE);
+               }
        }
-success:
+
        /* assign fields using values from superblock */
        bitmap->mddev->bitmap_info.chunksize = chunksize;
        bitmap->mddev->bitmap_info.daemon_sleep = daemon_sleep;