Received: by 2002:ab2:1347:0:b0:1f4:ac9d:b246 with SMTP id g7csp71987lqg; Wed, 10 Apr 2024 16:06:07 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCXNMDteUTZvqypdXd597k7CfMzSdoSrYkFoFSrwphFF9YL0WPvT/3FwD2ei6azCgF9MiUxCRFNTCTfGmBl6UIPhqwKdtZdoJ0AYatH4Sw== X-Google-Smtp-Source: AGHT+IEN3BVqPdhO0fbGRYUiFNc6+AAyudVn4wUp7udJeNU7pLWeydYsLoTb/cy4RmYDzb27mheA X-Received: by 2002:a17:902:82c8:b0:1e2:3e1e:3d9 with SMTP id u8-20020a17090282c800b001e23e1e03d9mr3948447plz.63.1712790367374; Wed, 10 Apr 2024 16:06:07 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1712790367; cv=pass; d=google.com; s=arc-20160816; b=tXG0gMIKxUZ3H46WKoqtzQ3QhohAePGtVUL4Gl8RCOm+Cann6IO6pfLka3j9dIo8wk m/oe2e4+Ou422YIXkk8LptZ9++qcXL5k0ZSed9+wcVrQ2GlfmF1wj+YxIoGqTosCzEId xIxjVF0q2WfxxH7ikC06j7/1CkbbtDjHGfYpSycIJaaqNKja37j3f0CT8Y+9KBoeiEJ5 QqVHd2oSG2akM/Ttx4X1aFSNlIzq6JObGhHhcethvxuoLr6UYL6PPnL71Tho8vzycOWs Oy8X83g389FRtoy5AI9Il3hczdo/ulJmfWntNSJo0Kp2CbhgroD/uZVoAzK0cSL6WbMr Ppjw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=date:mime-version:list-unsubscribe:list-subscribe:list-id :precedence:references:subject:cc:to:from:dkim-signature :dkim-signature:message-id; bh=kAkqE4xY1tktojAkL7ohXROJF3Nq4zDNsVY2SgjGpGc=; fh=joBs/8L+orz77ylyCy6wqNwwi35f7sYhLR+/TQjQjhI=; b=D8S3RIMTW5oxBN2BDgGV1BR7G9SOkxGrg5xqniFV4aQ7IkzEDrH1on5VqAR+8pBcR4 dIf0K4IY9uen5PWH9oGrA0gMVjhcyILAZ3lWF6E9eTeexoY6AMfw9q75taS9NBpI0ESG iUFKOvCQngKKLBoUTawN7YLmGLw22N4rSLxD3fgN/bT8aae9UiXbl09heTfn6Lh9t5bZ +iGWXgwZbW9xeVrggKP27YaRYawfxJgtx2nC5mC3QQ1f/RRFfUmGHHScX823bQIy6IZA EfWD6V7y9ac9LU7WoYtp8ekDE7RcJ9G6wp5dI29P/GNR/DlHHGuxGRqgGO2HXLF3liN7 8b7A==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=WtaJMplz; dkim=neutral (no key) header.i=@linutronix.de header.s=2020e header.b=MPTfS2Uj; arc=pass (i=1 spf=pass spfdomain=linutronix.de dkim=pass dkdomain=linutronix.de dmarc=pass fromdomain=linutronix.de); spf=pass (google.com: domain of linux-kernel+bounces-139502-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) smtp.mailfrom="linux-kernel+bounces-139502-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Return-Path: Received: from sy.mirrors.kernel.org (sy.mirrors.kernel.org. [147.75.48.161]) by mx.google.com with ESMTPS id h15-20020a170902680f00b001e10a526a94si120969plk.129.2024.04.10.16.06.06 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 10 Apr 2024 16:06:07 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-139502-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161; Authentication-Results: mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=WtaJMplz; dkim=neutral (no key) header.i=@linutronix.de header.s=2020e header.b=MPTfS2Uj; arc=pass (i=1 spf=pass spfdomain=linutronix.de dkim=pass dkdomain=linutronix.de dmarc=pass fromdomain=linutronix.de); spf=pass (google.com: domain of linux-kernel+bounces-139502-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) smtp.mailfrom="linux-kernel+bounces-139502-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sy.mirrors.kernel.org (Postfix) with ESMTPS id 008A4B269D6 for ; Wed, 10 Apr 2024 22:57:33 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id F38AB1F954; Wed, 10 Apr 2024 22:47:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linutronix.de header.i=@linutronix.de header.b="WtaJMplz"; dkim=permerror (0-bit key) header.d=linutronix.de header.i=@linutronix.de header.b="MPTfS2Uj" Received: from galois.linutronix.de (Galois.linutronix.de [193.142.43.55]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 06875137777 for ; Wed, 10 Apr 2024 22:47:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=193.142.43.55 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712789260; cv=none; b=CuQkwl85ENoE8/Er8ULSm//ij4Qvid3TKH9os8NJlk+4gFVXq7z013F7pbhTbur7KGaML+judeEN8KiKDN3HXxw8K3pbdTCPzkrbISo+2JIafWkj9vP01jtuhw+oEVGTi1jhJtLCdG3D43nOndVyWxkTMekm2tVy0GB1aSWRu+I= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712789260; c=relaxed/simple; bh=6+oMP4yhaB9snN9L9lq4a4BLEyD2V/UbAwBhcb8bH04=; h=Message-ID:From:To:Cc:Subject:References:MIME-Version: Content-Type:Date; b=g/v8Oufdg0r8FTqxRA66vF36U0odAoOdDO1qEqWvGW3uAu/LgTpYTsmozCRm+nz9EV4Eydwpxh5sJNRLbfjiBpcQ6AufJIk7n8vVyGYfd5wMijjh19wwy/3Wr6C9VmtXZ55S53/DY6PedSDck8FHzpR+NllJ2UW7YQ4RidMwkS0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linutronix.de; spf=pass smtp.mailfrom=linutronix.de; dkim=pass (2048-bit key) header.d=linutronix.de header.i=@linutronix.de header.b=WtaJMplz; dkim=permerror (0-bit key) header.d=linutronix.de header.i=@linutronix.de header.b=MPTfS2Uj; arc=none smtp.client-ip=193.142.43.55 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linutronix.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linutronix.de Message-ID: <20240410165553.944340294@linutronix.de> DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1712789257; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: references:references; bh=kAkqE4xY1tktojAkL7ohXROJF3Nq4zDNsVY2SgjGpGc=; b=WtaJMplzdx7bJXkeFHMbPAf76h1SeUpmsI11TmRfyPLfEYUY7faWJh3xSbrGMjxPix6Mqt U6b3oRqKFVWkd7umtHxX8fX1baV5H0/LJnZXdhYG8Aa5mCQTCDx9VFowzA5vUQXK1RfHCC sJ1gJMYsARzinQpHzcP3vCUl/lmrgEcxn7UPHgfCBJiDsxdQ+G7TP8Y0mQouAzsNzezkvk ScawoVFH3qpV2F54GoxG8UC867hg5thGhnWfceNroM/avu1S7StLo/gIjNh2bPtoa1HlkB bDNn859O+5lTxWkV1h9ygj9f4wcB2QmZZ2SAIbKzxkEGXjVKg+hPu3jdF59Gcg== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1712789257; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: references:references; bh=kAkqE4xY1tktojAkL7ohXROJF3Nq4zDNsVY2SgjGpGc=; b=MPTfS2UjRZCPYADveZKuYYz6s3QqCzNS3eD1D/mgQV8kzwUFYDmbCP1gZxj6frL35rJfId BSvYoSDIz4JjpFDw== From: Thomas Gleixner To: LKML Cc: Anna-Maria Behnsen , Frederic Weisbecker , John Stultz , Peter Zijlstra , Ingo Molnar , Stephen Boyd , Eric Biederman , Oleg Nesterov Subject: [patch V2 48/50] posix-timers: Cleanup SIG_IGN workaround leftovers References: <20240410164558.316665885@linutronix.de> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Date: Thu, 11 Apr 2024 00:47:36 +0200 (CEST) Now that ignored posix timer signals are requeued and the timers are rearmed on signal delivery the workaround to keep such timers alive and self rearm them is not longer required. Remove the relevant hacks and the not longer required return values from the related functions. The alarm timer workarounds will be cleaned up in a separate step. Signed-off-by: Thomas Gleixner --- include/linux/posix-timers.h | 2 - kernel/signal.c | 3 - kernel/time/alarmtimer.c | 37 +++--------------------- kernel/time/posix-cpu-timers.c | 18 ++--------- kernel/time/posix-timers.c | 63 +++-------------------------------------- kernel/time/posix-timers.h | 2 - 6 files changed, 18 insertions(+), 107 deletions(-) --- a/include/linux/posix-timers.h +++ b/include/linux/posix-timers.h @@ -111,7 +111,7 @@ static inline void posix_cputimers_rt_wa void posixtimer_rearm_itimer(struct task_struct *p); bool posixtimer_init_sigqueue(struct sigqueue *q); -int posixtimer_send_sigqueue(struct k_itimer *tmr); +void posixtimer_send_sigqueue(struct k_itimer *tmr); bool posixtimer_deliver_signal(struct kernel_siginfo *info); void posixtimer_free_timer(struct k_itimer *timer); --- a/kernel/signal.c +++ b/kernel/signal.c @@ -1968,7 +1968,7 @@ static inline struct task_struct *posixt return t; } -int posixtimer_send_sigqueue(struct k_itimer *tmr) +void posixtimer_send_sigqueue(struct k_itimer *tmr) { struct sigqueue *q = &tmr->sigq; int sig = q->info.si_signo; @@ -2053,7 +2053,6 @@ int posixtimer_send_sigqueue(struct k_it unlock_task_sighand(t, &flags); ret: rcu_read_unlock(); - return 0; } static inline void posixtimer_sig_ignore(struct task_struct *tsk, struct sigqueue *q) --- a/kernel/time/alarmtimer.c +++ b/kernel/time/alarmtimer.c @@ -198,27 +198,16 @@ static enum hrtimer_restart alarmtimer_f struct alarm *alarm = container_of(timer, struct alarm, timer); struct alarm_base *base = &alarm_bases[alarm->type]; unsigned long flags; - int ret = HRTIMER_NORESTART; - int restart = ALARMTIMER_NORESTART; spin_lock_irqsave(&base->lock, flags); alarmtimer_dequeue(base, alarm); spin_unlock_irqrestore(&base->lock, flags); if (alarm->function) - restart = alarm->function(alarm, base->get_ktime()); - - spin_lock_irqsave(&base->lock, flags); - if (restart != ALARMTIMER_NORESTART) { - hrtimer_set_expires(&alarm->timer, alarm->node.expires); - alarmtimer_enqueue(base, alarm); - ret = HRTIMER_RESTART; - } - spin_unlock_irqrestore(&base->lock, flags); + alarm->function(alarm, base->get_ktime()); trace_alarmtimer_fired(alarm, base->get_ktime()); - return ret; - + return HRTIMER_NORESTART; } ktime_t alarm_expires_remaining(const struct alarm *alarm) @@ -567,30 +556,16 @@ static enum alarmtimer_type clock2alarm( * * Return: whether the timer is to be restarted */ -static enum alarmtimer_restart alarm_handle_timer(struct alarm *alarm, - ktime_t now) +static enum alarmtimer_restart alarm_handle_timer(struct alarm *alarm, ktime_t now) { - struct k_itimer *ptr = container_of(alarm, struct k_itimer, - it.alarm.alarmtimer); - enum alarmtimer_restart result = ALARMTIMER_NORESTART; + struct k_itimer *ptr = container_of(alarm, struct k_itimer, it.alarm.alarmtimer); unsigned long flags; spin_lock_irqsave(&ptr->it_lock, flags); - - if (posix_timer_queue_signal(ptr) && ptr->it_interval) { - /* - * Handle ignored signals and rearm the timer. This will go - * away once we handle ignored signals proper. Ensure that - * small intervals cannot starve the system. - */ - ptr->it_overrun += __alarm_forward_now(alarm, ptr->it_interval, true); - ++ptr->it_signal_seq; - ptr->it_status = POSIX_TIMER_ARMED; - result = ALARMTIMER_RESTART; - } + posix_timer_queue_signal(ptr); spin_unlock_irqrestore(&ptr->it_lock, flags); - return result; + return ALARMTIMER_NORESTART; } /** --- a/kernel/time/posix-cpu-timers.c +++ b/kernel/time/posix-cpu-timers.c @@ -595,21 +595,11 @@ static void cpu_timer_fire(struct k_itim */ wake_up_process(timer->it_process); cpu_timer_setexpires(ctmr, 0); - } else if (!timer->it_interval) { - /* - * One-shot timer. Clear it as soon as it's fired. - */ + } else { posix_timer_queue_signal(timer); - cpu_timer_setexpires(ctmr, 0); - } else if (posix_timer_queue_signal(timer)) { - /* - * The signal did not get queued because the signal - * was ignored, so we won't get any callback to - * reload the timer. But we need to keep it - * ticking in case the signal is deliverable next time. - */ - posix_cpu_timer_rearm(timer); - ++timer->it_signal_seq; + /* Disable oneshot timers */ + if (!timer->it_interval) + cpu_timer_setexpires(ctmr, 0); } } --- a/kernel/time/posix-timers.c +++ b/kernel/time/posix-timers.c @@ -300,10 +300,9 @@ bool posixtimer_deliver_signal(struct ke return ret; } -int posix_timer_queue_signal(struct k_itimer *timr) +void posix_timer_queue_signal(struct k_itimer *timr) { enum posix_timer_state state = POSIX_TIMER_DISARMED; - int ret; lockdep_assert_held(&timr->it_lock); @@ -313,9 +312,7 @@ int posix_timer_queue_signal(struct k_it } timr->it_status = state; - ret = posixtimer_send_sigqueue(timr); - /* If we failed to send the signal the timer stops. */ - return ret > 0; + posixtimer_send_sigqueue(timr); } /* @@ -328,62 +325,12 @@ int posix_timer_queue_signal(struct k_it static enum hrtimer_restart posix_timer_fn(struct hrtimer *timer) { struct k_itimer *timr = container_of(timer, struct k_itimer, it.real.timer); - enum hrtimer_restart ret = HRTIMER_NORESTART; unsigned long flags; spin_lock_irqsave(&timr->it_lock, flags); - - if (posix_timer_queue_signal(timr)) { - /* - * The signal was not queued due to SIG_IGN. As a - * consequence the timer is not going to be rearmed from - * the signal delivery path. But as a real signal handler - * can be installed later the timer must be rearmed here. - */ - if (timr->it_interval != 0) { - ktime_t now = hrtimer_cb_get_time(timer); - - /* - * FIXME: What we really want, is to stop this - * timer completely and restart it in case the - * SIG_IGN is removed. This is a non trivial - * change to the signal handling code. - * - * For now let timers with an interval less than a - * jiffie expire every jiffie and recheck for a - * valid signal handler. - * - * This avoids interrupt starvation in case of a - * very small interval, which would expire the - * timer immediately again. - * - * Moving now ahead of time by one jiffie tricks - * hrtimer_forward() to expire the timer later, - * while it still maintains the overrun accuracy - * for the price of a slight inconsistency in the - * timer_gettime() case. This is at least better - * than a timer storm. - * - * Only required when high resolution timers are - * enabled as the periodic tick based timers are - * automatically aligned to the next tick. - */ - if (IS_ENABLED(CONFIG_HIGH_RES_TIMERS)) { - ktime_t kj = TICK_NSEC; - - if (timr->it_interval < kj) - now = ktime_add(now, kj); - } - - timr->it_overrun += hrtimer_forward(timer, now, timr->it_interval); - ret = HRTIMER_RESTART; - ++timr->it_signal_seq; - timr->it_status = POSIX_TIMER_ARMED; - } - } - - unlock_timer(timr, flags); - return ret; + posix_timer_queue_signal(timr); + spin_unlock_irqrestore(&timr->it_lock, flags); + return HRTIMER_NORESTART; } static struct pid *good_sigevent(sigevent_t * event) --- a/kernel/time/posix-timers.h +++ b/kernel/time/posix-timers.h @@ -42,7 +42,7 @@ extern const struct k_clock clock_proces extern const struct k_clock clock_thread; extern const struct k_clock alarm_clock; -int posix_timer_queue_signal(struct k_itimer *timr); +void posix_timer_queue_signal(struct k_itimer *timr); void common_timer_get(struct k_itimer *timr, struct itimerspec64 *cur_setting); int common_timer_set(struct k_itimer *timr, int flags,