From mboxrd@z Thu Jan 1 00:00:00 1970 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 EF325CD484A for ; Wed, 4 Sep 2024 14:45:04 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 85C4C6B042B; Wed, 4 Sep 2024 10:45:04 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 80C806B042C; Wed, 4 Sep 2024 10:45:04 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 6D50D6B042D; Wed, 4 Sep 2024 10:45:04 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 4EF4B6B042B for ; Wed, 4 Sep 2024 10:45:04 -0400 (EDT) Received: from smtpin21.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id EDB241C6197 for ; Wed, 4 Sep 2024 14:45:03 +0000 (UTC) X-FDA: 82527328086.21.4D815DA Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf02.hostedemail.com (Postfix) with ESMTP id 0B38880015 for ; Wed, 4 Sep 2024 14:45:00 +0000 (UTC) Authentication-Results: imf02.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=vMaeIFXV; spf=pass (imf02.hostedemail.com: domain of rafael@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=rafael@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1725461052; a=rsa-sha256; cv=none; b=JRvliRNENTRwSl47AkCSBHhaVtzFOZhReLcIpGA0056hSdd1sJfXtcOZSflSIh3MP26f7H fDc8ZeYSXUbKq9P4J/aZ1PAWL8jlKa7/zBhtU6C0giuOqjU8mlV6GjycwCv3JfKCVW3nKU pTm+okQJiMjH5FFRraVB5zD+6PPPMW8= ARC-Authentication-Results: i=1; imf02.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=vMaeIFXV; spf=pass (imf02.hostedemail.com: domain of rafael@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=rafael@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1725461052; 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:in-reply-to:references:references:dkim-signature; bh=Uu0nBMEsWQ03xP8pcUVRVc+x4Imnk32z5L6ejaG1Ms8=; b=lg1SPCoP8Qv4+qxQCTUaMMcucAr1up3W1pXloxIV35rWba3v2PCjYJtBlzg4QiaOjXlbuk 8PFTZI/Xu2YbQOSxOi1Hu13byW/hBq0ltrdxDdhaAXzB6of2hGhU6F75wnILdl+kUAAbIB cssjl3e+gvbihL5U4ohBjSvYgB7sg10= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id B825D5C57E6 for ; Wed, 4 Sep 2024 14:44:56 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D5BF0C4CECA for ; Wed, 4 Sep 2024 14:44:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1725461099; bh=ArAPCxI1Kj/v1qq50FpqqDgsTk6eZ/UAqDvPFj1T2/8=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=vMaeIFXVmG5TwnYwCPIyHpt12OpEsK/pI0cxjWT9uUKQHp1pWc4ZBU/WcIJSR0DZs QzSqsGdkj+OuQ9wXdNsaJvxdRuzx7e3Rz9elyL9vkhWYfhdY3U6Zer/Cy/8idp3bGL muQdcKV7rfHrAYEc67s94wy9Lkprb7fg0VbpeKve+qiPIbqPRDqhlIDDDj+MByqxQf Y1zUwjFiV5emnv2A9qfHNjTv1ELkKIhBAjN9WVVJTS0ZaVhTLaQyh5YkRMzHiDywrR XRVkQT8R+O7CVZ8tJC4KewPEtHaaBpJ+a8tPYK2FYZ9y7WDWjV/gQF9x/ubByzdmdx vd6yrc+Yj9Vdw== Received: by mail-oi1-f180.google.com with SMTP id 5614622812f47-3df0b748e08so499601b6e.0 for ; Wed, 04 Sep 2024 07:44:59 -0700 (PDT) X-Forwarded-Encrypted: i=1; AJvYcCWmm1Fd7jcOIQGjgR9YFuHMvt+FM0BfmeJqZYTzSMmLpam9lP9wPKFXNW05WPoX1UWiN8uwhQskTA==@kvack.org X-Gm-Message-State: AOJu0YxX6I0GK09M4o4vo7xq0+49C17DLWfSjy3qfFyM4p2UXCnP5zKz allUH93+kVrgA1Ij8ErE6zjVam3uwEp8MVQeaW3nfNhbOGyZI3fz11PfPgDpREIoP1Fjt4Jfdly Xcb+RQRiLJahZIAyUOh8vIdxadH8= X-Google-Smtp-Source: AGHT+IEv8U+fDbCjLqUFxz4//WKn6tB39IzLuCQnJWmLkgmLJZnbV32x+Njonp88P0xuQvY07t3qf0x9Ck/Ekt8R4EI= X-Received: by 2002:a05:6808:138a:b0:3db:50ed:e121 with SMTP id 5614622812f47-3e01315d6camr1093652b6e.0.1725461099193; Wed, 04 Sep 2024 07:44:59 -0700 (PDT) MIME-Version: 1.0 References: <20240904-devel-anna-maria-b4-timers-flseep-v1-0-e98760256370@linutronix.de> In-Reply-To: <20240904-devel-anna-maria-b4-timers-flseep-v1-0-e98760256370@linutronix.de> From: "Rafael J. Wysocki" Date: Wed, 4 Sep 2024 16:44:47 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 00/15] timers: Cleanup delay/sleep related mess To: Anna-Maria Behnsen Cc: Frederic Weisbecker , Thomas Gleixner , Jonathan Corbet , linux-kernel@vger.kernel.org, Len Brown , "Rafael J. Wysocki" , 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 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Stat-Signature: tr3a45nsqehzk8zo4oeq6yko9bds1a9i X-Rspamd-Queue-Id: 0B38880015 X-Rspam-User: X-Rspamd-Server: rspam10 X-HE-Tag: 1725461100-5625 X-HE-Meta: U2FsdGVkX1+DarFZsjRXPxGRegmT4Yr/ITmpgjpgbu/RS5xhZrbgzznKeISTrnurniBylcI2p2PDR30Y1yLK5shhnMXWFASGq2wt6QQBIV1KyUnP3x0dShhx3/qfRlOnOJw9rEJvnZfbTCE+4mI409QbcdKIfqghzsZC7NgDp3Qj1LX1aZoRqasEX7W2f/dI4rYzC2Z7r8Vlrr0gX5rn/ygqisICxWl+np+bEzH7hWNfI9t8f3tkW5/I59ClCHrHPBn+eJCCi7WaN1EmIsR0J5eYTNl+gzIotXSmCN0FIq7w41QglFtURFudRRuEJTWuf4vZrODN9KGxTeXdzauuruSy5V6GvFhWco431GeWmgxmcGlQxJisHHFZ1uNOc6lCLKsEFBUb2iMsriW8WPOfvyaNHij0CRIogbmiNdvyXlzy6+DtKgh7kilBYoEcRM7F4coYlZdZfusFOhNrxxYpI/6vDsRXlUy73aKooGN9L2L/yPP6yMuSro1193cjoPY1bUKvCFd6Gv5rNu2yD5jHRVPHtg6a/vzCgInLaLhNC70E6kEum7yaOOWgXw2c+fHOAagsGb9jDsdJCQqAT+Pk0c4CJsPxye5SlmaHeUS4NatK8gWTgcWv+ZmYL1ByQ0mRsVfSr90V64+e/MkcpkTaqqn35QQlHXZMd6bBfsFewE2UKvgrqQ5hmS+OCE2782qlvODRz4R1NZQZJEzJemFIy7wxNxJVDovuzxnXYmGzlAF8voI0Fyu1hS4agowOQLej1nMIBlqmIc/mZRYylZqd2M/zs/Cx6w9/SWH4SPsF7iZ9qUtl1i7o3lYJx386XiMbHJPyGCmF3EoL/hv+30yCJptA3Gp4YVhNBpv71c3W/F2SgaqOc2CME190i54GBE0Teipze6+j1kyUXdvwv2ugj0VPqDEHLtpMxMZ44/zz47PBfPa05gQNZDvMleg6tfftKdKehvsfgkAWFcaPEc2 EK70UcTa rcIPJuFN+Wmz5/aAvGEOAKPaB9iIVinMx1Sazj81khweDJfEC2gthcqQocYa3d60zRNdHoSJbdV5oVOZIyFUiNrwIGv8TJmMwdEU4wFtr7uq4h2Gab0NigWWikzDMdvhGvrqd3dHWoRXNeRn2Dr4YMrkYOFYi0t0rehTUoFt3tSTn0k61lLBV/mntM2W3l07GsTPG89xZ9GlPnBbq92dgs1cF7IpYgfhNbp7va7seuwQVrzVKcN+/ng6GHudRkxNVwHDJnULzzF2mwFCXDpFj68ZOynEdEKTW2DAmk49TSbufiSLdbXGQh8em3mgl5Z5rDIAatsjDjZwgSHOnHsuz1OAj6VlMSPeTfUQfVqfVE2J8akrNhj3Mj6GLyYfq4HpEBo9dGFIATyfoHMo= 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: On Wed, Sep 4, 2024 at 3:05=E2=80=AFPM Anna-Maria Behnsen wrote: > > 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 th= e > following things: > > - Minor changes (naming and typo fixes) > > - Split out all timeout and sleep related functions from hrtimer.c and ti= mer.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 outdat= ed > 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.gi= t 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 > > --- > 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 functio= ns > timers: Adjust flseep() to reflect reality > mm/damon/core: Use generic upper bound recommondation for usleep_ra= nge() > timers: Add a warning to usleep_range_state() for wrong order of ar= guments > 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 docum= entation > powerpc/rtas: Use fsleep() to minimize additional sleep duration > media: anysee: Fix link to outdated sleep function documentation > timers/Documentation: Cleanup delay/sleep documentation I like the changes, so Acked-by: Rafael J. Wysocki for the series.