From patchwork Wed Sep 4 13:04:50 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Anna-Maria Behnsen X-Patchwork-Id: 13790827 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0F972CD37B4 for ; Wed, 4 Sep 2024 13:05:37 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 7EB8C6B02CB; Wed, 4 Sep 2024 09:05:37 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 772CA6B02CD; Wed, 4 Sep 2024 09:05:37 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 5C6A86B02CB; Wed, 4 Sep 2024 09:05:37 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id 378AE6B02C8 for ; Wed, 4 Sep 2024 09:05:37 -0400 (EDT) Received: from smtpin27.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id C7C04C11D7 for ; Wed, 4 Sep 2024 13:05:36 +0000 (UTC) X-FDA: 82527077472.27.0DCA981 Received: from galois.linutronix.de (Galois.linutronix.de [193.142.43.55]) by imf10.hostedemail.com (Postfix) with ESMTP id CBD48C0023 for ; Wed, 4 Sep 2024 13:05:34 +0000 (UTC) Authentication-Results: imf10.hostedemail.com; dkim=pass header.d=linutronix.de header.s=2020 header.b=pKIctVY4; dkim=pass header.d=linutronix.de header.s=2020e header.b=E+eXvp9U; spf=pass (imf10.hostedemail.com: domain of anna-maria@linutronix.de designates 193.142.43.55 as permitted sender) smtp.mailfrom=anna-maria@linutronix.de; dmarc=pass (policy=none) header.from=linutronix.de ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1725455087; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding:in-reply-to: references:dkim-signature; bh=IcwSc34LvXMP/cT7noBpbT1jqfRVjScu2K4oFpyriwE=; b=Jp6rvXbIE2gpldXXtbOceZm+gqccur6TD5rD+JX8IbZL+XfdL8/GYrYz6ZYg97bW3gANiy YbMtf8Zr3a6EhB87Y+3H6U820kdtgA3ogbg3TQeVa5okjqNd7cHDcv8mjM2SOvcEQ+d2Pk /OxyXootfn8zUUe/IjBLkIvTnH4KLjE= ARC-Authentication-Results: i=1; imf10.hostedemail.com; dkim=pass header.d=linutronix.de header.s=2020 header.b=pKIctVY4; dkim=pass header.d=linutronix.de header.s=2020e header.b=E+eXvp9U; spf=pass (imf10.hostedemail.com: domain of anna-maria@linutronix.de designates 193.142.43.55 as permitted sender) smtp.mailfrom=anna-maria@linutronix.de; dmarc=pass (policy=none) header.from=linutronix.de ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1725455087; a=rsa-sha256; cv=none; b=GQ5eeb05dC8nXL0Bh1o63DF6gqg0sbU/V210GcH0KA1cabs6SKT6q0eVa1A80SEd12iiEj RGLxRnRFc5HV0IcZr623xD2rA/r2ypA3UEK1i7rxFDkIWaiTGaVroGDaxe+z0IR2HYTVqh IovgV8g6x47Q1lcylxjxMK+ewXoCDgs= From: Anna-Maria Behnsen DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1725455132; 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: content-transfer-encoding:content-transfer-encoding; bh=IcwSc34LvXMP/cT7noBpbT1jqfRVjScu2K4oFpyriwE=; b=pKIctVY4axfpzaDAkuW89nfEerNMQNXGr4oRrZz38wPjBOmrxBuAnOBWaEizzBAwauAjPh iSPCWacRZvzl6CCp1is+uBZAeRqXH7FmxMsCNUd2XW7fLBRd3R2v/gVjNOtuOAe97g20n4 iyKyv4M3SsAHlQyN0Bb5+/WPEPJl38gvPLOQnPCVAYjWpdHo54oO5T6wTZVXJJDlbqXkkE 3oyAKI4DEAwwq/8zn1lI0pDDGpx/h1p9ThiWQxIDIVUHt673XGvyw0iUaWqSDLv7yLyqpo Anfa2HXMo+/DXNdDrHE8hi9N5LqhVKzPFFuFJBaC1IHamnKd3WZ2Kp5fjH9bRQ== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1725455132; 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: content-transfer-encoding:content-transfer-encoding; bh=IcwSc34LvXMP/cT7noBpbT1jqfRVjScu2K4oFpyriwE=; b=E+eXvp9UB9r1HGGHMLtT0w+geizfWK2j8vOahsM5RXh4xyoJXjN+XNIwwk0C+aPRWMYALP cm7NexPhzPCUw2DQ== Subject: [PATCH 00/15] timers: Cleanup delay/sleep related mess Date: Wed, 04 Sep 2024 15:04:50 +0200 Message-Id: <20240904-devel-anna-maria-b4-timers-flseep-v1-0-e98760256370@linutronix.de> MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAPJa2GYC/x3NQQqDMBBG4avIrDsQ07QkvUrpYtTfdkBTmYgUx Ls3uPw27+1UYIpCj2Ynw6ZFv7mivTTUfyS/wTpUk3c+uOQCD9gwseQsPIupcBd41RlWeJwKsHC 8xltqfZfuKVLtLIZRf+fj+TqOPzF6qSBzAAAA To: Frederic Weisbecker , Thomas Gleixner , Jonathan Corbet Cc: linux-kernel@vger.kernel.org, Len Brown , "Rafael J. Wysocki" , Anna-Maria Behnsen , Peter Zijlstra , SeongJae Park , Andrew Morton , damon@lists.linux.dev, linux-mm@kvack.org, Arnd Bergmann , linux-arch@vger.kernel.org, Heiner Kallweit , "David S. Miller" , Andy Whitcroft , Joe Perches , Dwaipayan Ray , Liam Girdwood , Mark Brown , Andrew Lunn , Jaroslav Kysela , Takashi Iwai , netdev@vger.kernel.org, linux-sound@vger.kernel.org, Michael Ellerman , Nathan Lynch , linuxppc-dev@lists.ozlabs.org, Mauro Carvalho Chehab , linux-media@vger.kernel.org X-Rspamd-Server: rspam03 X-Rspam-User: X-Rspamd-Queue-Id: CBD48C0023 X-Stat-Signature: 9bodig6ksxjeafaihwj5dt8acffs3cr5 X-HE-Tag: 1725455134-269261 X-HE-Meta: U2FsdGVkX19wIgyJsAQzT/qElpTpjGZ6LGoYWgVCHrs2dw50Q3OB5y5DNgDr0d54Z2KGD8FK7USDLEcZX4ZC4AQFEjVpNsHD2caRc9/nEM8TieOyXoBxhJymRvvSL8eOjBWmO4zA/v9F8pBBijiReIJ4gqKuanrMzlPhiYaTuEIapTZvLUGXx00Jjcms2sctPXLnldRSLpq+tNAoRfL9T0fAnSnQrxCFcbFYFCZHuL+LuQL4KakcClTemxJdazlgpatjSbGsIZkwB81iW9b9EVEzM2tpSZ0yZg5VLmk0kiSS8euSaxunHwQOOzKXBqruRcREsGg+TWSejLu+4OqkoNrK6Z6d8V3DPX5+GBLibtqe738uE4G9+D16o+INRMp273308aVmW18tXkLnBTPfcH1xYp7EgYrEMjbA+kys63oPHKLoR4IrDEDrQOJpmXMHa2C5pVstsPYsWTS1BHcfy5RB/cVW0yIv/aENGiI1RLjbf8nhMUJxxZ5MJXkqB73u+BY4mdjREkDw+WhQB+yky8uDuAkrCvKMTGynCcPiFfZQB3+sio6470FwOU1stYxjh18vcYGUhdjvlZF4UUbXVLMmSyedDAGG7p7DgjspUgh7j58qmsQgwMX5p6g49yr+Ac/aQKeUXIbjMdSbggCoOJoC79cMufeEjtqddJIUNnx2UsdsAptMZtCKt+CYzyr0fKvIne6JLgQDjqvwTb4jrcNJkFm97cxkSiI2tu33AMcNgi81Ia/IeeS3gv89bKg7WPC+bx6sd++JM52HSTpwywa6EHLGlT5MKyk+qIX0w30edZUG3xQBg+sNnDCSl/3ICxwu9ljUJNuExKFFvZ4JRUs7+8o5ahIQjGdS7scatmu26LGzBdaCU8c8wPNmuBEYFktYduatynZBdlwLKqZRIQ5Yd8DLBrb8DfgpRqt0ElHiylhLbZyjsyIYAig07mQP3b4KA/8FVpcMdK0FLeh s6l3NenU VYdgldpCHOAl9dtwWM79brH88B/d2egNebPRtO/PhQ2mITaDlhhxxpgVdr7tWSMjc7K/dXbsrci/lAgqhZMqYZeHT6tHWJU9pOl+pcdyH9re3dkcpdHrMdRz0SV7pE1bWKx/jSgSzKoRa47xkSyivUdpqQ9bwn3AfgdXOuRfnCTRKz7H2/9BKJb6lVG3zHpIRZE+69MXVahNGU9FlaiIDgcH8V8Ng8OsYIhR8BX1U75b2ESuCL/0LZQeo0UijiHl+JE97e6EF7J3K3T0= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Hi, a question about which sleeping function should be used in acpi_os_sleep() started a discussion and examination about the existing documentation and implementation of functions which insert a sleep/delay. The result of the discussion was, that the documentation is outdated and the implemented fsleep() reflects the outdated documentation but doesn't help to reflect reality which in turns leads to the queue which covers the following things: - Minor changes (naming and typo fixes) - Split out all timeout and sleep related functions from hrtimer.c and timer.c into a separate file - Update function descriptions of sleep related functions - Change fsleep() to reflect reality - Rework all comments or users which obviously rely on the outdated documentation as they reference "Documentation/timers/timers-howto.rst" - Last but not least (as there are no more references): Update the outdated documentation and move it into a file with a self explaining file name The queue is available here and applies on top of tip/timers/core: git://git.kernel.org/pub/scm/linux/kernel/git/anna-maria/linux-devel.git timers/misc Cc: linux-kernel@vger.kernel.org Cc: Len Brown Cc: Rafael J. Wysocki To: Frederic Weisbecker To: Thomas Gleixner To: Jonathan Corbet Signed-off-by: Anna-Maria Behnsen Thanks, Anna-Maria Acked-by: Rafael J. Wysocki --- Anna-Maria Behnsen (15): timers: Rename next_expiry_recalc() to be unique cpu: Use already existing usleep_range() Comments: Fix wrong singular form of jiffies timers: Move *sleep*() and timeout functions into a separate file timers: Rename sleep_idle_range() to sleep_range_idle() timers: Update function descriptions of sleep/delay related functions timers: Adjust flseep() to reflect reality mm/damon/core: Use generic upper bound recommondation for usleep_range() timers: Add a warning to usleep_range_state() for wrong order of arguments checkpatch: Remove broken sleep/delay related checks regulator: core: Use fsleep() to get best sleep mechanism iopoll/regmap/phy/snd: Fix comment referencing outdated timer documentation powerpc/rtas: Use fsleep() to minimize additional sleep duration media: anysee: Fix link to outdated sleep function documentation timers/Documentation: Cleanup delay/sleep documentation Documentation/admin-guide/media/vivid.rst | 2 +- Documentation/dev-tools/checkpatch.rst | 6 - Documentation/timers/delay_sleep_functions.rst | 122 +++++++ Documentation/timers/index.rst | 2 +- Documentation/timers/timers-howto.rst | 115 ------- .../sp_SP/scheduler/sched-design-CFS.rst | 2 +- MAINTAINERS | 2 + arch/arm/mach-versatile/spc.c | 2 +- arch/m68k/q40/q40ints.c | 2 +- arch/powerpc/kernel/rtas.c | 21 +- arch/x86/kernel/cpu/mce/dev-mcelog.c | 2 +- drivers/char/ipmi/ipmi_ssif.c | 2 +- drivers/dma-buf/st-dma-fence.c | 2 +- drivers/gpu/drm/i915/gem/i915_gem_wait.c | 2 +- drivers/gpu/drm/i915/gt/selftest_execlists.c | 4 +- drivers/gpu/drm/i915/i915_utils.c | 2 +- drivers/gpu/drm/v3d/v3d_bo.c | 2 +- drivers/isdn/mISDN/dsp_cmx.c | 2 +- drivers/media/usb/dvb-usb-v2/anysee.c | 6 +- drivers/net/ethernet/marvell/mvmdio.c | 2 +- drivers/regulator/core.c | 33 +- fs/xfs/xfs_buf.h | 2 +- include/asm-generic/delay.h | 46 ++- include/linux/delay.h | 79 ++++- include/linux/iopoll.h | 24 +- include/linux/jiffies.h | 2 +- include/linux/phy.h | 7 +- include/linux/regmap.h | 18 +- include/linux/timekeeper_internal.h | 2 +- kernel/cpu.c | 2 +- kernel/time/Makefile | 2 +- kernel/time/alarmtimer.c | 2 +- kernel/time/clockevents.c | 2 +- kernel/time/hrtimer.c | 122 +------ kernel/time/posix-timers.c | 4 +- kernel/time/sleep_timeout.c | 363 +++++++++++++++++++++ kernel/time/timer.c | 210 +----------- lib/Kconfig.debug | 2 +- mm/damon/core.c | 5 +- net/batman-adv/types.h | 2 +- scripts/checkpatch.pl | 38 --- sound/soc/sof/ops.h | 6 +- 42 files changed, 668 insertions(+), 607 deletions(-)