From patchwork Tue Oct 25 13:58:41 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Anna-Maria Behnsen X-Patchwork-Id: 10798 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp1024517wru; Tue, 25 Oct 2022 07:02:51 -0700 (PDT) X-Google-Smtp-Source: AMsMyM7bb+YhD4K6n1mZqbfevJzkOiTF2womd3rvL15B6z1DXjH2bCD7k2n76/R9tkk/SkvWBwF+ X-Received: by 2002:a17:903:2285:b0:185:44df:d916 with SMTP id b5-20020a170903228500b0018544dfd916mr38775240plh.120.1666706571090; Tue, 25 Oct 2022 07:02:51 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1666706571; cv=none; d=google.com; s=arc-20160816; b=CRDlRC75CwjG50uX9OdXUTETdIQb1ceuBQhoovyYB8m6Sl32/Sb5XLOyCVz/Im260l rO5vTZjzayUSHJ9+MifdGOJJHGgWDAc42UdmpImeGDXwn/RJdTtysw6vhXB+TMSTMuxh RDrF5gY/reBbWEixY8qBZVX8xXMwfWoO4LG1GxHrchKC2MCvQYlh8cXFf8QHjqdCI8D8 zWnyHryH8g4+O8Sss/hM8EOU20+eSKvMXCdibix14GbIEZ+HUIe6Krb/RoSgrnQzhYV7 77DaeqzJEHWPl1Luy+7AnX9RtNApIpM6pNwbssFM6jfp/dZYbCs+CCvdN1ZoJEQzovIl 03Sg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:dkim-signature :dkim-signature:from; bh=o6uqNXLi9fS/j/t+vRTXJBiE22APnO3iPTi53EIarr4=; b=maYo+nFo4oMQV10sx+R7zHRHjmEKRwfyI1AwRXcTnlANCcjUl3WWH+DvDJKdhaPICM wxQXHYmho5U8zwN9Y9rR5L1uUNVKK1RVvPKcGK+Gy8FqB2SOpNqiIcJ98VDUez5YZqFX 9cjlkdDAzT+Jqj5slEiTqucmFmOoVNShic5X6N7zqaGJ1O7hufdl9qfYmsmY2iYBTU1L voq5Pq0A/3neQdWaH1iFh5lIcg/QWvp2Mwb1CtSXsHfWG+tmnECoJoYkJOZtkSUs16PE PVpcwxOToqR6vW8yTo8eZmuVWP/y60sVm4tDBl78Qio/XyHK42RuZ+7SNLwSMuMSnR6j 2bcQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=IADpQrO9; dkim=neutral (no key) header.i=@linutronix.de header.s=2020e header.b=8eofvPNN; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id lx2-20020a17090b4b0200b001ed40b70436si4188955pjb.155.2022.10.25.07.02.35; Tue, 25 Oct 2022 07:02:51 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=IADpQrO9; dkim=neutral (no key) header.i=@linutronix.de header.s=2020e header.b=8eofvPNN; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232547AbiJYOAN (ORCPT + 99 others); Tue, 25 Oct 2022 10:00:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47618 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231811AbiJYN7a (ORCPT ); Tue, 25 Oct 2022 09:59:30 -0400 Received: from galois.linutronix.de (Galois.linutronix.de [193.142.43.55]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 884D1BDF; Tue, 25 Oct 2022 06:59:26 -0700 (PDT) From: Anna-Maria Behnsen DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1666706365; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=o6uqNXLi9fS/j/t+vRTXJBiE22APnO3iPTi53EIarr4=; b=IADpQrO9FsdT2Wos+BkvFJ9hjjdCh4J+cOqRGOoYjt3bnRm3xDgwNx89+L80e+paMSqk2S SvQHqauzMLFnVh/2rsm27Lv9IQSDZLlIRZILg0M20RvYZcBw4HFGP1VwAmo4y4+wN+OSVg LI05beGqz/QfdGGNxfsn7oXrF7hqw2VhHXdEVm9xFjx8P8ZXWiz3bBIbi8z5OoYiXgwA96 QSCHO0iQJSYE2PiZfinnps6DexqXxPexByDGMjTmaIUwTRQ/+/2PVgm57jMKO8OoIdX9Co eP7PeXKJ5eaI8eCmj+CMpqI9HQLsWgbvpbgg4L7Xw2zG6PJH1oVv1HP7+pBTuA== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1666706365; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=o6uqNXLi9fS/j/t+vRTXJBiE22APnO3iPTi53EIarr4=; b=8eofvPNNbQD92dcU/AsphDAJ7l/BqOxnIQ7vR6FpjpkbEt7FENM3a0/xo/Yqp6+oniYnu8 k7KMGh2fs+K9qjCQ== To: linux-kernel@vger.kernel.org Cc: Peter Zijlstra , John Stultz , Eric Dumazet , Thomas Gleixner , "Rafael J. Wysocki" , linux-pm@vger.kernel.org, Arjan van de Ven , "Paul E. McKenney" , Frederic Weisbecker , Rik van Riel , Anna-Maria Behnsen Subject: [PATCH v3 08/17] timer: Rename get_next_timer_interrupt() Date: Tue, 25 Oct 2022 15:58:41 +0200 Message-Id: <20221025135850.51044-9-anna-maria@linutronix.de> In-Reply-To: <20221025135850.51044-1-anna-maria@linutronix.de> References: <20221025135850.51044-1-anna-maria@linutronix.de> MIME-Version: 1.0 X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1747668509542171524?= X-GMAIL-MSGID: =?utf-8?q?1747668509542171524?= get_next_timer_interrupt() does more than simply getting the next timer interrupt. The timer bases are forwarded and also marked as idle when possible and the next timer interrupt information is required for this. To get not confused, rename function to a more descriptive name. No functional change. Signed-off-by: Anna-Maria Behnsen --- kernel/time/tick-internal.h | 4 ++-- kernel/time/tick-sched.c | 16 ++++++++-------- kernel/time/timer.c | 10 +++++++--- 3 files changed, 17 insertions(+), 13 deletions(-) diff --git a/kernel/time/tick-internal.h b/kernel/time/tick-internal.h index fcb2d45c2934..6f5f164506d5 100644 --- a/kernel/time/tick-internal.h +++ b/kernel/time/tick-internal.h @@ -168,8 +168,8 @@ static inline void timers_update_nohz(void) { } DECLARE_PER_CPU(struct hrtimer_cpu_base, hrtimer_bases); -extern void get_next_timer_interrupt(unsigned long basej, u64 basem, - struct timer_events *tevt); +extern void forward_and_idle_timer_bases(unsigned long basej, u64 basem, + struct timer_events *tevt); void timer_clear_idle(void); #define CLOCK_SET_WALL \ diff --git a/kernel/time/tick-sched.c b/kernel/time/tick-sched.c index 78f172d1f3d2..7f7bfe8b498d 100644 --- a/kernel/time/tick-sched.c +++ b/kernel/time/tick-sched.c @@ -802,11 +802,11 @@ static ktime_t tick_nohz_next_event(struct tick_sched *ts, int cpu) * Keep the periodic tick, when RCU, architecture or irq_work * requests it. * Aside of that check whether the local timer softirq is - * pending. If so its a bad idea to call get_next_timer_interrupt() - * because there is an already expired timer, so it will request - * immediate expiry, which rearms the hardware timer with a - * minimal delta which brings us back to this place - * immediately. Lather, rinse and repeat... + * pending. If so its a bad idea to call + * forward_and_idle_timer_bases() because there is an already + * expired timer, so it will request immeditate expiry, which + * rearms the hardware timer with a minimal delta which brings us + * back to this place immediately. Lather, rinse and repeat... */ if (rcu_needs_cpu() || arch_needs_cpu() || irq_work_needs_cpu() || local_timer_softirq_pending()) { @@ -823,7 +823,7 @@ static ktime_t tick_nohz_next_event(struct tick_sched *ts, int cpu) * disabled this also looks at the next expiring * hrtimer. */ - get_next_timer_interrupt(basejiff, basemono, &tevt); + forward_and_idle_timer_bases(basejiff, basemono, &tevt); tevt.local = min_t(u64, tevt.local, tevt.global); ts->next_timer = tevt.local; } @@ -838,7 +838,7 @@ static ktime_t tick_nohz_next_event(struct tick_sched *ts, int cpu) if (delta <= (u64)TICK_NSEC) { /* * Tell the timer code that the base is not idle, i.e. undo - * the effect of get_next_timer_interrupt(): + * the effect of forward_and_idle_timer_bases(): */ timer_clear_idle(); /* @@ -1141,7 +1141,7 @@ void tick_nohz_idle_retain_tick(void) { tick_nohz_retain_tick(this_cpu_ptr(&tick_cpu_sched)); /* - * Undo the effect of get_next_timer_interrupt() called from + * Undo the effect of forward_and_idle_timer_bases() called from * tick_nohz_next_event(). */ timer_clear_idle(); diff --git a/kernel/time/timer.c b/kernel/time/timer.c index 069478e65230..1990eb1b2cc9 100644 --- a/kernel/time/timer.c +++ b/kernel/time/timer.c @@ -1705,7 +1705,7 @@ static unsigned long next_timer_interrupt(struct timer_base *base) } /** - * get_next_timer_interrupt + * forward_and_idle_timer_bases * @basej: base time jiffies * @basem: base time clock monotonic * @tevt: Pointer to the storage for the expiry values @@ -1713,9 +1713,13 @@ static unsigned long next_timer_interrupt(struct timer_base *base) * Stores the next pending local and global timer expiry values in the * struct pointed to by @tevt. If a queue is empty the corresponding field * is set to KTIME_MAX. + * + * If required, base->clk is forwarded and base is also marked as + * idle. Idle handling of timer bases is allowed only to be done by + * CPU itself. */ -void get_next_timer_interrupt(unsigned long basej, u64 basem, - struct timer_events *tevt) +void forward_and_idle_timer_bases(unsigned long basej, u64 basem, + struct timer_events *tevt) { unsigned long nextevt, nextevt_local, nextevt_global; struct timer_base *base_local, *base_global;