Commit ec523735 authored by Redha Gouicem's avatar Redha Gouicem Committed by Greg Kroah-Hartman

Staging: lustre: lnet: lnet: router.c: fix useless returns and elses

This patch removes useless returns and elses.
Signed-off-by: default avatarRedha Gouicem <redha.gouicem@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent e6157b1b
...@@ -270,7 +270,6 @@ static void lnet_shuffle_seed(void) ...@@ -270,7 +270,6 @@ static void lnet_shuffle_seed(void)
do_gettimeofday(&tv); do_gettimeofday(&tv);
cfs_srand(tv.tv_sec ^ seed[0], tv.tv_usec ^ seed[1]); cfs_srand(tv.tv_sec ^ seed[0], tv.tv_usec ^ seed[1]);
seeded = 1; seeded = 1;
return;
} }
/* NB expects LNET_LOCK held */ /* NB expects LNET_LOCK held */
...@@ -355,13 +354,12 @@ lnet_add_route(__u32 net, unsigned int hops, lnet_nid_t gateway, ...@@ -355,13 +354,12 @@ lnet_add_route(__u32 net, unsigned int hops, lnet_nid_t gateway,
LIBCFS_FREE(route, sizeof(*route)); LIBCFS_FREE(route, sizeof(*route));
LIBCFS_FREE(rnet, sizeof(*rnet)); LIBCFS_FREE(rnet, sizeof(*rnet));
if (rc == -EHOSTUNREACH) { /* gateway is not on a local net */ if (rc == -EHOSTUNREACH) /* gateway is not on a local net */
return 0; /* ignore the route entry */ return 0; /* ignore the route entry */
} else { CERROR("Error %d creating route %s %d %s\n", rc,
CERROR("Error %d creating route %s %d %s\n", rc, libcfs_net2str(net), hops,
libcfs_net2str(net), hops, libcfs_nid2str(gateway));
libcfs_nid2str(gateway));
}
return rc; return rc;
} }
...@@ -607,7 +605,6 @@ lnet_swap_pinginfo(lnet_ping_info_t *info) ...@@ -607,7 +605,6 @@ lnet_swap_pinginfo(lnet_ping_info_t *info)
__swab64s(&stat->ns_nid); __swab64s(&stat->ns_nid);
__swab32s(&stat->ns_status); __swab32s(&stat->ns_status);
} }
return;
} }
/** /**
...@@ -1011,7 +1008,6 @@ lnet_ping_router_locked(lnet_peer_t *rtr) ...@@ -1011,7 +1008,6 @@ lnet_ping_router_locked(lnet_peer_t *rtr)
} }
lnet_peer_decref_locked(rtr); lnet_peer_decref_locked(rtr);
return;
} }
int int
...@@ -1084,7 +1080,6 @@ lnet_router_checker_stop(void) ...@@ -1084,7 +1080,6 @@ lnet_router_checker_stop(void)
rc = LNetEQFree(the_lnet.ln_rc_eqh); rc = LNetEQFree(the_lnet.ln_rc_eqh);
LASSERT(rc == 0); LASSERT(rc == 0);
return;
} }
static void static void
...@@ -1582,7 +1577,6 @@ EXPORT_SYMBOL(lnet_notify); ...@@ -1582,7 +1577,6 @@ EXPORT_SYMBOL(lnet_notify);
void void
lnet_get_tunables(void) lnet_get_tunables(void)
{ {
return;
} }
#else #else
...@@ -1671,7 +1665,6 @@ lnet_router_checker(void) ...@@ -1671,7 +1665,6 @@ lnet_router_checker(void)
lnet_net_unlock(0); lnet_net_unlock(0);
running = 0; /* lock only needed for the recursion check */ running = 0; /* lock only needed for the recursion check */
return;
} }
/* NB lnet_peers_start_down depends on me, /* NB lnet_peers_start_down depends on me,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment