drbd: receive_protocol(): Make the program flow less confusing
authorAndreas Gruenbacher <agruen@linbit.com>
Fri, 15 Jul 2011 15:04:26 +0000 (17:04 +0200)
committerPhilipp Reisner <philipp.reisner@linbit.com>
Thu, 8 Nov 2012 15:57:55 +0000 (16:57 +0100)
Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com>
Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>
drivers/block/drbd/drbd_receiver.c

index 11f2e7273e13a17ceaeb0e4a0f61e760700adaff..06bbf0f79fc5683feedc2d5b22ae2114d4438390 100644 (file)
@@ -3083,55 +3083,52 @@ static int receive_protocol(struct drbd_tconn *tconn, struct packet_info *pi)
 
                synchronize_rcu();
                kfree(old_net_conf);
+       } else {
+               clear_bit(CONN_DRY_RUN, &tconn->flags);
 
-               return 0;
-       }
-
-       clear_bit(CONN_DRY_RUN, &tconn->flags);
+               if (cf & CF_DRY_RUN)
+                       set_bit(CONN_DRY_RUN, &tconn->flags);
 
-       if (cf & CF_DRY_RUN)
-               set_bit(CONN_DRY_RUN, &tconn->flags);
+               rcu_read_lock();
+               nc = rcu_dereference(tconn->net_conf);
 
-       rcu_read_lock();
-       nc = rcu_dereference(tconn->net_conf);
+               if (p_proto != nc->wire_protocol) {
+                       conn_err(tconn, "incompatible communication protocols\n");
+                       goto disconnect_rcu_unlock;
+               }
 
-       if (p_proto != nc->wire_protocol) {
-               conn_err(tconn, "incompatible communication protocols\n");
-               goto disconnect_rcu_unlock;
-       }
+               if (convert_after_sb(p_after_sb_0p) != nc->after_sb_0p) {
+                       conn_err(tconn, "incompatible after-sb-0pri settings\n");
+                       goto disconnect_rcu_unlock;
+               }
 
-       if (convert_after_sb(p_after_sb_0p) != nc->after_sb_0p) {
-               conn_err(tconn, "incompatible after-sb-0pri settings\n");
-               goto disconnect_rcu_unlock;
-       }
+               if (convert_after_sb(p_after_sb_1p) != nc->after_sb_1p) {
+                       conn_err(tconn, "incompatible after-sb-1pri settings\n");
+                       goto disconnect_rcu_unlock;
+               }
 
-       if (convert_after_sb(p_after_sb_1p) != nc->after_sb_1p) {
-               conn_err(tconn, "incompatible after-sb-1pri settings\n");
-               goto disconnect_rcu_unlock;
-       }
+               if (convert_after_sb(p_after_sb_2p) != nc->after_sb_2p) {
+                       conn_err(tconn, "incompatible after-sb-2pri settings\n");
+                       goto disconnect_rcu_unlock;
+               }
 
-       if (convert_after_sb(p_after_sb_2p) != nc->after_sb_2p) {
-               conn_err(tconn, "incompatible after-sb-2pri settings\n");
-               goto disconnect_rcu_unlock;
-       }
+               if (p_discard_my_data && nc->discard_my_data) {
+                       conn_err(tconn, "both sides have the 'discard_my_data' flag set\n");
+                       goto disconnect_rcu_unlock;
+               }
 
-       if (p_discard_my_data && nc->discard_my_data) {
-               conn_err(tconn, "both sides have the 'discard_my_data' flag set\n");
-               goto disconnect_rcu_unlock;
-       }
+               if (p_two_primaries != nc->two_primaries) {
+                       conn_err(tconn, "incompatible setting of the two-primaries options\n");
+                       goto disconnect_rcu_unlock;
+               }
 
-       if (p_two_primaries != nc->two_primaries) {
-               conn_err(tconn, "incompatible setting of the two-primaries options\n");
-               goto disconnect_rcu_unlock;
-       }
+               if (strcmp(integrity_alg, nc->integrity_alg)) {
+                       conn_err(tconn, "incompatible setting of the data-integrity-alg\n");
+                       goto disconnect_rcu_unlock;
+               }
 
-       if (strcmp(integrity_alg, nc->integrity_alg)) {
-               conn_err(tconn, "incompatible setting of the data-integrity-alg\n");
-               goto disconnect_rcu_unlock;
+               rcu_read_unlock();
        }
-
-       rcu_read_unlock();
-
        return 0;
 
 disconnect_rcu_unlock: