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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A626CCEACFA for ; Tue, 1 Oct 2024 19:12:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:MIME-Version: Content-Transfer-Encoding:Content-Type:In-Reply-To:From:References:Cc:To: Subject:Date:Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=ruttxAPmFHwQBMP5oGSnn8sHoE+Hdsk/Ioz+xukebOg=; b=0BfhGDIS1fCZyf/+bx6qmAC//Q 2kkMTlCUVZrC00HaK8z2pByRPEnTLETVLZWt6wsunsn4Ag5MafP74JbNhh8Yna7eO9htA2/5IoS6D UMZMYTh3F9ZfcTjbqZUekQlEGVZ4FZftG5JBNQq2LGMjQkYO1wVTz6C4142gyU4vpYX2u2Xnww42Z 995JoYgeWEmybAi1Szn661zC8Xdlzt8YP+2EZ0jza4TLUTTiAjruRr5IGP80FmoDWleMtaOTZ+9ER /ynkGbO+kmhdfBcspuKZ1piQQzXbWr9NvTWThl9K83EoCRJe4dtQ/gIiBfV+y6LQwx0tdHwz/EQgo M8fryBiQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1sviIW-00000003rO1-2WQN; Tue, 01 Oct 2024 19:12:36 +0000 Received: from desiato.infradead.org ([2001:8b0:10b:1:d65d:64ff:fe57:4e05]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1sviFO-00000003r81-0Es0 for linux-arm-kernel@bombadil.infradead.org; Tue, 01 Oct 2024 19:09:22 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=MIME-Version:Content-Transfer-Encoding :Content-Type:In-Reply-To:From:References:Cc:To:Subject:Date:Message-ID: Sender:Reply-To:Content-ID:Content-Description; bh=ruttxAPmFHwQBMP5oGSnn8sHoE+Hdsk/Ioz+xukebOg=; b=GtS/FkHip4VyY4c3jNS5zn0FwJ LhGhR4vaTFGrbMAgLOxUL8VAgUaGej3KUytPO+Utm4E17y+fQ4aOcDtG99QDLKfp8b55we0ckGw4O rgXKBYqh5SVLoYrq7CTXCym8PkcK+xNW+y/uQSsCXD8n4CKGJbIU1PU1oTEnWp+cmvU72bmzLR26g NW4dymw1mPwNledLP3D75iBmRCVmhFo4KoEzfeb8n3n0Vr9BCkbAGUqPP9Jmr7Vrx4G0yhsA/qPS2 hc6L7KZi+qt/KJEFutM2eEkKOpftGA8n/w7QtiyoJObgppz/cOQvWlZp+kQHNu6b/87cFtSbUn2uY 3UGcst9A==; Received: from mail-centralusazlp170110009.outbound.protection.outlook.com ([2a01:111:f403:c111::9] helo=DM5PR21CU001.outbound.protection.outlook.com) by desiato.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1sviFK-00000003Ifm-35r7 for linux-arm-kernel@lists.infradead.org; Tue, 01 Oct 2024 19:09:20 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=OazryjpGXReQqjBloKV5zJGB2/gH1fMp9dCBFkbEBIx1/PNPgohU6DlUCyziKUY9qIeoXo0GH26ydSVR4hZhcTcwWg6EFI8n4KF6Tm5xbIQ4MonVtLTe3RkpqwibCuo2f2xyF30zFUVFkmyQDOy5Pl6shTSgRL4639ri9Q9DUSouWeZJk2wPpvL3+y1l+rivSRfsw+NZoP47HV0IAz/PJiUCdZxqHhhK1lJK578YTOlV95a/mvg9oo8sVK/eZL/FCvNz+MdE2TNHeUCEDeF8dntasJInWKyN5S1xd5t/1oYdhg/0/aXlNYDFlxrt8FggOWAkB4jI7B0BC5Y/zItGCg== 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=ruttxAPmFHwQBMP5oGSnn8sHoE+Hdsk/Ioz+xukebOg=; b=NTMLbWiqO/130Y3sbUVOF3EPlz6PGW024BPqTsyQSNtPnNTlNI6LwTFt1AJovEWLLHIITE/k0KfdKV8wuQy8gKv4Za4y76ul3SFDr8x7nsKJXUoSxpk0TdBFOmp0TXOlBhvomWR6xfRfzRqNqn6Ow0ul2+malnY6xRhVt+HG8qh00b8XNivIMznjWfW98j2b1QHEOp0ESodgMtjkR96zVsWW1mzpnnIkbKEiW2TbCl1zVs63KnsQ1Yj05pD67AhkdHGOsfzKxnhaxo3n3/WnacbUol/8SuPqoA0MSKZn304CbEad3QRQNwA+I+u7xXTGVtRWpewYENDLdl3XmDBwBA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=os.amperecomputing.com; dmarc=pass action=none header.from=os.amperecomputing.com; dkim=pass header.d=os.amperecomputing.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=os.amperecomputing.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ruttxAPmFHwQBMP5oGSnn8sHoE+Hdsk/Ioz+xukebOg=; b=noKAoaZlhgCnqlob/cIn+BSu8orlK5kqfgsNZ32i7a8mhR1DbQE0D2pnFobHBTv+NfUYCos5XcHJMW6HDU8fs7Bthp8UyLT6dk5hrGjIlQVURMwUvRFiEZjIpvhD4bqBjqTIE8Mk/DZUPvMIIBI1YZ96cGGSNR8T42EJDKmDkY0= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=os.amperecomputing.com; Received: from CH0PR01MB6873.prod.exchangelabs.com (2603:10b6:610:112::22) by MW4PR01MB6386.prod.exchangelabs.com (2603:10b6:303:76::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7982.31; Tue, 1 Oct 2024 19:09:07 +0000 Received: from CH0PR01MB6873.prod.exchangelabs.com ([fe80::3850:9112:f3bf:6460]) by CH0PR01MB6873.prod.exchangelabs.com ([fe80::3850:9112:f3bf:6460%5]) with mapi id 15.20.7982.022; Tue, 1 Oct 2024 19:09:07 +0000 Message-ID: <45b97496-29a2-4111-ba38-3c8bcf9f8b4d@os.amperecomputing.com> Date: Tue, 1 Oct 2024 12:09:03 -0700 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] iommu/arm-smmu-v3: Fix L1 stream table index calculation for AmpereOne To: Nicolin Chen Cc: jgg@ziepe.ca, will@kernel.org, robin.murphy@arm.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20241001180346.1485194-1-yang@os.amperecomputing.com> Content-Language: en-US From: Yang Shi In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-ClientProxiedBy: CH0PR03CA0252.namprd03.prod.outlook.com (2603:10b6:610:e5::17) To CH0PR01MB6873.prod.exchangelabs.com (2603:10b6:610:112::22) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH0PR01MB6873:EE_|MW4PR01MB6386:EE_ X-MS-Office365-Filtering-Correlation-Id: 263b5287-fc8f-46a4-6474-08dce24c85a2 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|1800799024|366016; X-Microsoft-Antispam-Message-Info: =?utf-8?B?V1VUc2NESzBBVXVMREw2ZVhjTFRZZFZ5K0g4QzVyaFlqVG0weUVPMERFNncx?= =?utf-8?B?aTVlejN2RzhSQTVJcmtadmdZSjJFWXJUZjdGeEJGQ2FyMlkvamJ4aFVLaEhG?= =?utf-8?B?OW1oaFpDTkFtYnJQaVdSdHJOZWk0Ty9OYVlhR1dYSDNvNEFOMlY1UTN1V1Fz?= =?utf-8?B?WFBOQmRHdVFhcjZZNEJTNlZLL3p6TFN4dCtIaEZ0RCtiaW9YSDkwSzZESW9r?= =?utf-8?B?WUdLRFF6Z3BEUnFMNldaVUt3Vm5VRlJNU3RhcjNMbFFHTEVmRFJreUV2RlRy?= =?utf-8?B?Qi9yK091L0ZmWjVEbW1XWEVTQVVhN3ZSaW9yaDhHSEw4VzlFS01sZkY5b0VH?= =?utf-8?B?MGdiM1Nic2o1Z0t2RnNNN3hBaWg0elBKYjBrOVF0Rk9lc3crTTVZK0xvSzlv?= =?utf-8?B?QW5JdmI4amVIb0NSeEJjWlIwd0xlMjI5c29Ma3lkL2lzdHBmNHVlT2t0ZENz?= =?utf-8?B?VVNCQUw2cThZeE54cVVQYVl3MGpjbUlSRDI0cG01THBST1NLVmNTcThibUdQ?= =?utf-8?B?V2Jqem1pQ0VKUHZ4VmNKRFVhcnlObVo0eVN6ZVFkZjFQWjFjaE0yU0ZSa1V3?= =?utf-8?B?L1h3WTZFbGFva1MyaUNPT1U0cWFVbVVSZ1pYNHEwaHBNdW13YThYS0xqZmJW?= =?utf-8?B?QllmUE82WFVkRFpHRTl5NlNLM1oxTytYR0F5YjQ4Wi8wSHVsU0FKdnd4c2c5?= =?utf-8?B?ZlA1TFRoNFdBVklnVTNEVm9qMmlxU3VhM01FaUhGblliWm40VnRMS2lwNDVB?= =?utf-8?B?QjVKM0VKcXZtazBGa04vUVhuV1VBWGFmQllaeU8zZGM0VFRvaGVUc1BaZmdF?= =?utf-8?B?bENrbzZFSFZ4ZzNnOEZrdUJnN3JqTVFkcEJhSmlWY09iY2VIM2FnNXBzVmVM?= =?utf-8?B?QSt5UmJlUCt0TlVRSTRDby82WUdSMGduc2J4b3hhK2ZNVFRPcTRZR01UR1VU?= =?utf-8?B?cnNOaUxWY0l1M3A3cGVCLy9pU28wVlJVQ3Y2cTFrbkE1VHIzTmM1ZTJPUW5j?= =?utf-8?B?ZjdUUjl4YmtoTjJ0LzYvV2RydzByZ2JGejF0OWpDM1k2czZwK2pIdjRJNkFo?= =?utf-8?B?THBucndvbEVIcnI1VkRLNW5KQ0psbVRITHB6RFBocVJ6bHNDYXc0eWhXZmdB?= =?utf-8?B?NXQrMjFRdFpmdUVNYkxuK04zNnBKUEVKeWUydDgzOC9heE1VMW93V1R3a1ZN?= =?utf-8?B?WVFsWStYdjFUK0VDb0dLYmY2a21BRzNKSUtoa3VLdGVOc2lVS0pWVmh0dWxE?= =?utf-8?B?czZTcEp1bUF0ZVpycDd0UGN4cGRFekxHYytSRXprZ3lMQnNya3N1TmFRUTBx?= =?utf-8?B?L2p6d2NPQUJZS2NIY1NZSGJCVFRWRVFzS3pDMXh5RGNwQzdxdGVuK3V4a0N5?= =?utf-8?B?d0JxN2dXOFFFUzZmZGdFdUUxSk5aakdlaUtPeFNGVDZrUXpCQm9jN2J3bkpL?= =?utf-8?B?R3hCNUxZWDQ3RExaQVk2WHZqWUF4NGpUM1hKMWdyUWVjMGlQMnhFb1REeUdL?= =?utf-8?B?VzdDbTBZYzZYc05JcmhlZVB4d1J0T0YwNnVFbGwwNU9nTGRDOGg4ejQxdS9i?= =?utf-8?B?Rk44N0daSDNNRjFqdm04b2g4VWtUNVpmdm9IWkVSMVRGWjVHZHpHdDduYjJ0?= =?utf-8?B?b1VDMXAzYm9vdkNNYUNOc2V2UzAxUHNzdWxvSHBQUWlsS05kbVk2ckRvMnpz?= =?utf-8?B?elc4QW5MeS91dmZOZUxQYWF6QXcvZkVsYUJyOVF5NHRYU1hHb2FrTHJnPT0=?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:CH0PR01MB6873.prod.exchangelabs.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(1800799024)(366016);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ck0zNk1yYWZZOGZFNkV5enVUV29xN3N2Q3c1aUFMZHhsdDhrUHR3Vm1KaFFn?= =?utf-8?B?NlJtY0d2aHR0YXRWSTZ5QnNhWlhjTHFhYXZJNzlBbGYrZ3VPd3BQekhPdnB3?= =?utf-8?B?MmhodjNDVnQzVWZhYytKRWJyUmdIZUt3cGdySGMyS0J0MU8xRVRKcno2bHRQ?= =?utf-8?B?VENXell6cUdxOGw4YUJteU5wMmtYL2lFeVhQeXVUNXRxR3lpWERUOFl6ZUp3?= =?utf-8?B?bUovcjVCZzVDdSt6UE0xY01BMjJ0Z01aM1l4WElrYkRPVVlhZUwvQU1DTFBh?= =?utf-8?B?WjIyKzh1YVJCdjdJTFFrQUpLSngwLzA4RXExMlQrengrNHZMWVlibnlobzBJ?= =?utf-8?B?RnlDRXQ1bGN4aVQ1Vkxnc1NVdnN6ZmtIMStYTzU1SGZzNWJYeFIxVnBnQm9N?= =?utf-8?B?SUFnei9pREt5bGx0RGE1ZGdNN2Y3TDNHbTg1QnNFVzVHbGUrMkIzRS9lTGlP?= =?utf-8?B?VStHWGdJaW5uTkV4WXdOUVozMnZFVGdXTEJzdHloY3BNVHZpR1hNN0VPMlFn?= =?utf-8?B?VUNyWkI1eVdLdkdGZENMSndjZjRKTVN3OGVkbTVteWl2RTYvVjZMWngvQUI2?= =?utf-8?B?S3dLZGpINVFNbDNSbHhuM3pCaEtjMTZ1bjJybGVsNm1FUW13cDAybGRhY1ZD?= =?utf-8?B?Rm5TbjgzT2F6OXhEcVVDWnZwUU5EaUJXLzVjdXVpdHZDd29TdFdPOGw4b2hM?= =?utf-8?B?em10OFordFY2NC9YdEtia2tKOFRmQUw0cjA0SXVrQkwybXNtZFl6eGxLOFU2?= =?utf-8?B?bkZHVkNOY2ZSTWhKZ3ViWVBLbGdFUHNwWkwrVUVoQWVTZnVqN28rY0ExdFd0?= =?utf-8?B?MWxrZmVPSDgyQisvOUd5RGdDK0VDRmpwZnVnNG93ZVFreDZQNlhJcVdZcVlW?= =?utf-8?B?a0JiVjczelZEYStnNlZhWUJLK2pDQ1JHWG9uYXl5SlkzTUlWN0ZvSE9tWm5B?= =?utf-8?B?QUVnQmhZeG0wUGUrMWhBL1RMVE1sUG00UUM3UnhIeEYrYWNoNUlkTW10Mkwr?= =?utf-8?B?ekFUeDAyM2RzQ2JzVGpDWVhxTmF3T1BJZ0xjTjlNVDd5UTBCVUVXVnFyUE1k?= =?utf-8?B?WGo5TXkxM2lLSU0venNLWm5oSld5NlBtT1FUTUNOQ0t0WS9WZC9scTVjSUJz?= =?utf-8?B?ZE9kSGEwTW51K3hhY3d2M2ZJclBEV0VLandEMWl1d1hsWFFLcmJaWk54bFdE?= =?utf-8?B?ak40Y3VZeURnaGM1M0Q2aG1PejlFOWwrTFRTS0ZLSGI2bGN1M0JUcHhwWUdJ?= =?utf-8?B?K2pyalNOaXREZ0oyQWxUMVMzOTFxWGx0TzlFelg3U0RVOVBXRXdzbEFqNllj?= =?utf-8?B?SWdWQ0NObm1USnlXOUhZOUVXeGhjSHZVQWhxKzl2aGJnd210NnlkbHZkQmZ5?= =?utf-8?B?MWN4RmsyeUwwL05MeHp0Uis5bVdaeSt3MnlEQXZwREF0Tkh4YlJZT3JuTytr?= =?utf-8?B?NWVpeGF3RVQvaUhFSDBLZWVGMlBBMzVsRko4ZHZySEdqWHYzQzVXMi9BVHZW?= =?utf-8?B?YXJaUFhGMVZiM1pxRmlBbUtwYUo0ZUJiSjV1M1JXd1dDUytRdWduMlJLZ2ZY?= =?utf-8?B?YXg4Ymp2WVJTMS9HeVNjdHloVm12cWQrZ25vU1Rpc201aXo3d1BmSWM0bHda?= =?utf-8?B?ZUorNGJyeFlpUmJRdUxlR2NzSjRuazR5QWJJOUlSR2dXK1F5cTFWZ0oxSWFy?= =?utf-8?B?VkJKaEhYb1kwbEhZcFkwa3dMR0x2dHpwOTZMZzd3Q2NTc1FFTzVndzBFS0Fi?= =?utf-8?B?NVdNQnk1dEEvZ1lCQWdNVmlvMDJrb1dvcWxNK2thWHNoSzlYQWxUYk50OUdV?= =?utf-8?B?RXBsSFNjNERqVDVEQUkvK0JTVGFFUFg1c1ZuSnlocmRKMmozZ1pSLzRtZG9F?= =?utf-8?B?R0RvUkxYVHJ4MTJ3TXpucU5MRHQ5amRYODJ5bGp0SEV6d0FlcnJuZXZZekNU?= =?utf-8?B?eHVTUVBlRzFlMTlTSUxLVWpGczRhZUJid01aaXhrRjUrZHVhL0E5OUVITDRp?= =?utf-8?B?a3BacTJjeVJ4Z3ZiNTBYSWtiUHdRSnpMam5zc2FOYW54dkNnQm84VEFyWnFS?= =?utf-8?B?S1FOVnlBS2ZLQjhRZzZwQk1nSnFyQm1NWmM4VUJoQkxvL0ZyUFFQbElVR3Bv?= =?utf-8?B?cXluMG1KWkVWQldvM1pyMnRzVWRla25aeTV2ME1DTW5jTTRxM3MreGp3RXlo?= =?utf-8?Q?X0jLTv0PwfTIzmhaGkiUyEQ=3D?= X-OriginatorOrg: os.amperecomputing.com X-MS-Exchange-CrossTenant-Network-Message-Id: 263b5287-fc8f-46a4-6474-08dce24c85a2 X-MS-Exchange-CrossTenant-AuthSource: CH0PR01MB6873.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Oct 2024 19:09:06.8718 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: bzqTSI4juVSvjzfTYWBwtrVgqx4yOonG52V33+7fQc8tIUdrDVIOBQCHGy5Ri9er40C4i+UQAcFmqdIXzmr4EWw113Wa43P37OXgmdxoIc4= X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR01MB6386 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241001_200918_955387_0DD92680 X-CRM114-Status: GOOD ( 13.85 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 10/1/24 11:27 AM, Nicolin Chen wrote: > On Tue, Oct 01, 2024 at 11:03:46AM -0700, Yang Shi wrote: >> Using 64 bit immediate when doing shift can solve the problem. The >> disssembly after the fix looks like: > [...] > >> unsigned int last_sid_idx = >> - arm_smmu_strtab_l1_idx((1 << smmu->sid_bits) - 1); >> + arm_smmu_strtab_l1_idx((1UL << smmu->sid_bits) - 1); > Could a 32-bit build be a corner case where UL is no longer a > "64 bit" stated in the commit message? It shouldn't. Because smmu v3 depends on ARM64. config ARM_SMMU_V3         tristate "ARM Ltd. System MMU Version 3 (SMMUv3) Support"         depends on ARM64 > > Also, there are other places doing "1 << smmu->sid_bits", e.g. > arm_smmu_init_strtab_linear(). The disassembly shows it uses "sbfiz   x21, x20, 6, 32" instead of lsl. 1UL should be used if we want to take extra caution and don't prefer rely on compiler. > > Then, can ssid_bits/s1cdmax be a concern similarly? IIUC, ssid_bits is determined by IDR1_SSIDSIZE. It is GENMASK(10, 6). So it shouldn't be 32. IDR1_SIDSIZE is GENMASK(5, 0). > > Thanks > Nicolin