Whamcloud - gitweb
LU-7362 lnet: Remove LASSERTS from router checker 03/17003/2
authorDoug Oucharek <doug.s.oucharek@intel.com>
Fri, 30 Oct 2015 21:40:59 +0000 (14:40 -0700)
committerOleg Drokin <oleg.drokin@intel.com>
Fri, 13 Nov 2015 18:26:40 +0000 (18:26 +0000)
In lnet_router_checker(), there are two LASSERTS.  Neither protects
us from anything and one of them triggered for a customer crashing
the system unecessarily.  This patch removes them.

Signed-off-by: Doug Oucharek <doug.s.oucharek@intel.com>
Change-Id: If2732632e47103fb8fa63a263c4c5ef4a44142a3
Reviewed-on: http://review.whamcloud.com/17003
Tested-by: Jenkins
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Chris Horn <hornc@cray.com>
Reviewed-by: Matt Ezell <ezellma@ornl.gov>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lnet/lnet/router.c

index 9c945e9..072adcc 100644 (file)
@@ -1234,8 +1234,6 @@ lnet_router_checker(void *arg)
 
        cfs_block_allsigs();
 
-       LASSERT (the_lnet.ln_rc_state == LNET_RC_STATE_RUNNING);
-
        while (the_lnet.ln_rc_state == LNET_RC_STATE_RUNNING) {
                __u64   version;
                int     cpt;
@@ -1289,8 +1287,6 @@ rescan:
                                                         cfs_time_seconds(1));
        }
 
-       LASSERT(the_lnet.ln_rc_state == LNET_RC_STATE_STOPPING);
-
        lnet_prune_rc_data(1); /* wait for UNLINK */
 
        the_lnet.ln_rc_state = LNET_RC_STATE_SHUTDOWN;