diff options
-rw-r--r-- | kernel/posix-cpu-timers.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/kernel/posix-cpu-timers.c b/kernel/posix-cpu-timers.c index bf2a6502860a..d5dbef5e89e6 100644 --- a/kernel/posix-cpu-timers.c +++ b/kernel/posix-cpu-timers.c | |||
@@ -1272,10 +1272,6 @@ static inline int fastpath_timer_check(struct task_struct *tsk) | |||
1272 | { | 1272 | { |
1273 | struct signal_struct *sig; | 1273 | struct signal_struct *sig; |
1274 | 1274 | ||
1275 | /* tsk == current, ensure it is safe to use ->signal/sighand */ | ||
1276 | if (unlikely(tsk->exit_state)) | ||
1277 | return 0; | ||
1278 | |||
1279 | if (!task_cputime_zero(&tsk->cputime_expires)) { | 1275 | if (!task_cputime_zero(&tsk->cputime_expires)) { |
1280 | struct task_cputime task_sample = { | 1276 | struct task_cputime task_sample = { |
1281 | .utime = tsk->utime, | 1277 | .utime = tsk->utime, |
@@ -1308,6 +1304,7 @@ void run_posix_cpu_timers(struct task_struct *tsk) | |||
1308 | { | 1304 | { |
1309 | LIST_HEAD(firing); | 1305 | LIST_HEAD(firing); |
1310 | struct k_itimer *timer, *next; | 1306 | struct k_itimer *timer, *next; |
1307 | unsigned long flags; | ||
1311 | 1308 | ||
1312 | BUG_ON(!irqs_disabled()); | 1309 | BUG_ON(!irqs_disabled()); |
1313 | 1310 | ||
@@ -1318,7 +1315,8 @@ void run_posix_cpu_timers(struct task_struct *tsk) | |||
1318 | if (!fastpath_timer_check(tsk)) | 1315 | if (!fastpath_timer_check(tsk)) |
1319 | return; | 1316 | return; |
1320 | 1317 | ||
1321 | spin_lock(&tsk->sighand->siglock); | 1318 | if (!lock_task_sighand(tsk, &flags)) |
1319 | return; | ||
1322 | /* | 1320 | /* |
1323 | * Here we take off tsk->signal->cpu_timers[N] and | 1321 | * Here we take off tsk->signal->cpu_timers[N] and |
1324 | * tsk->cpu_timers[N] all the timers that are firing, and | 1322 | * tsk->cpu_timers[N] all the timers that are firing, and |
@@ -1340,7 +1338,7 @@ void run_posix_cpu_timers(struct task_struct *tsk) | |||
1340 | * that gets the timer lock before we do will give it up and | 1338 | * that gets the timer lock before we do will give it up and |
1341 | * spin until we've taken care of that timer below. | 1339 | * spin until we've taken care of that timer below. |
1342 | */ | 1340 | */ |
1343 | spin_unlock(&tsk->sighand->siglock); | 1341 | unlock_task_sighand(tsk, &flags); |
1344 | 1342 | ||
1345 | /* | 1343 | /* |
1346 | * Now that all the timers on our list have the firing flag, | 1344 | * Now that all the timers on our list have the firing flag, |