diff mbox series

[v2,net-next,12/12] net: dsa: tag_ar9331: let DSA core deal with TX reallocation

Message ID 20201030014910.2738809-13-vladimir.oltean@nxp.com
State New
Headers show
Series [v2,net-next,01/12] net: dsa: implement a central TX reallocation procedure | expand

Commit Message

Vladimir Oltean Oct. 30, 2020, 1:49 a.m. UTC
Now that we have a central TX reallocation procedure that accounts for
the tagger's needed headroom in a generic way, we can remove the
skb_cow_head call.

Cc: Per Forlin <per.forlin@axis.com>
Cc: Oleksij Rempel <linux@rempel-privat.de>
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
Changes in v2:
None.

 net/dsa/tag_ar9331.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

Oleksij Rempel Oct. 30, 2020, 9:21 a.m. UTC | #1
On Fri, Oct 30, 2020 at 03:49:10AM +0200, Vladimir Oltean wrote:
> Now that we have a central TX reallocation procedure that accounts for

> the tagger's needed headroom in a generic way, we can remove the

> skb_cow_head call.

> 

> Cc: Per Forlin <per.forlin@axis.com>

> Cc: Oleksij Rempel <linux@rempel-privat.de>

> Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>


Tested-by: Oleksij Rempel <linux@rempel-privat.de>


> ---

> Changes in v2:

> None.

> 

>  net/dsa/tag_ar9331.c | 3 ---

>  1 file changed, 3 deletions(-)

> 

> diff --git a/net/dsa/tag_ar9331.c b/net/dsa/tag_ar9331.c

> index 55b00694cdba..002cf7f952e2 100644

> --- a/net/dsa/tag_ar9331.c

> +++ b/net/dsa/tag_ar9331.c

> @@ -31,9 +31,6 @@ static struct sk_buff *ar9331_tag_xmit(struct sk_buff *skb,

>  	__le16 *phdr;

>  	u16 hdr;

>  

> -	if (skb_cow_head(skb, AR9331_HDR_LEN) < 0)

> -		return NULL;

> -

>  	phdr = skb_push(skb, AR9331_HDR_LEN);

>  

>  	hdr = FIELD_PREP(AR9331_HDR_VERSION_MASK, AR9331_HDR_VERSION);

> -- 

> 2.25.1

> 


Regards,
Oleksij

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
diff mbox series

Patch

diff --git a/net/dsa/tag_ar9331.c b/net/dsa/tag_ar9331.c
index 55b00694cdba..002cf7f952e2 100644
--- a/net/dsa/tag_ar9331.c
+++ b/net/dsa/tag_ar9331.c
@@ -31,9 +31,6 @@  static struct sk_buff *ar9331_tag_xmit(struct sk_buff *skb,
 	__le16 *phdr;
 	u16 hdr;
 
-	if (skb_cow_head(skb, AR9331_HDR_LEN) < 0)
-		return NULL;
-
 	phdr = skb_push(skb, AR9331_HDR_LEN);
 
 	hdr = FIELD_PREP(AR9331_HDR_VERSION_MASK, AR9331_HDR_VERSION);