From: Tommi Rantala <tommi.t.rantala@nokia.com>
To: Neil Horman <nhorman@tuxdriver.com>, Dmitry Vyukov <dvyukov@google.com>
Cc: Xin Long <lucien.xin@gmail.com>, David Ahern <dsahern@gmail.com>,
Daniel Borkmann <daniel@iogearbox.net>,
Cong Wang <xiyou.wangcong@gmail.com>,
David Miller <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Willem de Bruijn <willemb@google.com>,
Jakub Kicinski <jakub.kicinski@netronome.com>,
Rasmus Villemoes <linux@rasmusvillemoes.dk>,
netdev <netdev@vger.kernel.org>,
LKML <linux-kernel@vger.kernel.org>,
Alexey Kuznetsov <kuznet@ms2.inr.ac.ru>,
Hideaki YOSHIFUJI <yoshfuji@linux-ipv6.org>,
syzkaller <syzkaller@googlegroups.com>,
Dan Streetman <ddstreet@canonical.com>,
"Eric W. Biederman" <ebiederm@xmission.com>,
Alexey Kodanev <alexey.kodanev@oracle.com>,
Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>,
linux-sctp@vger.kernel.org
Subject: Re: net: hang in unregister_netdevice: waiting for lo to become free
Date: Wed, 21 Feb 2018 16:53:54 +0200 [thread overview]
Message-ID: <7d98027d-e810-a079-49c5-0bf8beef390e@nokia.com> (raw)
In-Reply-To: <20180220162622.GA32068@hmswarspite.think-freely.org>
On 20.02.2018 18:26, Neil Horman wrote:
> On Tue, Feb 20, 2018 at 09:14:41AM +0100, Dmitry Vyukov wrote:
>> On Tue, Feb 20, 2018 at 8:56 AM, Tommi Rantala
>> <tommi.t.rantala@nokia.com> wrote:
>>> On 19.02.2018 20:59, Dmitry Vyukov wrote:
>>>> Is this meant to be fixed already? I am still seeing this on the
>>>> latest upstream tree.
>>>>
>>>
>>> These two commits are in v4.16-rc1:
>>>
>>> commit 4a31a6b19f9ddf498c81f5c9b089742b7472a6f8
>>> Author: Tommi Rantala <tommi.t.rantala@nokia.com>
>>> Date: Mon Feb 5 21:48:14 2018 +0200
>>>
>>> sctp: fix dst refcnt leak in sctp_v4_get_dst
>>> ...
>>> Fixes: 410f03831 ("sctp: add routing output fallback")
>>> Fixes: 0ca50d12f ("sctp: fix src address selection if using secondary
>>> addresses")
>>>
>>>
>>> commit 957d761cf91cdbb175ad7d8f5472336a4d54dbf2
>>> Author: Alexey Kodanev <alexey.kodanev@oracle.com>
>>> Date: Mon Feb 5 15:10:35 2018 +0300
>>>
>>> sctp: fix dst refcnt leak in sctp_v6_get_dst()
>>> ...
>>> Fixes: dbc2b5e9a09e ("sctp: fix src address selection if using secondary
>>> addresses for ipv6")
>>>
>>>
>>> I guess we missed something if it's still reproducible.
>>>
>>> I can check it later this week, unless someone else beat me to it.
>>
>> Hi Tommi,
>>
>> Hmmm, I can't claim that it's exactly the same bug. Perhaps it's
>> another one then. But I am still seeing these:
>>
>> [ 58.799130] unregister_netdevice: waiting for lo to become free.
>> Usage count = 4
>> [ 60.847138] unregister_netdevice: waiting for lo to become free.
>> Usage count = 4
>> [ 62.895093] unregister_netdevice: waiting for lo to become free.
>> Usage count = 4
>> [ 64.943103] unregister_netdevice: waiting for lo to become free.
>> Usage count = 4
>>
>> on upstream tree pulled ~12 hours ago.
>>
> Can you write a systemtap script to probe dev_hold, and dev_put, printing out a
> backtrace if the device name matches "lo". That should tell us definitively if
> the problem is in the same location or not
Hi Dmitry, I tested with the reproducer and the kernel .config file that
you sent in the first email in this thread:
With 4.16-rc2 unable to reproduce.
With 4.15-rc9 bug reproducible, and I get "unregister_netdevice: waiting
for lo to become free. Usage count = 3"
With 4.15-rc9 and Alexey's "sctp: fix dst refcnt leak in
sctp_v6_get_dst()" cherry-picked on top, unable to reproduce.
Is syzkaller doing something else now to trigger the bug...?
Can you still trigger the bug with the same reproducer?
Tommi
next prev parent reply other threads:[~2018-02-21 14:54 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-30 12:09 net: hang in unregister_netdevice: waiting for lo to become free Dmitry Vyukov
2018-01-30 18:32 ` Cong Wang
2018-01-30 20:08 ` Daniel Borkmann
2018-01-30 20:57 ` David Ahern
2018-01-30 22:59 ` David Ahern
2018-02-01 0:49 ` Xin Long
2018-02-03 12:15 ` Xin Long
2018-02-19 18:59 ` Dmitry Vyukov
2018-02-20 7:56 ` Tommi Rantala
2018-02-20 8:14 ` Dmitry Vyukov
2018-02-20 16:26 ` Neil Horman
2018-02-21 14:53 ` Tommi Rantala [this message]
2018-04-12 12:15 ` Dmitry Vyukov
2018-04-13 11:37 ` Neil Horman
2018-04-13 12:43 ` Dan Streetman
2018-04-13 15:54 ` Dmitry Vyukov
2018-04-16 7:35 ` Dmitry Vyukov
2018-04-16 19:42 ` Dan Streetman
2018-05-10 6:46 ` Dmitry Vyukov
2018-05-10 10:23 ` Dan Streetman
2018-05-11 9:19 ` Dmitry Vyukov
2018-05-11 11:40 ` Dan Streetman
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=7d98027d-e810-a079-49c5-0bf8beef390e@nokia.com \
--to=tommi.t.rantala@nokia.com \
--cc=alexey.kodanev@oracle.com \
--cc=daniel@iogearbox.net \
--cc=davem@davemloft.net \
--cc=ddstreet@canonical.com \
--cc=dsahern@gmail.com \
--cc=dvyukov@google.com \
--cc=ebiederm@xmission.com \
--cc=edumazet@google.com \
--cc=jakub.kicinski@netronome.com \
--cc=kuznet@ms2.inr.ac.ru \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sctp@vger.kernel.org \
--cc=linux@rasmusvillemoes.dk \
--cc=lucien.xin@gmail.com \
--cc=marcelo.leitner@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=nhorman@tuxdriver.com \
--cc=syzkaller@googlegroups.com \
--cc=willemb@google.com \
--cc=xiyou.wangcong@gmail.com \
--cc=yoshfuji@linux-ipv6.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 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).