[ovs-dev] [PATCH 5/5] lib/packets: Remove packet_get_tcp_flags().

Jarno Rajahalme jrajahalme at nicira.com
Tue Mar 18 23:42:59 UTC 2014


This function has no more users, so remove it.

Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
---
 lib/packets.c |   18 +-----------------
 lib/packets.h |    1 -
 2 files changed, 1 insertion(+), 18 deletions(-)

diff --git a/lib/packets.c b/lib/packets.c
index 0c023fc..65ba3f6 100644
--- a/lib/packets.c
+++ b/lib/packets.c
@@ -895,22 +895,6 @@ packet_set_sctp_port(struct ofpbuf *packet, ovs_be16 src, ovs_be16 dst)
     sh->sctp_csum = old_csum ^ old_correct_csum ^ new_csum;
 }
 
-/* If 'packet' is a TCP packet, returns the TCP flags.  Otherwise, returns 0.
- *
- * 'flow' must be the flow corresponding to 'packet' and 'packet''s header
- * pointers must be properly initialized (e.g. with flow_extract()). */
-uint16_t
-packet_get_tcp_flags(const struct ofpbuf *packet, const struct flow *flow)
-{
-    if (dl_type_is_ip_any(flow->dl_type) &&
-        flow->nw_proto == IPPROTO_TCP && packet->l7) {
-        const struct tcp_header *tcp = packet->l4;
-        return TCP_FLAGS(tcp->tcp_ctl);
-    } else {
-        return 0;
-    }
-}
-
 const char *
 packet_tcp_flag_to_string(uint32_t flag)
 {
@@ -945,7 +929,7 @@ packet_tcp_flag_to_string(uint32_t flag)
 }
 
 /* Appends a string representation of the TCP flags value 'tcp_flags'
- * (e.g. obtained via packet_get_tcp_flags() or TCP_FLAGS) to 's', in the
+ * (e.g. from struct flow.tcp_flags or obtained via TCP_FLAGS) to 's', in the
  * format used by tcpdump. */
 void
 packet_format_tcp_flags(struct ds *s, uint16_t tcp_flags)
diff --git a/lib/packets.h b/lib/packets.h
index 24d61a9..18a3b17 100644
--- a/lib/packets.h
+++ b/lib/packets.h
@@ -666,7 +666,6 @@ void packet_set_tcp_port(struct ofpbuf *, ovs_be16 src, ovs_be16 dst);
 void packet_set_udp_port(struct ofpbuf *, ovs_be16 src, ovs_be16 dst);
 void packet_set_sctp_port(struct ofpbuf *, ovs_be16 src, ovs_be16 dst);
 
-uint16_t packet_get_tcp_flags(const struct ofpbuf *, const struct flow *);
 void packet_format_tcp_flags(struct ds *, uint16_t);
 const char *packet_tcp_flag_to_string(uint32_t flag);
 
-- 
1.7.10.4




More information about the dev mailing list