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 7F969C52D7F for ; Thu, 15 Aug 2024 22:55:56 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id DBE466B00BA; Thu, 15 Aug 2024 18:55:55 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id D6E456B00CF; Thu, 15 Aug 2024 18:55:55 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id BE8FA6B00DE; Thu, 15 Aug 2024 18:55:55 -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 A25BF6B00BA for ; Thu, 15 Aug 2024 18:55:55 -0400 (EDT) Received: from smtpin02.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 4B130418CA for ; Thu, 15 Aug 2024 22:55:55 +0000 (UTC) X-FDA: 82455989070.02.9E504E7 Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11on2054.outbound.protection.outlook.com [40.107.236.54]) by imf17.hostedemail.com (Postfix) with ESMTP id 63A384001C for ; Thu, 15 Aug 2024 22:55:52 +0000 (UTC) Authentication-Results: imf17.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=hvH3Aupd; dmarc=pass (policy=reject) header.from=nvidia.com; spf=pass (imf17.hostedemail.com: domain of apopple@nvidia.com designates 40.107.236.54 as permitted sender) smtp.mailfrom=apopple@nvidia.com; arc=pass ("microsoft.com:s=arcselector10001:i=1") ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1723762470; 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=4dAbU+SPNGkVUpPogll4h+jBfTzKTKy2RoVZqQJAYo8=; b=VdxjjJrw6Xab5mmQiyB9vhuVSFGQxVG31b0RbfeaCsCmzMCVIaU47uWxons5G9g6p/ciFl dUihC1V/IzdV0+vRF3RHye+Lh9yCE1tnnEcNMmYadzwp7qpWTWdsFK6OKxjotONuJgZr02 1ooeKEZ7XxBc7rjrdV7YND1C4veH3aA= ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1723762470; a=rsa-sha256; cv=pass; b=usAabDXWTvxotkRcmd+fJqGzKtu4X+RauxvInxKfHq89ADngjYjRc5I+hixrwsK639XMN/ FqsFteTA3jRPnCiDr/7/aZMbOtuy11AXLDnQP+/wcmoOz9AqCNrONDdta/q0aHn8GsGFfE udagATUyQU1gNOn5XNehPn/zcSKIpHw= ARC-Authentication-Results: i=2; imf17.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=hvH3Aupd; dmarc=pass (policy=reject) header.from=nvidia.com; spf=pass (imf17.hostedemail.com: domain of apopple@nvidia.com designates 40.107.236.54 as permitted sender) smtp.mailfrom=apopple@nvidia.com; arc=pass ("microsoft.com:s=arcselector10001:i=1") ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=fAX+/rXMOeHroE3nLw5PFk/S367Rsjm4N/UaCdfTT6AYjyibx0R3pPXgj8wqwdFavLYauDhZPPcw8vr16HfKxZsM8XUdc72X4BKT1awPqCONR5twA2VRObBFK169kPiNiTaALQX/6+Lo44Mz13tC8jyjkt/a6cZZlY4PR5E64NoR/GQOhuYrZWqUBRD8dd2AHrnxhM5aqy56HXjxkRTPV4kESZo4Nf5wiGs/Wxe+0R8RQMo6pbgrWWEwqBUy6qZStrxGJavkhx5n28JlUeTuYsC7XREOUtbkTfxb06c1sHXvbjfaSWc0HOs0lFx1YbHOdzRpEGcCzdN45On2xfyLBQ== 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=4dAbU+SPNGkVUpPogll4h+jBfTzKTKy2RoVZqQJAYo8=; b=COh1hXXNSkhTICn0BDqxYnkiis6vgGr0vbG6q1b/1t0/Lcq0gee3fQzYrfhA85pLnOMkHYeebbaPmDo9Y1+ZGOtATMLHP1+r5m+vuvb4beYH3kqDslvcAJeho21Ku+2k0oYVPeUXgm4k0wROjbjnU3PXIiwGKaYdfdrlN5R2zmIeu0zrfVNDCn0qrcJDISihqkRL+DdVqoX5OU/qnPoFAM3ZhylFkmB4IUVr+wlPi5o5XHmWJNurdUQVTqUw6ZJrNDwdUHX2VPnAPwkfJNEs2plVWrkawUksQlEu1f9eoaM2RJKLcmS9rzNaqGrL5rfBQ4tSLm6Fp2WQYemvv3pTlA== 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=4dAbU+SPNGkVUpPogll4h+jBfTzKTKy2RoVZqQJAYo8=; b=hvH3Aupdlg9Fblkb2+RdLJLGJMkeUVhwTiMZOlRMMDAFKpFbsVBjeKHAp57ai+j7dau9gAxDsTcrqSZ+NQ6G1tOZzbaH2iUJPzdewOnysDki+nbEm+7w9qRSY2mNRCpoan1AyDcPUoteoE7EFNNnL5eMvSO54flue8g2vqhVkzd3hYMmbzSjjkYOZfLK3BdQkPhvmdG+Q1RgoeXjPMp/ZuIRgfQlahpirxyo3ti2YPawOGe0Dn3VwF5rF29yIJb5gu/cwjQMBCdhaqQapFjb1FLz8htDdAXsgp3ygvR5ubHAuosaqXK2Uzx7BiCLbI/qXlWm1x8YkhThvZrfa/cB5g== Received: from DS0PR12MB7726.namprd12.prod.outlook.com (2603:10b6:8:130::6) by CY8PR12MB8313.namprd12.prod.outlook.com (2603:10b6:930:7d::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7875.17; Thu, 15 Aug 2024 22:55:48 +0000 Received: from DS0PR12MB7726.namprd12.prod.outlook.com ([fe80::953f:2f80:90c5:67fe]) by DS0PR12MB7726.namprd12.prod.outlook.com ([fe80::953f:2f80:90c5:67fe%2]) with mapi id 15.20.7875.016; Thu, 15 Aug 2024 22:55:48 +0000 References: <20240815205606.16051-2-max8rr8@gmail.com> User-agent: mu4e 1.10.8; emacs 29.1 From: Alistair Popple To: Max Ramanouski Cc: x86@kernel.org, dave.hansen@linux.intel.com, luto@kernel.org, peterz@infradead.org, linux-kernel@vger.kernel.org, jniethe@nvidia.com, jhubbard@nvidia.com, linux-mm@kvack.org, tglx@linutronix.de, hch@infradead.org, Christoph Hellwig Subject: Re: [PATCH v3] x86/ioremap: Use is_ioremap_addr() in iounmap() Date: Fri, 16 Aug 2024 08:55:02 +1000 In-reply-to: <20240815205606.16051-2-max8rr8@gmail.com> Message-ID: <87y14xpf9r.fsf@nvdebian.thelocal> Content-Type: text/plain X-ClientProxiedBy: SYBPR01CA0132.ausprd01.prod.outlook.com (2603:10c6:10:5::24) To DS0PR12MB7726.namprd12.prod.outlook.com (2603:10b6:8:130::6) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS0PR12MB7726:EE_|CY8PR12MB8313:EE_ X-MS-Office365-Filtering-Correlation-Id: c799ed6b-cfcc-4738-3f6b-08dcbd7d6748 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|7416014|1800799024|366016|376014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?nvfEWk6N+Lw9gJJzc90BCEtqc28lRIkD7fvJF+d5POST3YIPm+vhoUT3NOoY?= =?us-ascii?Q?lpr5zO2dVgZIGzLzqIDAMlT8vDw+fkhPSnM71+kdDQ1h4tuoc/Fu4kspVD5K?= =?us-ascii?Q?rUVRaRqP9bRyCPVrHG1MeOHjs3wcSbjt2zTCdiN4EbJ405Dhb/pLYyVzbPHZ?= =?us-ascii?Q?LIRPF7UQLJOWeJwsihhNPXchKePxzCPef3K/n2KwyQCUaLkklKjt/7jkbKic?= =?us-ascii?Q?+zyTJ/YbJ7x3bXo+uGbU27LBhrJOCp0eKSX67LGIWdtBt3qKzPzMxtAjD83v?= =?us-ascii?Q?Qa1pNvpTZBPYmuN0IU0dE3KArav9f4xl6jxLSROCmAJk3Aa3zf8Kzb72Sq6h?= =?us-ascii?Q?3EKNZL6NoV/bjcZT6040QDL55b2PwP9hGlnVyv6gbN6JGk55ZI14aHKNztSQ?= =?us-ascii?Q?uNCCjqOMoWqRt3PcR2HKSFOTHsovTr/o/nb62eWQCy1CVlTI+sPV9JC/q0Ft?= =?us-ascii?Q?K+9XAa7Rl3DTaOwgjc3DA+pgNmW0GEAocHblYQE4PEb1A0AnnDP0k+uC4F9a?= =?us-ascii?Q?7gz2Lx4DtwzAiHER/eWr75ZR2nFRLT98OrvDFHsIBE8nzuEcr/bDfuZ1Oafb?= =?us-ascii?Q?Iew0h6t14eoVnNZrdv56cOBoFHQ1LifQFol+OvlzVfMcFMYqTEjgZvQ3+dW1?= =?us-ascii?Q?oD6M79iCTdmhWg/Fi1t+Mhtg2WPVfBz76zEzYlP63vqWkZFgeRWurd2cf2j1?= =?us-ascii?Q?tZW93wf4T5el61/whyN9FNPrmtUZapLoaOI2DcvKaDugwwB99ygmC9PCsV4u?= =?us-ascii?Q?GRm/IhcHYxwf7fQ/+8JTDlNTghvVYW3nEUNaSWIa1ih4715Eop0SGAkGoocv?= =?us-ascii?Q?v1EApes8sLCgLQ8uHy7M9VhLPKDaz7iBhQ50pLcicWSmDuoebYafjNFchRPT?= =?us-ascii?Q?w+E+UTwD0SQTYFM1M+cBkAM8XO4SoAGny3v5uFc9OZKypjpQM/otsVSoBHpe?= =?us-ascii?Q?5mS4Ee7/BmgdAnu0YPoksXx0qvyp3t3aQjpVDCiyFwWWcJteM6VOiFc7iMK0?= =?us-ascii?Q?cuXI6l6Dz1AanyvTF83f3Ntim/uhBZFrrp5GazIbEsiz2z7PYS7U4Xh9UXeb?= =?us-ascii?Q?zrE/aASRRMlCW1l8w8VLd3AJZRnIgOiL+dI/l3DV0T+r2PRgyG3P8rLRNpqg?= =?us-ascii?Q?vBI83yeC4J1QUn96+faljCx4Ct9xkkNEgT0HDVv2nKf5UH2yLRHNvcF18KIZ?= =?us-ascii?Q?xIVGPp7U3xSVMI7gUzcRK0Nr/ZdtABJztoeSf7uqfJF7obRQwcrAuuyQVNyz?= =?us-ascii?Q?YadavU4ZlyQFwNB0pcL+nRIZdOW6ByUr37bvjawxJG0rF7QudXujg/wlWUH+?= =?us-ascii?Q?Ajx8Nrf0AMCpUFxXzckEvovS9GMD0baLzz4T9jzHI/Z5XQ=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DS0PR12MB7726.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(7416014)(1800799024)(366016)(376014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?tqHkmcVueBUJ2DFreF7LOEnzyxNvM+5GYhj92wymQpONZOQ6lOL3O6WC8fYY?= =?us-ascii?Q?7T9ixV/YnVoXNLUlpFCPeppR6cCOFWq5IXuEJ+CY2AadKhokoCriUj+e1KNb?= =?us-ascii?Q?ZQAvk7nr7J+4L6iKmf8gFXvlhB4E9GFnL+oxldDfFZS+JDKiTbGWzw3R3DyI?= =?us-ascii?Q?EGebQcJUUtbcud4/nWhEbrox70h6J7bJOcnjFJPaTXr6UahblWyfeGwuY83/?= =?us-ascii?Q?GC8HXE1J95yEt2SVenGL2xR8nNv93xPhSyxnMshbLw0gWJBss8YrW6J/dH5t?= =?us-ascii?Q?cRgI8ksVYIJN6tWR3giWaGb8jamZlnNmsN+4NkFqIXlUrqFYZN4gKmkfHRyN?= =?us-ascii?Q?j3myxI5wUVps2N6CZgB/0Xg43jzft9HPLNzNdMLBgk3cew/KrkL0HdkKp82B?= =?us-ascii?Q?q96IhL7cj4nj72STBtlLdQXNgZ9sFMOsnc/FBgot00ap0FzYsHh8EpD8xok7?= =?us-ascii?Q?N7Wg1bMHxFx2zenovyt/ogp9JlDxFzj57cZEugZ6I94VkDAKqSgALZBbiY+O?= =?us-ascii?Q?KDQvMc3yoK5rm6/dkg0J7P+5dkpu/r9trdYyKw3ma6zKVXjDmft5y1dX8crh?= =?us-ascii?Q?Nyo56E0u9V1341/4c9+IkTyOi7SXc7dQIlWhwbO5p4RyP03xXiTYvziKCEkA?= =?us-ascii?Q?XwGvammhSFbiMDhoebd/xUst31wEZIImXmSbnBrgNpB+wMHquuQNcwr/trZV?= =?us-ascii?Q?aADFB1sxxNNRow6NZR63RCoo5c4QPwrnpaKpSnJS8JETsLHM0eeSf4BCdmZj?= =?us-ascii?Q?7cY9p7EnF14vpXoHE1ETm2aRfKVo78SKJhY15BDr3wKcEV8epjFjdEU7QpB7?= =?us-ascii?Q?fVMRjWXJfKj8FPO6hxClqGn0zMGDNgpQAE7fqsojIWzvc93EpDMBa8OKrAlx?= =?us-ascii?Q?rL0FS8dpQ3wetTL6g/dX59Ph+J9juXZE+VoVJCsrmPNfb4DXkz1zEJGMsdC+?= =?us-ascii?Q?1fiBuawKxBzz5L99Hpl3MyMM1Rxl1qHFT2VWT6zeFMUmXUknIH14IUMR2XXl?= =?us-ascii?Q?t8R38MYhE7QkoFSPEVsfEd5k2JkuU1MxC3bJnc56+0K4+0/th4wbQplm6bl4?= =?us-ascii?Q?kC8rtpDZWidPKRj05ByuhpxMRE8MffKG58hlQ+n7jCuKxIu9KdXOy7SOpq75?= =?us-ascii?Q?b/JkvmvZqO2icrFgtjJhB9jy8hVw+NTI0Pz5EVLnTafbOsIYRHu/7AVlIipx?= =?us-ascii?Q?lJZYidguLgNzHPWC8CiX1dlkD7QGKx8Wjv9H4vJ3tRmPDlxjJCEb/KvwN7Dg?= =?us-ascii?Q?auMRW38haeQXZ8ZQ1rRm4xmnMvdYaTQJ1tnJtZ6peK6BtpGbzcyufMK0pKQf?= =?us-ascii?Q?K9HSekYUqTuv8JbiJtAHj2S+AvN1bWaxM5QmFWKhRhQaeVImJ+yDGMqBctSC?= =?us-ascii?Q?vViHVwmioYcOMMK6gfdOV9GjFfD9cf1Mweslly2m+xaTon/R3GNg3ei7eBsC?= =?us-ascii?Q?bXGZz80Giu9XM3SjgNtC5w7aq3dVUA10KFQT4WtOtwxJNFYceHg8Ny2/FdA6?= =?us-ascii?Q?rp2nbfwHoupNAghqq5Hrph5/NPOqIOWonux+Rg7Fyzrz5VmN4lXz7046ScP/?= =?us-ascii?Q?IBR7WlVcNI23Rnkkc5f4bPyGEFuRL0n6uLdR+iiT?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: c799ed6b-cfcc-4738-3f6b-08dcbd7d6748 X-MS-Exchange-CrossTenant-AuthSource: DS0PR12MB7726.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Aug 2024 22:55:48.2753 (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: 5Ha1tQBisUZVZp0osQ7ucyXM6RbQhiON+H5DpP+Kyfi6r/C4XEmgbFxh3L6YIAe/na6jgEcOQTE9Yg19NG16XQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR12MB8313 X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: 63A384001C X-Stat-Signature: cy6x4eh43q4sqnjat4h1tj4xqqe9mijm X-Rspam-User: X-HE-Tag: 1723762552-141310 X-HE-Meta: U2FsdGVkX1+jZfD4k56FTZqVzn4g7Uy8SbgQbE2EKUD07kAqGj5GC050aPKGPmovaWIxntDXt46Nzg10lMNTnDrDqw+OYFnlY8eWMyBvfj//7uOpBE95xnlHk21dBCRca8dqBZiA7TGpz2c+dRE8M/wUWStDDoioZZ3qR3w8Ro553UU5RDyp4GevgywGXqvBBdD4qQcN6AodoNzi+p+EOZH0XS1TMf7uvup+LuJbfN6u7sbLvtAep4BZb3vlfrjVQR57yly2wbVg4XD7DeIDYStasPT9pLmGFZa7wFOsVFAdEZdmjt1NmQlFLN632+KI6W568xNeagTFWsf8vrgOJAfboc4lmZ1vb7sLo4yJRgpHNk7EF0Lgi69IT9nJav0YzbeEXrAbb+uNo31RlV3a+57JqUITvqNLqRgHQlwfH59i7eRGXK77y7y/odE9uR5itCRhHtjuFKx6BRQOKhflH10INKrBkn74Xfrlx+HNnyW/Rpx2CqdFhva2rOGZa0QvsjfTOBwIiXs2MS+cbAJMFqNDEw/zKIhPdMz7HOXsX64rykPmYYLTAvBpuTit8k4A8hMHqJ2d/Owi5msNGNdSBm/APfixDYbeAmcBkzhXfMHTPy+47YVUrqhDdM0+Ag3ZMgoxhNfk1KrUeSYz2eid42T+9TPoWM7HYiWORAm14z7A4XqFzfpCmoVM0CdAKDmaKObuBjTRQkNVxNXhrcd+6Nrm+LHA8ePec51D7TpWaL69S85rs0DbIJtBMnkx/shHo/39ZXv5b226g2oTbvnBTd67xAfRcCGugRFvntmkUv4t02pOgq2oyUo4fHIdtPwp3j42ezezG9bpcRuaCjcLmLi56DvpfJ5tm/+4rMPdouIjdiYfgnLozEo61p75+x/Bi3oqHI9Ul1GYGxvV4PdVfjAcJde7dBBRnIzE5Vk4Xwc5kiDWPJreLGncKIyGkgjgIcBIAlyfGJTmWxsd8HY Auf8O5WW E5F35CAXK2zYXhOTk/Orwq2i+A37ocLaNX6c45e134D3kFG036z4sZZO9o0ackcKD+7oQtFS1MGi+AB+n/8WusuFWHYaY+mfz7fHT3HlzkTsZyAYM197drwLi4uSydHS4pQcIYx9UUzA6TQhvhA7fxasi2vOSrhQdrskk74h/eZ3iiGixRv/efunGTFFjC2bIfemNkgU3cGW329OO/EWJSvWQ/w== 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: Max Ramanouski writes: > Restrict iounmap() to memory allocated in ioremap region, by using > is_ioremap_addr(). Similarly to the generic iounmap() implementation. > > Additionally, add warning in case there is an attempt to iounmap() > invalid memory, instead of silently exiting, thus helping to > avoid incorrect usage of iounmap(). Looks good, thanks. Feel free to add: Reviewed-by: Alistair Popple > Reviewed-by: Christoph Hellwig > Signed-off-by: Max Ramanouski > --- > arch/x86/mm/ioremap.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/x86/mm/ioremap.c b/arch/x86/mm/ioremap.c > index aa7d279321ea..70b02fc61d93 100644 > --- a/arch/x86/mm/ioremap.c > +++ b/arch/x86/mm/ioremap.c > @@ -11,6 +11,7 @@ > #include > #include > #include > +#include > #include > #include > #include > @@ -457,7 +458,7 @@ void iounmap(volatile void __iomem *addr) > { > struct vm_struct *p, *o; > > - if ((void __force *)addr <= high_memory) > + if (WARN_ON_ONCE(!is_ioremap_addr((void __force *)addr))) > return; > > /*