Thread (29 messages) 29 messages, 4 authors, 2012-06-01

Re: [PATCH 05/20] hexagon: Do not call do_signal() with irqs disabled

From: Richard Kuo <hidden>
Date: 2012-06-01 19:02:05
Also in: lkml

On Sat, May 26, 2012 at 12:04:21AM +0200, Richard Weinberger wrote:
quoted hunk ↗ jump to hunk
get_signal_to_deliver() calls try_to_freeze() which might sleep.

Cc: Richard Kuo <redacted>
Cc: linux-hexagon@vger.kernel.org
Signed-off-by: Richard Weinberger <richard@nod.at>
---
 arch/hexagon/kernel/signal.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/arch/hexagon/kernel/signal.c b/arch/hexagon/kernel/signal.c
index 434866e..8403bfd 100644
--- a/arch/hexagon/kernel/signal.c
+++ b/arch/hexagon/kernel/signal.c
@@ -22,6 +22,7 @@
 #include <linux/syscalls.h>
 #include <linux/freezer.h>
 #include <linux/tracehook.h>
+#include <linux/irqflags.h>
 #include <asm/registers.h>
 #include <asm/thread_info.h>
 #include <asm/unistd.h>
@@ -267,6 +268,8 @@ no_restart:
 
 void do_notify_resume(struct pt_regs *regs, unsigned long thread_info_flags)
 {
+	local_irq_enable();
+
 	if (thread_info_flags & _TIF_SIGPENDING)
 		do_signal(regs);
 
-- 
1.7.7.3
Acked-by:  Richard Kuo <redacted>


-- 

Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help