* Corrupt Network Packets -- 3.17-rc5/rc6
@ 2014-09-22 11:30 Pete Clements
2014-09-22 15:53 ` Cong Wang
0 siblings, 1 reply; 6+ messages in thread
From: Pete Clements @ 2014-09-22 11:30 UTC (permalink / raw)
To: linux-kernel
Heads up fyi: system is i386 UP.
Started getting some pop3 reception errors from the above system.
Investigating the problem discovered scp of the mailbox failed
with "Corrupted MAC on input"
Problem started with rc5, still in rc6. Running rc4 with no
problem.
Sorry for lack of better definition.
--
Pete Clements
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Corrupt Network Packets -- 3.17-rc5/rc6
2014-09-22 11:30 Corrupt Network Packets -- 3.17-rc5/rc6 Pete Clements
@ 2014-09-22 15:53 ` Cong Wang
2014-09-22 17:24 ` Pete Clements
0 siblings, 1 reply; 6+ messages in thread
From: Cong Wang @ 2014-09-22 15:53 UTC (permalink / raw)
To: Pete Clements; +Cc: linux-kernel, Linux Kernel Network Developers
(Networking bug report needs to go netdev@)
On Mon, Sep 22, 2014 at 4:30 AM, Pete Clements
<clem@clem.clem-digital.net> wrote:
> Heads up fyi: system is i386 UP.
>
> Started getting some pop3 reception errors from the above system.
> Investigating the problem discovered scp of the mailbox failed
> with "Corrupted MAC on input"
>
> Problem started with rc5, still in rc6. Running rc4 with no
> problem.
>
> Sorry for lack of better definition.
Which driver are you using?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Corrupt Network Packets -- 3.17-rc5/rc6
2014-09-22 15:53 ` Cong Wang
@ 2014-09-22 17:24 ` Pete Clements
2014-09-22 17:38 ` David Miller
0 siblings, 1 reply; 6+ messages in thread
From: Pete Clements @ 2014-09-22 17:24 UTC (permalink / raw)
To: Cong Wang; +Cc: Pete Clements, linux-kernel, Linux Kernel Network Developers
Quoting Cong Wang
> <clem@clem.clem-digital.net> wrote:
> > Heads up fyi: system is i386 UP.
> >
> > Started getting some pop3 reception errors from the above system.
> > Investigating the problem discovered scp of the mailbox failed
> > with "Corrupted MAC on input"
> >
> > Problem started with rc5, still in rc6. Running rc4 with no
> > problem.
> >
> > Sorry for lack of better definition.
>
>
> Which driver are you using?
>
Here's the output from boot log:
PCI: setting IRQ 11 as level-triggered
3c59x 0000:00:0d.0: found PCI INT A -> IRQ 11
3c59x: Donald Becker and others.
0000:00:0d.0: 3Com PCI 3c900 Cyclone 10Mbps TPC at d8812000.
--
Pete Clements
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Corrupt Network Packets -- 3.17-rc5/rc6
2014-09-22 17:24 ` Pete Clements
@ 2014-09-22 17:38 ` David Miller
2014-09-22 18:30 ` Pete Clements
0 siblings, 1 reply; 6+ messages in thread
From: David Miller @ 2014-09-22 17:38 UTC (permalink / raw)
To: clem; +Cc: xiyou.wangcong, linux-kernel, netdev
From: Pete Clements <clem@clem.clem-digital.net>
Date: Mon, 22 Sep 2014 13:24:59 -0400 (EDT)
> Quoting Cong Wang
> > <clem@clem.clem-digital.net> wrote:
> > > Heads up fyi: system is i386 UP.
> > >
> > > Started getting some pop3 reception errors from the above system.
> > > Investigating the problem discovered scp of the mailbox failed
> > > with "Corrupted MAC on input"
> > >
> > > Problem started with rc5, still in rc6. Running rc4 with no
> > > problem.
> > >
> > > Sorry for lack of better definition.
> >
> >
> > Which driver are you using?
> >
> Here's the output from boot log:
>
> PCI: setting IRQ 11 as level-triggered
> 3c59x 0000:00:0d.0: found PCI INT A -> IRQ 11
> 3c59x: Donald Becker and others.
> 0000:00:0d.0: 3Com PCI 3c900 Cyclone 10Mbps TPC at d8812000.
Should be fixed by:
commit 8400dd029e764e30361a51a7ae35900ae0032b43
Author: Neil Horman <nhorman@tuxdriver.com>
Date: Wed Sep 17 09:04:45 2014 -0400
3c59x: Fix bad offset spec in skb_frag_dma_map
Recently aded the use of skb_frag_dma_map to 3c59x, but didn't realize it
automatically included the frag_offset internally, as well as provided an option
to specify an extra offset in the parameter list. We need to specify an offset
of 0 in the parameter list to avoid skb corruption that results in lost
connections.
Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
CC: Linux Kernel list <linux-kernel@vger.kernel.org>
CC: "David S. Miller" <davem@davemloft.net>
CC: Meelis Roos <mroos@linux.ee>
Tested-by: Meelis Roos <mroos@linux.ee>
diff --git a/drivers/net/ethernet/3com/3c59x.c b/drivers/net/ethernet/3com/3c59x.c
index 8ab87ff..8ca49f04 100644
--- a/drivers/net/ethernet/3com/3c59x.c
+++ b/drivers/net/ethernet/3com/3c59x.c
@@ -2186,7 +2186,7 @@ boomerang_start_xmit(struct sk_buff *skb, struct net_device *dev)
skb_frag_t *frag = &skb_shinfo(skb)->frags[i];
dma_addr = skb_frag_dma_map(&VORTEX_PCI(vp)->dev, frag,
- frag->page_offset,
+ 0,
frag->size,
DMA_TO_DEVICE);
if (dma_mapping_error(&VORTEX_PCI(vp)->dev, dma_addr)) {
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: Corrupt Network Packets -- 3.17-rc5/rc6
2014-09-22 17:38 ` David Miller
@ 2014-09-22 18:30 ` Pete Clements
2014-09-22 19:07 ` David Miller
0 siblings, 1 reply; 6+ messages in thread
From: Pete Clements @ 2014-09-22 18:30 UTC (permalink / raw)
To: David Miller; +Cc: clem, xiyou.wangcong, linux-kernel, netdev
Thanks. Does not patch. Code looks not to be in the rc6 that Linus
put out. Guess there is more to come. (This is what I believe is
the segment in rc6 -- at line 2179.)
vp->tx_ring[entry].frag[i+1].addr =
cpu_to_le32(skb_frag_dma_map(
&VORTEX_PCI(vp)->dev,
frag,
frag->page_offset, frag->size, DMA_TO_DEVICE));
--
Pete Clements
Quoting David Miller
> From: Pete Clements <clem@clem.clem-digital.net>
> Date: Mon, 22 Sep 2014 13:24:59 -0400 (EDT)
>
> > Quoting Cong Wang
> > > <clem@clem.clem-digital.net> wrote:
> > > > Heads up fyi: system is i386 UP.
> > > >
> > > > Started getting some pop3 reception errors from the above system.
> > > > Investigating the problem discovered scp of the mailbox failed
> > > > with "Corrupted MAC on input"
> > > >
> > > > Problem started with rc5, still in rc6. Running rc4 with no
> > > > problem.
> > > >
> > > > Sorry for lack of better definition.
> > >
> > >
> > > Which driver are you using?
> > >
> > Here's the output from boot log:
> >
> > PCI: setting IRQ 11 as level-triggered
> > 3c59x 0000:00:0d.0: found PCI INT A -> IRQ 11
> > 3c59x: Donald Becker and others.
> > 0000:00:0d.0: 3Com PCI 3c900 Cyclone 10Mbps TPC at d8812000.
>
> Should be fixed by:
>
> commit 8400dd029e764e30361a51a7ae35900ae0032b43
> Author: Neil Horman <nhorman@tuxdriver.com>
> Date: Wed Sep 17 09:04:45 2014 -0400
>
> 3c59x: Fix bad offset spec in skb_frag_dma_map
>
> Recently aded the use of skb_frag_dma_map to 3c59x, but didn't realize it
> automatically included the frag_offset internally, as well as provided an option
> to specify an extra offset in the parameter list. We need to specify an offset
> of 0 in the parameter list to avoid skb corruption that results in lost
> connections.
>
> Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
> CC: Linux Kernel list <linux-kernel@vger.kernel.org>
> CC: "David S. Miller" <davem@davemloft.net>
> CC: Meelis Roos <mroos@linux.ee>
> Tested-by: Meelis Roos <mroos@linux.ee>
>
> diff --git a/drivers/net/ethernet/3com/3c59x.c b/drivers/net/ethernet/3com/3c59x.c
> index 8ab87ff..8ca49f04 100644
> --- a/drivers/net/ethernet/3com/3c59x.c
> +++ b/drivers/net/ethernet/3com/3c59x.c
> @@ -2186,7 +2186,7 @@ boomerang_start_xmit(struct sk_buff *skb, struct net_device *dev)
> skb_frag_t *frag = &skb_shinfo(skb)->frags[i];
>
> dma_addr = skb_frag_dma_map(&VORTEX_PCI(vp)->dev, frag,
> - frag->page_offset,
> + 0,
> frag->size,
> DMA_TO_DEVICE);
> if (dma_mapping_error(&VORTEX_PCI(vp)->dev, dma_addr)) {
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Corrupt Network Packets -- 3.17-rc5/rc6
2014-09-22 18:30 ` Pete Clements
@ 2014-09-22 19:07 ` David Miller
0 siblings, 0 replies; 6+ messages in thread
From: David Miller @ 2014-09-22 19:07 UTC (permalink / raw)
To: clem; +Cc: xiyou.wangcong, linux-kernel, netdev
From: Pete Clements <clem@clem.clem-digital.net>
Date: Mon, 22 Sep 2014 14:30:36 -0400 (EDT)
>
> Thanks. Does not patch. Code looks not to be in the rc6 that Linus
> put out. Guess there is more to come. (This is what I believe is
> the segment in rc6 -- at line 2179.)
>
> vp->tx_ring[entry].frag[i+1].addr =
> cpu_to_le32(skb_frag_dma_map(
> &VORTEX_PCI(vp)->dev,
> frag,
> frag->page_offset, frag->size, DMA_TO_DEVICE));
Just simply change the frag->page_offset argument to zero.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-09-22 19:07 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-09-22 11:30 Corrupt Network Packets -- 3.17-rc5/rc6 Pete Clements
2014-09-22 15:53 ` Cong Wang
2014-09-22 17:24 ` Pete Clements
2014-09-22 17:38 ` David Miller
2014-09-22 18:30 ` Pete Clements
2014-09-22 19:07 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).