From patchwork Thu Oct 8 15:46:38 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 11823385 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id B60A71580 for ; Thu, 8 Oct 2020 15:48:38 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 6CFBF20739 for ; Thu, 8 Oct 2020 15:48:38 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="N1AjhMSd" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6CFBF20739 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=arndb.de Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:To:From: Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender :Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=grXjfI4VAEyxAhfPDAPXHuMap5EzVGbLbxVgNKNGvNc=; b=N1AjhMSdVbWEz4xiLt6ita9Znp SDQnaYLtOsL1Kwuao55tR0hJCN7mcH+pOEpSCiiuwhYyLmds/FIjIQOutHp6cOJVEeB+3ub0aFeY7 iWesrnuFejSqgIzhxlta1KbUImoimrOq/ByH/s1T1YC6I4mbKAuwQsVOT9pBdk+IXyPMWE6A3r2dI 5Ug65oTnmucSxbqg2K5xMHa1kRpD9E+y8A0xhrpj8WP41JeAb1Tty22DPcp+QyLeILirsG6S3N8gz FquZFFAdzgT52balLqASh9NOA+mZMQRgeJWux/OdFYjkaRwAm+JXCyW6K1lCrmHwYIjAcECQmTdi1 9IJdlF6w==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kQY9X-0002YX-SS; Thu, 08 Oct 2020 15:48:24 +0000 Received: from mout.kundenserver.de ([217.72.192.74]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kQY9J-0002Su-1n for linux-arm-kernel@lists.infradead.org; Thu, 08 Oct 2020 15:48:15 +0000 Received: from localhost.localdomain ([192.30.34.233]) by mrelayeu.kundenserver.de (mreue107 [212.227.15.145]) with ESMTPA (Nemesis) id 1M7KKA-1kIpyQ2KzM-007kBr; Thu, 08 Oct 2020 17:47:15 +0200 From: Arnd Bergmann To: linux-kernel@vger.kernel.org Subject: [PATCH 00/13] Clean up legacy clock tick users Date: Thu, 8 Oct 2020 17:46:38 +0200 Message-Id: <20201008154651.1901126-1-arnd@arndb.de> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 X-Provags-ID: V03:K1:ToCg0sXzfof6vBLW0FBjjVXywAK/gLyirWECWdzB4HS8Fcsvfc3 KICej28tXkzfQ4JRGwmkABm7jfD+Q1lU1BJw7T0Lwpe5f+9kcKbUsHe20Hl/PaNGaJm7f64 vzJ+b6TvkgNmTNmKF4q0otuZPBYaKl04cVbqXwwhtpd4LjYBTPzdyr5BMwVWRMEtd03OY9O 9R1EBRW62aNmSfzzkZ7hQ== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1;V03:K0:XzC6G7gG190=:Cqm3PvhpJLmyzbv8maC2Qh dnY6yR+InSfpI2H7h2qKLgqXY0N/nDpx+LyKwsDfWCfy30pv1/X179z3M18wDXjMbMsUpzZ6F 4B5NNcDcFVMTUCUPlRfy2d3qzVZht/dx1zUdmL1xXlE9E6pFkyUCwoA1J8rrxWA6pnghYo8Ah 0QSFCJMZvvnYyDhAejrhQIiHSmxWuUyB1azRxHYtMm8V4BzxflkuuFqtzYNFyr5b7P1Avsz1e vjXufHhD9qmSPjaXhc5RcMXIkJAI+tb9RmveUbqRO0gARB+S5JRJAOPu55715IQVquUYc+Y5C N5gF9BoV3MEe9uywKU5G6DC9AoyVV+MojMMrN4HTf3AH28dnEbcQyqhAAvff+j+KTTMO4Hukl xIaBIKOo129/+r06dLt9e+ttV49J6xE04WgPvF5KAVvx7q18DrfiB7sQ+FWwkxcvLNlkgfqbV p3QRrIypBFfQvLQYH2yK74by+sk2YmiWpDsQOvIMimQy5W7CyoB4zE8YWuxKhPxD6hec9R/yc DqXtXjbeI22nK835/2cu3jL/ee7OYxq153PK5R+xpWaWMUEfgHODmzl2dQKvdlKKK0EUA6ude Kx5FnaKQM2AQen2RtvDCMpnrIY6mfmE7JyR1UYiwOBxq94oNhfYrqF51lwlEk+2IqfKP8p1pq 2Qpt//BK9nvrV4/feo5U6lbqA3jqVxd4+3SBbYvE4zeNuixTzu60EDSpJLikef7pvt3YItvJw F9NzxPuwnT+XRGwnAbbJtv/0NyKifs6ul5oBFvy4znZCAaP/bkNaai/drzAigtOY0YH+Cevib CIPjT954uXTsZoTu+YGOFhXj/r9QB5+3/FxlRKqwpWINpJmbBojGsvZ5XZYaHlnDgvrQbsx X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201008_114809_304344_44EDA51A X-CRM114-Status: GOOD ( 20.62 ) X-Spam-Score: 0.0 (/) X-Spam-Report: SpamAssassin version 3.4.4 on merlin.infradead.org summary: Content analysis details: (0.0 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at https://www.dnswl.org/, no trust [217.72.192.74 listed in list.dnswl.org] 0.0 SPF_NONE SPF: sender does not publish an SPF Record 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record -0.0 RCVD_IN_MSPIKE_H2 RBL: Average reputation (+2) [217.72.192.74 listed in wl.mailspike.net] X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Sam Creasey , Fenghua Yu , Linus Walleij , Tony Luck , Thomas Gleixner , linux-parisc@vger.kernel.org, Arnd Bergmann , linux-ia64@vger.kernel.org, Stephen Boyd , Helge Deller , Daniel Lezcano , Russell King , Finn Thain , "James E.J. Bottomley" , linux-m68k@lists.linux-m68k.org, Geert Uytterhoeven , linux-arm-kernel@lists.infradead.org, John Stultz , Philip Blundell , Greg Ungerer , Joshua Thompson Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org When I created the patch removing CONFIG_ARCH_GETTIMEOFFSET, I also had a look at CONFIG_GENERIC_CLOCKEVENTS, which is selected by most, but not all, platforms today, each of the ones that lack it doing the timer tick slightly differently. The cleanups here make the old platforms a bit more consistent, in multiple ways: - rather than selecting GENERIC_CLOCKEVENTS on modern platforms, select LEGACY_TIMER_TICK on the old ones. - Hide some more of the internal implementation and only provide a single common entry point for the timer tick, which also makes the behavior more consistent. - Remove the m68k and arm specific infrastructure pieces and call the common helper directly from each timer interrupt function. I tested the series on m68k with the qemu q800 target platform, both with the legacy_timer_tick() implementation and after converting that to a periodic clockevent driver. Arnd Arnd Bergmann (13): timekeeping: add CONFIG_LEGACY_TIMER_TICK ia64: convert to legacy_timer_tick ARM: rpc: use legacy_timer_tick parisc: use legacy_timer_tick m68k: coldfire: use legacy_timer_tick() m68k: split heartbeat out of timer function m68k: sun3/sun3c: use legacy_timer_tick m68k: m68328: use legacy_timer_tick() m68k: change remaining timers to legacy_timer_tick m68k: remove timer_interrupt() function timekeeping: remove xtime_update timekeeping: default GENERIC_CLOCKEVENTS to enabled [RFC] m68k: mac: convert to generic clockevent .../time/clockevents/arch-support.txt | 8 ++-- arch/alpha/Kconfig | 1 - arch/arc/Kconfig | 1 - arch/arm/Kconfig | 13 +----- arch/arm/include/asm/mach/time.h | 2 - arch/arm/kernel/time.c | 14 ------ arch/arm/mach-rpc/time.c | 2 +- arch/arm64/Kconfig | 1 - arch/arm64/Kconfig.platforms | 1 - arch/c6x/Kconfig | 1 - arch/csky/Kconfig | 1 - arch/h8300/Kconfig | 1 - arch/hexagon/Kconfig | 1 - arch/ia64/Kconfig | 1 + arch/ia64/kernel/time.c | 36 ++++++--------- arch/m68k/68000/timers.c | 7 ++- arch/m68k/Kconfig.cpu | 34 +++++++++++--- arch/m68k/Kconfig.machine | 15 +++++++ arch/m68k/amiga/config.c | 11 +++-- arch/m68k/apollo/config.c | 11 +++-- arch/m68k/atari/config.c | 2 +- arch/m68k/atari/time.c | 9 ++-- arch/m68k/bvme6000/config.c | 9 ++-- arch/m68k/coldfire/Makefile | 32 ++++++++------ arch/m68k/coldfire/pit.c | 2 +- arch/m68k/coldfire/sltimers.c | 8 ++-- arch/m68k/coldfire/timers.c | 8 ++-- arch/m68k/hp300/time.c | 8 ++-- arch/m68k/hp300/time.h | 2 +- arch/m68k/include/asm/machdep.h | 12 +++-- arch/m68k/kernel/setup_mm.c | 2 +- arch/m68k/kernel/setup_no.c | 2 +- arch/m68k/kernel/time.c | 18 ++------ arch/m68k/mac/config.c | 6 +-- arch/m68k/mac/via.c | 44 +++++++++++++++---- arch/m68k/mvme147/config.c | 9 ++-- arch/m68k/mvme16x/config.c | 9 ++-- arch/m68k/q40/config.c | 2 +- arch/m68k/q40/q40ints.c | 10 ++--- arch/m68k/sun3/config.c | 4 +- arch/m68k/sun3/sun3ints.c | 3 +- arch/m68k/sun3x/time.c | 5 +-- arch/m68k/sun3x/time.h | 2 +- arch/microblaze/Kconfig | 1 - arch/mips/Kconfig | 1 - arch/nds32/Kconfig | 1 - arch/nios2/Kconfig | 1 - arch/openrisc/Kconfig | 1 - arch/parisc/Kconfig | 2 +- arch/parisc/kernel/time.c | 9 ++-- arch/powerpc/Kconfig | 1 - arch/riscv/Kconfig | 1 - arch/s390/Kconfig | 1 - arch/sh/Kconfig | 1 - arch/sparc/Kconfig | 1 - arch/um/Kconfig | 1 - arch/x86/Kconfig | 1 - arch/xtensa/Kconfig | 1 - include/linux/timekeeping.h | 5 +-- kernel/time/Kconfig | 9 +++- kernel/time/Makefile | 1 + kernel/time/tick-legacy.c | 37 ++++++++++++++++ kernel/time/timekeeping.c | 16 ------- kernel/time/timekeeping.h | 1 + 64 files changed, 242 insertions(+), 221 deletions(-) create mode 100644 kernel/time/tick-legacy.c Cc: Russell King Cc: Tony Luck Cc: Fenghua Yu Cc: Greg Ungerer Cc: Geert Uytterhoeven Cc: Finn Thain Cc: Philip Blundell Cc: Joshua Thompson Cc: Sam Creasey Cc: "James E.J. Bottomley" Cc: Helge Deller Cc: Thomas Gleixner Cc: Daniel Lezcano Cc: John Stultz Cc: Stephen Boyd Cc: Linus Walleij Cc: linux-kernel@vger.kernel.org Cc: linux-ia64@vger.kernel.org Cc: linux-parisc@vger.kernel.org Cc: linux-m68k@lists.linux-m68k.org Cc: linux-arm-kernel@lists.infradead.org Reviewed-by: Linus Walleij