linux-kernel - Re: INFO: trying to register non-static key in io_cqring_ev

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>] [day] [month] [year] [list]

Message-ID: <5a9c10c6-30b7-4177-146c-c2da2585ce5d@kernel.dk>
Date:   Tue, 31 Mar 2020 08:53:43 -0600
From:   Jens Axboe <axboe@...nel.dk>
To:     Hillf Danton <hdanton@...a.com>,
        syzbot <syzbot+0c3370f235b74b3cfd97@...kaller.appspotmail.com>
Cc:     io-uring@...r.kernel.org, linux-fsdevel@...r.kernel.org,
        linux-kernel@...r.kernel.org, syzkaller-bugs@...glegroups.com,
        viro@...iv.linux.org.uk
Subject: Re: INFO: trying to register non-static key in io_cqring_ev_posted
 (2)

On 3/31/20 5:44 AM, Hillf Danton wrote:
> 
> On Tue, 31 Mar 2020 04:14:03 -0700
>>
>> syzbot has bisected this bug to:
>>
>> commit b41e98524e424d104aa7851d54fd65820759875a
>> Author: Jens Axboe <axboe@...nel.dk>
>> Date:   Mon Feb 17 16:52:41 2020 +0000
>>
>>     io_uring: add per-task callback handler
>>
>> bisection log:  https://syzkaller.appspot.com/x/bisect.txt?x=115adadbe00000
>> start commit:   673b41e0 staging/octeon: fix up merge error
>> git tree:       upstream
>> final crash:    https://syzkaller.appspot.com/x/report.txt?x=135adadbe00000
>> console output: https://syzkaller.appspot.com/x/log.txt?x=155adadbe00000
>> kernel config:  https://syzkaller.appspot.com/x/.config?x=acf766c0e3d3f8c6
>> dashboard link: https://syzkaller.appspot.com/bug?extid=0c3370f235b74b3cfd97
>> syz repro:      https://syzkaller.appspot.com/x/repro.syz?x=13ac1b9de00000
>> C reproducer:   https://syzkaller.appspot.com/x/repro.c?x=10449493e00000
>>
>> Reported-by: syzbot+0c3370f235b74b3cfd97@...kaller.appspotmail.com
>> Fixes: b41e98524e42 ("io_uring: add per-task callback handler")
>>
>> For information about bisection process see: https://goo.gl/tpsmEJ#bisection
> 
> Looks like another line is missed in that work.
> 
> --- a/fs/io_uring.c
> +++ b/fs/io_uring.c
> @@ -5962,6 +5962,7 @@ static int io_sq_thread(void *data)
>  				}
>  				if (current->task_works) {
>  					task_work_run();
> +					finish_wait(&ctx->sqo_wait, &wait);
>  					continue;
>  				}
>  				if (signal_pending(current))

Can you send this as a properly formatted patch? That indeed looks like
the issue.

-- 
Jens Axboe