Re: [syzbot] [cgroups?] possible deadlock in __run_timer_base (2)

[Waiman Long <llong@redhat.com>]

On 2/14/25 5:19 AM, Peter Zijlstra wrote:
> On Tue, Feb 11, 2025 at 09:14:12PM -0500, Waiman Long wrote:
>> On 2/9/25 3:43 PM, syzbot wrote:
>>> Hello,
>>>
>>> syzbot found the following issue on:
>>>
>>> HEAD commit:    92514ef226f5 Merge tag 'for-6.14-rc1-tag' of git://git.ker..
>>> git tree:       upstream
>>> console output: https://syzkaller.appspot.com/x/log.txt?x=179453df980000
>>> kernel config:  https://syzkaller.appspot.com/x/.config?x=1909f2f0d8e641ce
>>> dashboard link: https://syzkaller.appspot.com/bug?extid=ed801a886dfdbfe7136d
>>> compiler:       Debian clang version 15.0.6, GNU ld (GNU Binutils for Debian) 2.40
>>>
>>> Unfortunately, I don't have any reproducer for this issue yet.
>>>
>>> Downloadable assets:
>>> disk image (non-bootable): https://storage.googleapis.com/syzbot-assets/7feb34a89c2a/non_bootable_disk-92514ef2.raw.xz
>>> vmlinux: https://storage.googleapis.com/syzbot-assets/c4d8b91f8769/vmlinux-92514ef2.xz
>>> kernel image: https://storage.googleapis.com/syzbot-assets/c24ec4365966/bzImage-92514ef2.xz
>>>
>>> IMPORTANT: if you fix the issue, please add the following tag to the commit:
>>> Reported-by: syzbot+ed801a886dfdbfe7136d@syzkaller.appspotmail.com
>> This problem should be fixed by the following upstream patch once it is
>> merged into mainline.
>>
>> https://lore.kernel.org/lkml/20250127013127.3913153-1-longman@redhat.com/
>>
> AFAICT all these lockdep reports are because of an earlier warning. Fix
> warning, report goes away. Notably:
>
>>>          _printk+0xd5/0x120 kernel/printk/printk.c:2457
>>>          __report_bug lib/bug.c:195 [inline]
>>>          report_bug+0x346/0x500 lib/bug.c:219
>>>          handle_bug+0x60/0x90 arch/x86/kernel/traps.c:285
>>>          exc_invalid_op+0x1a/0x50 arch/x86/kernel/traps.c:309
>>>          asm_exc_invalid_op+0x1a/0x20 arch/x86/include/asm/idtentry.h:621
>>>          expire_timers kernel/time/timer.c:1827 [inline]
> IOW I think we're focusing on the wrong thing here.
>
>> Peter, are you planning to merge this patch? This is another instance where
>> the old way of calling wake_up_process() inside the lock critical region can
>> lead to deadlock.
> I still don't love the Changelog, but yeah, I suppose I can pick it up.
> But I see Boqun took it and I'll get it eventually.
>
> No real hurry there I suppose.

I know my changelog is sometimes too verbose and may contain 
non-important information. I will try to improve that in the future.

Cheers,
Longman

>