Re: [net-next 1/5] qede: use reset to set network header

2016-12-01 Thread Mintz, Yuval
> Since offset is zero, it's not necessary to use set function. Reset
> function is straightforward, and will remove the unnecessary add
> operation in set function.

Thanks.

Acked-by: Yuval Mintz 

[For #2 for bnx2x as well]  

[net-next 1/5] qede: use reset to set network header

2016-12-01 Thread Zhang Shengju
Since offset is zero, it's not necessary to use set function. Reset
function is straightforward, and will remove the unnecessary add
operation in set function.

Signed-off-by: Zhang Shengju 
---
 drivers/net/ethernet/qlogic/qede/qede_main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/qlogic/qede/qede_main.c 
b/drivers/net/ethernet/qlogic/qede/qede_main.c
index b84a2c4..31c8449 100644
--- a/drivers/net/ethernet/qlogic/qede/qede_main.c
+++ b/drivers/net/ethernet/qlogic/qede/qede_main.c
@@ -1220,7 +1220,7 @@ static void qede_gro_receive(struct qede_dev *edev,
 
 #ifdef CONFIG_INET
if (skb_shinfo(skb)->gso_size) {
-   skb_set_network_header(skb, 0);
+   skb_reset_network_header(skb);
 
switch (skb->protocol) {
case htons(ETH_P_IP):
-- 
1.8.3.1