Skip to contentSkip to conversation

staging: lustre: lnet: lnet: do not initialise statics to 0 or NULL

PATCH In review Balavasu 2 participants 2 messages Oct 31, 2014

Subsystems (3)
lore
Share & tools
lore

Balavasu

author
#0 lore raw

PATCH 1/2 staging: lustre: lnet: lnet: do not initialise statics to 0 or NULL

This patch fixes the checkpatch.pl issue
Error: do not initialise statics to 0 or NULL for time

---
 drivers/staging/lustre/lnet/lnet/do not instalise 0 | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)
drivers/staging/lustre/lnet/lnet/router.c +5 -5
index b5b8fb5..cdeb246 100644
@@ -46,7 +46,7 @@ MODULE_PARM_DESC(small_router_buffers, "# of small (1 page) messages to buffer i
 static int large_router_buffers;
 module_param(large_router_buffers, int, 0444);
 MODULE_PARM_DESC(large_router_buffers, "# of large messages to buffer in the router");
-static int peer_buffer_credits = 0;
+static int peer_buffer_credits;
 module_param(peer_buffer_credits, int, 0444);
 MODULE_PARM_DESC(peer_buffer_credits, "# router buffer credits per peer");
 
@@ -80,7 +80,7 @@ lnet_peer_buffer_credits(lnet_ni_t *ni)
 
 #endif
 
-static int check_routers_before_use = 0;
+static int check_routers_before_use;
 module_param(check_routers_before_use, int, 0444);
 MODULE_PARM_DESC(check_routers_before_use, "Assume routers are down and ping them before use");
 
@@ -245,7 +245,7 @@ lnet_find_net_locked (__u32 net)
 
 static void lnet_shuffle_seed(void)
 {
-	static int seeded = 0;
+	static int seeded;
 	int lnd_type, seed[2];
 	struct timeval tv;
 	lnet_ni_t *ni;
@@ -1584,8 +1584,8 @@ lnet_notify (lnet_ni_t *ni, lnet_nid_t nid, int alive, unsigned long when)
 void
 lnet_router_checker (void)
 {
-	static time_t last = 0;
-	static int    running = 0;
+	static time_t last;
+	static int    running;
 
 	time_t	    now = get_seconds();
 	int	       interval = now - last;
Signature · 1 line
1.9.1
#1 lore raw
Quoted 6 lines · Balavasu

This patch fixes the checkpatch.pl issue
Error: do not initialise statics to 0 or NULL for time

---
 drivers/staging/lustre/lnet/lnet/do not instalise 0 | 10 +++++-----

That line doesn't look right, does it?

And why netdev and not the mailing list for the drivers/staging/ tree?

And again, I need a full name.

greg k-h