diff options
author | Johannes Berg <johannes.berg@intel.com> | 2019-05-24 16:02:44 -0400 |
---|---|---|
committer | Richard Weinberger <richard@nod.at> | 2019-07-02 17:27:19 -0400 |
commit | c7f04e87e444a4bdeced1b43ce961d31257414ab (patch) | |
tree | ebddb5af2e6c1a120e793d8a52539c48d4648e8f | |
parent | 80bf6ceaf9310b3f61934c69b382d4912deee049 (diff) |
um: Don't garbage collect in deactivate_all_fds()
My previous commit didn't actually address the whole issue with
lockdep shutdown, I had another local modification that disabled
lockdep but that wasn't sufficient alone, so had to do the other
change.
Another issue remained though - during kfree() we acquire locks
and lockdep tries to annotate those with exactly the same issue
in the other patch - we no longer have "current".
So, just remove the garbage collection. There's no value in it
anyway since we're going to shut down anyway and marking a slab
object as free is now not very useful anymore.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Richard Weinberger <richard@nod.at>
-rw-r--r-- | arch/um/kernel/irq.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/um/kernel/irq.c b/arch/um/kernel/irq.c index d532377f5808..efde1f16c603 100644 --- a/arch/um/kernel/irq.c +++ b/arch/um/kernel/irq.c | |||
@@ -401,7 +401,7 @@ int deactivate_all_fds(void) | |||
401 | ); | 401 | ); |
402 | to_free = to_free->next; | 402 | to_free = to_free->next; |
403 | } | 403 | } |
404 | garbage_collect_irq_entries(); | 404 | /* don't garbage collect - we can no longer call kfree() here */ |
405 | os_close_epoll_fd(); | 405 | os_close_epoll_fd(); |
406 | return 0; | 406 | return 0; |
407 | } | 407 | } |