Skip to content

Commit 11fbe6c

Browse files
committed
posix-timers: Remove pointless irqsafe from hash_lock
All usage of hash_lock is in thread context. No point in using spin_lock_irqsave()/irqrestore() for a single usage site. Signed-off-by: Thomas Gleixner <tglx@linutronix.de> Reviewed-by: Frederic Weisbecker <frederic@kernel.org> Link: https://lore.kernel.org/r/20230425183313.249063953@linutronix.de
1 parent 72786ff commit 11fbe6c

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

kernel/time/posix-timers.c

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -471,10 +471,9 @@ static void k_itimer_rcu_free(struct rcu_head *head)
471471
static void release_posix_timer(struct k_itimer *tmr, int it_id_set)
472472
{
473473
if (it_id_set) {
474-
unsigned long flags;
475-
spin_lock_irqsave(&hash_lock, flags);
474+
spin_lock(&hash_lock, flags);
476475
hlist_del_rcu(&tmr->t_hash);
477-
spin_unlock_irqrestore(&hash_lock, flags);
476+
spin_unlock(&hash_lock, flags);
478477
}
479478
put_pid(tmr->it_pid);
480479
sigqueue_free(tmr->sigq);

0 commit comments

Comments
 (0)