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 9C30BC77B7D for ; Mon, 15 May 2023 17:45:35 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 10F1A900004; Mon, 15 May 2023 13:45:35 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 0BF77900002; Mon, 15 May 2023 13:45:35 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E7AC3900004; Mon, 15 May 2023 13:45:34 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id D63A0900002 for ; Mon, 15 May 2023 13:45:34 -0400 (EDT) Received: from smtpin24.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 842B38069D for ; Mon, 15 May 2023 17:45:34 +0000 (UTC) X-FDA: 80793216588.24.49BD845 Received: from mx0a-00069f02.pphosted.com (mx0a-00069f02.pphosted.com [205.220.165.32]) by imf25.hostedemail.com (Postfix) with ESMTP id 05200A0011 for ; Mon, 15 May 2023 17:45:30 +0000 (UTC) Authentication-Results: imf25.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-03-30 header.b=PToCCFGN; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=qnaGPILQ; arc=pass ("microsoft.com:s=arcselector9901:i=1"); spf=pass (imf25.hostedemail.com: domain of mike.kravetz@oracle.com designates 205.220.165.32 as permitted sender) smtp.mailfrom=mike.kravetz@oracle.com; dmarc=pass (policy=none) header.from=oracle.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1684172731; a=rsa-sha256; cv=pass; b=B1cd6RfsrRMOXlFnmuOynlyY5LvqDukLfF3NSH6LJUAJc7S8OMZ5XMOcgdxJF/81Gom5YU G0IoGQbqGR07o1gYK3lVuyYJ2mMpTEz5JQLee/2bRLCcOUVBzMHgoY+NYBaxhgBw/f03o2 mPlq48hVrTi6fGUbXCbUF0LoDY9uOqk= ARC-Authentication-Results: i=2; imf25.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-03-30 header.b=PToCCFGN; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=qnaGPILQ; arc=pass ("microsoft.com:s=arcselector9901:i=1"); spf=pass (imf25.hostedemail.com: domain of mike.kravetz@oracle.com designates 205.220.165.32 as permitted sender) smtp.mailfrom=mike.kravetz@oracle.com; dmarc=pass (policy=none) header.from=oracle.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1684172731; 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=OqzH2ENGPMsdemzGYlEkYbF3LSfjeKzaE2XAp18b5OE=; b=gU8dbCSDr19eCyI4AKv/MyvoZP68oyvY8BO4M02a3N/TSstS6zaCIB2npDSGqMrZRWVnnK j6tndt1ETXZLJdb9uy8p8o0uxdI0l9wghBiCWYOXrTWnqO8r0j791AL1W8XYqbKFglA41T lhxdiApNidTTaIDJbClUPanVekyWIyg= Received: from pps.filterd (m0333521.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 34FGnqrF011049; Mon, 15 May 2023 17:45:24 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=date : from : to : cc : subject : message-id : references : content-type : in-reply-to : mime-version; s=corp-2023-03-30; bh=OqzH2ENGPMsdemzGYlEkYbF3LSfjeKzaE2XAp18b5OE=; b=PToCCFGNk1kxWhZs0IYJFnUdTP258i9NjVxJ1pNYuqnfR+X911ocZ3hf+MJUqzyUn3a1 U2EhuSGKfq32yzuQbgrKzdgM1TlPjvxYutXs1pU3SsZME0uGYXzfEWuPL0sfnPok728X yBWpvw9UKwHb08wFKANYKZ4cHpDZ0VtGtYpfZo3FLtxRq2YLiR3pX8Kyg1RzwsxgPnkW kL3lAJdjCY1Nz2YhjtTJWz0ULyDHYSQPHXG+TKyAr0wTQgl/tMK6INEBIYNqmgdDG5k4 25cEBGD0pKGsVjDKy1HAdZhokvyWjSD4of+NN4y7LK2wsABySMSZz7ucAdPcqVZI4qih Dg== Received: from phxpaimrmta02.imrmtpd1.prodappphxaev1.oraclevcn.com (phxpaimrmta02.appoci.oracle.com [147.154.114.232]) by mx0b-00069f02.pphosted.com (PPS) with ESMTPS id 3qj1fc0pbk-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 15 May 2023 17:45:24 +0000 Received: from pps.filterd (phxpaimrmta02.imrmtpd1.prodappphxaev1.oraclevcn.com [127.0.0.1]) by phxpaimrmta02.imrmtpd1.prodappphxaev1.oraclevcn.com (8.17.1.19/8.17.1.19) with ESMTP id 34FGOAso029507; Mon, 15 May 2023 17:45:22 GMT Received: from nam11-co1-obe.outbound.protection.outlook.com (mail-co1nam11lp2170.outbound.protection.outlook.com [104.47.56.170]) by phxpaimrmta02.imrmtpd1.prodappphxaev1.oraclevcn.com (PPS) with ESMTPS id 3qj102yupq-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 15 May 2023 17:45:22 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NQkk9x5Pg8YvOXIsCb+dEH7VqLj7BKxp4/CIe6x7Bm75WFbB1ps1DassoJ5tixMIU9sWceV1cc/21UsJQeKVxuQb2z8+L7sPEVpbLZJGpxa8DtdrsojB1yY7F4WEUwJbZJmXqDLdHrfA/OrNfmewQAiFHKJxc7EVp//7PJk75q8IcpS8mFnioERVSibyed9aaaKi/j6Mfa91yaZaAajppcT1G/Wa0AQuNXbbt2esjP86qSVJIPPmiNNqjGYwo3DICFEMBryOtuY5N7TmcyeN/NEk+Dn3dj57Q7RBCNhoKo19tkHHc2dbacJfda2Wy9rjfmHDbNVt4knTNM7Pa1jp2Q== 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=OqzH2ENGPMsdemzGYlEkYbF3LSfjeKzaE2XAp18b5OE=; b=dVv7zAuPGFN+cNIKqSdMgi+j6qFYP0pE0UXfa4MK9jRNUDclziQoZhKEBjKOeAZGjRboMbc2k8rezsusbB5aifl5eTKMiyJrud+gZR1XEJ2sw27rhgaEsdSPZuedKBNAXM9OglIJdRmYum73Xd/gIpDdnezmshddRdaUuRUEJctpaMfBMpFntcJeRWLDAe8FZaIKTM9CxXXR1qiHWJ/IELhTJzfOFCGkSXa7/XapAaeMrN50TGXVGLndAd0poNeU0BruOhCbNH4r30KKGr+n2GYUfgKhwiZjFxUBVn2vk7WxObQrU5rjaJxKQzFd+c4Gb1OzLLgnb9tK3sJp6ugkfg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=OqzH2ENGPMsdemzGYlEkYbF3LSfjeKzaE2XAp18b5OE=; b=qnaGPILQo1QYbOoRNJ4NL+fSPjc9eld+HBUF3BGU+y9kFCz1rAKygZviDIFRqwAual/q+m+dz5yIs4NdTHPFTsDzTo0ozsfwbnAwlpW/3i/ihhUWFTigFy6zXDqQO5vMuydErBeIZDxPAPv0Azslhtrw7ZsJJGZpk+tAlslXA+k= Received: from BY5PR10MB4196.namprd10.prod.outlook.com (2603:10b6:a03:20d::23) by BY5PR10MB4387.namprd10.prod.outlook.com (2603:10b6:a03:211::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6387.30; Mon, 15 May 2023 17:45:17 +0000 Received: from BY5PR10MB4196.namprd10.prod.outlook.com ([fe80::ffb:de39:b76b:52eb]) by BY5PR10MB4196.namprd10.prod.outlook.com ([fe80::ffb:de39:b76b:52eb%3]) with mapi id 15.20.6387.030; Mon, 15 May 2023 17:45:16 +0000 Date: Mon, 15 May 2023 10:45:13 -0700 From: Mike Kravetz To: Matthew Wilcox Cc: Tarun Sahu , linux-mm@kvack.org, akpm@linux-foundation.org, muchun.song@linux.dev, aneesh.kumar@linux.ibm.com, sidhartha.kumar@oracle.com, gerald.schaefer@linux.ibm.com, linux-kernel@vger.kernel.org, jaypatel@linux.ibm.com Subject: Re: [PATCH v2] mm/folio: Avoid special handling for order value 0 in folio_set_order Message-ID: <20230515174513.GB3848@monkey> References: <20230515170809.284680-1-tsahu@linux.ibm.com> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-ClientProxiedBy: MW4PR04CA0252.namprd04.prod.outlook.com (2603:10b6:303:88::17) To BY5PR10MB4196.namprd10.prod.outlook.com (2603:10b6:a03:20d::23) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BY5PR10MB4196:EE_|BY5PR10MB4387:EE_ X-MS-Office365-Filtering-Correlation-Id: 19df0e10-04f3-4042-210c-08db556c24d5 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 5Deg44HnmlRbFrAKTNaAY4ANMECKgEoq7oAxRUUephUDbE4HmS2NssKteBbA1H7VuF2KKdXFCW/mWg2H6ngeWifeA5zuSW5dlqpj4NoHKGY0qsnNKhQJX13HIA5YzYC5h71/CNkKJC5S/H+ZC6CO1X+2rvQHOgn/hkxPHc2PDIfuQ4w+v7Bj9+ugPACu3gm5YXlJ28yIxr6SErlsu2hoKCtEzGDZqCPSXpBjzplkp4ELd6h7oTyz1FPJPofw2M0/AMnCr88dgfjDGxXEhcyrSXmO1nHpcqmJz54UzB4fZl9OO/pH0CFWQBlwqTYrBs6a2/inMEs0jLhf+vydSXYzvTSrg9QeQfWh9hSs6s5aiU8SeNNW3ZGMlfakc/QyLGQ5Mb6f5zH7S6G2MKchl/QpTyU68bfv4z1n91mpaRtpjFoT6beVnZgJmJarr3kZarmkkE4atkkbljdQLsqXKVsON7w9PqnOTFg+ygTqSl6/5SEWi12/N7fDnPiqdPUIprcnXRtqBVALQTqBWnjen9BHxvJpQFeiiU03IBmlYMaqpT/jtdwRTqu0xLLvOleXTf3A X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BY5PR10MB4196.namprd10.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(7916004)(136003)(376002)(346002)(396003)(39860400002)(366004)(451199021)(6486002)(83380400001)(478600001)(6666004)(9686003)(6506007)(1076003)(26005)(6512007)(186003)(53546011)(33656002)(44832011)(33716001)(2906002)(6916009)(4326008)(38100700002)(66476007)(66556008)(41300700001)(66946007)(8936002)(8676002)(5660300002)(86362001)(316002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?VaKwbY3cmcF+fS52RZ9/BoJyWXYX1u1WNIZkWa25mbK6tJ0plNw2xok0c7i5?= =?us-ascii?Q?TOfpXsgnpKChVmPPpYSf+YgDt4YZU1xAa9GcNbipnZSLp5O9QXkv9hxnpT4Y?= =?us-ascii?Q?dsTuNLSiNqmn+0LXcRyFVP2sKmavC7Q+000JaljPLLseeMYPZIbrUZZRUNSC?= =?us-ascii?Q?Hj4c6VDtxvaf9MvJqJ7OHAit+5DhUA73q0LgaDAkDEU/mqhPggopPnUVdHpt?= =?us-ascii?Q?TNN59YyPBBJ9lnMbOjS4g8EwMY3pPLp7rqdCYWpoBoEL4M75U7iTy3QWwYMj?= =?us-ascii?Q?WuUlmwhIPVHUgksX/sI8M79fGtlwiIY0IF9mB58YKnYhTiQ1gW/aHDpf9Gtz?= =?us-ascii?Q?9AmCbdQ56JFim5/9Ob8OsySvRpZLeErVZpVrQ2Y/7/OHmSDoGAkdHGRPlCNY?= =?us-ascii?Q?5OmZILu+fbnEDEBVcCz9wfJujeAGVUO+cWP7825GolkSWLUoi5VpbfqWs2jV?= =?us-ascii?Q?IojvnUzivPU8cYpLi4vxxqByI3EVsg5BjsmtjCeA2Jhs+JwW5Jfy1v6XR1lh?= =?us-ascii?Q?lbaZFNvDDMA4yWT41ITtKR1/yNfB7o4YbXjubWgPqLlt1h7uXMu1jwMJ1bJF?= =?us-ascii?Q?bFXffg0EOWCWRsPvnVmzgvfdqyzsWy6lMovxF9KFm09fe5arg0vPBIt5Kvg7?= =?us-ascii?Q?23JRyHl+Q7U1EBEGTBwPVXOvVYNfNQ2KPWbOlu/CveDF/edXO6IPTf+QgyVs?= =?us-ascii?Q?wkNzR4a2F+rXXib7unP4jPHIG9NEe7H4yLeQo8F6scaccMaZVrikZswZu/Hd?= =?us-ascii?Q?PySY46Hr9PQh1nCPsy0qwP7zidIkiXe84v42el1QvfNLL9TGl0mPPBZqvKfn?= =?us-ascii?Q?3408REX5vF8O4VJMjKlCsv40ce5mDody4Tkzo5T/Lh/BFRy1/A3sUY6bWMCL?= =?us-ascii?Q?CN+PeoNaYsPmy26E6/f054/qO8KX3X7LK6VmCHqarWSu49QrEuoq5fbwBQIu?= =?us-ascii?Q?dlI0Q8X2lNVLS+nspHLIV339t1Z0adXVkI9VbC+whb43SbTKrpYz0IsRZEb6?= =?us-ascii?Q?1a4Vn46uNJeiKFoC7v6fDJ7n73FsFVNztHlrWnl5jdVqcyHtL/wrNf0OPIdF?= =?us-ascii?Q?E9QmY0xfReKaBWr+vQDuyNSpJwUJHs/43z7EtSUkNP+PcJHo33xmWmYy0UdA?= =?us-ascii?Q?e1izq4Zx65E1M4Q5SIJmf6VXeIQh/gRtEXFGgp43oG4YubwEjIVFgerOW6Q7?= =?us-ascii?Q?3sgjnSInM6w4Vsok4oV56jx1MXalyFDBrnTwZ6RE+8d6+AoCbcpHJq0wQpzW?= =?us-ascii?Q?pteMtRc8yXIeTJuHKtF1LC8LrtLKvqFHYRXGC+SeDloNE+g5obfRKg0aHcIK?= =?us-ascii?Q?bymUGadKU6WAFL50HTHH0VPxgd7yi0VgSeDp0AW3r7ynIFCF6dU3d3Fz6lJq?= =?us-ascii?Q?76iansabFmaNBWFiWeXSvJXm6QJPCz37WXCFgbokBDGCdthW4zHvKuySXR8Q?= =?us-ascii?Q?DLbfK8dppbNB8KnkomiEJkA9J0hZ81viflZmbVumHhb4PbfZpAkwk2uyZR2J?= =?us-ascii?Q?cAV9/zdafN8QhIkdH7jeSf1aCQQwhDiGRL566FgZzVZyBqu6GodHs7zyABsG?= =?us-ascii?Q?ErK/qsZmiA9CIPRCNbYCPvHk2bL+sLBEG/2wVHCJ?= X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: =?us-ascii?Q?+OGiWPFmUef92Q4dWwZBMmLrzzU5FB92uovIhPUvfC3eE1bOqZRoqhx5Yhl5?= =?us-ascii?Q?sA/b/LKpwwp4qmW2Sv9pUwxIiVw9CxqoO0AnL1WvqCSF1aQUxVwDpUO5BTyx?= =?us-ascii?Q?ztTUUDEL5oi43CW/x5am9pd2MVMkJ+YGq2Bw9QKDgBUi4/6727fdldDhgFhp?= =?us-ascii?Q?SrGdrzPw3D3GtHWOMpo3xK7CUnDvoAikO+6eaQ0LM/5hkvEKjFR/ZP8FcObW?= =?us-ascii?Q?QXME+fEzEZujBXjqeRI5CjZqwidHAi/akXK+r+XPwQo/jV9oDcKQdI/vT301?= =?us-ascii?Q?Iaam3LkY9eDtLVmMteedkzxIZ8z8oXJwcI7lpnnl+ooZJMr693sHsgF5xp7q?= =?us-ascii?Q?9Janbwky5OoJt4buuq9SiFidoRdNYAXf1ZkkeN4GcNHt4Luaw79DCDq/2nlp?= =?us-ascii?Q?Kg8pD3ziLbzzZ1EOdIJt1X1cKpoaKI06mCi6ea7eDgpjh9jdB6H2PPBoVkt3?= =?us-ascii?Q?su2Qo+QX+MUmNDxY7GGvHAcdFCjqSBnVuzSe6+Q5QUOABgyfqrEqXKB5nB7M?= =?us-ascii?Q?JMFPrJx+yvFZVKHn7XFHmlgzFiANZGQrSmXzw/HZtUlpAj5u/hJ/Biup4FSn?= =?us-ascii?Q?b56xHv/w+Yhnq55wWrSxXVTU33WoMt4bQSuOhrVo9gvAGV7IAOnUtRZBaVRX?= =?us-ascii?Q?4ZghyE0dwqq/L82NYA48Qf/8A6Be9JaaxtO2xXFnQLLXxJpsnr+b8pWBSvxj?= =?us-ascii?Q?m6uTqfNU1FEmXdOEoz3v8xc1v6EwdNqnzKW74wmRDiPmBr5bCBC03D8ET5kg?= =?us-ascii?Q?P30RQVKVqXqYFkXLWMhgTc2WFEp/Uly2y0qLlqPDb4mix2nczoKzIcCKNXzE?= =?us-ascii?Q?m3peERiue8pZSCiN/HhEeWCa74HBiGWfqmrwECZZmQnmrL+BiLmk0+r9mG60?= =?us-ascii?Q?NswLqw/2bl90qqPogBfgGLKk4nE4xzvrcjsD2qs55Bt5rhS9bT6Yk+jTYwPv?= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 19df0e10-04f3-4042-210c-08db556c24d5 X-MS-Exchange-CrossTenant-AuthSource: BY5PR10MB4196.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 May 2023 17:45:16.8080 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: Jtlp/8/PqCHgI2vgUY95ofJxXSFJgIxE/47J0VwZyfYo6Hwi+/TwMHNZ1vbpG1KobRmLQqAKU6fkDljJHeUoDA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR10MB4387 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.254,Aquarius:18.0.957,Hydra:6.0.573,FMLib:17.11.170.22 definitions=2023-05-15_16,2023-05-05_01,2023-02-09_01 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 mlxscore=0 phishscore=0 mlxlogscore=999 spamscore=0 adultscore=0 malwarescore=0 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2304280000 definitions=main-2305150149 X-Proofpoint-ORIG-GUID: ZNcqt4L6nGL5iv4QEeXq06V-hosfNhPc X-Proofpoint-GUID: ZNcqt4L6nGL5iv4QEeXq06V-hosfNhPc X-Rspam-User: X-Stat-Signature: 6cob169k7argg7zkot3upbkht38ubn3i X-Rspamd-Server: rspam08 X-Rspamd-Queue-Id: 05200A0011 X-HE-Tag: 1684172730-901003 X-HE-Meta: U2FsdGVkX1+6pXoen41E3nQFoFI8uDnhQHFeHgalgX4PGBdORryssmKptS6pOFLaqw7IiJ4rU90nS3+n9x9AcWsy+SHjwRWrZp5TJQkqnydXaKpLXC1QtZ2l5ktdjVYbuQW6qwtRmJwlOn94+ptsgC1ReAvF4YVOo6fdTE/yf7pUBWxnmh26hQfB9+bq+CE1raSfmYnV/cQPpvTOPk+tgSx1aQdZ5WnbQajfOXS7ksiSKe67nJ0LX5RKw2p84eshkdwdvLW9Rp+hvpxpRDzC3e0bpdnp+fucZuWHANszBmBvdTHzJyvDKTj5YMvaapTeNWs5BTT3C81nhCJbg47QQ0rTgbA7SNvfgi9fY0/Ly9UFo0qnDMmGh/QUYP1lfbHA7b7jdPvmLsKbPKyHeXRhuVpb3TaS9OKx0/RiHwdTrgRyinxNbIRn4GcCJJcFlYlg8yrrSXJ2E4aBh8zKf+74+TJHRX1uGwMtPmGg3jWOP4AWPwZex435rTmicOEYhfym7twDcfGaRtYGEJAFaSlByaFQQn5dwJDIR87EdE8uuC/BdFZzC4JJSMYv7X1ggH+5RKdLn8j6Iim3oM13Alb6YIBMHp9PHq2ojKU2QeEJOBUrL8D56LPTJmfGG5CgsSDjujA0LyAE8MpSMwQMfCa52b9aOOSoTF+2xk0vEPnZAMh2ukgfAlBlPCeauIu7RzLVOqWCyJGcNAtBIRnOux34ePM5SjKmLTzvAJwMfQtE1AxHt7mjvqH+rTm/S5WgmHRp3zSe989cWNSyNLc0IKH9DFxAnzgA/wj4ueYaPg/cQv492dTQ343Nd7UxCvGybIuDQ4FtwwXLx+n+6TVgoL+X97YHM+VMckOH3dBP6KYyksoPOQDcJguxB+FIpSn6whL3CheGNR7Bv26od1yKgh5eowrodrqru3j/puSCPNC/E9gsp/s2mHgaD5XVmw6+BGecks/9ym8G5YPlMKNGxHE EVGoBZOe jW3LcBWvl1XrvpcdJNqOs2PILSZKd8Yn67fcBPw0A0pVtSyFrHKxZEFoGAKNuEd5NS+X7KAi02XO5lJ+HsBZd4bv8cKNBPlx0raO8GYinV6KLLow3lvrRLNUI1TF1HNxFbXhxsuvmljlGb30S+SelJLT4Nu1whC9vKWIkKYXgDDY6HI9uUu5vUc36DiaclNde3F8bkNLvgrZL14uo2FajXpQGLnKgOHIWTm7+Elnm8xJU4346BIgnnp3PB5Zy6L1+37L6udFB71PIKOPEUyWC8Odp3RNMryS+ljxdz2FfhV2P7C0QGUaR/VAl3ahPyyUMLfIj194K5aHwhW7R5IMTjvACbBOxCVjgV4oUwfK1OeZdNZD+79Uk+7O1LAxYrsR9N4f44UUTDiFO6zm5517hP7homzwvrLXvtQov X-Bogosity: Ham, tests=bogofilter, spamicity=0.000178, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On 05/15/23 18:16, Matthew Wilcox wrote: > On Mon, May 15, 2023 at 10:38:09PM +0530, Tarun Sahu wrote: > > @@ -1951,9 +1950,6 @@ static bool __prep_compound_gigantic_folio(struct folio *folio, > > struct page *p; > > > > __folio_clear_reserved(folio); > > - __folio_set_head(folio); > > - /* we rely on prep_new_hugetlb_folio to set the destructor */ > > - folio_set_order(folio, order); > > for (i = 0; i < nr_pages; i++) { > > p = folio_page(folio, i); > > > > @@ -1999,6 +1995,9 @@ static bool __prep_compound_gigantic_folio(struct folio *folio, > > if (i != 0) > > set_compound_head(p, &folio->page); > > } > > + __folio_set_head(folio); > > + /* we rely on prep_new_hugetlb_folio to set the destructor */ > > + folio_set_order(folio, order); > > This makes me nervous, as I said before. This means that > compound_head(tail) can temporarily point to a page which is not marked > as a head page. That's different from prep_compound_page(). You need to > come up with some good argumentation for why this is safe, and no amount > of testing you do can replace it -- any race in this area will be subtle. I added comments supporting this approach in the first version of the patch. My argument was that this is actually safer than the existing code. That is because we freeze the page (ref count 0) before setting compound_head(tail). So, nobody should be taking any speculative refs on those tail pages. In the existing code, we set the compound page order in the head before freezing the head or any tail pages. Therefore, speculative refs can be taken on any of the pages while in this state. If we want prep_compound_gigantic_folio to work like prep_compound_page we would need to take two passes through the pages. In the first pass, freeze all the pages and in the second set up the compound page. -- Mike Kravetz