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 8FB7DC4829E for ; Thu, 15 Feb 2024 10:50:28 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 207CC8D0025; Thu, 15 Feb 2024 05:50:28 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 1B90C8D0024; Thu, 15 Feb 2024 05:50:28 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 07E5C8D0025; Thu, 15 Feb 2024 05:50:27 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id E54F38D0024 for ; Thu, 15 Feb 2024 05:50:27 -0500 (EST) Received: from smtpin20.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 89BE2C0FBB for ; Thu, 15 Feb 2024 10:50:27 +0000 (UTC) X-FDA: 81793719294.20.F8C7F16 Received: from out-183.mta1.migadu.com (out-183.mta1.migadu.com [95.215.58.183]) by imf06.hostedemail.com (Postfix) with ESMTP id AA515180020 for ; Thu, 15 Feb 2024 10:50:25 +0000 (UTC) Authentication-Results: imf06.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b=Vm9wgtZz; dmarc=pass (policy=none) header.from=linux.dev; spf=pass (imf06.hostedemail.com: domain of muchun.song@linux.dev designates 95.215.58.183 as permitted sender) smtp.mailfrom=muchun.song@linux.dev ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1707994226; a=rsa-sha256; cv=none; b=A+sfDWNDxbCArsRiBVWcB4LI0W/fdqs3X2YrxS36NjosK4gbKTrbI1AN50q+PTOvWMnO0W BnocUfpUABarsILzMW4h7Kasji1H6zysJZDLJv64XzqK0N6Flh5qwPVAzARanFHuOZJnSA acIs3e/iFp94706rMKg47XK2smMq5kc= ARC-Authentication-Results: i=1; imf06.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b=Vm9wgtZz; dmarc=pass (policy=none) header.from=linux.dev; spf=pass (imf06.hostedemail.com: domain of muchun.song@linux.dev designates 95.215.58.183 as permitted sender) smtp.mailfrom=muchun.song@linux.dev ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1707994226; 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=tDrgJC7hzHurmZ2XnlYtAZDoBfMM4U/XsubK9yFFoCo=; b=r99PUPgcwMEodJM0K34EOW2M2ccdhoZJCp0zZJVA7CzLM8/9CnG2fCDhV7idqimgSKfHZp NnUJ7NH9sLhYk7YNuuvnyghN/g+c2VAJoYousqabFZ/S01N59o3J3vCOcQ4UDHC874Fex6 iGbRdSrIsUOYsjToOOudq72iyo5KXzw= Content-Type: text/plain; charset=utf-8 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1707994222; 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: in-reply-to:in-reply-to:references:references; bh=tDrgJC7hzHurmZ2XnlYtAZDoBfMM4U/XsubK9yFFoCo=; b=Vm9wgtZzKYFyuphTro/pr6m6ImxEcuFX8SzrfmEdR+feYaOKmgKKECxQGgT9bJW4r6pZup eydUOoFabVf9XQeh1ayo4PdPDH2Gf/rzyEZT4MWldGjqa8jZwVDAIZg9QTGVulWBHGuZ9e 1z/zZ/u+0jZlbvLzPbozgmThbZqveu8= Content-Transfer-Encoding: quoted-printable X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Muchun Song Mime-Version: 1.0 Subject: Re: [PATCH v1 1/2] padata: downgrade padata_do_multithreaded to serial execution for non-SMP Date: Thu, 15 Feb 2024 18:49:46 +0800 Message-Id: <05F7392A-FCC9-48E3-92B3-3A43A9D0C987@linux.dev> References: <146c8a41-8864-4cff-80ce-173812441844@linux.dev> Cc: Andrew Morton , Steffen Klassert , Daniel Jordan , Jane Chu , "Paul E . McKenney" , Randy Dunlap , kernel test robot , Gang Li , linux-kernel@vger.kernel.org, linux-mm@kvack.org In-Reply-To: <146c8a41-8864-4cff-80ce-173812441844@linux.dev> To: Gang Li X-Migadu-Flow: FLOW_OUT X-Rspam-User: X-Rspamd-Server: rspam06 X-Rspamd-Queue-Id: AA515180020 X-Stat-Signature: h4ugo1ex1qe3jwqzkhwyma9bzzkpoera X-HE-Tag: 1707994225-425295 X-HE-Meta: U2FsdGVkX18ewId6OejC1Er3enBeESHc5LUQ3g6s58jZxVmuZKXFvnxQ1AW8WR0F0ghhMt259lA/KhD9dSNrpdHYMskBYUHKb/Q3uayoXkQz8zlqT0Bb34aJsztIRIdLv1Gbh0qsYfW4wwbNMaL6Tzcmm0mRs9hLWpDOZsY7QiY8883tgfDKLsxsWVbN5hG2dlIiQs7HsrJ+kUsw87VeqVcHtVhWfSNC1CiKmhWCzLeIqs5G2/CXywBp7B8jA6v46G+ATpq6NlWlyCB9f7ynoVLtVWYHl4L9jTVmPuE4obRnsllabzM0dl9sknzD0VmjnyGKSPdg/EZmTeiKXoFwigPTpjDqSLX4IifAoFbqFSHQakpGhjx1SZSiH5TotQMVme+RCpRAtdhAHF+3sz8+vThaBBU0tunEw/xKRdSmxqQpKSao5jB4ZAQUFDdm37KgfThRTvYKPsLEeXKVJxsEpAX4FZJooKnsp0Rv7E32jJdN3rlbT+i7xRYwjGtCYx1C35UDkMhsJrEXhkEeCw28XktYZqqGcphkJ9PmiZU1RRqi1HNIdnS/HaTtm6w/iVlm/Ikh0veuHzi9pXj7iomYE+2PHvsuf6qEXvB18ZsQVnERX69NwQGbQa2jbwk73yzJuyKVTxSrbLEqTgOP6O5G935c4JTX6BAo7zxg1ODfaYRivTsfIfiMWk8vjPzEabfjE30Pdv/bY3zt1tVDcCjZyZaSa6Du8tjPiV5u+v6QCt+/I5nxEeM0c1AVy6dQrKrmQIvLdqLcuvSvRlJsd49FHc/S6lqc3EmFsnkK5561B3+NzupZuxWzW141b6IznLX07kdtRO9oWJNHGdcH7e+1J3G44NLcWWq4W9/6WU2fZtKNLDaXRTLoAQ== 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 Feb 13, 2024, at 23:15, Gang Li wrote: >=20 > =EF=BB=BF >=20 >> On 2024/2/13 22:52, Muchun Song wrote: >>> On 2024/2/13 19:13, Gang Li wrote: >>> Randy Dunlap and kernel test robot reported a warning: >>>=20 >>> ``` >>> WARNING: unmet direct dependencies detected for PADATA >>> Depends on [n]: SMP [=3Dn] >>> Selected by [y]: >>> - HUGETLBFS [=3Dy] && (X86 [=3Dy] || SPARC64 || ARCH_SUPPORTS_HUGETLB= FS [=3Dn] || BROKEN [=3Dn]) && (SYSFS [=3Dy] || SYSCTL [=3Dn]) >>> ``` >>>=20 >>> hugetlb parallelization depends on PADATA, and PADATA depends on SMP. >>>=20 >>> PADATA consists of two distinct functionality: One part is >>> padata_do_multithreaded which disregards order and simply divides >>> tasks into several groups for parallel execution. Hugetlb >>> init parallelization depends on padata_do_multithreaded. >>>=20 >>> The other part is composed of a set of APIs that, while handling data in= >>> an out-of-order parallel manner, can eventually return the data with >>> ordered sequence. Currently Only `crypto/pcrypt.c` use them. >>>=20 >>> All users of PADATA of non-SMP case currently only use >>> padata_do_multithreaded. It is easy to implement a serial one in >>> include/linux/padata.h. And it is not necessary to implement another >>> functionality unless the only user of crypto/pcrypt.c does not depend on= >>> SMP in the future. >>>=20 >>> Fixes: a2cefb08be66 ("hugetlb: have CONFIG_HUGETLBFS select CONFIG_PADAT= A") >>> Reported-by: Randy Dunlap >>> Closes: https://lore.kernel.org/lkml/ec5dc528-2c3c-4444-9e88-d2c48395b43= 3@infradead.org/ >>> Reported-by: kernel test robot >>> Closes: https://lore.kernel.org/oe-kbuild-all/202402020454.6EPkP1hi-lkp@= intel.com/ >>> Signed-off-by: Gang Li >>> --- >>> fs/Kconfig | 2 +- >>> include/linux/padata.h | 13 +++++++++---- >>> 2 files changed, 10 insertions(+), 5 deletions(-) >>>=20 >>> diff --git a/fs/Kconfig b/fs/Kconfig >>> index 4a51331f172e5..7963939592d70 100644 >>> --- a/fs/Kconfig >>> +++ b/fs/Kconfig >>> @@ -261,7 +261,7 @@ menuconfig HUGETLBFS >>> depends on X86 || SPARC64 || ARCH_SUPPORTS_HUGETLBFS || BROKEN >>> depends on (SYSFS || SYSCTL) >>> select MEMFD_CREATE >>> - select PADATA >>> + select PADATA if SMP >> I'd like to drop this dependence since HugeTLB does not depend >> on PADATA anymore. If some users take care about the kernel >> image size, it also can disable PADATA individually. >=20 > Only CRYPTO_PCRYPT, HUGETLBFS and DEFERRED_STRUCT_PAGE_INIT select > PADATA. If drop this dependence, hugetlb init parallelization may not > work at all. Oh, right. In this case, maybe current choice is better. >=20 > Maybe we can set PADATA enabled on default? >=20 >>> help >>> hugetlbfs is a filesystem backing for HugeTLB pages, based on >>> ramfs. For architectures that support it, say Y here and read >>> diff --git a/include/linux/padata.h b/include/linux/padata.h >>> index 8f418711351bc..7b84eb7d73e7f 100644 >>> --- a/include/linux/padata.h >>> +++ b/include/linux/padata.h >>> @@ -180,10 +180,6 @@ struct padata_instance { >>> #ifdef CONFIG_PADATA >>> extern void __init padata_init(void); >>> -#else >>> -static inline void __init padata_init(void) {} >>> -#endif >>> - >>> extern struct padata_instance *padata_alloc(const char *name); >>> extern void padata_free(struct padata_instance *pinst); >>> extern struct padata_shell *padata_alloc_shell(struct padata_instance *= pinst); >>> @@ -194,4 +190,13 @@ extern void padata_do_serial(struct padata_priv *pa= data); >>> extern void __init padata_do_multithreaded(struct padata_mt_job *job);= >>> extern int padata_set_cpumask(struct padata_instance *pinst, int cpuma= sk_type, >>> cpumask_var_t cpumask); >>> +#else >>> +static inline void __init padata_init(void) {} >>> +static inline void __init padata_do_multithreaded(struct padata_mt_job *= job) >>> +{ >>> + if (job->size) >> I think we could drop this check, at least now there is no users will >> pass a zero of ->size to this function, and even if someone does in the >> future, I think it is really a corner case, it is unnecessary to optimize= >> it and ->thread_fn is supporsed to handle case of zero size if it dose >> pass a zero size. >> Thanks. >>> + job->thread_fn(job->start, job->start + job->size, job->fn_arg)= ; >>> +} >>> +#endif >>> + >>> #endif