[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20240913121109.289-1-hdanton@sina.com>
Date: Fri, 13 Sep 2024 20:11:09 +0800
From: Hillf Danton <hdanton@...a.com>
To: Frederic Weisbecker <frederic@...nel.org>
Cc: syzbot <syzbot+943d34fa3cf2191e3068@...kaller.appspotmail.com>,
linux-kernel@...r.kernel.org,
syzkaller-bugs@...glegroups.com,
Thomas Gleixner <tglx@...utronix.de>,
Tejun Heo <tj@...nel.org>
Subject: Re: [PATCH] kthread: Unpark only parked kthreads (was Re: [syzbot] [wireguard?] WARNING in kthread_unpark (2))
Test Frederic's idea.
#syz test upstream master
--- a/kernel/kthread.c
+++ b/kernel/kthread.c
@@ -623,6 +623,8 @@ void kthread_unpark(struct task_struct *
{
struct kthread *kthread = to_kthread(k);
+ if (!test_bit(KTHREAD_SHOULD_PARK, &kthread->flags))
+ return;
/*
* Newly created kthread was parked when the CPU was offline.
* The binding was lost and we need to set it again.
--- l/drivers/input/misc/yealink.c
+++ y/drivers/input/misc/yealink.c
@@ -438,7 +438,7 @@ static void urb_irq_callback(struct urb
yealink_do_idle_tasks(yld);
- if (!yld->shutdown) {
+ if (!yld->shutdown && status != -EPROTO) {
ret = usb_submit_urb(yld->urb_ctl, GFP_ATOMIC);
if (ret && ret != -EPERM)
dev_err(&yld->intf->dev,
@@ -460,13 +460,13 @@ static void urb_ctl_callback(struct urb
case CMD_KEYPRESS:
case CMD_SCANCODE:
/* ask for a response */
- if (!yld->shutdown)
+ if (!yld->shutdown && status != -EPROTO)
ret = usb_submit_urb(yld->urb_irq, GFP_ATOMIC);
break;
default:
/* send new command */
yealink_do_idle_tasks(yld);
- if (!yld->shutdown)
+ if (!yld->shutdown && status != -EPROTO)
ret = usb_submit_urb(yld->urb_ctl, GFP_ATOMIC);
break;
}
--
Powered by blists - more mailing lists