aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Howells <dhowells@redhat.com>2011-08-22 14:09:28 +0100
committerJames Morris <jmorris@namei.org>2011-08-23 09:57:36 +1000
commitd199798bdf969873f78d48140600ff0a98a87e69 (patch)
treefb0fbfe0eda27054eae9c9efe0240ace297c3661
parentb072e9bc2fe9aeff4e104e80e479160349f474a9 (diff)
KEYS: The dead key link reaper should be non-reentrant
The dead key link reaper should be non-reentrant as it relies on global state to keep track of where it's got to when it returns to the work queue manager to give it some air. Signed-off-by: David Howells <dhowells@redhat.com> Signed-off-by: James Morris <jmorris@namei.org>
-rw-r--r--security/keys/gc.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/security/keys/gc.c b/security/keys/gc.c
index b23db3fbb32..d67e88b791f 100644
--- a/security/keys/gc.c
+++ b/security/keys/gc.c
@@ -51,7 +51,7 @@ void key_schedule_gc(time_t gc_at)
kenter("%ld", gc_at - now);
if (gc_at <= now) {
- schedule_work(&key_gc_work);
+ queue_work(system_nrt_wq, &key_gc_work);
} else if (gc_at < key_gc_next_run) {
expires = jiffies + (gc_at - now) * HZ;
mod_timer(&key_gc_timer, expires);
@@ -65,7 +65,7 @@ static void key_gc_timer_func(unsigned long data)
{
kenter("");
key_gc_next_run = LONG_MAX;
- schedule_work(&key_gc_work);
+ queue_work(system_nrt_wq, &key_gc_work);
}
/*
@@ -206,7 +206,7 @@ gc_released_our_lock:
key_gc_new_timer = new_timer;
key_gc_again = true;
clear_bit(0, &key_gc_executing);
- schedule_work(&key_gc_work);
+ queue_work(system_nrt_wq, &key_gc_work);
kleave(" [continue]");
return;