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 99914C4708E for ; Thu, 8 Dec 2022 02:27:48 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 247BA8E0006; Wed, 7 Dec 2022 21:27:48 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 21DCC8E0001; Wed, 7 Dec 2022 21:27:48 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 0E6188E0006; Wed, 7 Dec 2022 21:27:48 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id 000FF8E0001 for ; Wed, 7 Dec 2022 21:27:47 -0500 (EST) Received: from smtpin29.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id C33BF120455 for ; Thu, 8 Dec 2022 02:27:47 +0000 (UTC) X-FDA: 80217553374.29.FEDD370 Received: from NAM10-MW2-obe.outbound.protection.outlook.com (mail-mw2nam10on2044.outbound.protection.outlook.com [40.107.94.44]) by imf17.hostedemail.com (Postfix) with ESMTP id C54CE40002 for ; Thu, 8 Dec 2022 02:27:44 +0000 (UTC) Authentication-Results: imf17.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=C+tnESFA; spf=pass (imf17.hostedemail.com: domain of jhubbard@nvidia.com designates 40.107.94.44 as permitted sender) smtp.mailfrom=jhubbard@nvidia.com; arc=pass ("microsoft.com:s=arcselector9901:i=1"); dmarc=pass (policy=reject) header.from=nvidia.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1670466465; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=Kdoa9y69mXXGjQfWWtkPtvumDpGsHjUX4HaerkUp3RE=; b=Fwr/1V21RGODmeAp6e3+UHObRK9E2vvW7JtE/7d8PFQDpcLnqoBd4i/c1Zf3mGzclK9Thy WyMr6s41qFAmq/W1pSPcZ/Yd0suabyY0E9E58t1PnyB0+amsIyVt2qHhCLmpgCbTAh9ndv OAWD4B4uSOdMZa23CX705JMlTuzs6bA= ARC-Authentication-Results: i=2; imf17.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=C+tnESFA; spf=pass (imf17.hostedemail.com: domain of jhubbard@nvidia.com designates 40.107.94.44 as permitted sender) smtp.mailfrom=jhubbard@nvidia.com; arc=pass ("microsoft.com:s=arcselector9901:i=1"); dmarc=pass (policy=reject) header.from=nvidia.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1670466465; a=rsa-sha256; cv=pass; b=DqlfsP82MEw7zrbcGBxCxkv/X+YSkPYaYYMF6eyV/ELG/xDpdUrOM8HnPb/iiA+UYnSXny vMv6rfR6E+ZoPGzTRhiD2WArg/dK/VvTj5OnOfFTlIKRmeYBwhN8l6Gf4+JGZ/wfX3HHeD seFPS/ZPTQNVtmB4eYLGuJXkKz2EhXM= ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=nitoRDSbyPuxvWVtO799Uvaij9lRlhu61rb1nL17EDjiCHvKu48FT5OjU3sbGhIJ/PbLjI2JjRhY7IBTjzi4pxhkAbghdtT9AmumbjpzDhpde7eXBxTwzJO6YJuYSs4XpLkZrAGQ3cGRoTOuTIoQkPaeXywCc+CMiP5NbFMcQJE05VmdmrMeVtCS7/GGcDsZ9Z1o2Wm7SWEyDfpkFdfdDeANLu/JLQIn6Nt14QHW5UG9+eews7sdDmb2cccdxKxvqCsjAR5gHG5p/rGWUxHRMgiIg3YZGAcvmjh4WddL/D0oeilcZq7xsy7Y5gAoV88uVbgwCr2zHmrmOj9y+Etiuw== 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=Kdoa9y69mXXGjQfWWtkPtvumDpGsHjUX4HaerkUp3RE=; b=jzrI4/3DFVu+rSpdQ5pfVYsPzJunPngXdWQ75oGYL8TYCS5u0/ISC8SpeMgYlluGeD5vLxQyHZGb5Ui1kBZ8cIRG3v+uNILkkLlwDPxMP4MoZORVYByWHvShfkncz+PiPvUO5vluXKCmPaCJ6uJdBp9DQt0W4qv+WVOkIKdn4BzduharR9EYsyAp/o9DHuohvU6gqPgdsm5gygmFPOp5/yw06jtuD2rbf4NmRBsBlUI5kbve9J6NtRQQC42Ksx8cLJmgoymLyRJXJr4NYBEaQMhcRQFem1Tkrqt2LcQmslr7GdCwoUiwS7/JmVOPnRnNHqvNLAU6nZAjeFhN5lN/0Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.117.161) smtp.rcpttodomain=oracle.com smtp.mailfrom=nvidia.com; dmarc=pass (p=reject sp=reject pct=100) action=none header.from=nvidia.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Kdoa9y69mXXGjQfWWtkPtvumDpGsHjUX4HaerkUp3RE=; b=C+tnESFATnlRVSg36JCymKtaoZZ6iyXSgrJ2kspRLm7lMn03s0JKsn4nepq6LmlkvjI8XzJ+si8Pd/0uMAHVezHTMHvPSF1GWXrXXq8wI+sRANAAAOv2iizJVcK+YXHG47DQhwcmt5uWraXpy7NayY7vZBXfZHuuCTOYFSfcmAJ4TtqpHjHYN8+Jh8caKdkUoSzJvNA1LbcdMPBlA9zknihBm7Vd+AGiDtI9u2TXkbL2d+jcRTZ74eSxdz77p9W8qdgMqfVTfjPOv+9mY4uxpm2DOjZpQ6/aSvj77QPJnkak5MiA9IFW/hBOWuQUxeF49G4+6TiGvu9Z7Obru++rKg== Received: from DM6PR14CA0069.namprd14.prod.outlook.com (2603:10b6:5:18f::46) by MN2PR12MB4064.namprd12.prod.outlook.com (2603:10b6:208:1d3::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5880.14; Thu, 8 Dec 2022 02:27:42 +0000 Received: from DM6NAM11FT040.eop-nam11.prod.protection.outlook.com (2603:10b6:5:18f:cafe::25) by DM6PR14CA0069.outlook.office365.com (2603:10b6:5:18f::46) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5880.14 via Frontend Transport; Thu, 8 Dec 2022 02:27:42 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.117.161) smtp.mailfrom=nvidia.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 216.228.117.161 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.117.161; helo=mail.nvidia.com; pr=C Received: from mail.nvidia.com (216.228.117.161) by DM6NAM11FT040.mail.protection.outlook.com (10.13.173.133) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5901.16 via Frontend Transport; Thu, 8 Dec 2022 02:27:42 +0000 Received: from rnnvmail201.nvidia.com (10.129.68.8) by mail.nvidia.com (10.129.200.67) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.36; Wed, 7 Dec 2022 18:27:30 -0800 Received: from [10.110.48.28] (10.126.230.37) by rnnvmail201.nvidia.com (10.129.68.8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.36; Wed, 7 Dec 2022 18:27:30 -0800 Message-ID: Date: Wed, 7 Dec 2022 18:27:29 -0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.5.1 Subject: Re: [PATCH mm-unstable] mm: clarify folio_set_compound_order() zero support Content-Language: en-US To: Sidhartha Kumar , , CC: , , , , , References: <20221207223731.32784-1-sidhartha.kumar@oracle.com> <92965844-c430-8b8e-d9f1-705d7578bceb@nvidia.com> From: John Hubbard In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.126.230.37] X-ClientProxiedBy: rnnvmail203.nvidia.com (10.129.68.9) To rnnvmail201.nvidia.com (10.129.68.8) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DM6NAM11FT040:EE_|MN2PR12MB4064:EE_ X-MS-Office365-Filtering-Correlation-Id: 7475e76b-4ad7-4513-32e5-08dad8c3c89d X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: wTUjpcFP+XW0J136e5qzvyOoEG45UtE8ft6a3/itpQlbarusIvNvIP837UyKGdDZl8WbRHsd0r9Rj/ZFl3NfZ4naX1j2vX8pnRy8MiGmN7YAk9A/v/VSU65J1IYlycQJhihNtSSLvDXcqsWzUvZRglRgIUCYt7sRg4mPJFTwn7ahq+aJNWdmXRwoxJC9L5XkkBXdnNiX83RdNW9exVuUKb/s9Bj0Q2/EkMD20mT+4/W2/TI9CN7zbYN2fx9WzhTSNgU69wLP0aYlUA8Zwl+Ul7R52RUVvlHpMqlatWv7ikf3ZU11tqo2DPBKHXfleHo4WpqbnKv/ombajn9jBD9TZVuhW/xE4lehZhOckq7AMCmdSCTQKTYNI9K/Q1fkQ8/dzd5T6eZOPGDY1/b1j09EUKucvdELHub/nYmfzzbUeWZf6kh3NfPKIqRP6kV6Vl/9/lrPK65iXVuk3oieQ84DXGfqyM8DwBFuw5oGZ07Fi/hTviKqbpmrkOnoWLyfY+mFmjWaXO0Nz8O9noZEtKVXqz+fVG5u3Cvsp5RQUSKv3PJk50vAy72EHJCnL0T7peEIm6qPWM6EnRnmdkEppCTXPfk/MG8xWJ9/MAPFY/sNKoqIkf1zN8EliEaHhwqMjZpiZGqXGYGSI16y8wPk35MbTGiexWkspUbm80yCM5XQ0D/ipJ9g6thlOh83gENsBaAFdCuPeveRKkzkU7ohv5NLcuzuGPW+7nEcWWqXy6zizkN9OepymLwN2qFI5FUNN2tx X-Forefront-Antispam-Report: CIP:216.228.117.161;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:mail.nvidia.com;PTR:dc6edge2.nvidia.com;CAT:NONE;SFS:(13230022)(4636009)(39860400002)(396003)(136003)(346002)(376002)(451199015)(36840700001)(40470700004)(46966006)(31696002)(41300700001)(36756003)(70206006)(16526019)(186003)(40480700001)(40460700003)(70586007)(86362001)(336012)(4326008)(8936002)(8676002)(5660300002)(47076005)(2616005)(316002)(16576012)(53546011)(54906003)(426003)(478600001)(110136005)(7636003)(356005)(26005)(82310400005)(82740400003)(36860700001)(2906002)(83380400001)(31686004)(43740500002)(2101003);DIR:OUT;SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Dec 2022 02:27:42.0751 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 7475e76b-4ad7-4513-32e5-08dad8c3c89d X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a;Ip=[216.228.117.161];Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: DM6NAM11FT040.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR12MB4064 X-Rspam-User: X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: C54CE40002 X-Stat-Signature: 3ysikiex3j86qfhgfkdn9c3qh93jrf8g X-HE-Tag: 1670466464-240313 X-HE-Meta: U2FsdGVkX19DXTiziig09aZwqkaMPTY0FJaEEZPXv7CPyX3DzrMdcEUcR1bexk02ojvxsg7Aw8iUDvGky9mQCHpVUDDv5rna/AWRINX2rW2PgVCui6igYQJCRR8Xbvwq4Mjmj1Aljcmyf3/GmK4btbDCc7xk8darw5ER4Inc8bwISwhDrJNtHnJ0l/Nggn8bqMJxNXYMwigb460oA12dlkMHgx+dkOVa7t4shgD9n44dqFZ7IzkmefuFjlYlonGxEyvCnafpkIPFfYYwYmJ8h1v3wmg/OlHSL5g7ERkY9bOUCO6HMzxIW8Bl1XU5c7A+b1voM5NF6vT2St0IuHzr/LL1CizXwzIJ+AD2VWtnKcote/eKJeyz4SvRwkSRv9lc0wd5YxcwanZIvren2/utwOqVu4ZQsAZAsO3IEFTkFPXey8xUK9/QgDIBNNa/vTrllWho341xzi3DAtTqoWIphbKAx5e02rRoqB+EdENxkC7qC279IuUdvBF8bkPLxKrzUTIHSM+ByfXOEK/yh0mM5FkTlsOqWlhC9sfLXIFBxK0T2hP64OgsUkTkDjun3kvURPyEsQVdUUEpchaAVu3hYuxWVQU4/X3XtZjnLMLTu2S2AUVYBTMaKA7phyiPgJcybGKOuJqDVkRdAOFd2824ifUkVFMsmL7Xf6Q9A6OzfLDUwEB8suRMP4WYOhaQ3NIs9Gfl0TwJujmOcbfE7/MtzagA1BMVlJK/QifiZ60w9nhrGH/t9BXdAcIunQt6Ro8u76zJ3hUmvSUIwcVMh6Rl0/w4MeO1/n0tgW+XfpeJQBSnwSILNMxrOf/PPdfnxisCDCn8WnbWbIvSYxPOhrtF8R8rVMTgsOxpLm3VUNf4Vm7JYJg/u5Qw2D3BEkF3PleSdbP44iOy/FUlKlQXfoO61H64kK+gUDFHGhDy/HC1FD+WMSvgoE1sKnw80WxbCGVLywsOvkAKDpFjsKug/ly tJP/An+o 5JrabxthJ6qHvOzwT94RGh6Vl/Q== 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 12/7/22 17:42, Sidhartha Kumar wrote: >> Wouldn't it be better to instead just create a new function for that >> case, such as: >> >>      dissolve_large_folio() >> > > Prior to the folio conversion, the helper function __destroy_compound_gigantic_page() did: > >     set_compound_order(page, 0); > #ifdef CONFIG_64BIT >     page[1].compound_nr = 0; > #endif > > as part of dissolving the page. My goal for this patch was to create a function that would encapsulate that segment of code with a single call of folio_set_compound_order(folio, 0). set_compound_order() does not set compound_nr to 0 when 0 is passed in to the order argument so explicitly setting it is required. I don't think a separate dissolve_large_folio() function for the hugetlb case is needed as __destroy_compound_gigantic_folio() is pretty concise as it is. > Instead of "this is abusing function X()" comments, we should prefer well-named functions that do something understandable. And you can get that by noticing that folio_set_compound_order() collapses down to nearly nothing in the special "order 0" case. So just inline that code directly into __destroy_compound_gigantic_folio(), taking a moment to fill in and consolidate the CONFIG_64BIT missing parts in mm.h. And now you can get rid of this cruft and "abuse" comment, and instead just end up with two simple lines of code that are crystal clear--as they should be, in a "__destroy" function. Like this: diff --git a/include/linux/mm.h b/include/linux/mm.h index 105878936485..cf227ed00945 100644 --- a/include/linux/mm.h +++ b/include/linux/mm.h @@ -1754,6 +1754,7 @@ static inline void set_page_links(struct page *page, enum zone_type zone, #endif } +#ifdef CONFIG_64BIT /** * folio_nr_pages - The number of pages in the folio. * @folio: The folio. @@ -1764,13 +1765,32 @@ static inline long folio_nr_pages(struct folio *folio) { if (!folio_test_large(folio)) return 1; -#ifdef CONFIG_64BIT return folio->_folio_nr_pages; +} + +static inline void folio_set_nr_pages(struct folio *folio, long nr_pages) +{ + folio->_folio_nr_pages = nr_pages; +} #else +/** + * folio_nr_pages - The number of pages in the folio. + * @folio: The folio. + * + * Return: A positive power of two. + */ +static inline long folio_nr_pages(struct folio *folio) +{ + if (!folio_test_large(folio)) + return 1; return 1L << folio->_folio_order; -#endif } +static inline void folio_set_nr_pages(struct folio *folio, long nr_pages) +{ +} +#endif + /** * folio_next - Move to the next physical folio. * @folio: The folio we're currently operating on. diff --git a/mm/hugetlb.c b/mm/hugetlb.c index e3500c087893..b507a98063e6 100644 --- a/mm/hugetlb.c +++ b/mm/hugetlb.c @@ -1344,7 +1344,8 @@ static void __destroy_compound_gigantic_folio(struct folio *folio, set_page_refcounted(p); } - folio_set_compound_order(folio, 0); + folio->_folio_order = 0; + folio_set_nr_pages(folio, 0); __folio_clear_head(folio); } Yes? thanks, -- John Hubbard NVIDIA