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 06943D17129 for ; Mon, 21 Oct 2024 19:52:16 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 2EB716B0096; Mon, 21 Oct 2024 15:52:16 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 29C956B0098; Mon, 21 Oct 2024 15:52:16 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 114F06B0099; Mon, 21 Oct 2024 15:52:16 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id E9E9F6B0096 for ; Mon, 21 Oct 2024 15:52:15 -0400 (EDT) Received: from smtpin11.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id D74DC1C7732 for ; Mon, 21 Oct 2024 19:51:57 +0000 (UTC) X-FDA: 82698655536.11.8B041EF Received: from NAM02-BN1-obe.outbound.protection.outlook.com (mail-bn1nam02on2072.outbound.protection.outlook.com [40.107.212.72]) by imf17.hostedemail.com (Postfix) with ESMTP id C16CE40010 for ; Mon, 21 Oct 2024 19:52:00 +0000 (UTC) Authentication-Results: imf17.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=i08p+7mE; spf=pass (imf17.hostedemail.com: domain of jhubbard@nvidia.com designates 40.107.212.72 as permitted sender) smtp.mailfrom=jhubbard@nvidia.com; dmarc=pass (policy=reject) header.from=nvidia.com; arc=pass ("microsoft.com:s=arcselector10001:i=1") ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1729540256; a=rsa-sha256; cv=pass; b=8RIwlotkp+ZXM8P9oa6H2FMnfqjrEQtkLlpUD1CVz7F6VjWPf1SY2S9XNleWSJrB2vJlU3 QEt19OsLO6Xs36nFmWxUXVkttqRVql2jNtqWX0rOBfFbD3+waa4HDqMWLDCZnYRmnqtxhW G1WE+s8n8RZYOgeBqxtZJSBp6q+QNLM= ARC-Authentication-Results: i=2; imf17.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=i08p+7mE; spf=pass (imf17.hostedemail.com: domain of jhubbard@nvidia.com designates 40.107.212.72 as permitted sender) smtp.mailfrom=jhubbard@nvidia.com; dmarc=pass (policy=reject) header.from=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=1729540256; 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=w+AligqO/KjBVIYzXyITw53MPFiEmYDFy35nqilTkKk=; b=Zvp9kHu5wqJGjWibMGMdkVCWiXOOH58wDSNVxhPMHLIR1ikh/rqMd0SVpUGv3DmG2jjvx1 2bbD/GEr0Ld6QauOh5JZqOfQjb0N9T8ydRpgh0jdjJvIywCdzxl4Hpfpk0g/oYGWQiIox3 RR9vXP48L/VWZnSpR9Jbt0KMBpwK/Ds= ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=vJMeDrNBf7InvFHxdQqQSG3/gJxEv+WMkIfjC3ZEnivmhnyghpOX07Y6wRxCDfzTB1GHJfUpqop2GdPZlcBqfVzZ+rK1ZWCcAUAAEYfsIvIinqjdcgjlIug+9379cTvNzHh6m8EaK5Tmjc8qJiBQsQ4IGH2I+dYAOPR23V54TjfcxB62aKV+6WFZpYLHE1QsDKQ7mioIE36h+RuaohhdV9eW/SGRGzeTXwJNi+a1J7rTczoGcfOsid9obSV9VEm3enSgQgzMFmdQxvRvSOBNInsRBo/bauwxGY6Pufptf5ULo/GccyqDGtEDhQaJ/wX5P7idUbmpQefMTTMThe7eJg== 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=w+AligqO/KjBVIYzXyITw53MPFiEmYDFy35nqilTkKk=; b=D8Sj1VPcg1ULcXjnklltkh5NNoeMLcrUVDFYxRjLv5JPq/7P087mmvfHcMCSi8q3mHaEnqleqKQM6ZnblpkLBXdxEtiat7+OezG/QChsEfo9+wSpkxiwoAtFwx0Ogj36AzGM0+E9Xo7ZZuwvZbXRisdyQx1QCBh1bfW9ESHKjVrUVmccyH7/Wd0m8ZJG6ymr5+VTnTouLsAFETMcNX3QeCngHq9CAiYP7MAdCkEpsslVYughnzAUbK7QMygQET2U8K/8z4/leC/9WEmZX5SfbGCmMNVN8ooFX2UgRbMt/lVR/wCYB1HnwuKukqrhJ4lmVbR3ds/S2RPvHYZdA5njDg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.117.160) smtp.rcpttodomain=ryhl.io 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 (0) 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=w+AligqO/KjBVIYzXyITw53MPFiEmYDFy35nqilTkKk=; b=i08p+7mE6U3Aexdyfi0WbmCRtc1stHJ91uXgg/IPv0v6XZJ5UeTBlrMQPYcvkL1MbHQ3mFAKf8ml8UwZTm3TUBs3hJ5m2Tjb6I8PUzPgzs0xu5hWr0qFH7DDTExZF4GPNxcTRu3/trd7CyeLzCbgraU4elDGLjWnodvfEUsoxs+WqKa5wnkDFwpUogWaoS5Twp9+Tc1mtob143OSUdEirNIllB0O5mg6mJerVxF6e6TX+xtBpe/UmHi46nNZgTw8wGkB2MUMIkgV+RzbgG537bf6etK8fSkggI/2tvPbfaNRLR0TyGgmADXtMW/lgdr+5ozJ3yi8vXa8VOnww8NxuA== Received: from BY3PR10CA0002.namprd10.prod.outlook.com (2603:10b6:a03:255::7) by DS7PR12MB6237.namprd12.prod.outlook.com (2603:10b6:8:97::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8069.28; Mon, 21 Oct 2024 19:52:05 +0000 Received: from SJ1PEPF000023D8.namprd21.prod.outlook.com (2603:10b6:a03:255:cafe::18) by BY3PR10CA0002.outlook.office365.com (2603:10b6:a03:255::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8069.28 via Frontend Transport; Mon, 21 Oct 2024 19:52:04 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.117.160) 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.160 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.117.160; helo=mail.nvidia.com; pr=C Received: from mail.nvidia.com (216.228.117.160) by SJ1PEPF000023D8.mail.protection.outlook.com (10.167.244.73) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8114.2 via Frontend Transport; Mon, 21 Oct 2024 19:52:04 +0000 Received: from rnnvmail201.nvidia.com (10.129.68.8) by mail.nvidia.com (10.129.200.66) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.4; Mon, 21 Oct 2024 12:51:45 -0700 Received: from [10.110.48.28] (10.126.231.35) 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.1544.4; Mon, 21 Oct 2024 12:51:45 -0700 Message-ID: <41f8b0d0-0ef6-4883-bbad-52e77b13700b@nvidia.com> Date: Mon, 21 Oct 2024 12:51:44 -0700 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2] rust: page: add Rust version of PAGE_ALIGN To: Alice Ryhl , Alice Ryhl CC: Miguel Ojeda , Miguel Ojeda , Andrew Morton , "Matthew Wilcox (Oracle)" , Alex Gaynor , Boqun Feng , Gary Guo , =?UTF-8?Q?Bj=C3=B6rn_Roy_Baron?= , Benno Lossin , Andreas Hindborg , "Trevor Gross" , , , References: <20241016-page-align-v2-1-e0afe85fc4b4@google.com> <81e9b289-b95c-4671-b442-1a0ac3dae466@nvidia.com> <682c97a0-9877-4e31-b180-c1e38d3bc883@nvidia.com> <7d708e79-1a94-499e-a0e8-c3431aeaea3d@nvidia.com> Content-Language: en-US From: John Hubbard In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.126.231.35] X-ClientProxiedBy: rnnvmail202.nvidia.com (10.129.68.7) To rnnvmail201.nvidia.com (10.129.68.8) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ1PEPF000023D8:EE_|DS7PR12MB6237:EE_ X-MS-Office365-Filtering-Correlation-Id: 39a34323-0053-4272-2692-08dcf209d689 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|7416014|36860700013|376014|82310400026|1800799024; X-Microsoft-Antispam-Message-Info: =?utf-8?B?VXAxMTF3aDRNVTY4M2VreXJFQnV1b1lDNVhaQUlkTHhJTGZQbGs5NldPblF3?= =?utf-8?B?WG5ieWd2Wktwb1Fua3NEbG16NldrM1VabUc3cEJNUEhuT2s3VVorQmZOQngw?= =?utf-8?B?RFdYN2xuamU3MGJFbEdWeEkxaWZXVHJsZjhhT2o4T2gveG5HaG5GZVEyQ3U5?= =?utf-8?B?Ly9IUUU4b1pCbWhwMlZpbURuN1ovU0FqdnBKb1JLRXA0aXBSeXBpYXRteEly?= =?utf-8?B?ak1iS2xDZzVVbUpEZkVtdG5jbTM3WlZ4VFBqTmtWTkZDWXpYaktiaTEyTDcy?= =?utf-8?B?ZVF4MXl1NW95R0hXNVU3WllMcFdyOVNXM3BpRjZTSVc2dDhkeFV1bWs4enBR?= =?utf-8?B?RUUvMjc2VTFJY3dJMTRjR2NiRGRVcVJwVERQSnpzbklaZS9iSklQZGhxeWRs?= =?utf-8?B?cEZPSVBhYTdMeFNzcmtxaFZTenkyUDI0VENHL2l0R2FHVU5EVU8wcXZxQUlK?= =?utf-8?B?T2ZXcHVBSkQyMlNaWlViZjF0Y0M1UzlVd0ZWSUdHUEZOM3piQ0pBbG5RSi9l?= =?utf-8?B?NmE4eFJyeURDNXpaVnFuQlVBKzlObzBmcXlPbTU4c2VLcng5ekY1RkZnd3l6?= =?utf-8?B?eUc4MU0xejlCMWh0ZE04UkJuRENGcFBPSG5VTDRHZDN1ZnpocktrMzJibGdx?= =?utf-8?B?eGY5cXpKaUtGb0UzU2NuT3kzMWhNV0Z4UDNwR1JWUE1aOWl1SFk4ODYrejZB?= =?utf-8?B?UElhRXZVMWpMUis1TzhUNThaUUNZbkZQQndUU3BDbmJ3cDI4Q1pmV1JRMnJZ?= =?utf-8?B?NXJnOE5FNGlDTGhXL3Rqd1FnRThCV2RxTEJmMjVaakJITUt4VnMza0JsSjc2?= =?utf-8?B?eHVUcG13UnlnSEtRZFpDSHRwQndXcS82ejJVMUt6UlluY1FHeWR6cGwrZFBD?= =?utf-8?B?UHhwUjZSREJHWVYvOXNTWHJqTlpkVFJSdkRtWERxMTl0R25xdCtEcGYxNTkv?= =?utf-8?B?TlgzY0NSSGxJZkx2cE9FUVB3YTA5dHN3SWNEdGl3cXNOZFZkbkNXTG1OeHFY?= =?utf-8?B?aGNPQjVoWHV1ZG84c3l1bFNNNHBESk9tbWtjUCtFSEtLd3o3TWhPZUk2WERp?= =?utf-8?B?WFNLU3h0OHNCU2tNTCtDMVJISGVRQXgvK2MyWGRualJSZnIzNU9tVExjY0Y3?= =?utf-8?B?bG92OXMwVnJMU2RBK1FzbFZKK2dzMStSYWIweS9Cdkx2MzNjNndOTDF1cDhn?= =?utf-8?B?M0VDOG1ZZnNRYVdmYjAzc21NemxtbEdiTVh6bmd1Z0twVlhyMWttRitVakR5?= =?utf-8?B?SWFubVo1OGd5MTFlQm04VFFXUy94NWZJWHVLSHkwSUE5RGtvaUx0T1ZFb1NB?= =?utf-8?B?SFZxc2NXUGdOVXB4TCs5Y1VzNmExanV1UFg4cGNleVVWTi9IMGkyK0puMXpQ?= =?utf-8?B?ck53Y1YwaXkxSzJMUEt0cTVOd2N1clRHTE9KdkoxRnhvMnV1MFlzVi9zaWZr?= =?utf-8?B?TlBZeWpXa0tLQjNFWmJqUmJ5TG8xbWsxZDh4eVRjRzRrYlF1MzVFRUFTYy92?= =?utf-8?B?UzY2dXkvVW0rUTMxV29zbnUzV1F2bzBPMlUwNjNsbVR0WDVtU1JyTGZYdVo5?= =?utf-8?B?TDJmT2NKS0hPaENXcGtoSVJrbVV3elk4dTAxZTZ0ekV5Z1AvZWJjL2g1Z3o0?= =?utf-8?B?emF4eXJpZ0lpRkx6RFFEQ0kreHI2cnNRTWtSNVIwS2hrMXNzWmJzbnFIYVJV?= =?utf-8?B?czhSNkxwMTNlTVJLa0ZibXk2ZElaRDZqRjlrR2JlOWtwSGQxcXM4Sis1ZU1q?= =?utf-8?B?MmpnTHBaSlprSktFN211bXFnZTVaR3NvcmUvbDA3STVHQW4zUnY2QXFiZW9J?= =?utf-8?B?ZExSQXZGZURqVnNEa0NnMGtJMFZFYWhrblJWTkVsK2oxMEpGVWQ5TFVJaDdS?= =?utf-8?B?WkFkWVhOVk5rakUxRGNlR0lla0NZb0RXcEw4VTNJc3R5WEE9PQ==?= X-Forefront-Antispam-Report: CIP:216.228.117.160;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:mail.nvidia.com;PTR:dc6edge1.nvidia.com;CAT:NONE;SFS:(13230040)(7416014)(36860700013)(376014)(82310400026)(1800799024);DIR:OUT;SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Oct 2024 19:52:04.7395 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 39a34323-0053-4272-2692-08dcf209d689 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.160];Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: SJ1PEPF000023D8.namprd21.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB6237 X-Stat-Signature: 89e7w4e6z4d1cskk5p8nmtq9dh4764ou X-Rspamd-Queue-Id: C16CE40010 X-Rspam-User: X-Rspamd-Server: rspam10 X-HE-Tag: 1729540320-979435 X-HE-Meta: U2FsdGVkX1+tyDUtGDJwHWpP9MrL/RM+jZUFjktgpaG1xpH8jl8EJb2D6X8nTlXMIQClJGJ0SH+LdOXVOzry+L7UweAZdG9yhQoRWPbaZul+9AFYE+lhqvEVf/ATrb2wyr4ExSnqH8abNSWyJB4XFHHyVJUecVmt3mvyjGTwuk2Bic1yVVd74Mfdf7/4p8uuCQntpI5amw6yH1t20f6IYk4SGlGd4CGODqD85SyrjCBddegoG+SiBItD4Cy4evkohTFWeDRGSTCIey4psBbjxy54AKId2XOdsAikpXIIsfiySM3JCmcGSvsGNsyW3bAlhBap8iNDrPn8HaSpchn3Pci3l0NSgLvmbcPfv5vYA8GN/2ycmFlObEOISfr8Z6ikEyytbH9706xMjUkKHTD6mIV6CcwgDxWctLua+FGDk7tCQ2AVMsggWLIDU1I/Q3Dku2cZe/zRUSBDvtXyWMorImVlz149OL7/vUMN+YdcfgV3oGPvW5OoPOc+AXdyCOeCWITyeMgZannjMxKCd1JBxhZ/Bgg/yGMkCivcHDNGmK2L7LUJB6owca5pUMrvVOnsAn65gJ9jhgSUOA9wmYisOqkItjCYeakegkETAvA8DR9lOdru27+Ee6ZpfuiUuYXqz2Tvx8AlVRn09UGIHDbnOKuR/AV60MUFMliX6dbpNCJEHgAsWCxDNwBKr2hzicp47g0QFyj2fE8jgi7nKr8VDj65TiUzOGmvgU3ckEtIp8j+BsTBP5j/doU32lSZZDcmKYRrNoT7G8qxxEj3gCC4jthSXSV2/yOiNLNqKQoHNOTZiSPHACd32UDxGBTiz9m3HkJQcVUt61bkER79q/choFOs7tVFEWtseJqufpU1iommo/nAy4mpAzHg3GxumQDO9eWR9GWh7wtZSkDLm+X66c7UiFBSJdLZMdM/sIaK+7rR0tJJIb9e14UmeVrst1+HsSd4a9Guk2ArJg88ehH vHH++6nf QngEmKu6xpoOLNoULEyO9U7hz8H5G9Rw0ULU38L2NELFI/dT2JCrQ02P6Zh+Tr1FtB2+/8+Df7yNwSY8bacfeAEs+B6geUVy6arNWpGOGHb0GRqqPbEEatqqDq0oMLAOKv8TX3ceoHKdupyhuHgGaH8qaZTElrPvS55C8qkNtPrvQ8eVcvJT4C2GwYlkLHESTagAqXO3hgtbYS3cMaJooT1QK5S9hXFe+LrhdG8eGhiFaR/qYDR9D6SF2dc7ADC8M+5eM8e/SiKeb9WnPgt1x73/8Kpwq0aJP2WixBZsxyFSDMIURsbFsTyrBsmWbkIunLGSJ6Y0O8F/P8SXyHeaGh9TAAuKohHrkh4qcrhy/WYpYgnaotuMDLb8j7jL5qATVGkae/mJP1vexLPJUnNWtTRf9YvVMJT2xrIzf4XJ0B3ueuq3zi0CRbWVRiCZGsLh+L3RvvOHuh0qL6HMASgwdHldKjvVPOqkM9dthS354Ysxl1T6dIG2c0lf/clcicbGqZQB+iulbfyRxjANZ/O2ETr5zNBaBPiGzSpfVdRxsO3EmpJmy0Zb14xf3MAELsrOOddSiwlLXy/uT8TM= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000539, 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 10/21/24 12:49 PM, Alice Ryhl wrote: > On 10/21/24 9:34 PM, John Hubbard wrote: >> On 10/21/24 12:26 PM, Alice Ryhl wrote: >>> On Mon, Oct 21, 2024 at 9:09 PM John Hubbard wrote: ... >>> Rust macros can use different types of brackets. For example, the >>> `assert!(1 < 2)` macro uses round parenthesises, the `vec![1,2,3]` >>> macro uses square parenthesises, and the `thread_local! { ... }` macro >>> uses curly parenthesies. The round and square brackets are used for >>> expression-like things, and the curlies are used for things that >>> expand to top-level items such as global variables or functions. >>> >>> Macros cannot use any other delimiter than those three. So e.g. <> >>> wouldn't work. >> >> That answers my implicit "are there any cases in which you would >> want to collectively refer to all three types of...bracket?", yes. >> >> For the original point, though, we are not in a Rust macro. Is it >> actually allowable to use [] or {} here: >> >> +    // Brackets around PAGE_SIZE-1 to avoid triggering overflow sanitizers in the wrong cases. >> +    (addr + (PAGE_SIZE - 1)) & PAGE_MASK >> >> ? Is that why you were not seeing a difference between saying "brackets" >> vs. "parentheses" there? If so, this would be yet another case of my >> Rust newbie-ness being inflicted on you. :) > You can use both () and {}, but you can only use brackets if you're European. ;) I *knew* I was missing out! hahaha > > Using {} to create a block works because a block evaluates to the value of the last expression in the block. It would be super weird to define a block here, though. > No argument there. :) thanks, -- John Hubbard