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 3B9D9C433FE for ; Thu, 3 Nov 2022 16:02:19 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id A893A6B0072; Thu, 3 Nov 2022 12:02:18 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id A38646B0074; Thu, 3 Nov 2022 12:02:18 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 9007E6B0075; Thu, 3 Nov 2022 12:02:18 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 832996B0072 for ; Thu, 3 Nov 2022 12:02:18 -0400 (EDT) Received: from smtpin19.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 2B4031C5B52 for ; Thu, 3 Nov 2022 16:02:18 +0000 (UTC) X-FDA: 80092597956.19.91CA0BA Received: from mx0a-001b2d01.pphosted.com (mx0a-001b2d01.pphosted.com [148.163.156.1]) by imf19.hostedemail.com (Postfix) with ESMTP id C4B251A0034 for ; Thu, 3 Nov 2022 16:02:14 +0000 (UTC) Received: from pps.filterd (m0098409.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.17.1.5/8.17.1.5) with ESMTP id 2A3FcXSI032562; Thu, 3 Nov 2022 16:02:11 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=date : from : to : cc : subject : message-id : references : content-type : in-reply-to : mime-version; s=pp1; bh=RHGYzL6HBFwRpP58PRBhJnwTukk2/WWX9VndeIDpZVw=; b=LY9LENNnzi4xScCfX6Key+ykkH1uLDmNxacpvRr8VnwFktw4AgbU0RM9dVV2OTD1xI/Z DZmYi80tgllupU5nLzzGM3m5Rm8E+PwR+fxDSClcsaygDt8b32V/oV1OlarXLZyI2g5B yYJ5EGlFvwO3pre1FxfxC6zM+iBw9wPDrdrNtverQnc7MmYb31MbFpGPVUV2gNHSVXdJ M7N1xPmEG2KcoRhscPGM+eDzQY1lqr0OUiIPGA58ntxfB/t8uOglsfot5xAzNE8y1tQu lqw3IdMpn4YqhTdYZZt0+Urz3Em+NWx4C3oLiyes8ifsX9uCQ5tx7lafc2YMRV2u//PM kw== Received: from ppma04ams.nl.ibm.com (63.31.33a9.ip4.static.sl-reverse.com [169.51.49.99]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 3kmf9r4h5v-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 03 Nov 2022 16:02:10 +0000 Received: from pps.filterd (ppma04ams.nl.ibm.com [127.0.0.1]) by ppma04ams.nl.ibm.com (8.16.1.2/8.16.1.2) with SMTP id 2A3F6dTJ022637; Thu, 3 Nov 2022 16:02:07 GMT Received: from b06avi18878370.portsmouth.uk.ibm.com (b06avi18878370.portsmouth.uk.ibm.com [9.149.26.194]) by ppma04ams.nl.ibm.com with ESMTP id 3kgut991j4-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 03 Nov 2022 16:02:07 +0000 Received: from d06av26.portsmouth.uk.ibm.com (d06av26.portsmouth.uk.ibm.com [9.149.105.62]) by b06avi18878370.portsmouth.uk.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 2A3G2fkF51511786 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 3 Nov 2022 16:02:41 GMT Received: from d06av26.portsmouth.uk.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 5F421AE04D; Thu, 3 Nov 2022 16:02:04 +0000 (GMT) Received: from d06av26.portsmouth.uk.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id EB016AE045; Thu, 3 Nov 2022 16:02:03 +0000 (GMT) Received: from localhost (unknown [9.171.56.84]) by d06av26.portsmouth.uk.ibm.com (Postfix) with ESMTPS; Thu, 3 Nov 2022 16:02:03 +0000 (GMT) Date: Thu, 3 Nov 2022 17:02:02 +0100 From: Vasily Gorbik To: Muchun Song , Andrew Morton Cc: Muchun Song , Mike Kravetz , Gerald Schaefer , Linux Memory Management List , linux-kernel@vger.kernel.org Subject: Re: [PATCH] mm: hugetlb_vmemmap: include missing linux/moduleparam.h Message-ID: References: <71BFC75C-483D-42C0-8248-59FC8DBE148C@linux.dev> Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <71BFC75C-483D-42C0-8248-59FC8DBE148C@linux.dev> X-TM-AS-GCONF: 00 X-Proofpoint-GUID: k7APG_JBHhj9KPhONqzItEdIaQauVAIG X-Proofpoint-ORIG-GUID: k7APG_JBHhj9KPhONqzItEdIaQauVAIG X-Proofpoint-UnRewURL: 0 URL was un-rewritten MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.205,Aquarius:18.0.895,Hydra:6.0.545,FMLib:17.11.122.1 definitions=2022-11-03_04,2022-11-03_01,2022-06-22_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 clxscore=1011 lowpriorityscore=0 malwarescore=0 bulkscore=0 impostorscore=0 spamscore=0 mlxscore=0 priorityscore=1501 mlxlogscore=803 suspectscore=0 phishscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2210170000 definitions=main-2211030103 ARC-Authentication-Results: i=1; imf19.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b=LY9LENNn; spf=pass (imf19.hostedemail.com: domain of gor@linux.ibm.com designates 148.163.156.1 as permitted sender) smtp.mailfrom=gor@linux.ibm.com; dmarc=pass (policy=none) header.from=ibm.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1667491335; a=rsa-sha256; cv=none; b=Uj1xA1aElA7lcO3dkzBDAO4kwO6IZ2r4D5LTYvTES/HRPf2dzEP0XGtWTbwOEXhqGoqiKI z5mP169WJV4mNbxdnZBicD0X5syvDoo8S/wkh0/VoHJ7UW/r1ciFLKdli8CjfXkAPqFrps UnsQgO7xCgbQRgAM9GpGl2ch9aem8OA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1667491335; 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=RHGYzL6HBFwRpP58PRBhJnwTukk2/WWX9VndeIDpZVw=; b=poAPhEcfpp4a436pwFfn3/WZS1wenUa2a9tJakWrDMo+CoblOCP5bgEawqZEToWcZiuXGS IY9YRuaSRHn8GmDQ3RlIziciRk7SNtIla5yN0JR3xEgu8nwwL+H3i+u3sNlH7Zg7aYM/d8 UruIMuL+htxgI0sawvi7szUqFGYOk6E= X-Stat-Signature: in8rt8aa3a98g9tdmgh6qq7tdr53z5o8 X-Rspamd-Queue-Id: C4B251A0034 Authentication-Results: imf19.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b=LY9LENNn; spf=pass (imf19.hostedemail.com: domain of gor@linux.ibm.com designates 148.163.156.1 as permitted sender) smtp.mailfrom=gor@linux.ibm.com; dmarc=pass (policy=none) header.from=ibm.com X-Rspamd-Server: rspam03 X-Rspam-User: X-HE-Tag: 1667491334-831772 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: On Thu, Nov 03, 2022 at 10:51:59AM +0800, Muchun Song wrote: > > > > On Nov 3, 2022, at 02:09, Vasily Gorbik wrote: > > > > The kernel test robot reported build failures with a 'randconfig' on s390: > >>> mm/hugetlb_vmemmap.c:421:11: error: a function declaration without a > > prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes] > > core_param(hugetlb_free_vmemmap, vmemmap_optimize_enabled, bool, 0); > > ^ > > > > Link: https://lore.kernel.org/linux-mm/202210300751.rG3UDsuc-lkp@intel.com/ > > Reported-by: kernel test robot > > Fixes: 30152245c63b ("mm: hugetlb_vmemmap: replace early_param() with core_param()") > > Thanks for your fixing. However, the correct commit that should be fixed is > 42deddf3ef5c ("s390: select ARCH_WANT_HUGETLB_PAGE_OPTIMIZE_VMEMMAP"). > > Reviewed-by: Muchun Song Even on x86 the inclusion chain looks like: CC mm/hugetlb_vmemmap.o In file included from ./include/linux/module.h:22, from ./include/linux/device/driver.h:21, from ./include/linux/device.h:32, from ./include/linux/blk_types.h:11, from ./include/linux/writeback.h:13, from ./include/linux/memcontrol.h:22, from ./include/linux/swap.h:9, from ./include/asm-generic/hugetlb.h:5, from ./arch/x86/include/asm/hugetlb.h:6, from ./include/linux/hugetlb.h:802, from mm/hugetlb_vmemmap.h:11, from mm/hugetlb_vmemmap.c:18: ./include/linux/moduleparam.h:2:2: warning: #warning included from here With s390 defconfig: CC mm/hugetlb_vmemmap.o In file included from ./include/linux/module.h:22, from ./include/linux/device/driver.h:21, from ./include/linux/device.h:32, from ./include/linux/blk_types.h:11, from ./include/linux/writeback.h:13, from ./include/linux/memcontrol.h:22, from ./include/linux/swap.h:9, from ./include/linux/userfaultfd_k.h:18, <-- randconfig disabled this path (CONFIG_USERFAULTFD) from ./include/linux/hugetlb.h:14, from mm/hugetlb_vmemmap.h:11, from mm/hugetlb_vmemmap.c:17: ./include/linux/moduleparam.h:2:2: warning: #warning included from here 42deddf3ef5c ("s390: select ARCH_WANT_HUGETLB_PAGE_OPTIMIZE_VMEMMAP") is just a one liner activating this feature and only present in linux-next and s390 features branch targeting v6.2. So, if you don't want to have Fixes: 30152245c63b ("mm: hugetlb_vmemmap: replace early_param() with core_param()") fine with me. But then just dropping Fixes tag is probably a better option. I hope that won't prevent it from going into some v6.1-rcX. @Andrew should I resend? or would you be so kind to drop Fixes tag and add Reviewed-by without it? Thank you