* [PATCH 01/11] lance: remove unnecessary setting of skb->dev
@ 2012-07-10 0:09 Jon Mason
2012-07-10 0:09 ` [PATCH 02/11] enic: " Jon Mason
` (10 more replies)
0 siblings, 11 replies; 15+ messages in thread
From: Jon Mason @ 2012-07-10 0:09 UTC (permalink / raw)
To: David S. Miller; +Cc: netdev
skb->dev is being unnecessarily set during ring init. It is already being set
to the proper value when eth_type_trans is called on packet receive, and the
skb->dev is not referenced anywhere else in the code.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
---
drivers/net/ethernet/amd/lance.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/net/ethernet/amd/lance.c b/drivers/net/ethernet/amd/lance.c
index a6e2e84..5c72843 100644
--- a/drivers/net/ethernet/amd/lance.c
+++ b/drivers/net/ethernet/amd/lance.c
@@ -873,10 +873,9 @@ lance_init_ring(struct net_device *dev, gfp_t gfp)
skb = alloc_skb(PKT_BUF_SZ, GFP_DMA | gfp);
lp->rx_skbuff[i] = skb;
- if (skb) {
- skb->dev = dev;
+ if (skb)
rx_buff = skb->data;
- } else
+ else
rx_buff = kmalloc(PKT_BUF_SZ, GFP_DMA | gfp);
if (rx_buff == NULL)
lp->rx_ring[i].base = 0;
--
1.7.9.5
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 02/11] enic: remove unnecessary setting of skb->dev
2012-07-10 0:09 [PATCH 01/11] lance: remove unnecessary setting of skb->dev Jon Mason
@ 2012-07-10 0:09 ` Jon Mason
2012-07-12 21:19 ` Jon Mason
2012-07-10 0:09 ` [PATCH 03/11] netxen: " Jon Mason
` (9 subsequent siblings)
10 siblings, 1 reply; 15+ messages in thread
From: Jon Mason @ 2012-07-10 0:09 UTC (permalink / raw)
To: David S. Miller
Cc: netdev, Christian Benvenuti, Roopa Prabhu, Neel Patel, Nishank Trivedi
skb->dev is being unnecessarily set after calling eth_type_trans.
eth_type_trans already sets skb->dev to the proper value, thus making this
unnecessary.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
Cc: Christian Benvenuti <benve@cisco.com>
Cc: Roopa Prabhu <roprabhu@cisco.com>
Cc: Neel Patel <neepatel@cisco.com>
Cc: Nishank Trivedi <nistrive@cisco.com>
---
drivers/net/ethernet/cisco/enic/enic_main.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/net/ethernet/cisco/enic/enic_main.c b/drivers/net/ethernet/cisco/enic/enic_main.c
index 8132c78..ad1468b 100644
--- a/drivers/net/ethernet/cisco/enic/enic_main.c
+++ b/drivers/net/ethernet/cisco/enic/enic_main.c
@@ -1300,8 +1300,6 @@ static void enic_rq_indicate_buf(struct vnic_rq *rq,
skb->ip_summed = CHECKSUM_COMPLETE;
}
- skb->dev = netdev;
-
if (vlan_stripped)
__vlan_hwaccel_put_tag(skb, vlan_tci);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 03/11] netxen: remove unnecessary setting of skb->dev
2012-07-10 0:09 [PATCH 01/11] lance: remove unnecessary setting of skb->dev Jon Mason
2012-07-10 0:09 ` [PATCH 02/11] enic: " Jon Mason
@ 2012-07-10 0:09 ` Jon Mason
2012-07-10 0:09 ` [PATCH 04/11] lantiq_etop: " Jon Mason
` (8 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Jon Mason @ 2012-07-10 0:09 UTC (permalink / raw)
To: David S. Miller; +Cc: netdev, Sony Chacko, Rajesh Borundia
skb->dev is being unnecessarily set by the driver on packet recieve.
eth_type_trans already sets skb->dev to the proper value and it is not
referenced anywhere else in the dirver, thus making its setting unnecessary.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
Cc: Sony Chacko <sony.chacko@qlogic.com>
Cc: Rajesh Borundia <rajesh.borundia@qlogic.com>
---
.../net/ethernet/qlogic/netxen/netxen_nic_init.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c b/drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c
index 8694124..b2c1b676 100644
--- a/drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c
+++ b/drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c
@@ -1532,8 +1532,6 @@ static struct sk_buff *netxen_process_rxbuf(struct netxen_adapter *adapter,
} else
skb->ip_summed = CHECKSUM_NONE;
- skb->dev = adapter->netdev;
-
buffer->skb = NULL;
no_skb:
buffer->state = NETXEN_BUFFER_FREE;
--
1.7.9.5
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 04/11] lantiq_etop: remove unnecessary setting of skb->dev
2012-07-10 0:09 [PATCH 01/11] lance: remove unnecessary setting of skb->dev Jon Mason
2012-07-10 0:09 ` [PATCH 02/11] enic: " Jon Mason
2012-07-10 0:09 ` [PATCH 03/11] netxen: " Jon Mason
@ 2012-07-10 0:09 ` Jon Mason
2012-07-10 0:09 ` [PATCH 05/11] ksz884x: " Jon Mason
` (7 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Jon Mason @ 2012-07-10 0:09 UTC (permalink / raw)
To: David S. Miller; +Cc: netdev
skb->dev is being unnecessarily set before calling eth_type_trans.
eth_type_trans already sets skb->dev to the proper value, thus making this
unnecessary.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
---
drivers/net/ethernet/lantiq_etop.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/net/ethernet/lantiq_etop.c b/drivers/net/ethernet/lantiq_etop.c
index 5dc9cbd..9fa39eb 100644
--- a/drivers/net/ethernet/lantiq_etop.c
+++ b/drivers/net/ethernet/lantiq_etop.c
@@ -149,7 +149,6 @@ ltq_etop_hw_receive(struct ltq_etop_chan *ch)
spin_unlock_irqrestore(&priv->lock, flags);
skb_put(skb, len);
- skb->dev = ch->netdev;
skb->protocol = eth_type_trans(skb, ch->netdev);
netif_receive_skb(skb);
}
--
1.7.9.5
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 05/11] ksz884x: remove unnecessary setting of skb->dev
2012-07-10 0:09 [PATCH 01/11] lance: remove unnecessary setting of skb->dev Jon Mason
` (2 preceding siblings ...)
2012-07-10 0:09 ` [PATCH 04/11] lantiq_etop: " Jon Mason
@ 2012-07-10 0:09 ` Jon Mason
2012-07-10 0:09 ` [PATCH 06/11] qlcnic: " Jon Mason
` (6 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Jon Mason @ 2012-07-10 0:09 UTC (permalink / raw)
To: David S. Miller; +Cc: netdev
skb->dev is being unnecessarily set during ring init. It is already being set
to the proper value when eth_type_trans is called on packet receive, and the
skb->dev is not referenced anywhere else in the code.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
---
drivers/net/ethernet/micrel/ksz884x.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/net/ethernet/micrel/ksz884x.c b/drivers/net/ethernet/micrel/ksz884x.c
index eaf9ff0..2b59e39 100644
--- a/drivers/net/ethernet/micrel/ksz884x.c
+++ b/drivers/net/ethernet/micrel/ksz884x.c
@@ -4480,14 +4480,12 @@ static void ksz_init_rx_buffers(struct dev_info *adapter)
dma_buf->len = adapter->mtu;
if (!dma_buf->skb)
dma_buf->skb = alloc_skb(dma_buf->len, GFP_ATOMIC);
- if (dma_buf->skb && !dma_buf->dma) {
- dma_buf->skb->dev = adapter->dev;
+ if (dma_buf->skb && !dma_buf->dma)
dma_buf->dma = pci_map_single(
adapter->pdev,
skb_tail_pointer(dma_buf->skb),
dma_buf->len,
PCI_DMA_FROMDEVICE);
- }
/* Set descriptor. */
set_rx_buf(desc, dma_buf->dma);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 06/11] qlcnic: remove unnecessary setting of skb->dev
2012-07-10 0:09 [PATCH 01/11] lance: remove unnecessary setting of skb->dev Jon Mason
` (3 preceding siblings ...)
2012-07-10 0:09 ` [PATCH 05/11] ksz884x: " Jon Mason
@ 2012-07-10 0:09 ` Jon Mason
2012-07-10 0:09 ` [PATCH 07/11] qlge: " Jon Mason
` (5 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Jon Mason @ 2012-07-10 0:09 UTC (permalink / raw)
To: David S. Miller; +Cc: netdev, Anirban Chakraborty, Sony Chacko, linux-driver
skb->dev is being unnecessarily set before calling eth_type_trans.
eth_type_trans already sets skb->dev to the proper value, thus making this
unnecessary.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
Cc: Anirban Chakraborty <anirban.chakraborty@qlogic.com>
Cc: Sony Chacko <sony.chacko@qlogic.com>
Cc: linux-driver@qlogic.com
---
drivers/net/ethernet/qlogic/qlcnic/qlcnic_init.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/net/ethernet/qlogic/qlcnic/qlcnic_init.c b/drivers/net/ethernet/qlogic/qlcnic/qlcnic_init.c
index 8620b69..0bcda9c 100644
--- a/drivers/net/ethernet/qlogic/qlcnic/qlcnic_init.c
+++ b/drivers/net/ethernet/qlogic/qlcnic/qlcnic_init.c
@@ -1488,8 +1488,6 @@ static struct sk_buff *qlcnic_process_rxbuf(struct qlcnic_adapter *adapter,
skb_checksum_none_assert(skb);
}
- skb->dev = adapter->netdev;
-
buffer->skb = NULL;
return skb;
--
1.7.9.5
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 07/11] qlge: remove unnecessary setting of skb->dev
2012-07-10 0:09 [PATCH 01/11] lance: remove unnecessary setting of skb->dev Jon Mason
` (4 preceding siblings ...)
2012-07-10 0:09 ` [PATCH 06/11] qlcnic: " Jon Mason
@ 2012-07-10 0:09 ` Jon Mason
2012-07-10 0:09 ` [PATCH 08/11] sunbmac: " Jon Mason
` (4 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Jon Mason @ 2012-07-10 0:09 UTC (permalink / raw)
To: David S. Miller
Cc: netdev, Anirban Chakraborty, Jitendra Kalsaria, Ron Mercer, linux-driver
skb->dev is being unnecessarily set by the driver on packet recieve.
eth_type_trans already sets skb->dev to the proper value and it is not
referenced anywhere else in the dirver, thus making its setting unnecessary.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
Cc: Anirban Chakraborty <anirban.chakraborty@qlogic.com>
Cc: Jitendra Kalsaria <jitendra.kalsaria@qlogic.com>
Cc: Ron Mercer <ron.mercer@qlogic.com>
Cc: linux-driver@qlogic.com
---
drivers/net/ethernet/qlogic/qlge/qlge_main.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/net/ethernet/qlogic/qlge/qlge_main.c b/drivers/net/ethernet/qlogic/qlge/qlge_main.c
index 3c3499d..ca427eb 100644
--- a/drivers/net/ethernet/qlogic/qlge/qlge_main.c
+++ b/drivers/net/ethernet/qlogic/qlge/qlge_main.c
@@ -1619,7 +1619,6 @@ static void ql_process_mac_rx_skb(struct ql_adapter *qdev,
}
prefetch(skb->data);
- skb->dev = ndev;
if (ib_mac_rsp->flags1 & IB_MAC_IOCB_RSP_M_MASK) {
netif_printk(qdev, rx_status, KERN_DEBUG, qdev->ndev,
"%s Multicast.\n",
@@ -1934,7 +1933,6 @@ static void ql_process_mac_split_rx_intr(struct ql_adapter *qdev,
}
prefetch(skb->data);
- skb->dev = ndev;
if (ib_mac_rsp->flags1 & IB_MAC_IOCB_RSP_M_MASK) {
netif_printk(qdev, rx_status, KERN_DEBUG, qdev->ndev, "%s Multicast.\n",
(ib_mac_rsp->flags1 & IB_MAC_IOCB_RSP_M_MASK) ==
--
1.7.9.5
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 08/11] sunbmac: remove unnecessary setting of skb->dev
2012-07-10 0:09 [PATCH 01/11] lance: remove unnecessary setting of skb->dev Jon Mason
` (5 preceding siblings ...)
2012-07-10 0:09 ` [PATCH 07/11] qlge: " Jon Mason
@ 2012-07-10 0:09 ` Jon Mason
2012-07-10 0:09 ` [PATCH 09/11] sungem: " Jon Mason
` (3 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Jon Mason @ 2012-07-10 0:09 UTC (permalink / raw)
To: David S. Miller; +Cc: netdev
skb->dev is being unnecessarily set during ring init and skb alloc in rx. It is
already being set to the proper value when eth_type_trans is called on packet
receive, and the skb->dev is not referenced anywhere else in the code.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
---
drivers/net/ethernet/sun/sunbmac.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/net/ethernet/sun/sunbmac.c b/drivers/net/ethernet/sun/sunbmac.c
index 2a83fc5..967fe8c 100644
--- a/drivers/net/ethernet/sun/sunbmac.c
+++ b/drivers/net/ethernet/sun/sunbmac.c
@@ -233,7 +233,6 @@ static void bigmac_init_rings(struct bigmac *bp, int from_irq)
continue;
bp->rx_skbs[i] = skb;
- skb->dev = dev;
/* Because we reserve afterwards. */
skb_put(skb, ETH_FRAME_LEN);
@@ -838,7 +837,6 @@ static void bigmac_rx(struct bigmac *bp)
RX_BUF_ALLOC_SIZE - 34,
DMA_FROM_DEVICE);
bp->rx_skbs[elem] = new_skb;
- new_skb->dev = bp->dev;
skb_put(new_skb, ETH_FRAME_LEN);
skb_reserve(new_skb, 34);
this->rx_addr =
--
1.7.9.5
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 09/11] sungem: remove unnecessary setting of skb->dev
2012-07-10 0:09 [PATCH 01/11] lance: remove unnecessary setting of skb->dev Jon Mason
` (6 preceding siblings ...)
2012-07-10 0:09 ` [PATCH 08/11] sunbmac: " Jon Mason
@ 2012-07-10 0:09 ` Jon Mason
2012-07-10 0:09 ` [PATCH 10/11] sunhme: " Jon Mason
` (2 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Jon Mason @ 2012-07-10 0:09 UTC (permalink / raw)
To: David S. Miller; +Cc: netdev
skb->dev is being unnecessarily set by the driver's skb alloc routine (which is
called in init and during rx). It is already being set to the proper value when
eth_type_trans is called on packet receive, and the skb->dev is not referenced
anywhere else in the code.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
---
drivers/net/ethernet/sun/sungem.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/net/ethernet/sun/sungem.c b/drivers/net/ethernet/sun/sungem.c
index 3cf4ab7..9ae12d0 100644
--- a/drivers/net/ethernet/sun/sungem.c
+++ b/drivers/net/ethernet/sun/sungem.c
@@ -752,7 +752,6 @@ static __inline__ struct sk_buff *gem_alloc_skb(struct net_device *dev, int size
if (likely(skb)) {
unsigned long offset = ALIGNED_RX_SKB_ADDR(skb->data);
skb_reserve(skb, offset);
- skb->dev = dev;
}
return skb;
}
--
1.7.9.5
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 10/11] sunhme: remove unnecessary setting of skb->dev
2012-07-10 0:09 [PATCH 01/11] lance: remove unnecessary setting of skb->dev Jon Mason
` (7 preceding siblings ...)
2012-07-10 0:09 ` [PATCH 09/11] sungem: " Jon Mason
@ 2012-07-10 0:09 ` Jon Mason
2012-07-10 0:09 ` [PATCH 11/11] ll_temac: " Jon Mason
2012-07-11 6:18 ` [PATCH 01/11] lance: " David Miller
10 siblings, 0 replies; 15+ messages in thread
From: Jon Mason @ 2012-07-10 0:09 UTC (permalink / raw)
To: David S. Miller; +Cc: netdev
skb->dev is being unnecessarily set during ring init and skb alloc in rx. It is
already being set to the proper value when eth_type_trans is called on packet
receive, and the skb->dev is not referenced anywhere else in the code.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
---
drivers/net/ethernet/sun/sunhme.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/net/ethernet/sun/sunhme.c b/drivers/net/ethernet/sun/sunhme.c
index dfc00c4..73f341b 100644
--- a/drivers/net/ethernet/sun/sunhme.c
+++ b/drivers/net/ethernet/sun/sunhme.c
@@ -1249,7 +1249,6 @@ static void happy_meal_clean_rings(struct happy_meal *hp)
static void happy_meal_init_rings(struct happy_meal *hp)
{
struct hmeal_init_block *hb = hp->happy_block;
- struct net_device *dev = hp->dev;
int i;
HMD(("happy_meal_init_rings: counters to zero, "));
@@ -1270,7 +1269,6 @@ static void happy_meal_init_rings(struct happy_meal *hp)
continue;
}
hp->rx_skbs[i] = skb;
- skb->dev = dev;
/* Because we reserve afterwards. */
skb_put(skb, (ETH_FRAME_LEN + RX_OFFSET + 4));
@@ -2031,7 +2029,6 @@ static void happy_meal_rx(struct happy_meal *hp, struct net_device *dev)
}
dma_unmap_single(hp->dma_dev, dma_addr, RX_BUF_ALLOC_SIZE, DMA_FROM_DEVICE);
hp->rx_skbs[elem] = new_skb;
- new_skb->dev = dev;
skb_put(new_skb, (ETH_FRAME_LEN + RX_OFFSET + 4));
hme_write_rxd(hp, this,
(RXFLAG_OWN|((RX_BUF_ALLOC_SIZE-RX_OFFSET)<<16)),
--
1.7.9.5
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 11/11] ll_temac: remove unnecessary setting of skb->dev
2012-07-10 0:09 [PATCH 01/11] lance: remove unnecessary setting of skb->dev Jon Mason
` (8 preceding siblings ...)
2012-07-10 0:09 ` [PATCH 10/11] sunhme: " Jon Mason
@ 2012-07-10 0:09 ` Jon Mason
2012-07-11 6:18 ` [PATCH 01/11] lance: " David Miller
10 siblings, 0 replies; 15+ messages in thread
From: Jon Mason @ 2012-07-10 0:09 UTC (permalink / raw)
To: David S. Miller; +Cc: netdev
skb->dev is being unnecessarily set by the driver on packet recieve.
eth_type_trans already sets skb->dev to the proper value and it is not
referenced anywhere else in the dirver, thus making its setting unnecessary.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
---
drivers/net/ethernet/xilinx/ll_temac_main.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/net/ethernet/xilinx/ll_temac_main.c b/drivers/net/ethernet/xilinx/ll_temac_main.c
index 1eaf712..705146d 100644
--- a/drivers/net/ethernet/xilinx/ll_temac_main.c
+++ b/drivers/net/ethernet/xilinx/ll_temac_main.c
@@ -768,7 +768,6 @@ static void ll_temac_recv(struct net_device *ndev)
DMA_FROM_DEVICE);
skb_put(skb, length);
- skb->dev = ndev;
skb->protocol = eth_type_trans(skb, ndev);
skb_checksum_none_assert(skb);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 01/11] lance: remove unnecessary setting of skb->dev
2012-07-10 0:09 [PATCH 01/11] lance: remove unnecessary setting of skb->dev Jon Mason
` (9 preceding siblings ...)
2012-07-10 0:09 ` [PATCH 11/11] ll_temac: " Jon Mason
@ 2012-07-11 6:18 ` David Miller
10 siblings, 0 replies; 15+ messages in thread
From: David Miller @ 2012-07-11 6:18 UTC (permalink / raw)
To: jdmason; +Cc: netdev
All 11 patches applied, thanks.
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 02/11] enic: remove unnecessary setting of skb->dev
2012-07-10 0:09 ` [PATCH 02/11] enic: " Jon Mason
@ 2012-07-12 21:19 ` Jon Mason
2012-07-12 21:23 ` Christian Benvenuti (benve)
0 siblings, 1 reply; 15+ messages in thread
From: Jon Mason @ 2012-07-12 21:19 UTC (permalink / raw)
To: David S. Miller; +Cc: netdev, Christian Benvenuti, Neel Patel, Nishank Trivedi
On Mon, Jul 9, 2012 at 5:09 PM, Jon Mason <jdmason@kudzu.us> wrote:
> skb->dev is being unnecessarily set after calling eth_type_trans.
> eth_type_trans already sets skb->dev to the proper value, thus making this
> unnecessary.
>
> Signed-off-by: Jon Mason <jdmason@kudzu.us>
> Cc: Christian Benvenuti <benve@cisco.com>
> Cc: Roopa Prabhu <roprabhu@cisco.com>
This address bounced. Anyone know of an updated address for Roopa or
should they be removed from the MAINTAINERS list?
> Cc: Neel Patel <neepatel@cisco.com>
> Cc: Nishank Trivedi <nistrive@cisco.com>
> ---
> drivers/net/ethernet/cisco/enic/enic_main.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/net/ethernet/cisco/enic/enic_main.c b/drivers/net/ethernet/cisco/enic/enic_main.c
> index 8132c78..ad1468b 100644
> --- a/drivers/net/ethernet/cisco/enic/enic_main.c
> +++ b/drivers/net/ethernet/cisco/enic/enic_main.c
> @@ -1300,8 +1300,6 @@ static void enic_rq_indicate_buf(struct vnic_rq *rq,
> skb->ip_summed = CHECKSUM_COMPLETE;
> }
>
> - skb->dev = netdev;
> -
> if (vlan_stripped)
> __vlan_hwaccel_put_tag(skb, vlan_tci);
>
> --
> 1.7.9.5
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* RE: [PATCH 02/11] enic: remove unnecessary setting of skb->dev
2012-07-12 21:19 ` Jon Mason
@ 2012-07-12 21:23 ` Christian Benvenuti (benve)
2012-07-13 4:56 ` Neel Patel (neepatel)
0 siblings, 1 reply; 15+ messages in thread
From: Christian Benvenuti (benve) @ 2012-07-12 21:23 UTC (permalink / raw)
To: Jon Mason, David S. Miller
Cc: netdev, Neel Patel (neepatel), Nishank Trivedi (nistrive)
Jon,
Roopa is not in the maintainer list anymore.
Sorry about that.
We will update MAINTAINERS.
BTW, the change in enic_main.c looks good.
/Chris
> -----Original Message-----
> From: Jon Mason [mailto:jdmason@kudzu.us]
> Sent: Thursday, July 12, 2012 2:19 PM
> To: David S. Miller
> Cc: netdev@vger.kernel.org; Christian Benvenuti (benve); Neel Patel (neepatel); Nishank Trivedi
> (nistrive)
> Subject: Re: [PATCH 02/11] enic: remove unnecessary setting of skb->dev
>
> On Mon, Jul 9, 2012 at 5:09 PM, Jon Mason <jdmason@kudzu.us> wrote:
> > skb->dev is being unnecessarily set after calling eth_type_trans.
> > eth_type_trans already sets skb->dev to the proper value, thus making this
> > unnecessary.
> >
> > Signed-off-by: Jon Mason <jdmason@kudzu.us>
> > Cc: Christian Benvenuti <benve@cisco.com>
> > Cc: Roopa Prabhu <roprabhu@cisco.com>
>
> This address bounced. Anyone know of an updated address for Roopa or
> should they be removed from the MAINTAINERS list?
>
> > Cc: Neel Patel <neepatel@cisco.com>
> > Cc: Nishank Trivedi <nistrive@cisco.com>
> > ---
> > drivers/net/ethernet/cisco/enic/enic_main.c | 2 --
> > 1 file changed, 2 deletions(-)
> >
> > diff --git a/drivers/net/ethernet/cisco/enic/enic_main.c
> b/drivers/net/ethernet/cisco/enic/enic_main.c
> > index 8132c78..ad1468b 100644
> > --- a/drivers/net/ethernet/cisco/enic/enic_main.c
> > +++ b/drivers/net/ethernet/cisco/enic/enic_main.c
> > @@ -1300,8 +1300,6 @@ static void enic_rq_indicate_buf(struct vnic_rq *rq,
> > skb->ip_summed = CHECKSUM_COMPLETE;
> > }
> >
> > - skb->dev = netdev;
> > -
> > if (vlan_stripped)
> > __vlan_hwaccel_put_tag(skb, vlan_tci);
> >
> > --
> > 1.7.9.5
> >
^ permalink raw reply [flat|nested] 15+ messages in thread
* RE: [PATCH 02/11] enic: remove unnecessary setting of skb->dev
2012-07-12 21:23 ` Christian Benvenuti (benve)
@ 2012-07-13 4:56 ` Neel Patel (neepatel)
0 siblings, 0 replies; 15+ messages in thread
From: Neel Patel (neepatel) @ 2012-07-13 4:56 UTC (permalink / raw)
To: Christian Benvenuti (benve), Jon Mason, David S. Miller
Cc: netdev, Nishank Trivedi (nistrive)
Jon,
The change in enic_main.c looks good.
_ Neel
-----Original Message-----
From: Christian Benvenuti (benve)
Sent: Thursday, July 12, 2012 2:23 PM
To: Jon Mason; David S. Miller
Cc: netdev@vger.kernel.org; Neel Patel (neepatel); Nishank Trivedi (nistrive)
Subject: RE: [PATCH 02/11] enic: remove unnecessary setting of skb->dev
Jon,
Roopa is not in the maintainer list anymore.
Sorry about that.
We will update MAINTAINERS.
BTW, the change in enic_main.c looks good.
/Chris
> -----Original Message-----
> From: Jon Mason [mailto:jdmason@kudzu.us]
> Sent: Thursday, July 12, 2012 2:19 PM
> To: David S. Miller
> Cc: netdev@vger.kernel.org; Christian Benvenuti (benve); Neel Patel
> (neepatel); Nishank Trivedi
> (nistrive)
> Subject: Re: [PATCH 02/11] enic: remove unnecessary setting of
> skb->dev
>
> On Mon, Jul 9, 2012 at 5:09 PM, Jon Mason <jdmason@kudzu.us> wrote:
> > skb->dev is being unnecessarily set after calling eth_type_trans.
> > eth_type_trans already sets skb->dev to the proper value, thus
> > making this unnecessary.
> >
> > Signed-off-by: Jon Mason <jdmason@kudzu.us>
> > Cc: Christian Benvenuti <benve@cisco.com>
> > Cc: Roopa Prabhu <roprabhu@cisco.com>
>
> This address bounced. Anyone know of an updated address for Roopa or
> should they be removed from the MAINTAINERS list?
>
> > Cc: Neel Patel <neepatel@cisco.com>
> > Cc: Nishank Trivedi <nistrive@cisco.com>
> > ---
> > drivers/net/ethernet/cisco/enic/enic_main.c | 2 --
> > 1 file changed, 2 deletions(-)
> >
> > diff --git a/drivers/net/ethernet/cisco/enic/enic_main.c
> b/drivers/net/ethernet/cisco/enic/enic_main.c
> > index 8132c78..ad1468b 100644
> > --- a/drivers/net/ethernet/cisco/enic/enic_main.c
> > +++ b/drivers/net/ethernet/cisco/enic/enic_main.c
> > @@ -1300,8 +1300,6 @@ static void enic_rq_indicate_buf(struct vnic_rq *rq,
> > skb->ip_summed = CHECKSUM_COMPLETE;
> > }
> >
> > - skb->dev = netdev;
> > -
> > if (vlan_stripped)
> > __vlan_hwaccel_put_tag(skb, vlan_tci);
> >
> > --
> > 1.7.9.5
> >
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2012-07-13 5:06 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-10 0:09 [PATCH 01/11] lance: remove unnecessary setting of skb->dev Jon Mason
2012-07-10 0:09 ` [PATCH 02/11] enic: " Jon Mason
2012-07-12 21:19 ` Jon Mason
2012-07-12 21:23 ` Christian Benvenuti (benve)
2012-07-13 4:56 ` Neel Patel (neepatel)
2012-07-10 0:09 ` [PATCH 03/11] netxen: " Jon Mason
2012-07-10 0:09 ` [PATCH 04/11] lantiq_etop: " Jon Mason
2012-07-10 0:09 ` [PATCH 05/11] ksz884x: " Jon Mason
2012-07-10 0:09 ` [PATCH 06/11] qlcnic: " Jon Mason
2012-07-10 0:09 ` [PATCH 07/11] qlge: " Jon Mason
2012-07-10 0:09 ` [PATCH 08/11] sunbmac: " Jon Mason
2012-07-10 0:09 ` [PATCH 09/11] sungem: " Jon Mason
2012-07-10 0:09 ` [PATCH 10/11] sunhme: " Jon Mason
2012-07-10 0:09 ` [PATCH 11/11] ll_temac: " Jon Mason
2012-07-11 6:18 ` [PATCH 01/11] lance: " David Miller
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.