mei: vsc: Call wake_up() in the threaded IRQ handler
BugLink: https://bugs.launchpad.net/bugs/2060097
[ Upstream commit 058a38acba15fd8e7b262ec6e17c4204cb15f984 ]
The hard IRQ handler vsc_tp_irq() is called with a raw spinlock taken.
wake_up() acquires a spinlock, a sleeping lock on PREEMPT_RT. This leads
to sleeping in atomic context.
Move the wake_up() call to the threaded IRQ handler vsc_tp_thread_isr()
where it can be safely called.
Fixes: 566f5ca976 ("mei: Add transport driver for IVSC device")
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Tested-and-Reviewed-by: Wentong Wu <wentong.wu@intel.com>
Link: https://lore.kernel.org/r/20240219195807.517742-2-sakari.ailus@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
(cherry picked from commit 76a5e0951a4f12685261c65f2cab6372c64cf363)
Signed-off-by: Paolo Pisati <paolo.pisati@canonical.com>
This commit is contained in:
committed by
Roxana Nicolescu
parent
db186a7970
commit
fd68576054
@@ -416,8 +416,6 @@ static irqreturn_t vsc_tp_isr(int irq, void *data)
|
||||
|
||||
atomic_inc(&tp->assert_cnt);
|
||||
|
||||
wake_up(&tp->xfer_wait);
|
||||
|
||||
return IRQ_WAKE_THREAD;
|
||||
}
|
||||
|
||||
@@ -425,6 +423,8 @@ static irqreturn_t vsc_tp_thread_isr(int irq, void *data)
|
||||
{
|
||||
struct vsc_tp *tp = data;
|
||||
|
||||
wake_up(&tp->xfer_wait);
|
||||
|
||||
if (tp->event_notify)
|
||||
tp->event_notify(tp->event_notify_context);
|
||||
|
||||
|
||||
Reference in New Issue
Block a user