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 9E655CD128A for ; Thu, 11 Apr 2024 12:25:40 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 1DFC96B0082; Thu, 11 Apr 2024 08:25:40 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 18FDF6B0083; Thu, 11 Apr 2024 08:25:40 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id F246A6B0087; Thu, 11 Apr 2024 08:25:39 -0400 (EDT) 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 D4AF06B0083 for ; Thu, 11 Apr 2024 08:25:39 -0400 (EDT) Received: from smtpin04.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id 55F56140A72 for ; Thu, 11 Apr 2024 12:25:39 +0000 (UTC) X-FDA: 81997171998.04.C643113 Received: from NAM04-MW2-obe.outbound.protection.outlook.com (mail-mw2nam04on2051.outbound.protection.outlook.com [40.107.101.51]) by imf05.hostedemail.com (Postfix) with ESMTP id 617E7100026 for ; Thu, 11 Apr 2024 12:25:36 +0000 (UTC) Authentication-Results: imf05.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=dCtpO4UJ; dmarc=pass (policy=reject) header.from=nvidia.com; arc=pass ("microsoft.com:s=arcselector9901:i=1"); spf=pass (imf05.hostedemail.com: domain of jgg@nvidia.com designates 40.107.101.51 as permitted sender) smtp.mailfrom=jgg@nvidia.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1712838336; 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=ficEjny52fzs0IMfRfVTOVPLqPakGktR5kcEbGvV30s=; b=pjnYhAIHrIzF9MMwtr0HKr/InwsvdrJ4eSWi+ibwKlhY94y87rtEJ9Zjd15e6DHyjGoldn 589WVdme3m6ouzvyrHntBn83kG7qwfgM3Vrme4AnWDKQBWdm6hk/x4kEykYeTVVwAgu9Pz VjlsYXmkFsLf3jHe+Rn8+rbTT1d14mM= ARC-Authentication-Results: i=2; imf05.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=dCtpO4UJ; dmarc=pass (policy=reject) header.from=nvidia.com; arc=pass ("microsoft.com:s=arcselector9901:i=1"); spf=pass (imf05.hostedemail.com: domain of jgg@nvidia.com designates 40.107.101.51 as permitted sender) smtp.mailfrom=jgg@nvidia.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1712838336; a=rsa-sha256; cv=pass; b=Kcs6WSH6LDiK0t0vKVe41TZJLCzUofK+bPutee9X1FBKpPWz6MgzHi8rsj3UNRpZt3bfW3 usxD0aLypd2Q2ibqFem+Iy67vKzlwvkFDaxy10fpFEAJ90Cxqan+KHygYa71SO1tT/8ooo QZLFrZ0WUN65MGJK67yCopGODZF71j0= ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZvbqQ2zLf9bzC8rp7KUaidbLXIoVZF3ERYNLFhWIKONZg1Mqzj/s5Ti1ycqFlRIJj0fvivvn9yK9zMnBeJs4+mSadbiO5LuH3ieEQwjTpZdzTVX2HMpuqF6qNS7KWaRPhO+ITggHGHecAQ56XZGzXB9t+S4jkHW6JX3WLYaOdp3FPxGCQJAvI07sLYa1s39hBC2PmIyCrWRrY1GLlfNCv/Va1Vx/Q3Rvow30mEYQGVIbQDNsLRdsFbxn9zfDzQCU8yfjnn81XO0I2UBKt9QB7t6FoBlsrWqygTBO+bpzueuT+a0APY56Hu3jM9Ir1elNSf0FfqXVzhVUG5A/iITNLw== 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=ficEjny52fzs0IMfRfVTOVPLqPakGktR5kcEbGvV30s=; b=ZUGqHT6VVfwbXcDp9b1NAVZDCulF3fx9SiWd+VviCC4csnmlH6+E72sTn5t1DDEvGBPMT58RwvqxUVi+A/wTh2WgkOE34NOSxdPVkU6VA2y5ilpxqFVOvtz6Z+OMPfLuvupgT8nrBP+VTQLv+fBw1iwPEVhhQnFtaZyZMtopsErjr1+IR6ngudicfOfpeLSp9Li0hi7bcaM1BdhzOhaAZF7KNMUTIuFd10TMMD4QEcGyMiqazTC1DeSUsswH+mEhxPnkhSzJA6SzJOtWWLJJFWSjjLvnWQqFOMB347Ti2qYSbHJ6G+z4HyE7uCzZZdRICdlvmY4gqnlCAhT4TJUj+Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; 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=ficEjny52fzs0IMfRfVTOVPLqPakGktR5kcEbGvV30s=; b=dCtpO4UJEc7/JE0P4BuG9LWXLkrhCjqk10+27SEjsWcRqpvUM3DnoFrK+Ou8xOGBVBxIhMAYPqzDc4mkB6kbn6lfFW9lm8j/ukTkwmw9x8K1s0Pp1mbGUXGU0iEivCkJ8L7Xd7YKaAy3YWhqOt0Are1G7fUm0gawOzAdS+al4OptA6CFRXcDrYtqKuilXKSOw+3UOM9L0i22P/VcLwnK5HHPfs/Os09MlREH2d8us1x5NSHo/URqPHXRAnnNi3d4U0Fv5dwv0qWUAWSSBUYs0z74AzXYk+Fj/b6IlpQr/ddNsF5gtNlEdlMtMsyQdJxaXbpxq5oS15gfCFiWAQT16w== Received: from DM6PR12MB3849.namprd12.prod.outlook.com (2603:10b6:5:1c7::26) by DM4PR12MB5987.namprd12.prod.outlook.com (2603:10b6:8:6a::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7409.46; Thu, 11 Apr 2024 12:25:31 +0000 Received: from DM6PR12MB3849.namprd12.prod.outlook.com ([fe80::6aec:dbca:a593:a222]) by DM6PR12MB3849.namprd12.prod.outlook.com ([fe80::6aec:dbca:a593:a222%5]) with mapi id 15.20.7409.053; Thu, 11 Apr 2024 12:25:31 +0000 Date: Thu, 11 Apr 2024 09:25:30 -0300 From: Jason Gunthorpe To: Alistair Popple , peterx@redhat.com Cc: linux-mm@kvack.org, david@fromorbit.com, dan.j.williams@intel.com, jhubbard@nvidia.com, rcampbell@nvidia.com, willy@infradead.org, linux-fsdevel@vger.kernel.org, jack@suse.cz, djwong@kernel.org, hch@lst.de, david@redhat.com, ruansy.fnst@fujitsu.com, nvdimm@lists.linux.dev, linux-xfs@vger.kernel.org, linux-ext4@vger.kernel.org, jglisse@redhat.com Subject: Re: [RFC 02/10] mm/hmm: Remove dead check for HugeTLB and FS DAX Message-ID: <20240411122530.GQ5383@nvidia.com> References: Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-ClientProxiedBy: MN2PR13CA0020.namprd13.prod.outlook.com (2603:10b6:208:160::33) To DM6PR12MB3849.namprd12.prod.outlook.com (2603:10b6:5:1c7::26) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DM6PR12MB3849:EE_|DM4PR12MB5987:EE_ X-MS-Office365-Filtering-Correlation-Id: bc053445-de45-4a9b-58a3-08dc5a227ae3 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: bP3j3/4B4+6ZFPhPBDbLSYXOJTOAv51xdrpEH6frWb0umDnWU8lKKb2QzKWBS2qHT3Zu65XOwh4AK090I43DFiZXWfy5KyvA7rLF4zvHdtHp01WqzjRCgeqoO/4Bb619rv1E79elL8W4VsObntZmFoSpW6pAyA9lA0DHU0um9gZRF5LltIlzqqlQP0/sLrEVa1Pb0LhvVNeAo5x+VsYEWMBQhc5O5yMiJZkcNCbtuqprLvH2vc5KlwtKh6BLQ3s5SWe+FQs7lc1iIQinQUaSOjMk2znKAsWCHYnTCRatupyU6QlIIGWq3mbPvgYL/cIHeE/k2OZeOq1KHEx+FLhvA7KLWUMy5AX0hkHgsBqM4BjbbeSxHE4o9fpc05eBMyXtaINq12vWIUAbARKKtCHLw/s4eHnv62ZjN0K605mBrfMPMrfyuQxoD4V6vvwdo5zjvcF7CPCuawGiYqcg+VA4F1A4ysT6GdRaSLWmb0FojZzgxbv+fpvSx1p/xaJpmxeAUXeiL/4Hm1Lu60gHsHvAYyk3KWL1LdI/0bKOMG5eVjnD4kPSv4c63szN3DzZvgGMx5tuoyl3IoQiwgb07/vhLy+H33RlaPSh3NifcNvpTVog3VyizQJKxdAgJY9vlEZzqWvht4+0HnA+G8WbUPZtLsb7SkxeFW1jP7gEtFiO36Y= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DM6PR12MB3849.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(366007)(376005)(1800799015)(7416005);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?6LAoaX6MAKUWG7RHQNFz/KIM0RUU9f/UhiiH27Tl0Cth5RYz9J1aSnLqt/O/?= =?us-ascii?Q?MM2jVH5ad9DaFZfvDGvQrjzp9G525HwYtn7Hxg3s+pAJpXDMXRp2WGWlgB+z?= =?us-ascii?Q?tIpKAL6EyQd+EkenP1KSqPYE+y139Fx7sdY0I/0XPM8gzD2ZnaDGEuQ6NYMu?= =?us-ascii?Q?0Unrm7hjwVvpdAbX/WIM0504OHGYaUMgLWV/0mf6EDpY+FhW5U94R8Dk18wd?= =?us-ascii?Q?AGnJ1JkwciLnny1hAzdRW096Gp8yVOyZ/iWvzKWDuacdtDh8X3meYf/H46DV?= =?us-ascii?Q?wOPKP0HFgw6K6VBqStx7v/7anZh/P3ry3M9lqQBPPJ+L4si2+n1yaC9yvC2E?= =?us-ascii?Q?+yldcQJlcNDssNIhTyEqdY7k+148x43gdCvm9QGiPOvzF05eI9vFlRYUMdp5?= =?us-ascii?Q?xnHIpqhg19MnFDLYLOF6dVn6we3MHYtJfvFys7XbcfGYMXP/NrkMNuLzZxX5?= =?us-ascii?Q?1/Whwfz0EfNxw9/111GsOK2cNU0tqiyBSXFCLhPSryEJqJmuBk6JA66njT1Z?= =?us-ascii?Q?FLv7CeIhvevMh7WP5aa6FxFJkeGM6PYtuSEn3MoCu20XIatPCD08AaXRByqg?= =?us-ascii?Q?cSeq7tGCkf8eUQMN7SNmIz1vTtb9HsBCcpLSvHdat08VwzX383uNHTOHzu+f?= =?us-ascii?Q?QwQ7qJ2u/QKXjeqcMiAJj+0qrDC6y4kFJmkqL7czljhKYUTArNj981vAFekm?= =?us-ascii?Q?G39b3MAc9jH3+fkej000i9p1DVIKHyCee+UWwK05J7s6/ieU7ABJ0PZbuBG9?= =?us-ascii?Q?aAujn9SBrnF6hAGuEfRcFySO3s6WYP7f18zNuXTvzwPdafRiqv6mw+nHfHDp?= =?us-ascii?Q?Z3Tphl/0RD83QvhwgU6fShezGTbRg/YccxX/eTjk/oF+HcJk6cT3RU/RT211?= =?us-ascii?Q?g0+cBxG1asV7816EjC7DtDucgU7MygUGI+8QPQIS1Fb+gvqtZOdIawnUPnRP?= =?us-ascii?Q?ObCMddAKhkWT2jYiZwL/baVrrkBUaGZ2bcWRzEZRgDN01ehLc2XiNzle/Dea?= =?us-ascii?Q?vNx+F93Ub4FsAIqrIHDcwTGdfYQ7TQ0WHefKZIJsyt15eWaM/hkqFORE3HcC?= =?us-ascii?Q?heZlMOmmk8LWHUEEwpdJ0o/Q2yf1TIn1JBOq8U3qC/5JTe+QNquV4n/AEbxB?= =?us-ascii?Q?TzUJooRueQItZEOSeSE/bpKn/Yja4OuMZ1TjXOAguMV4MPuFcdlrBI+H9C76?= =?us-ascii?Q?ah9iOaxPpaE2Mi+p/1Q9YX6ef/MjNkojIdO7PgCHWKaByn8NFdN0qwDf7ngs?= =?us-ascii?Q?lnLEwQdgn02RI8m+CGFadORl6+5tfEsB8xDNoT550V1BplZfDWiT3kaX07FH?= =?us-ascii?Q?6bPVzuOYjZYlpFpIdrVrMMhpPSZqaKv3qetsrvaJoyag4/yPytOtzPXfutvW?= =?us-ascii?Q?Mh0vAK7iIBotNmK7VPTrv9cX7/Rhu9nQn0+zEq1cbFojQ2+ZSHnELbl4b3tl?= =?us-ascii?Q?yMT74n/Azwy/hiARYiTnSfPZBKDb4gRyHuMOx3LyaB31Ju1x8TXD6HTYke+9?= =?us-ascii?Q?1mhjDAU5+ENeHTFQVpE8AmUm+BkHODvXmDvES3IRRrcqFBKtqdbwskZR7G19?= =?us-ascii?Q?ttQAbpAumbBNDULTtQI4pmSNc5z2RHOWSfZiBYtY?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: bc053445-de45-4a9b-58a3-08dc5a227ae3 X-MS-Exchange-CrossTenant-AuthSource: DM6PR12MB3849.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Apr 2024 12:25:31.8247 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: oUYDJOCEJla0TXK9HQlvWt7Q+Tpxqz5kOBu1PPVezglagM8OLMDcFSnP1Nh07tyO X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB5987 X-Rspam-User: X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: 617E7100026 X-Stat-Signature: tsh18ffmgpig44gnm8z4ciky61gks5q1 X-HE-Tag: 1712838336-802182 X-HE-Meta: U2FsdGVkX19V1Khppfyx7VhMPfPNXjEir3Jyth5e9DEXV8cLiUW0YE7pPGIlUd1w5jt1OUAI/rtpmyrB3RDfEuAuxeShyxVQB6n3QrYUgWqFYgtDw5Vl+KFlj6sd5oNwVIAEHzo93vNMXJstQA0ErYBXwf+MyadlEH3o8Xollqg2se4IwPTMu0A6mZCaatwGxn0+PLHt0kv3vASbosk8R57QY5jMCRtrVXyCbyOkP7QsKxQ2vQSP80HxTE8C2IGXiFaA5gbbb7/noa+KP+YlYUhlKlSwOw7gKylWAv60rHbxCSgwZy62OY0eKTGUXdUUrnk5X1UmcwQ1uWqh552vxbuxFAehCQaR5YndppF6FI2ou4fsbLc3a9jDZrVbNzZi25ejQMPfSvWaZx3/KV9itD9+eL5G6eUtUUSRzhkhKmI9AHPTNDG9Y56tjn+fbh5cVTdK6BZHciZgn0gvOC3H9O7CHHKFluQ1r8rkimJQ5oA8H7XVMw1R58uiWy94mWFXGii4hGmcXdujuVSAu1rjGIhtcvJy8dZFZrkr+sJYfPSlJxh7ZCqsNs2uIQt/pP4q5KWAqJsDlHZBzw1ZqqBvv5EqWzi57mVPh6c64mmS7Zh1Loo28/g6+B6RAXY70uHVfvx3KObEx+/KuKoJi646zPMLVoZnTFWztpJkZEtsmJX/LoB4VdB4CdtCO1wvY+ElRLKA2KzdPH9lDoh50EZn1dxAZqFUu9CNO130YMu/MP4QfAMfwkJJfjovs0/5wz8t2na+4ScJFUKyZJqRXsy2wyqoc0hXHboaS1t/PGRKb/BfUhkYKjLx6TnhUomE9UiDZck4naVAzVW6Qd9NvuYnhP0+GiosgmTWLarNt9UDSkB+oog8aAdnuAuA3yUCa73b4sPoQyXyelBXLrUxIrfZGtay0733y9MhIPzVbFf5w2qcbAbj8NhwiETqZrbkS9iHhi8kMPAz9YtSjbkaNmk dI1LEcNs OZfqMnoVH7eKggK1khUA0MFBJB/+9kt/+lFpuw62+pMnUjkkCzB/HKsX9wnbrWwpngCXNH4VQT2pCnh+XizAzD8SjwB4dIp1vdhhDKNXEPOKTEhuZxqp89XNlGIjZisAksxK6EFKQbe/+MiQC+7Q54RWmwzoeZ82oIHNIt0PjBPCufnIqcS5UaNw10S3sIeRJKOSY0mfl6iF3cBSmqfpgB88v22qmtBVqgxL2vskZl8083gY7IcLMol4Tt3YsyOfS7kX31eGCCWpV0POhTfJx1Y43y7Om4/t0m1iX X-Bogosity: Ham, tests=bogofilter, spamicity=0.000023, 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 Thu, Apr 11, 2024 at 10:57:23AM +1000, Alistair Popple wrote: > pud_huge() returns true only for a HugeTLB page. pud_devmap() is only > used by FS DAX pages. These two things are mutually exclusive so this > code is dead code and can be removed. I'm not sure this is true.. pud_huge() is mostly a misspelling of pud_leaf().. > - if (pud_huge(pud) && pud_devmap(pud)) { I suspect this should be written as: if (pud_leaf(pud) && pud_devmap(pud)) { In line with Peter's work here: https://lore.kernel.org/linux-mm/20240321220802.679544-1-peterx@redhat.com/ Jason