netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH net-next 1/2] tipc: make local function static
@ 2014-01-04 21:47 Stephen Hemminger
  2014-01-04 21:49 ` [PATCH net-next 2/2] tipc: remove unused code Stephen Hemminger
  2014-01-05  1:20 ` [PATCH net-next 1/2] tipc: make local function static David Miller
  0 siblings, 2 replies; 4+ messages in thread
From: Stephen Hemminger @ 2014-01-04 21:47 UTC (permalink / raw)
  To: Jon Maloy, David Miller; +Cc: netdev


Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>

---
 net/tipc/link.c |    2 +-
 net/tipc/link.h |    1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

--- a/net/tipc/link.c	2014-01-04 13:39:54.308221241 -0800
+++ b/net/tipc/link.c	2014-01-04 13:42:25.189725016 -0800
@@ -1169,7 +1169,7 @@ reject:
 /*
  * tipc_link_push_packet: Push one unsent packet to the media
  */
-u32 tipc_link_push_packet(struct tipc_link *l_ptr)
+static u32 tipc_link_push_packet(struct tipc_link *l_ptr)
 {
 	struct sk_buff *buf = l_ptr->first_out;
 	u32 r_q_size = l_ptr->retransm_queue_size;
--- a/net/tipc/link.h	2014-01-04 13:39:54.308221241 -0800
+++ b/net/tipc/link.h	2014-01-04 13:42:25.189725016 -0800
@@ -221,7 +221,6 @@ void tipc_link_send_duplicate(struct tip
 void tipc_link_reset_fragments(struct tipc_link *l_ptr);
 int tipc_link_is_up(struct tipc_link *l_ptr);
 int tipc_link_is_active(struct tipc_link *l_ptr);
-u32 tipc_link_push_packet(struct tipc_link *l_ptr);
 void tipc_link_stop(struct tipc_link *l_ptr);
 struct sk_buff *tipc_link_cmd_config(const void *req_tlv_area, int req_tlv_space, u16 cmd);
 struct sk_buff *tipc_link_cmd_show_stats(const void *req_tlv_area, int req_tlv_space);

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH net-next 2/2] tipc: remove unused code
  2014-01-04 21:47 [PATCH net-next 1/2] tipc: make local function static Stephen Hemminger
@ 2014-01-04 21:49 ` Stephen Hemminger
  2014-01-05  1:20   ` David Miller
  2014-01-05  1:20 ` [PATCH net-next 1/2] tipc: make local function static David Miller
  1 sibling, 1 reply; 4+ messages in thread
From: Stephen Hemminger @ 2014-01-04 21:49 UTC (permalink / raw)
  To: Jon Maloy, David Miller; +Cc: netdev

Remove dead code;
       tipc_bearer_find_interface
       tipc_node_redundant_links

This may break out of tree version of TIPC if there still is one.
But that maybe a good thing :-)

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>

---
 net/tipc/bearer.c |   19 -------------------
 net/tipc/bearer.h |    1 -
 net/tipc/node.c   |    5 -----
 net/tipc/node.h   |    1 -
 4 files changed, 26 deletions(-)

--- a/net/tipc/bearer.c	2014-01-04 13:39:54.308221241 -0800
+++ b/net/tipc/bearer.c	2014-01-04 13:42:27.553685918 -0800
@@ -185,25 +185,6 @@ struct tipc_bearer *tipc_bearer_find(con
 }
 
 /**
- * tipc_bearer_find_interface - locates bearer object with matching interface name
- */
-struct tipc_bearer *tipc_bearer_find_interface(const char *if_name)
-{
-	struct tipc_bearer *b_ptr;
-	char *b_if_name;
-	u32 i;
-
-	for (i = 0, b_ptr = tipc_bearers; i < MAX_BEARERS; i++, b_ptr++) {
-		if (!b_ptr->active)
-			continue;
-		b_if_name = strchr(b_ptr->name, ':') + 1;
-		if (!strcmp(b_if_name, if_name))
-			return b_ptr;
-	}
-	return NULL;
-}
-
-/**
  * tipc_bearer_get_names - record names of bearers in buffer
  */
 struct sk_buff *tipc_bearer_get_names(void)
--- a/net/tipc/bearer.h	2014-01-04 13:39:54.308221241 -0800
+++ b/net/tipc/bearer.h	2014-01-04 13:42:27.553685918 -0800
@@ -190,7 +190,6 @@ struct sk_buff *tipc_bearer_get_names(vo
 void tipc_bearer_add_dest(struct tipc_bearer *b_ptr, u32 dest);
 void tipc_bearer_remove_dest(struct tipc_bearer *b_ptr, u32 dest);
 struct tipc_bearer *tipc_bearer_find(const char *name);
-struct tipc_bearer *tipc_bearer_find_interface(const char *if_name);
 struct tipc_media *tipc_media_find(const char *name);
 int tipc_bearer_setup(void);
 void tipc_bearer_cleanup(void);
--- a/net/tipc/node.c	2014-01-04 13:39:54.308221241 -0800
+++ b/net/tipc/node.c	2014-01-04 13:42:27.553685918 -0800
@@ -235,11 +235,6 @@ int tipc_node_active_links(struct tipc_n
 	return n_ptr->active_links[0] != NULL;
 }
 
-int tipc_node_redundant_links(struct tipc_node *n_ptr)
-{
-	return n_ptr->working_links > 1;
-}
-
 int tipc_node_is_up(struct tipc_node *n_ptr)
 {
 	return tipc_node_active_links(n_ptr);
--- a/net/tipc/node.h	2014-01-04 13:39:54.308221241 -0800
+++ b/net/tipc/node.h	2014-01-04 13:42:27.553685918 -0800
@@ -115,7 +115,6 @@ void tipc_node_detach_link(struct tipc_n
 void tipc_node_link_down(struct tipc_node *n_ptr, struct tipc_link *l_ptr);
 void tipc_node_link_up(struct tipc_node *n_ptr, struct tipc_link *l_ptr);
 int tipc_node_active_links(struct tipc_node *n_ptr);
-int tipc_node_redundant_links(struct tipc_node *n_ptr);
 int tipc_node_is_up(struct tipc_node *n_ptr);
 struct sk_buff *tipc_node_get_links(const void *req_tlv_area, int req_tlv_space);
 struct sk_buff *tipc_node_get_nodes(const void *req_tlv_area, int req_tlv_space);

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH net-next 1/2] tipc: make local function static
  2014-01-04 21:47 [PATCH net-next 1/2] tipc: make local function static Stephen Hemminger
  2014-01-04 21:49 ` [PATCH net-next 2/2] tipc: remove unused code Stephen Hemminger
@ 2014-01-05  1:20 ` David Miller
  1 sibling, 0 replies; 4+ messages in thread
From: David Miller @ 2014-01-05  1:20 UTC (permalink / raw)
  To: stephen; +Cc: jon.maloy, netdev

From: Stephen Hemminger <stephen@networkplumber.org>
Date: Sat, 4 Jan 2014 13:47:48 -0800

> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>

Applied.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH net-next 2/2] tipc: remove unused code
  2014-01-04 21:49 ` [PATCH net-next 2/2] tipc: remove unused code Stephen Hemminger
@ 2014-01-05  1:20   ` David Miller
  0 siblings, 0 replies; 4+ messages in thread
From: David Miller @ 2014-01-05  1:20 UTC (permalink / raw)
  To: stephen; +Cc: jon.maloy, netdev

From: Stephen Hemminger <stephen@networkplumber.org>
Date: Sat, 4 Jan 2014 13:49:14 -0800

> Remove dead code;
>        tipc_bearer_find_interface
>        tipc_node_redundant_links
> 
> This may break out of tree version of TIPC if there still is one.
> But that maybe a good thing :-)
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>

Applied.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-01-05  1:20 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-01-04 21:47 [PATCH net-next 1/2] tipc: make local function static Stephen Hemminger
2014-01-04 21:49 ` [PATCH net-next 2/2] tipc: remove unused code Stephen Hemminger
2014-01-05  1:20   ` David Miller
2014-01-05  1:20 ` [PATCH net-next 1/2] tipc: make local function static David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).