[patch net-next 1/2] rocker: enable support for scattered packets

Subsystems: networking drivers, rocker driver, the rest

STALE4024d

6 messages, 3 authors, 2015-08-03 · open the first message on its own page

[patch net-next 1/2] rocker: enable support for scattered packets

From: Jiri Pirko <jiri@resnulli.us>
Date: 2015-08-02 18:56:41

From: Ido Schimmel <redacted>

rocker supports the transmission of scattered packets, so let the kernel
know about it by setting the NETIF_F_SG bit in the device's features.

Signed-off-by: Ido Schimmel <redacted>
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
---
 drivers/net/ethernet/rocker/rocker.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/rocker/rocker.c b/drivers/net/ethernet/rocker/rocker.c
index 7b4c347..730bdc0 100644
--- a/drivers/net/ethernet/rocker/rocker.c
+++ b/drivers/net/ethernet/rocker/rocker.c
@@ -4970,7 +4970,7 @@ static int rocker_probe_port(struct rocker *rocker, unsigned int port_number)
 		       NAPI_POLL_WEIGHT);
 	rocker_carrier_init(rocker_port);
 
-	dev->features |= NETIF_F_NETNS_LOCAL;
+	dev->features |= NETIF_F_NETNS_LOCAL | NETIF_F_SG;
 
 	err = register_netdev(dev);
 	if (err) {
-- 
1.9.3

[patch net-next 2/2] rocker: linearize skb in case frags would not fit into tx descriptor

From: Jiri Pirko <jiri@resnulli.us>
Date: 2015-08-02 18:56:43

Suggested-by: Scott Feldman <redacted>
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
---
 drivers/net/ethernet/rocker/rocker.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/rocker/rocker.c b/drivers/net/ethernet/rocker/rocker.c
index 730bdc0..4cd5a71 100644
--- a/drivers/net/ethernet/rocker/rocker.c
+++ b/drivers/net/ethernet/rocker/rocker.c
@@ -4157,8 +4157,11 @@ static netdev_tx_t rocker_port_xmit(struct sk_buff *skb, struct net_device *dev)
 					  skb->data, skb_headlen(skb));
 	if (err)
 		goto nest_cancel;
-	if (skb_shinfo(skb)->nr_frags > ROCKER_TX_FRAGS_MAX)
-		goto nest_cancel;
+	if (skb_shinfo(skb)->nr_frags > ROCKER_TX_FRAGS_MAX) {
+		err = skb_linearize(skb);
+		if (err)
+			goto unmap_frags;
+	}
 
 	for (i = 0; i < skb_shinfo(skb)->nr_frags; i++) {
 		const skb_frag_t *frag = &skb_shinfo(skb)->frags[i];
-- 
1.9.3

Re: [patch net-next 1/2] rocker: enable support for scattered packets

From: Scott Feldman <hidden>
Date: 2015-08-03 18:30:58

On Sun, Aug 2, 2015 at 11:56 AM, Jiri Pirko [off-list ref] wrote:
From: Ido Schimmel <redacted>

rocker supports the transmission of scattered packets, so let the kernel
know about it by setting the NETIF_F_SG bit in the device's features.

Signed-off-by: Ido Schimmel <redacted>
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
Acked-by: Scott Feldman <redacted>

Re: [patch net-next 2/2] rocker: linearize skb in case frags would not fit into tx descriptor

From: Scott Feldman <hidden>
Date: 2015-08-03 18:32:11

On Sun, Aug 2, 2015 at 11:56 AM, Jiri Pirko [off-list ref] wrote:
Suggested-by: Scott Feldman <redacted>
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
Acked-by: Scott Feldman <redacted>

Re: [patch net-next 1/2] rocker: enable support for scattered packets

From: David Miller <davem@davemloft.net>
Date: 2015-08-03 21:22:35

From: Jiri Pirko <jiri@resnulli.us>
Date: Sun,  2 Aug 2015 20:56:37 +0200
From: Ido Schimmel <redacted>

rocker supports the transmission of scattered packets, so let the kernel
know about it by setting the NETIF_F_SG bit in the device's features.

Signed-off-by: Ido Schimmel <redacted>
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
Applied.

Re: [patch net-next 2/2] rocker: linearize skb in case frags would not fit into tx descriptor

From: David Miller <davem@davemloft.net>
Date: 2015-08-03 21:22:40

From: Jiri Pirko <jiri@resnulli.us>
Date: Sun,  2 Aug 2015 20:56:38 +0200
Suggested-by: Scott Feldman <redacted>
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
Applied.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help