[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20240815214527.2100137-11-tom@herbertland.com>
Date: Thu, 15 Aug 2024 14:45:25 -0700
From: Tom Herbert <tom@...bertland.com>
To: davem@...emloft.net,
kuba@...nel.org,
edumazet@...gle.com,
netdev@...r.kernel.org,
felipe@...anda.io,
willemdebruijn.kernel@...il.com
Cc: Tom Herbert <tom@...bertland.com>
Subject: [PATCH net-next v2 10/12] gtp: Move gtp_parse_exthdrs into net/gtp.h
gtp_parse_exthdrs is a generic function, move into a header file
so we can call it outside of the GTP driver (specifically, we can
call it from flow dissector)
Signed-off-by: Tom Herbert <tom@...bertland.com>
---
drivers/net/gtp.c | 37 -------------------------------------
include/net/gtp.h | 38 ++++++++++++++++++++++++++++++++++++++
2 files changed, 38 insertions(+), 37 deletions(-)
diff --git a/drivers/net/gtp.c b/drivers/net/gtp.c
index 427b91aca50d..61d5dfd48c88 100644
--- a/drivers/net/gtp.c
+++ b/drivers/net/gtp.c
@@ -742,43 +742,6 @@ static int gtp1u_handle_echo_resp(struct gtp_dev *gtp, struct sk_buff *skb)
msg, 0, GTP_GENL_MCGRP, GFP_ATOMIC);
}
-static int gtp_parse_exthdrs(struct sk_buff *skb, unsigned int *hdrlen)
-{
- struct gtp_ext_hdr *gtp_exthdr, _gtp_exthdr;
- unsigned int offset = *hdrlen;
- __u8 *next_type, _next_type;
-
- /* From 29.060: "The Extension Header Length field specifies the length
- * of the particular Extension header in 4 octets units."
- *
- * This length field includes length field size itself (1 byte),
- * payload (variable length) and next type (1 byte). The extension
- * header is aligned to to 4 bytes.
- */
-
- do {
- gtp_exthdr = skb_header_pointer(skb, offset, sizeof(*gtp_exthdr),
- &_gtp_exthdr);
- if (!gtp_exthdr || !gtp_exthdr->len)
- return -1;
-
- offset += gtp_exthdr->len * 4;
-
- /* From 29.060: "If no such Header follows, then the value of
- * the Next Extension Header Type shall be 0."
- */
- next_type = skb_header_pointer(skb, offset - 1,
- sizeof(_next_type), &_next_type);
- if (!next_type)
- return -1;
-
- } while (*next_type != 0);
-
- *hdrlen = offset;
-
- return 0;
-}
-
static int gtp1u_udp_encap_recv(struct gtp_dev *gtp, struct sk_buff *skb)
{
unsigned int hdrlen = sizeof(struct udphdr) +
diff --git a/include/net/gtp.h b/include/net/gtp.h
index c0253c8702d0..a513aa1c7394 100644
--- a/include/net/gtp.h
+++ b/include/net/gtp.h
@@ -83,4 +83,42 @@ struct gtp_ext_hdr {
__u8 data[];
};
+static inline int gtp_parse_exthdrs(const struct sk_buff *skb,
+ unsigned int *hdrlen)
+{
+ struct gtp_ext_hdr *gtp_exthdr, _gtp_exthdr;
+ unsigned int offset = *hdrlen;
+ __u8 *next_type, _next_type;
+
+ /* From 29.060: "The Extension Header Length field specifies the length
+ * of the particular Extension header in 4 octets units."
+ *
+ * This length field includes length field size itself (1 byte),
+ * payload (variable length) and next type (1 byte). The extension
+ * header is aligned to 4 bytes.
+ */
+
+ do {
+ gtp_exthdr = skb_header_pointer(skb, offset, sizeof(*gtp_exthdr),
+ &_gtp_exthdr);
+ if (!gtp_exthdr || !gtp_exthdr->len)
+ return -1;
+
+ offset += gtp_exthdr->len * 4;
+
+ /* From 29.060: "If no such Header follows, then the value of
+ * the Next Extension Header Type shall be 0."
+ */
+ next_type = skb_header_pointer(skb, offset - 1,
+ sizeof(_next_type), &_next_type);
+ if (!next_type)
+ return -1;
+
+ } while (*next_type != 0);
+
+ *hdrlen = offset;
+
+ return 0;
+}
+
#endif
--
2.34.1
Powered by blists - more mailing lists