All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Breuer <mbreuer@majjas.com>
To: Jarek Poplawski <jarkao2@gmail.com>
Cc: Stephen Hemminger <shemminger@linux-foundation.org>,
	David Miller <davem@davemloft.net>,
	akpm@linux-foundation.org, flyboy@gmail.com,
	linux-kernel@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: [PATCH] af_packet: Don't use skb after dev_queue_xmit()
Date: Mon, 18 Jan 2010 16:39:24 -0500	[thread overview]
Message-ID: <4B54D50C.90608@majjas.com> (raw)
In-Reply-To: <20100118212516.GE3157@del.dom.local>

On 1/18/2010 4:25 PM, Jarek Poplawski wrote:
> On Mon, Jan 18, 2010 at 03:56:45PM -0500, Michael Breuer wrote:
>    
>> On 1/18/2010 3:46 PM, Jarek Poplawski wrote:
>>      
>>> On Mon, Jan 18, 2010 at 11:29:31AM -0500, Michael Breuer wrote:
>>>        
>>>> Ok - up on the two patches, no DMAR. Some early observations:
>>>>
>>>> 1. There's an early on MMAP oops (see below). This happens once, at
>>>> the completion of the transition to runlevel 5 (I've seen it
>>>> entering runlevel 3 as well). This does not recur when runlevels are
>>>> subsequently changed. I do not see this when running with DMAR
>>>> enabled.
>>>>          
>>> OK, you mentioned this oops (actually a warning only) happened during
>>> previous tests too.
>>>        
>> Yes - dk if it's significant or not. Only obvious difference between
>> DMAR and not.
>>      
> OK, let's try (as long as possible) if it can break so hard as with
> DMAR.
>
>    
>>>> 2. The dropped tx packet (DHCP) is a bit harder to recreate, but it
>>>> still happens.
>>>>          
>>> Btw, I guess you improved the test because you didn't mention it here,
>>> even after my explicit question?:
>>> http://permalink.gmane.org/gmane.linux.network/149171
>>>        
>> I had been focusing on the hangs - dhcp causing the initial crash
>> from December. After things stabilized with the af patch&  skb may
>> pull I started noticing the dropped tx packets. I reported the TX
>> loss on the 16th of January after confirming the issue.
>>      
> OK, but we need to establish some status quo after these patches
> before any new things (including DMAR), so I'd suggest trying this
> config really longer and harder.
>
>    
>>>> Interestingly, I initially saw no dropped packets
>>>> with ping - but after I went the DCHP route and eventually
>>>> reconnected, I could then cause dropped tx packets with ping. To
>>>> clarify:
>>>>
>>>> a) start throughput
>>>> b) ping device - no packet loss - this was true for the entire test run.
>>>> c) start throughput again
>>>> d) ping - no loss.
>>>> e) drop wifi on the device&   restart - first attempt worked. Repeat
>>>> attempt yielded the dropped DHCPOFFER packets. After about 6 tries,
>>>> the device reconnected to wifi.
>>>> f) ping again (after the reconnection) - packet loss rate about 80%.
>>>> g) simultaneously ping the wifi router - no loss.
>>>> h) After a while, packets are no longer dropped during ping. If I
>>>> manage to cause the dhcp drop again, and then ping after the device
>>>> finally reconnects, packet loss is significant for a while (maybe 30
>>>> sec to a minute). Then things return to normal. Note that the packet
>>>> loss continues even if the reported throughput drops to nil.
>>>> i) I can't cause the initial packet loss at RX rates below about
>>>> 30,000KBPS (as reported by nethogs).  At rates over 40 I can
>>>> reproduce this on this set of patches&   config about 60% of the
>>>> time.
>>>>          
>>> I forgot to mention, but did you try to check if these lost ping
>>> packets are "being dropped somewhere after wireshark sees them and
>>> before hitting the wire" like DHCPOFFER? Aren't there any sky2
>>> warnings/resets while this happens?
>>>
>>> Jarek P.
>>>        
>> Yes. There are no errors, and no statistics anywhere that I know to
>> look reflect the loss. Nothing in netstat; ethtool -S; etc. The only
>> loss reported is RX. The recent TX warnings/resets happened while
>> the machine was up for several days and while unattended and under
>> high RX load.
>>      
> Please check "tc -s qdisc" each time as well.
>
> Jarek P
>    

Some output from tc -s qdisc:

Before test:
qdisc pfifo_fast 0: dev eth0 root refcnt 2 bands 3 priomap  1 2 2 2 1 2 
0 0 1 1 1 1 1 1 1 1
  Sent 35279532 bytes 291080 pkt (dropped 0, overlimits 0 requeues 0)
  rate 0bit 0pps backlog 0b 0p requeues 0
qdisc pfifo_fast 0: dev eth1 root refcnt 2 bands 3 priomap  1 2 2 2 1 2 
0 0 1 1 1 1 1 1 1 1
  Sent 377308 bytes 3107 pkt (dropped 0, overlimits 0 requeues 0)
  rate 0bit 0pps backlog 0b 0p requeues 0

During test (after initial observed packet loss):
qdisc pfifo_fast 0: dev eth0 root refcnt 2 bands 3 priomap  1 2 2 2 1 2 
0 0 1 1 1 1 1 1 1 1
  Sent 123389424 bytes 1781403 pkt (dropped 0, overlimits 0 requeues 0)
  rate 0bit 0pps backlog 0b 0p requeues 0
qdisc pfifo_fast 0: dev eth1 root refcnt 2 bands 3 priomap  1 2 2 2 1 2 
0 0 1 1 1 1 1 1 1 1
  Sent 400862 bytes 3250 pkt (dropped 0, overlimits 0 requeues 0)
  rate 0bit 0pps backlog 0b 0p requeues 0

During test - while packet loss occuring:
qdisc pfifo_fast 0: dev eth0 root refcnt 2 bands 3 priomap  1 2 2 2 1 2 
0 0 1 1 1 1 1 1 1 1
  Sent 150518974 bytes 2138312 pkt (dropped 0, overlimits 0 requeues 0)
  rate 0bit 0pps backlog 0b 0p requeues 0
qdisc pfifo_fast 0: dev eth1 root refcnt 2 bands 3 priomap  1 2 2 2 1 2 
0 0 1 1 1 1 1 1 1 1
  Sent 422003 bytes 3432 pkt (dropped 0, overlimits 0 requeues 0)
  rate 0bit 0pps backlog 0b 0p requeues 0

After the conclusion of the test:
qdisc pfifo_fast 0: dev eth0 root refcnt 2 bands 3 priomap  1 2 2 2 1 2 
0 0 1 1 1 1 1 1 1 1
  Sent 244900497 bytes 3416350 pkt (dropped 0, overlimits 0 requeues 0)
  rate 0bit 0pps backlog 0b 0p requeues 0
qdisc pfifo_fast 0: dev eth1 root refcnt 2 bands 3 priomap  1 2 2 2 1 2 
0 0 1 1 1 1 1 1 1 1
  Sent 564380 bytes 4708 pkt (dropped 0, overlimits 0 requeues 0)
  rate 0bit 0pps backlog 0b 0p requeues 0


During the test, 8.9GB received; 232.9MB sent).

I also connected a second device through the wifi router. I was able to 
ping that device w/o loss while DHCP packets were being dropped to the 
other connected device.

Last note: just moved to 2.6.32.4 from .3 for this test (from git).


  reply	other threads:[~2010-01-18 21:40 UTC|newest]

Thread overview: 145+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-21 23:52 sky2 panic in 2.6.32.1 under load Berck E. Nash
2009-12-22  0:09 ` Michael Breuer
2009-12-22 18:50   ` Michael Breuer
2009-12-23 22:54     ` sky2 panic in 2.6.32.1 under load (new oops) Michael Breuer
2009-12-24  7:01       ` Andrew Morton
2009-12-24 19:18         ` Michael Breuer
2009-12-24 22:27           ` Stephen Hemminger
2009-12-25 16:28         ` Michael Breuer
2009-12-25 23:22           ` Stephen Hemminger
2009-12-26  3:23             ` Michael Breuer
2009-12-26 17:57               ` Stephen Hemminger
2009-12-26 20:37                 ` Michael Breuer
2009-12-26 22:05                   ` [PATCH] sky2: make sure ethernet header is in transmit skb Stephen Hemminger
2009-12-27  3:44                     ` David Miller
2009-12-27  4:11                       ` David Miller
2010-01-04  5:32                         ` David Miller
2010-01-04 16:40                           ` Stephen Hemminger
2010-01-04 17:02                             ` Michael Breuer
2010-01-05 23:07                         ` [PATCH] af_packet: Don't use skb after dev_queue_xmit() Jarek Poplawski
2010-01-05 23:16                           ` Michael Breuer
2010-01-05 23:29                             ` Jarek Poplawski
2010-01-06  2:36                           ` Michael Breuer
2010-01-06  7:22                             ` Jarek Poplawski
2010-01-06  9:15                               ` [PATCH alt.2] " Jarek Poplawski
2010-01-06 14:49                                 ` Stephen Hemminger
2010-01-06 19:40                                   ` Jarek Poplawski
2010-01-06 19:49                               ` [PATCH] " Michael Breuer
2010-01-06 20:22                                 ` Jarek Poplawski
2010-01-06 20:33                                   ` Michael Breuer
2010-01-06 21:09                                     ` Jarek Poplawski
2010-01-06 21:32                                       ` Michael Breuer
2010-01-06 21:10                                 ` Stephen Hemminger
2010-01-06 21:20                                   ` Michael Breuer
2010-01-06 23:26                                   ` Michael Breuer
2010-01-07  2:42                                     ` Michael Breuer
2010-01-07  4:00                                       ` Michael Breuer
2010-01-07  4:53                                         ` Stephen Hemminger
2010-01-07  5:10                                           ` Michael Breuer
2010-01-07  5:32                                           ` Michael Breuer
2010-01-07  5:54                                             ` Michael Breuer
2010-01-07  7:20                                               ` Michael Breuer
2010-01-07  7:47                                                 ` Jarek Poplawski
2010-01-07  7:55                                                   ` Michael Breuer
2010-01-07  8:21                                                     ` Jarek Poplawski
2010-01-07 15:03                                                       ` Michael Breuer
2010-01-07 17:56                                                         ` Jarek Poplawski
2010-01-07 18:17                                                           ` Jarek Poplawski
2010-01-07 15:05                                                     ` Michael Breuer
2010-01-07 18:01                                                       ` Jarek Poplawski
2010-01-07 18:19                                                         ` Michael Breuer
2010-01-07 18:35                                                           ` Jarek Poplawski
2010-01-07 18:40                                                             ` Michael Breuer
2010-01-07 18:43                                                             ` Michael Breuer
2010-01-07 18:50                                                               ` Jarek Poplawski
2010-01-07 19:36                                                                 ` Jarek Poplawski
2010-01-07 19:55                                                                   ` Michael Breuer
2010-01-07 20:22                                                                     ` Jarek Poplawski
2010-01-07 23:11                                                                 ` Michael Breuer
2010-01-08  7:45                                                                   ` Jarek Poplawski
2010-01-08 16:40                                                                     ` Michael Breuer
2010-01-08 21:29                                                                       ` Jarek Poplawski
2010-01-08 21:48                                                                         ` Michael Breuer
2010-01-08 22:02                                                                           ` Jarek Poplawski
2010-01-09  4:45                                                                           ` Michael Breuer
2010-01-09  5:44                                                                             ` Michael Breuer
2010-01-09 12:28                                                                             ` Jarek Poplawski
2010-01-09 18:34                                                                               ` Michael Breuer
2010-01-13 20:39                                                                                 ` Michael Breuer
2010-01-13 21:09                                                                                   ` Jarek Poplawski
2010-01-13 21:16                                                                                     ` Michael Breuer
2010-01-13 21:34                                                                                       ` Jarek Poplawski
2010-01-17 16:26                                                                                       ` Michael Breuer
2010-01-17 22:17                                                                                         ` Jarek Poplawski
2010-01-17 22:34                                                                                           ` Michael Breuer
2010-01-17 23:05                                                                                             ` Jarek Poplawski
2010-01-17 23:15                                                                                               ` Michael Breuer
2010-01-18  7:30                                                                                                 ` Jarek Poplawski
2010-01-18 16:29                                                                                                   ` Michael Breuer
2010-01-18 20:46                                                                                                     ` Jarek Poplawski
2010-01-18 20:56                                                                                                       ` Michael Breuer
2010-01-18 21:00                                                                                                         ` Stephen Hemminger
2010-01-18 21:06                                                                                                           ` Jarek Poplawski
2010-01-18 21:24                                                                                                           ` Michael Breuer
2010-01-18 21:50                                                                                                             ` Jarek Poplawski
2010-01-18 21:25                                                                                                         ` Jarek Poplawski
2010-01-18 21:39                                                                                                           ` Michael Breuer [this message]
2010-01-18 22:08                                                                                                             ` Jarek Poplawski
2010-01-18 22:17                                                                                                               ` Jarek Poplawski
2010-01-18 22:47                                                                                                                 ` Michael Breuer
2010-01-19  5:46                                                                                                                   ` Michael Breuer
2010-01-19  8:41                                                                                                                     ` Jarek Poplawski
2010-01-19 15:28                                                                                                                       ` Michael Breuer
2010-01-21 19:48                                                                                                                       ` Michael Breuer
2010-01-19 10:47                                                                                                                     ` Jarek Poplawski
2010-01-19 15:47                                                                                                                       ` Michael Breuer
2010-01-19 19:59                                                                                                                         ` Jarek Poplawski
2010-01-19 20:06                                                                                                                           ` Michael Breuer
2010-01-19 20:29                                                                                                                             ` Jarek Poplawski
2010-01-19 22:45                                                                                                                             ` Jarek Poplawski
2010-01-20  1:01                                                                                                                               ` Michael Breuer
2010-01-20  1:10                                                                                                                                 ` Stephen Hemminger
2010-01-21 16:14                                                                                                                                   ` Stefan Richter
2010-01-21 16:50                                                                                                                                     ` Stefan Richter
2010-01-18 22:25                                                                                                               ` Michael Breuer
2010-01-18 22:40                                                                                                                 ` Jarek Poplawski
2009-12-27 17:03                   ` sky2 panic in 2.6.32.1 under load (new oops) Michael Breuer
2009-12-27 18:22                     ` Stephen Hemminger
2009-12-27 19:39                       ` Michael Breuer
2009-12-29 17:30                         ` Stephen Hemminger
2009-12-29 17:39                           ` Michael Breuer
2009-12-29 18:38                           ` Michael Breuer
2009-12-29 18:54                           ` Michael Breuer
2009-12-29 19:49                             ` Stephen Hemminger
2009-12-29 20:41                               ` Michael Breuer
2009-12-30  7:23                               ` Michael Breuer
2009-12-30  7:58                                 ` Stephen Hemminger
2009-12-30 17:49                                   ` Michael Breuer
2009-12-30 19:15                                     ` audit.c skb - tty race condition - was " Michael Breuer
2009-12-30 20:44                                       ` Michael Breuer
2009-12-30 21:15                                         ` Michael Breuer
2009-12-30 21:21                                           ` Michael Breuer
2009-12-30  7:59                                 ` Stephen Hemminger
2009-12-30 15:40                                   ` Michael Breuer
2009-12-30 18:10                                     ` Stephen Hemminger
2009-12-30 18:37                                       ` Michael Breuer
2009-12-31 18:09                                       ` Michael Breuer
2009-12-31 18:24                                         ` Stephen Hemminger
2010-01-01 17:42                                         ` Michael Breuer
2010-01-01 19:26                                           ` sky2 panic in 2.6.32.1 under load (tty NULL write) Michael Breuer
2010-01-01 20:34                                             ` Michael Breuer
2010-01-02 21:42                                               ` Michael Breuer
2009-12-29 19:15           ` sky2 panic in 2.6.32.1 under load (new oops) Jarek Poplawski
2009-12-29 19:20             ` Michael Breuer
2009-12-30  8:07         ` Stephen Hemminger
2009-12-30 15:36           ` Michael Breuer
2009-12-22  0:52 ` sky2 panic in 2.6.32.1 under load Daniel Hazelton
2009-12-24  6:58 ` Andrew Morton
2009-12-24 16:03   ` Berck Nash
2009-12-24 16:28     ` Daniel Hazelton
2009-12-24 22:21       ` Stephen Hemminger
2009-12-24 22:42         ` Michael Breuer
2009-12-25  0:06           ` Daniel Hazelton
2009-12-24 16:10   ` Michael Breuer
2009-12-24 16:16     ` Berck Nash
2009-12-24 16:26       ` Michael Breuer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4B54D50C.90608@majjas.com \
    --to=mbreuer@majjas.com \
    --cc=akpm@linux-foundation.org \
    --cc=davem@davemloft.net \
    --cc=flyboy@gmail.com \
    --cc=jarkao2@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=shemminger@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.