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 24AB1C3DA49 for ; Tue, 30 Jul 2024 03:25:02 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id B4D436B0089; Mon, 29 Jul 2024 23:25:01 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id AFCF46B008A; Mon, 29 Jul 2024 23:25:01 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 99E1E6B008C; Mon, 29 Jul 2024 23:25:01 -0400 (EDT) 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 7DF536B0089 for ; Mon, 29 Jul 2024 23:25:01 -0400 (EDT) Received: from smtpin16.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id 299B41401F2 for ; Tue, 30 Jul 2024 03:25:01 +0000 (UTC) X-FDA: 82394977602.16.BED380B Received: from APC01-SG2-obe.outbound.protection.outlook.com (mail-sgaapc01on2069.outbound.protection.outlook.com [40.107.215.69]) by imf17.hostedemail.com (Postfix) with ESMTP id C098C40010 for ; Tue, 30 Jul 2024 03:24:57 +0000 (UTC) Authentication-Results: imf17.hostedemail.com; dkim=pass header.d=oppo.com header.s=selector1 header.b=S+y4Bz0R; arc=pass ("microsoft.com:s=arcselector10001:i=1"); dmarc=pass (policy=quarantine) header.from=oppo.com; spf=pass (imf17.hostedemail.com: domain of hailong.liu@oppo.com designates 40.107.215.69 as permitted sender) smtp.mailfrom=hailong.liu@oppo.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1722309854; a=rsa-sha256; cv=pass; b=IvCG1i1ll0u6mjSwY9a3fj+Bp6DWKXZ4zFACTm50kSaDItXVSsuEcUMaQ/xE7aboW+OFGq 0zdqkx3CvQQov6JAIZzWfJJg+9GQKYRprJPYqBCaFywggUyhTJctstmLS/Nzn/D2PSk8NA 1BKlstJNLbaRSyD6Fq04g3ucKg7CFEA= ARC-Authentication-Results: i=2; imf17.hostedemail.com; dkim=pass header.d=oppo.com header.s=selector1 header.b=S+y4Bz0R; arc=pass ("microsoft.com:s=arcselector10001:i=1"); dmarc=pass (policy=quarantine) header.from=oppo.com; spf=pass (imf17.hostedemail.com: domain of hailong.liu@oppo.com designates 40.107.215.69 as permitted sender) smtp.mailfrom=hailong.liu@oppo.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1722309854; 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=5RH2E0B+oMV3na90A4cQXLYfVz8Bz39ACULOQRj7QDg=; b=0Zq46VGVA4nwMjY8vzwX5qn/7+inHi6rZPkms69LwJBaf8fqQY4mewsZJXgzCOB4EXSWOO Ned+v/hH8IMQuDMiJYVtcMq81fdBN2liZ9MMhjpBC0fhPVJyesgBE4vaQe2zx7hwoBOmmf 1vuy+onwoDeqnSIBwHhkUDsjM1lHBi0= ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=KaA34OCkIZskZ5uNUGFXoxgoof/qQ+sFSpgkaSkuiDSPPg/DdSDFKnxBkVP2cqlnDmf3TJS7QXewxWXJe8hdPKaI/V2qvaLmoxXWpEeQ6GdGR9ikFvegUP65+Rwz6BSNOnr1CxprdnwC8uTtuDX6ofoeMR8So04seYQD+CEg230ys664jU/RgiKWhC9QzVm7T2lWpIXFSvN4liV9Oo2eaP6XdD/5hbm2Oc6d1wgMq2lAUFP/7TMygOiHwb8TyBEDwC/UJ6k06FhQIrovmOUEh3RrRsFlFQx6pEFoN/JCklKQeHeMIa9/LFCgJiKvGUmCbN4wSEbd+8rHGluYhm1nGA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=5RH2E0B+oMV3na90A4cQXLYfVz8Bz39ACULOQRj7QDg=; b=PKGDkAr/dGXr5q7buw/0cuMTm+OtbZSk0+lLjUhgTJfzIdDwnqtVGlk0q9wv1z241ekzHxNuIZ93C8BMqkBr3WGf8EAh9wTfaSS+ezx5mbTB0LCLxoI4hkzwKTBZr7XPGeZfB2dib9Kv5UcQ/qXV89CGDHdg1hjf/bEsmnXtqTOEBdWZ4FD17p9Du3CIbyKrMPVmKAc9bSKaYRRfaHa5iw/ZtuCk0UJld3wZs7HodItHxmzEjMcYw1irFhwScRiso/tmhhFyr2Q2jgwchpw2ccFMQ/04UGhUsA2x9O2p07yGz1jxdf9TyfeYnOZsCoWVPf0M3fFE9qrpHPvdwMASKA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 58.252.5.68) smtp.rcpttodomain=redhat.com smtp.mailfrom=oppo.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=oppo.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oppo.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=5RH2E0B+oMV3na90A4cQXLYfVz8Bz39ACULOQRj7QDg=; b=S+y4Bz0RSL+VF48Ej+eZK8/Ltrk1aES449BJznhxRincP9YBBQtymJ6RprWTYqFeer1nQPmkdRZqpWEyBKzk8x1SGqZfjlJ8WeJhsTsZXHWo7LYzbCOas/yB3Lje3jnnjH1qBRQ5HtRlAf6WxOGMZBuzGScbA0t4VbsQbtIkkp4= Received: from SL2P216CA0190.KORP216.PROD.OUTLOOK.COM (2603:1096:101:1a::16) by SEYPR02MB7198.apcprd02.prod.outlook.com (2603:1096:101:1d9::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7807.23; Tue, 30 Jul 2024 03:24:48 +0000 Received: from HK2PEPF00006FB5.apcprd02.prod.outlook.com (2603:1096:101:1a:cafe::eb) by SL2P216CA0190.outlook.office365.com (2603:1096:101:1a::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7784.35 via Frontend Transport; Tue, 30 Jul 2024 03:24:48 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 58.252.5.68) smtp.mailfrom=oppo.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=oppo.com; Received-SPF: Pass (protection.outlook.com: domain of oppo.com designates 58.252.5.68 as permitted sender) receiver=protection.outlook.com; client-ip=58.252.5.68; helo=mail.oppo.com; pr=C Received: from mail.oppo.com (58.252.5.68) by HK2PEPF00006FB5.mail.protection.outlook.com (10.167.8.11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7828.19 via Frontend Transport; Tue, 30 Jul 2024 03:24:48 +0000 Received: from oppo.com (172.16.40.118) by mailappw31.adc.com (172.16.56.198) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Tue, 30 Jul 2024 11:24:47 +0800 Date: Tue, 30 Jul 2024 11:24:47 +0800 From: Hailong Liu To: Baoquan He CC: Barry Song <21cnbao@gmail.com>, Andrew Morton , Uladzislau Rezki , Christoph Hellwig , Lorenzo Stoakes , Vlastimil Babka , Michal Hocko , Matthew Wilcox , Tangquan Zheng , , Subject: Re: [RFC PATCH v2] mm/vmalloc: fix incorrect __vmap_pages_range_noflush() if vm_area_alloc_pages() from high order fallback to order0 Message-ID: <20240730032447.avaoa5uhcxdkcjzs@oppo.com> References: <20240725035318.471-1-hailong.liu@oppo.com> <20240725164003.ft6huabwa5dqoy2g@oppo.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: X-Originating-IP: [172.16.40.118] X-ClientProxiedBy: mailappw30.adc.com (172.16.56.197) To mailappw31.adc.com (172.16.56.198) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: HK2PEPF00006FB5:EE_|SEYPR02MB7198:EE_ X-MS-Office365-Filtering-Correlation-Id: 611191ac-e4c1-46f1-e3a3-08dcb0472ab1 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|82310400026|1800799024|36860700013; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?Zt9jWdpiQ3nePCJqgQCnleV/01pkydUJ7F73ezCPQpT6uWCRps4+M9kJ/9fM?= =?us-ascii?Q?CvI0QjlEXNOxAzSSC4Ldw9fUbgApPMi2yr1A+xhUyjY61v1XTlQgSggv2gCB?= =?us-ascii?Q?rORJFMeT4nXlpQoKgaleplGE+YvIW3UZplQLsmcQCjIhDpvJv628aNuGC3Um?= =?us-ascii?Q?zhTkpVXkVrt5FZx6cb7G7peBzz3WlbmuRbIwLgSoIegLOjJeIJQYcZWQeVQR?= =?us-ascii?Q?0+9BVgXaUS7UGt23c21C73jbdgWrwPwnSNxH6XNkt8HbkafdpGq+6XdqeTgJ?= =?us-ascii?Q?vss8w2RRgv+TPL+JoAmy6f56Bur4mEgcqLqNxRAXhKtfCTKE8Os+fP/pbFF9?= =?us-ascii?Q?aXXKua7aklUcVfamdUc7YoPZC0vSxLtXr4ZCmGeh0d/i56a/sIl6+ORxWikH?= =?us-ascii?Q?ZkzttabL4p7hmOOO8h0QDBZuDVatG7zdli8YWPvA/8v/6mwFNfnRl52Qs0kU?= =?us-ascii?Q?NUwMYl17odcCn5oiiwq6HIcsDGCOQLBZb3Y0fhv+wfuApkZ1id/et+4EJ8Xd?= =?us-ascii?Q?kF0tbopbiwuxybEiHiytaONCClUV0Res6sPD5IkxjaVk0BikaPFHnUlrVxI4?= =?us-ascii?Q?7qqLUFYsB94Up4eoxs36vvWC7zBU7WuP0sXKn2Vu8bCwsWwEl3oj+oOY+bPH?= =?us-ascii?Q?7un35RGX/l3nbeahHWIv1DyCMCE14I1grfpsBrPhgM51sqMZ5Ma2votn6+Wg?= =?us-ascii?Q?VQ2zoSfzUHgxMZy12zKRoSWY6evjQ92LlWhCIjw8MsjGI9jtW4pP/34f5JFd?= =?us-ascii?Q?kIp2rr/iV5mRnhEmkTIyIYQdT31JP8OSWLgL9ZaCuf4zSsA0NuBoPUFfjGHc?= =?us-ascii?Q?qjRycO+97F+qLk06+yWy230cbE8J58QKvKhRgIklgKF5f7+eX67Ac9+7SDmI?= =?us-ascii?Q?MEJadwpQBa4nbhJRDjEJ4kVTIo0fdoTDvGdG+wleZRHWTPPRMtkXyQ2TU/bt?= =?us-ascii?Q?K4KCYj9TLVKWg24M4oMpqv8F/VgdLIO/1OXp0T4+Vxe5mSGOgXZP+659FfYM?= =?us-ascii?Q?/I8GTloBGJ0jiKWOhVlYjiCcKk3hprvPqQrFxO1DlWhUoPxXO04I4iAjAvZI?= =?us-ascii?Q?gJUhW8z4zLjY6oNDYQYi3nHAFFC7VlQS6pq1rc2srDLj/+uU/LB8C8dltY5A?= =?us-ascii?Q?dyN0ff+Du1PDQEgg4iF0AUIoWtFn/K3xbqUZeq50ONXVHrveoS04SAFKH1Lx?= =?us-ascii?Q?DD+dFG52bH208+HYQOJcuZLMFcyGqdVb42ilyKKsgqnJbAt+WRM84ODERCnL?= =?us-ascii?Q?0lOsGhmGPCNkPhGO3qSUmufuErDGxW0o53zVu08igbEGJiKaW5rS9B3V+3B2?= =?us-ascii?Q?JIiaQsRKeO7ub/kWNtuopLkutJMNU8ZbC5mj6xlAuVkX8FS/jYAbyFj7TNH6?= =?us-ascii?Q?fsxUsSrav7kmSUwXyvC7TUJhiok4w4vDGGtWbyVpRYmMCxPqjn7bVQLAlAGE?= =?us-ascii?Q?2HFXK1dcloyf1nTD2YlyNkX1iKY6jUxv?= X-Forefront-Antispam-Report: CIP:58.252.5.68;CTRY:CN;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:mail.oppo.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(376014)(7416014)(82310400026)(1800799024)(36860700013);DIR:OUT;SFP:1101; X-OriginatorOrg: oppo.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Jul 2024 03:24:48.2891 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 611191ac-e4c1-46f1-e3a3-08dcb0472ab1 X-MS-Exchange-CrossTenant-Id: f1905eb1-c353-41c5-9516-62b4a54b5ee6 X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f1905eb1-c353-41c5-9516-62b4a54b5ee6;Ip=[58.252.5.68];Helo=[mail.oppo.com] X-MS-Exchange-CrossTenant-AuthSource: HK2PEPF00006FB5.apcprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: SEYPR02MB7198 X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: C098C40010 X-Stat-Signature: ua6c8jkujqy4qwqx4sozq3df5odpyrh1 X-Rspam-User: X-HE-Tag: 1722309897-417065 X-HE-Meta: U2FsdGVkX19zXcwBNQkW9245npIiR0GCLn14IqeWSQMF5x16dv96cOyhC0TI4gsUK7g3CbN5wSizWBf7ZaLD1fQE4BDR8WaN6MeSk834CfcMpMmwejxIVnzl/vnwOEmLdX5XFluZlZ3dkDXwSZ5vuWsu8AER+vrbBzWseFf/+odLUb8f8gMH6fLQNQ9lwcSh51yz81/OdrEAyG+wwLKdRQuHI99MGW0YIHbWPbaPV0gli9d7t0HhwlfP3qFKd5SOaSGirkhxG/9I6o3fjAXhnx1TGPsTO5OWxHK/ZdZZ8Rt86LcF/WCv5t8vccYsR24CSVj7adm/YgUh3YaRuw2A1qsnC7NVoQsbCy8BTI+d09rXSQgcv7Sz+4kCwypwWzAyfhd7YeoWMaavoSdtghP/wU/aHRtgWip9ArH8thBN8W+TN39LH0RMUIDuQ/nu1HLrkzbu3WRKmpwG3d5weN0A3ow1sxphsoioYvthPDoWv52lnlbboJXpHUfZqjJsynstHP7fKrBUGzvXF6juMSb5ui5YFFAqbOE28jKUfUlWsyjjjdogg7P3fKGU7KqSLN27sFAXXu1l+7+NyOdkkOhUa0JUH3y5wr4S+9qtLJlndAJJzjZ6O3lkFXbkyckirsMkliorD5FRmhTO8D0AtKhE0NkjRvcOZI3NeAcrJWWNh8NLb538vQGgAEpw/9WBqt7c0ITmiDm5ULwJaB/yk23pumTc+W5ruC59uVQ6ueEndH6LKse/iXtr+uUzZ0tyduQrReuNtQi0HgbI/dLXaZi5Bn1H9+tgtB6TZnaEqW5R3pICqxjKczV7qqtcsMvRz6YAVfvIO1vz9bhsFcDa1GiKsSdrN2nvornUpkPavBvQTuUftCRpO2X3qF84Q7zmNHAhPC1sglQnbcF0TfWI/5KpQ6IsObrU5wK2bostsDI349jXP3lqlroP1w6nSUlUEacM62GUK8dfMpk+76H6x5z 8tDgiosc Y+BuG9DE0UqALtOJk6ojtP5gh0EWlX/RjrxWYawO7PHf6JVWZd9UVSdFadATsGZXcdDkjl2J0G7tHHkR2Nh0ugSAdLW4JBSctpK5AvquPhW8CY0k4g0DRXiWmZ/jwXU9ond844rpWN0BtepWAqph6ZggEzqsWIwBvoo2RBuZ0anTiXDfb4LfZ4HV96Y/pg9CmSeSEjxn4WGvoc5QADCD28ThI6lCSooIDN1nZgsmxA9SMVfaa7Yh/daTMWIGtYHgf6OtBbbcSyMQSp3uGVxV02Tg7J0jrI3tnSHAUdNsRgmmr3T6SJKDTAIo/bjvg79Vb9T6qoA8mM3diIvWTXHxd6cZ7GtMJ+Clsa8Jc/jiZ98ZZL33/ukhv7dgMnrzpnzLKZZ2XNdalQyIcmIQ+MH8yG978XgkgzHc6aetYWfcZiduBLoKsMUwxk2+ULqP6/luU/5Zp4q8fl+wLGD4dBeq0Pm5uCbvGMGV40PU3NK6qOv0Y2A4= 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, 29. Jul 09:48, Baoquan He wrote: [...] > Hi Hailong, > > Please feel free to collect them to post formal patch, maybe two > patches, one is to allow non-nofail to fallback to order-0 in > vm_area_alloc_pages(), the other is passing out the fallbacked > page_order to vmap_pages_range() if it's OK. Sorry for late response. I personally prefer to revert part of the problematic patch. There are several reasons: - Save memory usage if high order allocation failed. - If nofail and fallback to order0 in vmalloc huge allocation, actually the allocation is alighed with PMD_SIZE or not PAGE_SHIFT. You might be concerned about performance issues. But IMO, - If we fallback to order0, we can make use of bulk allocator. - Maybe we can remove VM_ALLOW_HUGE_VMAP in kvmalloc. I am not sure if I have misunderstood anything. If I have, please let me know. > > Thanks > Baoquan > > -- help you, help me, Hailong.