Received-SPF: pass (google.com: domain of linux-kernel+bounces-153197-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Precedence: bulk
MIME-Version: 1.0
References: <0000000000004792a90615a1dde0@google.com> <CAADnVQKoPfHC_o7jSa0W-gC=fqodmNDeoRO8eaTPN_NxBuXD6w@mail.gmail.com>
 <ZiLzUgbW6dw-FYtf@google.com>
In-Reply-To: <ZiLzUgbW6dw-FYtf@google.com>
From: =?UTF-8?B?QmrDtnJuIFTDtnBlbA==?= <bjorn@kernel.org>
Date: Mon, 22 Apr 2024 12:36:48 +0200
Message-ID: <CAJ+HfNhNB9uMzri1xcyKmdEnDCm8YetoUWU6r_ms+aiqo3j8EQ@mail.gmail.com>
Subject: Re: [syzbot] [bpf?] BUG: unable to handle kernel paging request in
 bpf_prog_ADDR (2)
To: Stanislav Fomichev <sdf@google.com>
Cc: Alexei Starovoitov <alexei.starovoitov@gmail.com>, 
	syzbot <syzbot+838346b979830606c854@syzkaller.appspotmail.com>, 
	Andrii Nakryiko <andrii@kernel.org>, Alexei Starovoitov <ast@kernel.org>, bpf <bpf@vger.kernel.org>, 
	Daniel Borkmann <daniel@iogearbox.net>, Eddy Z <eddyz87@gmail.com>, Hao Luo <haoluo@google.com>, 
	John Fastabend <john.fastabend@gmail.com>, Jiri Olsa <jolsa@kernel.org>, 
	KP Singh <kpsingh@kernel.org>, LKML <linux-kernel@vger.kernel.org>, 
	Martin KaFai Lau <martin.lau@linux.dev>, Song Liu <song@kernel.org>, 
	syzkaller-bugs <syzkaller-bugs@googlegroups.com>, Yonghong Song <yonghong.song@linux.dev>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Sat, 20 Apr 2024 at 00:42, Stanislav Fomichev <sdf@google.com> wrote:
>
> On 04/19, Alexei Starovoitov wrote:
> > On Mon, Apr 8, 2024 at 8:53=E2=80=AFPM syzbot
> > <syzbot+838346b979830606c854@syzkaller.appspotmail.com> wrote:
> > >
> > > Hello,
> > >
> > > syzbot found the following issue on:
> > >
> > > HEAD commit:    fe46a7dd189e Merge tag 'sound-6.9-rc1' of git://git.k=
ernel..
> > > git tree:       upstream
> > > console+strace: https://syzkaller.appspot.com/x/log.txt?x=3D125962231=
80000
> > > kernel config:  https://syzkaller.appspot.com/x/.config?x=3D4d90a36f0=
cab495a
> > > dashboard link: https://syzkaller.appspot.com/bug?extid=3D838346b9798=
30606c854
> > > compiler:       Debian clang version 15.0.6, GNU ld (GNU Binutils for=
 Debian) 2.40
> > > syz repro:      https://syzkaller.appspot.com/x/repro.syz?x=3D134ecbb=
5180000
> > > C reproducer:   https://syzkaller.appspot.com/x/repro.c?x=3D141a8b3d1=
80000
> > >
> > > Downloadable assets:
> > > disk image: https://storage.googleapis.com/syzbot-assets/f6c04726a2ae=
/disk-fe46a7dd.raw.xz
> > > vmlinux: https://storage.googleapis.com/syzbot-assets/09c26ce901ea/vm=
linux-fe46a7dd.xz
> > > kernel image: https://storage.googleapis.com/syzbot-assets/134acf7f53=
22/bzImage-fe46a7dd.xz
> > >
> > > IMPORTANT: if you fix the issue, please add the following tag to the =
commit:
> > > Reported-by: syzbot+838346b979830606c854@syzkaller.appspotmail.com
> > >
> > > BUG: unable to handle page fault for address: 0000001000000112
> > > #PF: supervisor read access in kernel mode
> > > #PF: error_code(0x0000) - not-present page
> > > PGD 800000002e7b1067 P4D 800000002e7b1067 PUD 0
> > > Oops: 0000 [#1] PREEMPT SMP KASAN PTI
> > > CPU: 0 PID: 5060 Comm: syz-executor351 Not tainted 6.8.0-syzkaller-08=
951-gfe46a7dd189e #0
> > > Hardware name: Google Google Compute Engine/Google Compute Engine, BI=
OS Google 03/27/2024
> > > RIP: 0010:bpf_prog_a8e24a805b35c61b+0x19/0x1e
> > > Code: cc cc cc cc cc cc cc cc cc cc cc cc cc cc cc cc cc f3 0f 1e fa =
0f 1f 44 00 00 66 90 55 48 89 e5 f3 0f 1e fa 31 c0 48 8b 7f 18 <8b> 7f 00 c=
9 c3 cc cc cc cc cc cc 40 03 00 00 cc cc cc cc cc cc cc
> > > RSP: 0018:ffffc90003b07b30 EFLAGS: 00010246
> > > RAX: 0000000000000000 RBX: ffffc90000ace048 RCX: ffff88802aa89e00
> > > RDX: 0000000000000000 RSI: ffffc90000ace048 RDI: 0000001000000112
> > > RBP: ffffc90003b07b30 R08: ffffffff81bf633c R09: 1ffffffff2595ca0
> > > R10: dffffc0000000000 R11: ffffffffa000095c R12: ffffc90000ace030
> > > R13: ffff88802ac3ae28 R14: dffffc0000000000 R15: ffff88802ac3ae28
> > > FS:  000055558f759380(0000) GS:ffff8880b9400000(0000) knlGS:000000000=
0000000
> > > CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> > > CR2: 0000001000000112 CR3: 0000000077cfa000 CR4: 00000000003506f0
> > > DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> > > DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
> > > Call Trace:
> > >  <TASK>
> > >  bpf_dispatcher_nop_func include/linux/bpf.h:1234 [inline]
> > >  __bpf_prog_run include/linux/filter.h:657 [inline]
> > >  bpf_prog_run include/linux/filter.h:664 [inline]
> > >  bpf_prog_run_array_cg kernel/bpf/cgroup.c:51 [inline]
> > >  __cgroup_bpf_run_filter_setsockopt+0x6fa/0x1040 kernel/bpf/cgroup.c:=
1830
> > >  do_sock_setsockopt+0x6b4/0x720 net/socket.c:2293
> > >  __sys_setsockopt+0x1ae/0x250 net/socket.c:2334
> > >  __do_sys_setsockopt net/socket.c:2343 [inline]
> > >  __se_sys_setsockopt net/socket.c:2340 [inline]
> > >  __x64_sys_setsockopt+0xb5/0xd0 net/socket.c:2340
> > >  do_syscall_64+0xfb/0x240
> > >  entry_SYSCALL_64_after_hwframe+0x6d/0x75
> >
> > This one looks interesting.
> > But I cannot reproduce it.
> >
> > Bjorn or Stan,
> >
> > Could you take a look?
> >
> > Probably a race in xdp dispatcher setup or the way cgroup-lsm
> > logic is doing it.
>
> Managed to repro it by hacking the C reproducer to attach bpf prog
> to /sys/fs/cgroup instead of syzkallers custom path. Will try to
> poke it a bit more..

Stan, did you get anywhere? Please share your hack, where you manage
to reproduce the issue.


Cheers,
Bj=C3=B6rn