netdev - Re: [syzbot] WARNING: locking bug in inet

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Tue, 3 Jan 2023 23:12:21 +0100
From:   Eric Dumazet <edumazet@...gle.com>
To:     Tetsuo Handa <penguin-kernel@...ove.sakura.ne.jp>
Cc:     Felix Kuehling <felix.kuehling@....com>,
        Waiman Long <longman@...hat.com>, jakub@...udflare.com,
        syzkaller-bugs@...glegroups.com, netdev@...r.kernel.org,
        syzbot <syzbot+94cc2a66fc228b23f360@...kaller.appspotmail.com>,
        Alexander.Deucher@....com, Christian.Koenig@....com,
        David1.Zhou@....com, Evan.Quan@....com, Harry.Wentland@....com,
        Oak.Zeng@....com, Ray.Huang@....com, Yong.Zhao@....com,
        airlied@...ux.ie, ast@...nel.org, boqun.feng@...il.com,
        daniel@...ll.ch, daniel@...earbox.net, davem@...emloft.net,
        dsahern@...nel.org, gautammenghani201@...il.com, kafai@...com,
        kuba@...nel.org, kuznet@....inr.ac.ru, mingo@...hat.com,
        ozeng@....com, pabeni@...hat.com, peterz@...radead.org,
        rex.zhu@....com, songliubraving@...com, will@...nel.org,
        yhs@...com, yoshfuji@...ux-ipv6.org
Subject: Re: [syzbot] WARNING: locking bug in inet_autobind

On Tue, Jan 3, 2023 at 11:08 PM Tetsuo Handa
<penguin-kernel@...ove.sakura.ne.jp> wrote:
>
> On 2023/01/04 1:20, Felix Kuehling wrote:
> >
> > Am 2023-01-03 um 11:05 schrieb Waiman Long:
> >> On 1/3/23 10:39, Felix Kuehling wrote:
> >>> The regression point doesn't make sense. The kernel config doesn't enable CONFIG_DRM_AMDGPU, so there is no way that a change in AMDGPU could have caused this regression.
> >>>
> >> I agree. It is likely a pre-existing problem or caused by another commit that got triggered because of the change in cacheline alignment caused by commit c0d9271ecbd ("drm/amdgpu: Delete user queue doorbell variable").
> > I don't think the change can affect cache line alignment. The entire amdgpu driver doesn't even get compiled in the kernel config that was used, and the change doesn't touch any files outside drivers/gpu/drm/amd/amdgpu:
> >
> > # CONFIG_DRM_AMDGPU is not set
> >
> > My guess would be that it's an intermittent bug that is confusing bisect.
> >
> > Regards,
> >   Felix
>
> This was already explained in https://groups.google.com/g/syzkaller-bugs/c/1rmGDmbXWIw/m/nIQm0EmxBAAJ .
>
> Jakub Sitnicki suggested
>
>   What if we revisit Eric's lockdep splat fix in 37159ef2c1ae ("l2tp: fix
>   a lockdep splat") and:
>
>   1. remove the lockdep_set_class_and_name(...) call in l2tp; it looks
>      like an odd case within the network stack, and
>
>   2. switch to bh_lock_sock_nested in l2tp_xmit_core so that we don't
>      break what has been fixed in 37159ef2c1ae.
>
> and we are waiting for response from Eric Dumazet.
>

Eric Dumazet has been very busy.

Send a patch, instead of an idea/description.

Thanks.