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 22E95C71136 for ; Thu, 12 Jun 2025 11:22:30 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 9CA8E6B007B; Thu, 12 Jun 2025 07:22:29 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 9A17F6B0088; Thu, 12 Jun 2025 07:22:29 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8B63E6B008C; Thu, 12 Jun 2025 07:22:29 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 6D0CB6B007B for ; Thu, 12 Jun 2025 07:22:29 -0400 (EDT) Received: from smtpin07.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 1E3601D79CF for ; Thu, 12 Jun 2025 11:22:29 +0000 (UTC) X-FDA: 83546510418.07.7062834 Received: from mx0b-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) by imf22.hostedemail.com (Postfix) with ESMTP id C9C12C0017 for ; Thu, 12 Jun 2025 11:22:26 +0000 (UTC) Authentication-Results: imf22.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b=S+0Ier0z; spf=pass (imf22.hostedemail.com: domain of hca@linux.ibm.com designates 148.163.158.5 as permitted sender) smtp.mailfrom=hca@linux.ibm.com; dmarc=pass (policy=none) header.from=ibm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1749727346; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=3STLFyJlkAtONNswVp9fcHwUoZ12uc3WwYjMNueQd2c=; b=2Ccz13u7ha4mxbXG1nac/P8qI/0hG3k+P2D8drKoQ88dFooMSfIHBKKo0/qq+bAUvawdZh yAUfpxpAkigRS7z+dUF3soje8ZFDL61CmGbqSCIlr6kKLrDv086smwNbX3IhHZFfSz/yKP q8Hy1eJNXCstqTeoyRGSqmJaH38CMEc= ARC-Authentication-Results: i=1; imf22.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b=S+0Ier0z; spf=pass (imf22.hostedemail.com: domain of hca@linux.ibm.com designates 148.163.158.5 as permitted sender) smtp.mailfrom=hca@linux.ibm.com; dmarc=pass (policy=none) header.from=ibm.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1749727346; a=rsa-sha256; cv=none; b=sXOHovMs3SovdsI2GobT6hjboZ1mctwUEULjBFOFN56CnRFZKtK7PaU2fejgds7GrH5ili GKt3Sb45jwlkHo+V6yZtqAgImX0WZkODCc1Kb8WwQ2tI6HKlkhYsR4RjOlTMRGLd0nDhcD tfdPMWxeoSPwLI7B+remyIIFcdLuG6g= Received: from pps.filterd (m0360072.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 55C4AaaR027824; Thu, 12 Jun 2025 11:22:23 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=cc :content-type:date:from:in-reply-to:message-id:mime-version :references:subject:to; s=pp1; bh=3STLFyJlkAtONNswVp9fcHwUoZ12uc 3WwYjMNueQd2c=; b=S+0Ier0zEIGIw5PjDxw6VYXIDMZoNTokcmPlzb0mpYVaoQ TH+GKYaAIi9YTm9H96oBwmVlzv0GiPt7tzO1mciJbpqlOPq2lI7JYJlKmfi1fTfv NwbCaTyLWQlKJ+TIDh2dFtt7gxRynLQ4pM+nW92vuoQAuBK+zg5yH8uRrTBMHdxA caZraOnjhek2O5xICThS7KstoKCeeh+3nrlCEHO4unOofwTxAD7XceVgVCzfGqKS laqBAOJ1Cb+xH8oMfBsangs060yK5e0n/uy4Y/XrHbbF4QuquGW7vs90PK1rQisK nxMhJekZR76gI5pylyjhxXix0pym8FeGvug3CtEA== Received: from pps.reinject (localhost [127.0.0.1]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 474x4mfda8-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 12 Jun 2025 11:22:23 +0000 (GMT) Received: from m0360072.ppops.net (m0360072.ppops.net [127.0.0.1]) by pps.reinject (8.18.0.8/8.18.0.8) with ESMTP id 55CBLWRc029468; Thu, 12 Jun 2025 11:22:22 GMT Received: from ppma11.dal12v.mail.ibm.com (db.9e.1632.ip4.static.sl-reverse.com [50.22.158.219]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 474x4mfda5-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 12 Jun 2025 11:22:22 +0000 (GMT) Received: from pps.filterd (ppma11.dal12v.mail.ibm.com [127.0.0.1]) by ppma11.dal12v.mail.ibm.com (8.18.1.2/8.18.1.2) with ESMTP id 55C8AAnb019573; Thu, 12 Jun 2025 11:22:22 GMT Received: from smtprelay05.fra02v.mail.ibm.com ([9.218.2.225]) by ppma11.dal12v.mail.ibm.com (PPS) with ESMTPS id 4752f2m4uf-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 12 Jun 2025 11:22:21 +0000 Received: from smtpav03.fra02v.mail.ibm.com (smtpav03.fra02v.mail.ibm.com [10.20.54.102]) by smtprelay05.fra02v.mail.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 55CBMITF55837168 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 12 Jun 2025 11:22:18 GMT Received: from smtpav03.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 654EC2005A; Thu, 12 Jun 2025 11:22:18 +0000 (GMT) Received: from smtpav03.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 49AE82004B; Thu, 12 Jun 2025 11:22:17 +0000 (GMT) Received: from osiris (unknown [9.87.144.171]) by smtpav03.fra02v.mail.ibm.com (Postfix) with ESMTPS; Thu, 12 Jun 2025 11:22:17 +0000 (GMT) Date: Thu, 12 Jun 2025 13:22:15 +0200 From: Heiko Carstens To: Hao Ge Cc: Andrew Morton , David Hildenbrand , Lorenzo Stoakes , "Liam R. Howlett" , Vlastimil Babka , Suren Baghdasaryan , Mike Rapoport , Richard Henderson , Matt Turner , Dennis Zhou , Tejun Heo , Christoph Lameter , Vasily Gorbik , Alexander Gordeev , Christian Borntraeger , Sven Schnelle , Kent Overstreet , linux-mm@kvack.org, linux-kernel@vger.kernel.org, linux-alpha@vger.kernel.org, linux-s390@vger.kernel.org, Hao Ge Subject: Re: [PATCH 2/5] alpha: Modify the definition logic of WEAK_PER_CPU Message-ID: <20250612112215.10868Da1-hca@linux.ibm.com> References: <4d78498def57e0df4c768ad9eb672cac68fb51dc.1749715979.git.gehao@kylinos.cn> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4d78498def57e0df4c768ad9eb672cac68fb51dc.1749715979.git.gehao@kylinos.cn> X-TM-AS-GCONF: 00 X-Authority-Analysis: v=2.4 cv=Y4X4sgeN c=1 sm=1 tr=0 ts=684ab86f cx=c_pps a=aDMHemPKRhS1OARIsFnwRA==:117 a=aDMHemPKRhS1OARIsFnwRA==:17 a=kj9zAlcOel0A:10 a=6IFa9wvqVegA:10 a=1XWaLZrsAAAA:8 a=qgZoHugHpSLCC5hwMT8A:9 a=CjuIK1q_8ugA:10 X-Proofpoint-GUID: cFaCM5ioo6oV4W252CJQ6_AbeOYbv0MG X-Proofpoint-ORIG-GUID: 4eZjzSqrsFgGxPf8TxMGwJ5ywHDFaxa2 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUwNjEyMDA4NiBTYWx0ZWRfX9kqynoea00ca dw1P8MwVNzix+KaE7qPljPCZbvhxGYU11FDgEnQxCjnYKwnhhBpm6fg/TMDNf8nRMXYA4IH7jLz b/0gfJm57nuOFAAmBPDYe8kjqj8H+YhlZiZx0BnJrwt3vBG2hyR7cZ5Jn2W3L4/6EsWfg6ZugBK GlofawOkvsQrAH6/iknx8NlH7TiIFqihJyZ1LoZH8Hu4M63F5KvnDMW5ZgR973eoeBr8at+FhXx wQrWzesedZuQh86muM9uEUfojqVJnAq2ii5JB9/N77UpzOzOhm8NQadzFwQarApvM0ghLC29AQ9 JSdoBGYous175eq60+1nPaoxMKnBdOeGZ0Zbuh2IdAZCfo8yf/hDvgHj2fdY4wJTU+Ci9OwUnee sMm6IGCPItF/A8/mN44Ex4TmPJ18WEyVZcHcHFBveYf0UDDbTcABy1vTLhigl/KGmV7POEmN X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1099,Hydra:6.0.736,FMLib:17.12.80.40 definitions=2025-06-12_07,2025-06-10_01,2025-03-28_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 mlxlogscore=716 spamscore=0 clxscore=1011 lowpriorityscore=0 malwarescore=0 bulkscore=0 adultscore=0 impostorscore=0 suspectscore=0 phishscore=0 mlxscore=0 priorityscore=1501 classifier=spam authscore=0 authtc=n/a authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.19.0-2505280000 definitions=main-2506120086 X-Rspamd-Queue-Id: C9C12C0017 X-Rspamd-Server: rspam03 X-Rspam-User: X-Stat-Signature: 7b3gj7y39prwwg7cwjjubwz4wtzirjdn X-HE-Tag: 1749727346-807337 X-HE-Meta: U2FsdGVkX1/WbKqCs4NXc52y3QzmxkHXg+j/UvO4Jm8VzFLsYAiGcMEyYhDMqF/TyvLgk85k0c/bLJmXjEJxPkYMSwgPl0b0QOnVBsBl8kcmiUG43vCS9Pe7KlFLE+VPGe9UkDD5rFDvrJEZwMzhepQW6zGBRb00rw6HE6IRHef74UKk0NVvgbcUylfS3ATP5PotY8yIoqYThMrc5oOr10xPWFtQmI8MbsKR2ih3dzpvdQRynx7omLhtXEEEq+8CslH8WPJ9DPBNTUWiuS6a6B05lMi9L9+7xjjPxlb3F7V+KZ/oJVWF4q1Nh7t5d1dudulP65+e7HsNarzbhJEX49LyvJ8Ozp0UKnIR8Pj5IXva3TY3sefGrTUY6C071xHPd6xMUSy5+ZJ8bkrWNRmQmmvOSxO3aPQtXU1wnB+A+ZRzqdjnzvRb8YNDPqeaw3AL52mxyHPrPG6M6YoHjDSjRfZYV0kUxBrK4hEUDH4OGXbKpcQZkf25u/+VEWk5S7jhmAA3wc5igOxAMJGX/yEE4ztHMKUi8N3pDYPdSLrVHQEAPwH0JTdqhAI6thgOgGFjFKmxAgQW3VTGWJSTRtVnRb1zw8XG8pEjHJbOgGT4wuAYhqkiMXXEq8rvbdlcyN5zNap10BxksjTUThTiDNrL1/wselFM6BKCwXO8Bl9mYgNq/SFYlyw03OaQlKsNhg3gnfTDdcATESvwTF2Xl7H0dj/vR5vTlajJsrYAFV/38y+GU9E0lH9viwpd6p2GSgq7aIP0z9E+qYCT+qm7AZ7AB3nLDY8UuA0HGCKoT3FKherb4/CVfE+C96ulFzpK1SsN1fM83/f5YW8Yj82BSAC1jjs/JiN9K1yxi8MoQuVMHAVExPkCzeWmUNvl9XwkldGWU0r2xMiktf9DtJOFPYFk5zb4FTz2JqS6bFSB4tx2zG6uhNpLpUixHvGJt16wM56CeZMJHGCtuneTL3boe91 6dj0531Z S4Y/exrzMsxawTIz4utDTEr19hOhY8tG3H1mhwLwhLUt9cRD9UJFSPdQwGleeUZqamw8xT6PffdixbHGi3dksowd0vvleY2A4L93fsvglItOaHz8gt70GEHI487Q/YRmMQygAAJPJaoqFzvz7vFVi2/MW7L27tRNg8E6LvHxZPOPYYI5wSGPPesE4g4SHJFTmmTqWt/ciWy8toRrqYXxKHr23MzKC9eSDqIdZrQ0WSB7kWKOLc9GxB+F0Eum3dUOZpN2yCECnfPPIUMBdiHxl++vh6wUgDNQdi7g7DC4cAaMcvVOiv6qCyshylx5husgd792m7v/P4vVWM9F/b6Ku2lvaFvTqW4sUBs80yxbhg9h7sbSr5I6UTKWP84x+LgFjpdBBEhPrAWXzrA1V+NUqqhBe1nal8D6mkSK2 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 Thu, Jun 12, 2025 at 04:27:27PM +0800, Hao Ge wrote: > From: Hao Ge > > As stated in the first patch of this patch series, > we make ARCH_NEEDS_WEAK_PER_CPU a Kconfig option, > and replace all instances of ARCH_NEEDS_WEAK_PER_CPU > in the kernel code with MODULE_NEEDS_WEAK_PER_CPU, > gated by #ifdef CONFIG_ARCH_NEEDS_WEAK_PER_CPU. > > We'll make corresponding changes for the alpha > architecture. > > Suggested-by: Suren Baghdasaryan > Signed-off-by: Hao Ge > --- > arch/alpha/Kconfig | 1 + > arch/alpha/include/asm/percpu.h | 4 ++-- > 2 files changed, 3 insertions(+), 2 deletions(-) ... > + select ARCH_NEEDS_WEAK_PER_CPU > select ARCH_NO_PREEMPT > select ARCH_NO_SG_CHAIN > select ARCH_USE_CMPXCHG_LOCKREF > diff --git a/arch/alpha/include/asm/percpu.h b/arch/alpha/include/asm/percpu.h > index 6923249f2d49..5d8b75ec3bf2 100644 > --- a/arch/alpha/include/asm/percpu.h > +++ b/arch/alpha/include/asm/percpu.h > @@ -10,8 +10,8 @@ > * > * Always use weak definitions for percpu variables in modules. > */ > -#if defined(MODULE) && defined(CONFIG_SMP) > -#define ARCH_NEEDS_WEAK_PER_CPU > +#if defined(MODULE) && defined(CONFIG_SMP) && defined(CONFIG_ARCH_NEEDS_WEAK_PER_CPU) > +#define MODULE_NEEDS_WEAK_PER_CPU CONFIG_ARCH_NEEDS_WEAK_PER_CPU is always set with the above select. So there is no point in adding this to the check. Furthermore this removes ARCH_NEEDS_WEAK_PER_CPU and defines MODULE_NEEDS_WEAK_PER_CPU while the common code conversion happens only with patch 4. Or in other words: if patches are split like this things break. Same is true for patch 3. Just merging patches 2-4 would be the easiest solution to this problem.