summaryrefslogtreecommitdiffstats
path: root/arch/um/kernel
diff options
context:
space:
mode:
authorJohannes Berg <johannes.berg@intel.com>2019-05-24 21:37:00 +0200
committerRichard Weinberger <richard@nod.at>2019-07-02 23:27:05 +0200
commit8eacd6fca4044f470fc2eb0552ce6b8eab977e6c (patch)
tree2cd02a0b1c23e329c6e0e9aec857473335d21ee8 /arch/um/kernel
parentum: Timer code cleanup (diff)
downloadlinux-8eacd6fca4044f470fc2eb0552ce6b8eab977e6c.tar.xz
linux-8eacd6fca4044f470fc2eb0552ce6b8eab977e6c.zip
um: Remove locking in deactivate_all_fds()
Not only does the locking contradict the comment, and as the comment says is pointless and actually harmful (all the actual OS threads have exited already), but it also causes crashes when lockdep is enabled, because calling into the spinlock calls into lockdep, which then tries to determine the current task, which no longer exists. Remove the locking to let UML shut down cleanly in case lockdep is enabled. Signed-off-by: Johannes Berg <johannes.berg@intel.com> Signed-off-by: Richard Weinberger <richard@nod.at>
Diffstat (limited to 'arch/um/kernel')
-rw-r--r--arch/um/kernel/irq.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/arch/um/kernel/irq.c b/arch/um/kernel/irq.c
index b40dac71e25b..d532377f5808 100644
--- a/arch/um/kernel/irq.c
+++ b/arch/um/kernel/irq.c
@@ -384,10 +384,8 @@ EXPORT_SYMBOL(deactivate_fd);
*/
int deactivate_all_fds(void)
{
- unsigned long flags;
struct irq_entry *to_free;
- spin_lock_irqsave(&irq_lock, flags);
/* Stop IO. The IRQ loop has no lock so this is our
* only way of making sure we are safe to dispose
* of all IRQ handlers
@@ -404,7 +402,6 @@ int deactivate_all_fds(void)
to_free = to_free->next;
}
garbage_collect_irq_entries();
- spin_unlock_irqrestore(&irq_lock, flags);
os_close_epoll_fd();
return 0;
}