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 E8119C47422 for ; Wed, 17 Jan 2024 05:26:43 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 801E46B00BD; Wed, 17 Jan 2024 00:26:43 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 7B1E46B00BE; Wed, 17 Jan 2024 00:26:43 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 6536A6B00BF; Wed, 17 Jan 2024 00:26:43 -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 570D56B00BD for ; Wed, 17 Jan 2024 00:26:43 -0500 (EST) Received: from smtpin01.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay07.hostedemail.com (Postfix) with ESMTP id 28F69160B59 for ; Wed, 17 Jan 2024 05:26:43 +0000 (UTC) X-FDA: 81687668286.01.042672D Received: from NAM04-MW2-obe.outbound.protection.outlook.com (mail-mw2nam04on2054.outbound.protection.outlook.com [40.107.101.54]) by imf03.hostedemail.com (Postfix) with ESMTP id 335A420008 for ; Wed, 17 Jan 2024 05:26:38 +0000 (UTC) Authentication-Results: imf03.hostedemail.com; dkim=pass header.d=memverge.com header.s=selector2 header.b=sa2rKhAf; dmarc=pass (policy=none) header.from=memverge.com; arc=pass ("microsoft.com:s=arcselector9901:i=1"); spf=pass (imf03.hostedemail.com: domain of gregory.price@memverge.com designates 40.107.101.54 as permitted sender) smtp.mailfrom=gregory.price@memverge.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1705469199; 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=RxLc7vwXB9DzWQ5rRlwGTHtyleOkBH3StC1nqcr43OQ=; b=md6pjv6AhvAkoW8/e72udFG64HRVwVLqbxLgdhZxmw1PSUif6/k7/OJ7mXHXDsaUoYhH1Z WCwNrtVMvTes1Hz0ALWwAUBEs1+24HAp0Hy6vijmDgPc7hdze8S6YTt/Bp4RnyAFGUBgFu /u+fv3152ihlVqPy0awcXYB9NMf8rdM= ARC-Authentication-Results: i=2; imf03.hostedemail.com; dkim=pass header.d=memverge.com header.s=selector2 header.b=sa2rKhAf; dmarc=pass (policy=none) header.from=memverge.com; arc=pass ("microsoft.com:s=arcselector9901:i=1"); spf=pass (imf03.hostedemail.com: domain of gregory.price@memverge.com designates 40.107.101.54 as permitted sender) smtp.mailfrom=gregory.price@memverge.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1705469199; a=rsa-sha256; cv=pass; b=WGTehZtPyjqod1n/pCHbVGwN5TpsnRUB3FETagjgAJMy3nz6HcxYHjRrJZ9UFGpdi44/sX vUyV25NNpnIx+gp0dxBgmMYPGfC0C/Jx6xxRyEVHgrPRys3X9cqzh1Rh0RrefmC4WuuiRg LOMrPTtuC3d1fOa7ns30u55n1+1IXg0= ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=KAMTzL9aJkZemzhr5cyBCqMa5ZMjJC95CUtqvN8AsuS5mBi0y7zau20rHZmomy2z4IzqSQPMb7EWePViBJcM/wDQ3HOGfl1NUPKXm8ZbdhVhSbjaerykZFS6ATfp/kvKRLtNkuJe4QAOH8qbrW3kDv0pwAE65K9QuswivMILfeFsexOe5Py8nq+/uYg9q0LzlTzehpowZnZQyd83amwSkDPpha9LCYa0HDEhKRF3WKYSlUgVxyoAFHK70YMSJXShsHUP2BCoYeNTvsR1LoK2dZAvlehVtGOQG5W41WvZEmXKa/jWjm+4yirJJKxADe6CEtd+bABU6s8JFV4+VvpYow== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=RxLc7vwXB9DzWQ5rRlwGTHtyleOkBH3StC1nqcr43OQ=; b=lfFJWaTDTutZae6eN/nqIdSvjYWf36QyPOx40oinn7Xrm1JnBn63QFIyss1fyvR/26c7yKjw9YerDe1GUEyEWp+ARHiEWkn7W0qHJ94qRmZETQCXRupQ+ooD2l2WB3tmJesZJaQuXbIVAwgyDlbF14/+Yz0EAqO1AYi7h8xzb38X+g36snIMGPsJmegoTb5TbIZSgeBXYtbbLMybK1XqPlDZx+rwVgt9c5pvuPXYuI5NtoTPMd8DLVYUbg68RSHZYwrDo7dvFVCOCuWeTIOSOl6TSavhCacdhRea8MgrUx5+KEf54Ghjt8H5KMaf2wc6on65AC2aZbdNHU1j1Sk/Nw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=memverge.com; dmarc=pass action=none header.from=memverge.com; dkim=pass header.d=memverge.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=memverge.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=RxLc7vwXB9DzWQ5rRlwGTHtyleOkBH3StC1nqcr43OQ=; b=sa2rKhAfQy6iy0KAjHe3uc+hwS4keleFIGwjteL220HWAvY/x1WR9jWw2rVoNBbKin6vXVbSuwUGsIbDn7dYC5Bt9HZSjd70i/iq01bVTy+Mq1bM5oOO8BxciLrdqTwhhZtm1hj21Gz39Os3VH5kZ7muexJiinNU3HhqT17ApUk= Received: from SJ0PR17MB5512.namprd17.prod.outlook.com (2603:10b6:a03:394::19) by SN7PR17MB6514.namprd17.prod.outlook.com (2603:10b6:806:323::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7181.30; Wed, 17 Jan 2024 05:26:36 +0000 Received: from SJ0PR17MB5512.namprd17.prod.outlook.com ([fe80::7a04:dc86:2799:2f15]) by SJ0PR17MB5512.namprd17.prod.outlook.com ([fe80::7a04:dc86:2799:2f15%5]) with mapi id 15.20.7181.027; Wed, 17 Jan 2024 05:26:36 +0000 Date: Wed, 17 Jan 2024 00:26:20 -0500 From: Gregory Price To: "Huang, Ying" Cc: Gregory Price , linux-mm@kvack.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-api@vger.kernel.org, corbet@lwn.net, akpm@linux-foundation.org, honggyu.kim@sk.com, rakie.kim@sk.com, hyeongtak.ji@sk.com, mhocko@kernel.org, vtavarespetr@micron.com, jgroves@micron.com, ravis.opensrc@micron.com, sthanneeru@micron.com, emirakhur@micron.com, Hasan.Maruf@amd.com, seungjun.ha@samsung.com, hannes@cmpxchg.org, dan.j.williams@intel.com Subject: Re: [PATCH 2/3] mm/mempolicy: refactor a read-once mechanism into a function for re-use Message-ID: References: <20240112210834.8035-1-gregory.price@memverge.com> <20240112210834.8035-3-gregory.price@memverge.com> <87h6jf1bfx.fsf@yhuang6-desk2.ccr.corp.intel.com> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <87h6jf1bfx.fsf@yhuang6-desk2.ccr.corp.intel.com> X-ClientProxiedBy: PH8PR20CA0019.namprd20.prod.outlook.com (2603:10b6:510:23c::25) To SJ0PR17MB5512.namprd17.prod.outlook.com (2603:10b6:a03:394::19) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR17MB5512:EE_|SN7PR17MB6514:EE_ X-MS-Office365-Filtering-Correlation-Id: ff886a70-4f24-4281-6298-08dc171cdfda X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: IIHFvejNoU3tMsphhLKQFpJSdrqVzj4qI1OG7isFim2fYUhXuGqwHn79jkfiePeFuArdzmi4N3LkGqNeKV+qW9pR4DO58ch+TjWKduzhe/LTX8CI/vgZETLHSgTu9dlPIRV+SFMydpuBOlVgaU+dXHcE6bAMA9mp22Xhd0fWYffeIVOb68ux3jnkttMyEJCL5eQyl7OqkugHyl0k2nAKo31GTJBFe2D8NyHuz0eiDo5ycbjcUrGvIlzjnfd+iBtrLSakg28FmjUwmTxzoYkk/UflQ/q/LeGs1AM+AwFPE21ENKxWYFlfnTz5Nw6vGyJNJkG4JIcAf8hkIKxoNnAUMlZ47xDl67XHILMaDzFxWa2u3Z1HwN4F0Ys8xxRE6YySovDXTRZ9LlMA73EreoIQTr3sG0Sq5CvdnVFUFUjU7b5J9AO/RiOpU2wFyTDN+6fYOpKKy/KTLyxg6OjBvm5MzYwJGj1CgQqfEiG8UbWGp2fU7t28I9q5Dg9TqTsrceMYmyCxlEV661a32S/b+KMVBQ4SPvmUOx9KryaxSX7VH6CX6rAAQBx8vfWCdcHO6DBz X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SJ0PR17MB5512.namprd17.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(346002)(396003)(376002)(136003)(39840400004)(366004)(230922051799003)(1800799012)(64100799003)(186009)(451199024)(4326008)(8936002)(8676002)(7416002)(4744005)(66476007)(5660300002)(66556008)(2906002)(6486002)(26005)(2616005)(36756003)(86362001)(6916009)(66946007)(316002)(44832011)(478600001)(6666004)(6506007)(6512007)(41300700001)(38100700002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?Vm76eHIBCSAvX6RFUelF+rDdh4siq+RBnoXsa78kWbEIRDYXYnUItUxWHI5J?= =?us-ascii?Q?L/t8q8G67sBTD2H03PNJBAdW6B9TmT7YjpA6Suwp0k6FlANEy77I6ydFYr3x?= =?us-ascii?Q?dH+X399j5lAs45nii8NUnd8qcY7pJp5ErfSXB1WFIMNAZE8wJWTOmHTtHqMf?= =?us-ascii?Q?7wGHDBUBvtfrvvolFkQTwsI0zM3HWGCh0uP8PmIbiUGbDMAmxu58Datttevd?= =?us-ascii?Q?DNHm0A2CBGbaiK0zOxL2a24JPN9vs08O96dsVitN0h6ZIEDJ4Xs40gZnPG8r?= =?us-ascii?Q?rZxkqzGhcZ17g7Kf+GIBKT41TSlUZXL0n2TZLH+E1TJ7KqvOFmMBamriKib+?= =?us-ascii?Q?OAJL4CwusvTpmD9crl06GMfJWUcTXsbRf9sMaMm6/1GWN9Jz0O41M/SFrlHm?= =?us-ascii?Q?tUXTx8/3AthJd/MJq5PIBer5qvYYcdo1yd68YRAJ3+UTe8NNxYOs3+qAyzyb?= =?us-ascii?Q?ytU/KW7AfDUcJDHfpeZ7axmkGsLpaQKqZM3bEVtMnkicuMbdjl5iywBIEQF8?= =?us-ascii?Q?HrO1LA4y6YJMIo3QhNGJX+GObeJtVv0jZ+rjzo+eG53v8QT591hkG6w85EQr?= =?us-ascii?Q?Wt3Wg75926FjoM3wUKH5x8zkaF3mZvFQ7fy9LI0CobZVpDBfIth0TwiendsF?= =?us-ascii?Q?+FHs/6Rxupa2Gu+Sr/CHrnkWMF7kRow/17EjEGKkQAcHH0U1BPx42ogCKpTn?= =?us-ascii?Q?RoVzN4CZBJ2XxitM2F4VmZChmNnDP/g5nzCpmejH6mk/ligcb2xkd1lIQZOc?= =?us-ascii?Q?/gonZBk9ecuWPlxnzmXcUJrA1XRK9RdrV7JY4SdykBywpK+kw46HH3dTgYQq?= =?us-ascii?Q?E7DRLx2hr+xXPZnUjLTrWhrDd3eoDOxaai3kvT7V1tLEVVprbTj0YY8pZj01?= =?us-ascii?Q?wEdME1Qz2IA/NpZ/NAckpMiZH4hDEuvvn5zt6WladKRbKB3MTTgSDwI+s9xW?= =?us-ascii?Q?51w8Z6tODh1MRr/HKp+0jfM2ybkt/NnWKdvq1Hwyd1S2LIYtBZwWrHAiujF1?= =?us-ascii?Q?OZOGXh+0o1dmnTi/boXMl3Rbe4LYrNhkgyYVO+nO4tXYjnBTKYpbfHOrCsQu?= =?us-ascii?Q?/17NhuLvejRPqdKFMHBTrzvf+8PzNLvugqqr9QWBCtl299wiJAS/PJ6r7ZZf?= =?us-ascii?Q?ODxy1FfeqwbwMckcYsFSlDF/S7Foe++/nHQItbVOukps8h6VFZF8rFu8L0FS?= =?us-ascii?Q?9ag8vDy4JOJ8TL0qYB3jaWeyGhiIOGioIfsYhZ0pQ2zc2m5kPvn1r9YPImiH?= =?us-ascii?Q?WK/Lj8/lkYr//Cq3ViEk4GBFXpb1sr7r6N8cjcCRFaVwVmYTTFeOwLcCyUKk?= =?us-ascii?Q?w2E8EAHqawIdnRBzL/pQa3bTxqiamPoM+FuKjYXhiz5Z8lYAfdm+XZ5pxJwK?= =?us-ascii?Q?nX5A30MOyhbIQvoH3CJg6UXeb35i+yamZNOAK27BowpY9nodXnl5dYDZIqzo?= =?us-ascii?Q?weB+demjCDQxBszdD+ktD8BUcNWwXsbon4Tou3AkytQHv4qHL2w4+VNcBFxw?= =?us-ascii?Q?NPu1yKYEmX2L09JfeE1NDbp8I7JFbhDcS/v0Tt/mP32JVgfABOhRO4eFY0VJ?= =?us-ascii?Q?tNX/8RHryMiqAk+BI1ahhjgh9NllTuTfn6IWe7P8ckLdjROZc9Am0ndZ66k8?= =?us-ascii?Q?OQ=3D=3D?= X-OriginatorOrg: memverge.com X-MS-Exchange-CrossTenant-Network-Message-Id: ff886a70-4f24-4281-6298-08dc171cdfda X-MS-Exchange-CrossTenant-AuthSource: SJ0PR17MB5512.namprd17.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Jan 2024 05:26:36.2737 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 5c90cb59-37e7-4c81-9c07-00473d5fb682 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: ISoJKJEbzdl+5CzTYLUK4Efs+TEdRZH34jKf42svSildHgbC94HIf5EarEikz2dDqjogT4SZO9IFpBxREQgKCAmKrl5oLspPkT4E4WXXrdk= X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR17MB6514 X-Rspam-User: X-Stat-Signature: b8ih768t4xf73nptc6aaix746dqubisw X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: 335A420008 X-HE-Tag: 1705469198-847260 X-HE-Meta: U2FsdGVkX1937CZiXMTDfSv8TPBE8q2bjgo3VYnVXwqD8eQovlBP0ohMpLBu2RXkcuVmdBwUAiosI07aVS6yDQXea4iAlvE4Uz9HovOFnKSQWZCG/l0RYcUTvsFwDl1Rrby18cp+R/9kR9ctBxsDenOdKOw+SeN5gTT8v+H9jiHK2CP2Ef9OIw8bolsTzzRQV+Pb34pg0jm8soviPlcCF2wmM+U5Ckv9tnxrGI91s3qeTkJCw8m+QB2nZKiE3jltMoMeRv3lXuIxfZdRNvr6jTfrvJQ73XwUgTfzwU2oOSDGh2+RRuHELW/MVwsB8em7HDKgTu4qA/IU3ams28Z9wpLcV+8yDNy7NaOc107+ppWMTXLBOkfytzCGA3MB2NAlXMKtM5GlnnjQ2lIYd+nT0TIqG7hhntMSt8qkWv/0BKUhZz9ITbgASWGvW6VYs8txCK3Pivo1/H+nPrR6yXs8HAVgTS4Hq6gMLALC+M/n+71kqcqtkpaEOkGaDcoFv1UyPYYWXtsayJc+pDtGE0dtvgHioz1JJUVIePZm1c2cFtelz85dYnhoANe6p7WVZAx9sswVyI5wqrRjQapmQvcQT1/jPoXrW6NmHdzDs7jYATC8YEzmClDjJCPod7r9F/MLRtWe/GSDK8eIOvk6h6RihhhUXQ+/buK6RxMcwEpYZTSbkIHb73vgxk4FsKQqtoapL2V/3c/F7LLftGUyYw4c5tNrYQnFmC82FPLb/FimtVaGq56RfqCeQTrTpOM5rCN+YqoXflpbrBUAPxurwDeIWGD5gAdCGsvBrBSGLzWgN4l6bTMFX2rTABETUj+RJa1Hr9dGyaW1L2f36iXm6c9ldGv2BHsX0fjH7jWOk73Pkt1iT+FMOEz/6qtfsFhaQE8dtG3XAjE7KwiP/f50LxiL151r9R36aMbfNELgFX9RkXJrLUfCmjWhhmMNQAFT0y6jUtEO7soQTC9RgGXPHFf vqq7bRgd nsglxiJVrGw19wy0Hj/0hg3KPJUgCDaQbJuVofBc5ylU0UZfVQ8XkVUGkgZaEFLkYeGtt0Y+lVBSl2jd2lJLWRXa0RdWuNDtebvDXTLI81i9wmxghP6ttuBMrCS6HYte3jge7DjAgsK120bhi8ehBBnye8xZx1BfMo0+QVhern+GbmikMBXPY8iN+3pG/G8vJsfYr70aEKRaOLzLD+kIZOJEJSRvPWfM857RTre+8kUObsKOq684ZQUSLo+I5W6pw5vHz8gECPbNVdoUcjPh4x3Dfxi1sKVjSOpp/YMVncJ9syFQP+crM8hpoN31nOj4qeNcZh/GT4dyfHrzFftoQ5rUYmA== 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 Mon, Jan 15, 2024 at 12:13:06PM +0800, Huang, Ying wrote: > Gregory Price writes: > > > > > +static unsigned int read_once_policy_nodemask(struct mempolicy *pol, > > + nodemask_t *mask) > > It may be more useful if we define this as memcpy_once(). That can be > used not only for nodemask, but also other data structure. > Seemed better to do this is an entirely separate patch line to avoid scope creep on reviews and such. > > + barrier(); > > + __builtin_memcpy(mask, &pol->nodes, sizeof(nodemask_t)); > > We don't use __builtin_memcpy() in kernel itself directly. Although it > is used in kernel tools. So, I think it's better to use memcpy() here. > ack. ~Gregory