diff options
-rw-r--r-- | kernel/exit.c | 8 | ||||
-rw-r--r-- | kernel/posix-cpu-timers.c | 36 |
2 files changed, 18 insertions, 26 deletions
diff --git a/kernel/exit.c b/kernel/exit.c index e95b93282210..e06d0c10a24e 100644 --- a/kernel/exit.c +++ b/kernel/exit.c | |||
@@ -881,14 +881,6 @@ fastcall NORET_TYPE void do_exit(long code) | |||
881 | 881 | ||
882 | tsk->flags |= PF_EXITING; | 882 | tsk->flags |= PF_EXITING; |
883 | 883 | ||
884 | /* | ||
885 | * Make sure we don't try to process any timer firings | ||
886 | * while we are already exiting. | ||
887 | */ | ||
888 | tsk->it_virt_expires = cputime_zero; | ||
889 | tsk->it_prof_expires = cputime_zero; | ||
890 | tsk->it_sched_expires = 0; | ||
891 | |||
892 | if (unlikely(in_atomic())) | 884 | if (unlikely(in_atomic())) |
893 | printk(KERN_INFO "note: %s[%d] exited with preempt_count %d\n", | 885 | printk(KERN_INFO "note: %s[%d] exited with preempt_count %d\n", |
894 | current->comm, current->pid, | 886 | current->comm, current->pid, |
diff --git a/kernel/posix-cpu-timers.c b/kernel/posix-cpu-timers.c index 9d9169aa2e24..4882bf1e094a 100644 --- a/kernel/posix-cpu-timers.c +++ b/kernel/posix-cpu-timers.c | |||
@@ -1288,30 +1288,30 @@ void run_posix_cpu_timers(struct task_struct *tsk) | |||
1288 | 1288 | ||
1289 | #undef UNEXPIRED | 1289 | #undef UNEXPIRED |
1290 | 1290 | ||
1291 | BUG_ON(tsk->exit_state); | ||
1292 | |||
1293 | /* | 1291 | /* |
1294 | * Double-check with locks held. | 1292 | * Double-check with locks held. |
1295 | */ | 1293 | */ |
1296 | read_lock(&tasklist_lock); | 1294 | read_lock(&tasklist_lock); |
1297 | spin_lock(&tsk->sighand->siglock); | 1295 | if (likely(tsk->signal != NULL)) { |
1296 | spin_lock(&tsk->sighand->siglock); | ||
1298 | 1297 | ||
1299 | /* | 1298 | /* |
1300 | * Here we take off tsk->cpu_timers[N] and tsk->signal->cpu_timers[N] | 1299 | * Here we take off tsk->cpu_timers[N] and tsk->signal->cpu_timers[N] |
1301 | * all the timers that are firing, and put them on the firing list. | 1300 | * all the timers that are firing, and put them on the firing list. |
1302 | */ | 1301 | */ |
1303 | check_thread_timers(tsk, &firing); | 1302 | check_thread_timers(tsk, &firing); |
1304 | check_process_timers(tsk, &firing); | 1303 | check_process_timers(tsk, &firing); |
1305 | 1304 | ||
1306 | /* | 1305 | /* |
1307 | * We must release these locks before taking any timer's lock. | 1306 | * We must release these locks before taking any timer's lock. |
1308 | * There is a potential race with timer deletion here, as the | 1307 | * There is a potential race with timer deletion here, as the |
1309 | * siglock now protects our private firing list. We have set | 1308 | * siglock now protects our private firing list. We have set |
1310 | * the firing flag in each timer, so that a deletion attempt | 1309 | * the firing flag in each timer, so that a deletion attempt |
1311 | * that gets the timer lock before we do will give it up and | 1310 | * that gets the timer lock before we do will give it up and |
1312 | * spin until we've taken care of that timer below. | 1311 | * spin until we've taken care of that timer below. |
1313 | */ | 1312 | */ |
1314 | spin_unlock(&tsk->sighand->siglock); | 1313 | spin_unlock(&tsk->sighand->siglock); |
1314 | } | ||
1315 | read_unlock(&tasklist_lock); | 1315 | read_unlock(&tasklist_lock); |
1316 | 1316 | ||
1317 | /* | 1317 | /* |