netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: nuclearcat@nuclearcat.com
To: Cong Wang <xiyou.wangcong@gmail.com>
Cc: Linux Kernel Network Developers <netdev@vger.kernel.org>
Subject: Re: 4.6.3, pppoe + shaper workload, skb_panic / skb_push / ppp_start_xmit
Date: Tue, 12 Jul 2016 21:13:11 +0300	[thread overview]
Message-ID: <61e8cde07f8d12680d1eb01cc024451b@nuclearcat.com> (raw)
In-Reply-To: <CAM_iQpX65H3SasBCqBcu6qcH00OW3WjZgfWkq7=rrQzyBvKF9A@mail.gmail.com>

On 2016-07-12 21:05, Cong Wang wrote:
> On Tue, Jul 12, 2016 at 11:03 AM,  <nuclearcat@nuclearcat.com> wrote:
>> On 2016-07-12 20:31, Cong Wang wrote:
>>> 
>>> On Mon, Jul 11, 2016 at 12:45 PM,  <nuclearcat@nuclearcat.com> wrote:
>>>> 
>>>> Hi
>>>> 
>>>> On latest kernel i noticed kernel panic happening 1-2 times per day. 
>>>> It
>>>> is
>>>> also happening on older kernel (at least 4.5.3).
>>>> 
>>> ...
>>>> 
>>>>  [42916.426463] Call Trace:
>>>>  [42916.426658]  <IRQ>
>>>> 
>>>>  [42916.426719]  [<ffffffff81843786>] skb_push+0x36/0x37
>>>>  [42916.427111]  [<ffffffffa00e8ce5>] ppp_start_xmit+0x10f/0x150
>>>> [ppp_generic]
>>>>  [42916.427314]  [<ffffffff81853467>] 
>>>> dev_hard_start_xmit+0x25a/0x2d3
>>>>  [42916.427516]  [<ffffffff818530f2>] ?
>>>> validate_xmit_skb.isra.107.part.108+0x11d/0x238
>>>>  [42916.427858]  [<ffffffff8186dee3>] sch_direct_xmit+0x89/0x1b5
>>>>  [42916.428060]  [<ffffffff8186e142>] __qdisc_run+0x133/0x170
>>>>  [42916.428261]  [<ffffffff81850034>] net_tx_action+0xe3/0x148
>>>>  [42916.428462]  [<ffffffff810c401a>] __do_softirq+0xb9/0x1a9
>>>>  [42916.428663]  [<ffffffff810c4251>] irq_exit+0x37/0x7c
>>>>  [42916.428862]  [<ffffffff8102b8f7>] 
>>>> smp_apic_timer_interrupt+0x3d/0x48
>>>>  [42916.429063]  [<ffffffff818cb15c>] apic_timer_interrupt+0x7c/0x90
>>> 
>>> 
>>> Interesting, we call a skb_cow_head() before skb_push() in
>>> ppp_start_xmit(),
>>> I have no idea why this could happen.
>>> 
>>> Do you have any tc qdisc, filter or actions on this ppp device?
>> 
>> Yes, i have policing filters for incoming traffic (ingress), and also 
>> on
>> egress htb + pfifo + filters.
> 
> Does it make any difference if you remove the egress qdisc and/or
> filters? If yes, please share the `tc qd show...` and `tc filter show 
> ...`?
> 
> Thanks!

It is not easy, because it is NAS with approx 5000 users connected (and 
they are constantly connecting/disconnecting), and crash can't be 
reproduced easily. If i will remove qdisc/filters users will get 
unlimited speed and this will cause serious service degradation.
But maybe i can add some debug lines and run some test kernel if 
necessary (if it will not cause serious performance overhead).

  reply	other threads:[~2016-07-12 18:13 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-11 19:45 4.6.3, pppoe + shaper workload, skb_panic / skb_push / ppp_start_xmit nuclearcat
2016-07-12 17:31 ` Cong Wang
2016-07-12 18:03   ` nuclearcat
2016-07-12 18:05     ` Cong Wang
2016-07-12 18:13       ` nuclearcat [this message]
2016-07-28 11:09   ` Guillaume Nault
2016-07-28 11:28     ` Denys Fedoryshchenko
2016-08-01 20:54       ` Guillaume Nault
2016-08-01 20:59       ` Guillaume Nault
2016-08-01 22:52         ` Denys Fedoryshchenko
2016-08-08 11:25         ` Denys Fedoryshchenko
2016-08-08 21:05           ` Guillaume Nault
2016-08-17 11:54             ` Denys Fedoryshchenko

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=61e8cde07f8d12680d1eb01cc024451b@nuclearcat.com \
    --to=nuclearcat@nuclearcat.com \
    --cc=netdev@vger.kernel.org \
    --cc=xiyou.wangcong@gmail.com \
    /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 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).