[net-next,14/23] tipc: Eliminate useless memset operations in Ethernet media support

Submitted by Paul Gortmaker on Dec. 27, 2011, 5:39 p.m.


Message ID 1325007582-31610-15-git-send-email-paul.gortmaker@windriver.com
State Accepted
Delegated to: David Miller
Headers show

Commit Message

Paul Gortmaker Dec. 27, 2011, 5:39 p.m.
From: Allan Stephens <allan.stephens@windriver.com>

Gets rid of two pointless operations that zero out the array used to
record information about TIPC's Ethernet bearers. There is no need to
initialize the array on start up since it is a global variable that is
already zero'd out, and there is no need to zero it out on exit because
the array is never referenced again.

Signed-off-by: Allan Stephens <allan.stephens@windriver.com>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
 net/tipc/eth_media.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

Patch hide | download patch | download mbox

diff --git a/net/tipc/eth_media.c b/net/tipc/eth_media.c
index 236155c..cd0a4b8 100644
--- a/net/tipc/eth_media.c
+++ b/net/tipc/eth_media.c
@@ -371,8 +371,6 @@  int tipc_eth_media_start(void)
 	if (eth_started)
 		return -EINVAL;
-	memset(eth_bearers, 0, sizeof(eth_bearers));
 	res = tipc_register_media(&eth_media_info);
 	if (res)
 		return res;
@@ -396,6 +394,5 @@  void tipc_eth_media_stop(void)
-	memset(&eth_bearers, 0, sizeof(eth_bearers));
 	eth_started = 0;