Commit 763087da authored by Eric Dumazet's avatar Eric Dumazet Committed by Jakub Kicinski

net: add skb_set_end_offset() helper

We have multiple places where this helper is convenient,
and plan using it in the following patch.
Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 0ebea8f9
...@@ -1536,6 +1536,11 @@ static inline unsigned int skb_end_offset(const struct sk_buff *skb) ...@@ -1536,6 +1536,11 @@ static inline unsigned int skb_end_offset(const struct sk_buff *skb)
{ {
return skb->end; return skb->end;
} }
static inline void skb_set_end_offset(struct sk_buff *skb, unsigned int offset)
{
skb->end = offset;
}
#else #else
static inline unsigned char *skb_end_pointer(const struct sk_buff *skb) static inline unsigned char *skb_end_pointer(const struct sk_buff *skb)
{ {
...@@ -1546,6 +1551,11 @@ static inline unsigned int skb_end_offset(const struct sk_buff *skb) ...@@ -1546,6 +1551,11 @@ static inline unsigned int skb_end_offset(const struct sk_buff *skb)
{ {
return skb->end - skb->head; return skb->end - skb->head;
} }
static inline void skb_set_end_offset(struct sk_buff *skb, unsigned int offset)
{
skb->end = skb->head + offset;
}
#endif #endif
/* Internal */ /* Internal */
......
...@@ -201,7 +201,7 @@ static void __build_skb_around(struct sk_buff *skb, void *data, ...@@ -201,7 +201,7 @@ static void __build_skb_around(struct sk_buff *skb, void *data,
skb->head = data; skb->head = data;
skb->data = data; skb->data = data;
skb_reset_tail_pointer(skb); skb_reset_tail_pointer(skb);
skb->end = skb->tail + size; skb_set_end_offset(skb, size);
skb->mac_header = (typeof(skb->mac_header))~0U; skb->mac_header = (typeof(skb->mac_header))~0U;
skb->transport_header = (typeof(skb->transport_header))~0U; skb->transport_header = (typeof(skb->transport_header))~0U;
...@@ -1736,11 +1736,10 @@ int pskb_expand_head(struct sk_buff *skb, int nhead, int ntail, ...@@ -1736,11 +1736,10 @@ int pskb_expand_head(struct sk_buff *skb, int nhead, int ntail,
skb->head = data; skb->head = data;
skb->head_frag = 0; skb->head_frag = 0;
skb->data += off; skb->data += off;
skb_set_end_offset(skb, size);
#ifdef NET_SKBUFF_DATA_USES_OFFSET #ifdef NET_SKBUFF_DATA_USES_OFFSET
skb->end = size;
off = nhead; off = nhead;
#else
skb->end = skb->head + size;
#endif #endif
skb->tail += off; skb->tail += off;
skb_headers_offset_update(skb, nhead); skb_headers_offset_update(skb, nhead);
...@@ -6044,11 +6043,7 @@ static int pskb_carve_inside_header(struct sk_buff *skb, const u32 off, ...@@ -6044,11 +6043,7 @@ static int pskb_carve_inside_header(struct sk_buff *skb, const u32 off,
skb->head = data; skb->head = data;
skb->data = data; skb->data = data;
skb->head_frag = 0; skb->head_frag = 0;
#ifdef NET_SKBUFF_DATA_USES_OFFSET skb_set_end_offset(skb, size);
skb->end = size;
#else
skb->end = skb->head + size;
#endif
skb_set_tail_pointer(skb, skb_headlen(skb)); skb_set_tail_pointer(skb, skb_headlen(skb));
skb_headers_offset_update(skb, 0); skb_headers_offset_update(skb, 0);
skb->cloned = 0; skb->cloned = 0;
...@@ -6186,11 +6181,7 @@ static int pskb_carve_inside_nonlinear(struct sk_buff *skb, const u32 off, ...@@ -6186,11 +6181,7 @@ static int pskb_carve_inside_nonlinear(struct sk_buff *skb, const u32 off,
skb->head = data; skb->head = data;
skb->head_frag = 0; skb->head_frag = 0;
skb->data = data; skb->data = data;
#ifdef NET_SKBUFF_DATA_USES_OFFSET skb_set_end_offset(skb, size);
skb->end = size;
#else
skb->end = skb->head + size;
#endif
skb_reset_tail_pointer(skb); skb_reset_tail_pointer(skb);
skb_headers_offset_update(skb, 0); skb_headers_offset_update(skb, 0);
skb->cloned = 0; skb->cloned = 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment