From nobody Mon Jun 8 05:26:39 2026 Received: from BYAPR05CU005.outbound.protection.outlook.com (mail-westusazon11010049.outbound.protection.outlook.com [52.101.85.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 47BBF3ED13F for ; Mon, 1 Jun 2026 20:45:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.85.49 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780346728; cv=fail; b=TcxYKwErnnzBdeL+PKH/7rztqYjQ61bpZ5Wm0EpuBC2dVLSUXnNuH0+9vrmHtXSA8q9X1uJAuNnPFJ88tF3+fx6IoYz5Jnh21P1i0zodxjQA3eXzGWX2wqe880jDqnhUmaCuS/DqstsPk3z/8869h0YMYf1KhXKQ5fSJmNhDddk= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780346728; c=relaxed/simple; bh=EXM4jqd4vzs3BNtX0aXlXUjp86LIY4767PoZzD63XDE=; h=From:Date:Subject:Content-Type:Message-Id:References:In-Reply-To: To:Cc:MIME-Version; b=mb/PlQDmfCEl7jGT79JpOSudrp+DzAHdI1YAa9bDdvHjc0OTN+6nWuUucG2XFz/CrdXH47PByrwk7kUyzQdu2CxLB5iBdLZGWnLfjSwkPioq8wfY0lhuIKbhXG+nXbBqNJ5TXfsbV8G5na/aM8WnY6LIpf+dnRWmr0mgVowdKw8= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=nvidia.com; spf=fail smtp.mailfrom=nvidia.com; dkim=pass (2048-bit key) header.d=Nvidia.com header.i=@Nvidia.com header.b=ROTKX22B; arc=fail smtp.client-ip=52.101.85.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=nvidia.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=nvidia.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=Nvidia.com header.i=@Nvidia.com header.b="ROTKX22B" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=lojtGvPhyRER6xTyd7munzzwJwcn8Dz+uqZP5JCITFKzBbZ5idCTKZda5SESUONRPvj+HkflA5optbEcqv5OUUr9uDDlwLgrCZFW7bw6BSUC7E55qMrfNll++2CRYtZbft+vXflI6zIbacqkfQt5zhg1WaWkmFf4MrtKltxTvkqZnpD+DUMGSsibr12sapmMHJifdhJIezx+aNNpFS4cE8+eYJo42O5SjD4r5udM8X5MQZ1ifPxhTsdbDohpoZIVqHGYdJ5xGIU12sZjBNrUxx4YgIBVInpVyDMuxfXGQlKXRu6o+u1eYvO7vUtE9P5YvaTyrhlOPQ05KhEmW9EfQw== 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=LPdJovlkw4BiJWaR3ZQrXyGhuFCOoZ/QhCpmbwd4qIA=; b=BSfliIlEtd9jAwA+KpFKD5fIgS3QQTHO554zph5m7/CaaXBf9Lr2phpdNWpf8F8rE5eToy6HHZ31YIm8/4LGapSgY9DN4Y/an2D8KIsACBZMdirLaVCCxvqjPXEgpjPQu6TTzzC+U28sJsZLXhE9A8iGUlRrdUcRgAWr5SoBShYQ0Jn8FHBqMwa7kPJuEcYTyAxcAL2q70Ta+5wbT/KPKMz5jXXvq15fZiTSwmNQmBOqcLcpsmpwbabChMLo9tu2qUHCpVr5ZjHffwRdc9+3wj+9r+VooNuROHIXMNutnKqGCydu0t2gt9BIPzFhDTU56IB2Xfxcx6+gP9PKuzpoEg== 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=LPdJovlkw4BiJWaR3ZQrXyGhuFCOoZ/QhCpmbwd4qIA=; b=ROTKX22BO+zZiIbzBtrnjQYSBceQGoxGip2EUbHdBl3gXNE7GipREE4fNzfqPKVwuPGNvFOO3ETwY3lWvyzfqYVwW02+Wn/7QZ1RG9hwpUmMA6WFNmZULg5zlKPVFdMqoFnefNfRTa8JYtmxDLVjdzIad8RY4YcVFEQlIZ3vA+HfN8sdIsBueKNZ8QNh+FN4wRhPhq148Rd/PTmaL15Z2Kz/iE5qz9npm0y4NfokxE95H9DIU2+/NgRhWwTiwis3kzGgogn+HFP2W4AIHjTP1PxGWTixAyzw8Zbulx//2CCLQXgiAfmRJyJIvZzEfWHTz907rhDVzmRHJYQDi5z3qg== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; Received: from DS0PR12MB8768.namprd12.prod.outlook.com (2603:10b6:8:14f::20) by SJ0PR12MB5674.namprd12.prod.outlook.com (2603:10b6:a03:42c::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.21.71.17; Mon, 1 Jun 2026 20:45:21 +0000 Received: from DS0PR12MB8768.namprd12.prod.outlook.com ([fe80::c16b:c351:2fc7:95e5]) by DS0PR12MB8768.namprd12.prod.outlook.com ([fe80::c16b:c351:2fc7:95e5%7]) with mapi id 15.21.0048.016; Mon, 1 Jun 2026 20:45:20 +0000 From: Seth Forshee Date: Mon, 01 Jun 2026 15:45:11 -0500 Subject: [PATCH 1/2] firmware: arm_ffa: Honor maximum RX/TX buffer size Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260601-b4-ffa-rxtx-map-fixes-v1-1-c071b12ae05c@nvidia.com> References: <20260601-b4-ffa-rxtx-map-fixes-v1-0-c071b12ae05c@nvidia.com> In-Reply-To: <20260601-b4-ffa-rxtx-map-fixes-v1-0-c071b12ae05c@nvidia.com> To: Sudeep Holla , Sebastian Ene Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Seth Forshee X-Mailer: b4 0.15.2 X-ClientProxiedBy: DS1P222CA0006.NAMP222.PROD.OUTLOOK.COM (2603:10b6:8:454::17) To DS0PR12MB8768.namprd12.prod.outlook.com (2603:10b6:8:14f::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS0PR12MB8768:EE_|SJ0PR12MB5674:EE_ X-MS-Office365-Filtering-Correlation-Id: fcb8c184-6045-4007-8a99-08dec01eb269 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|1800799024|366016|56012099006|5023799004|18002099003|22082099003|11063799006|6133799003; X-Microsoft-Antispam-Message-Info: 6AQ9aRBUVha9n/UNl8tezS6O5WefJSvBjPYm5WAEkuAWH2GEcFuTxdO45Byv7bO/5n8Vp1NzKO+L6m6FySoVIeY5rda0qp1R4pC6JVm3KRpeibR6XzIcaAULD9cnMNv+LC/dlBkHD517LGaovdLiw5YKkQoCvGiOYKEATLQVOukYb1xuQ/PIq7Z0Bmv+/IhLbnPe0klqWYdXfw/MKyvSokiBSoszfWKNAqbwPTE1k5MztjY5mnECWO5sORtnNboVpUzKoqzrYT57j+w9+n8DEHYXMLUPoPaLfl9g4YKpT/bBZmJ01tKwMJws/LB93joEA4/OOpAXIyhLgAZCERmjp+lyuw8jWjNGUS5bRBtkOD9G2KWF1le/W6pYZY5emYqs4D+vx0wbFRYXOjNzz3coayFNjTWFMzDxG+lCUR7YH5iMOvEmDqZktEQMakK3C8cTUw4+VBz1nr5bb4pa17eqlHRcg2Y+lGfAFNFAp6ln4ThQDr/x9jGmDIUd0CN1iTRXa5t047uIHFtIlhRNBqY0xGAkvsl7G2mVaTrnAmcz59uouiZfDJ7JoavXomQvif8au4eh9P1DKMlI7rTy5b02PmeeIcslyf0HgVjZ17SohxVv4e9JL8lHoKoQWedIgUNf8ZCrTuixvGNEaH6eIR/Zki9dfHNSha8UeGjpfjiO4qRVaabfBLFUUBHmmDzsl2SY X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DS0PR12MB8768.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(1800799024)(366016)(56012099006)(5023799004)(18002099003)(22082099003)(11063799006)(6133799003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?dEhnTXg2bndodmhKa3VDQTJHRVJxMTc0cW5lU1dVZGNDeEZIcmhuYVdwcWt3?= =?utf-8?B?OVc3cE5VZk9HVGE1UFBmQVUvbkthZEttVzZHMUt4TW8rVVZTWFJXd1ZlQU5T?= =?utf-8?B?dnRkWHRrVy9acWVkcVp2by9zMHpyZ1ZOdXhRcGlCb05uTmZmb29rQ001dnFM?= =?utf-8?B?bWh2ZVF1ZHA2dWVTclF5MCs1a2J4TlJhWlNnYkJCUlN4S2RPWVJRdUZZSUcx?= =?utf-8?B?SitMVGd6WTg2eXE2czhwZmlzaXJMcGRiTHdZRkIyU0NxT3JtUEhTWmxLbkwv?= =?utf-8?B?d3FMUmJQeG56K0haZEh3QjZXdG1lMFZ1VzNpUlphUW1TaENpKy8vRzB6c3g0?= =?utf-8?B?M2FrNG1vUDJRWjFla2JhK0VpWGMvSU4xelQxOXluZFg5U2gvd25ySTUwaTRO?= =?utf-8?B?enlKT2xQS24xVjkvelJYNHJwemcyYURqK1N3RzRkemMzOXpldzU2NUZYN054?= =?utf-8?B?MU50SHU3cTRWcnQxbWJJOHkyZ0dzUThjR2U3S2s4Ny9ibUE3SitNQnFrV1lL?= =?utf-8?B?WGkweGwvSWNBbmQ2NWhYSFozMzBKMUZjYWxZUmNRaEdjcWIyM1VIaGYvUmtT?= =?utf-8?B?L0RqRXQrOFhPeTgwYXkvQTVVRW9rL3BSZ2xoYkluTUFLbVBRMlVYb3B3WHlz?= =?utf-8?B?M0MrZzUrNHh6dCtlSmk0aStieG9KU05tNnprZVlqQXRFd1BaazNNNTk2SG5a?= =?utf-8?B?eGdKbHoxR1pXMFdkYytRK1BrdFNBVlYrMGVtWEU4Ri8xMURXRGpMNFAzQm5y?= =?utf-8?B?R01lYkxhUVdyOHphN1VLbkRpbm41N0xtUDVKRk1qVnpUaXdVVkxjK1N5ZE1u?= =?utf-8?B?QVIwREk5ZlJQaUJJOEdZSm5RTWY3Sm9OclpSUEtOcUlLZkJtUTJlREZHQTJN?= =?utf-8?B?TVU5emxvMlNRLzA2Tjl4SFB1dmhaY3JHaUswcVA4Wm13a1R5QWRaT1N2ZjZ1?= =?utf-8?B?eFVCV0VSNkJqTWhyRGtjVERqMnVOYjBEdGxYZ0dwWTYvYVBzTEk1Nkt4S1Z6?= =?utf-8?B?aXQrdE9mY0JRdnErWDFDVjcvWnhkZXhFY2FtWDY3Skk2dHlETmw2bFRIclg3?= =?utf-8?B?UUQxMy9NWHluaENqK1gwWHlldEtpWjNsK05YZlNYTjd2bzV5c2p5MC9jN0Ey?= =?utf-8?B?dFMvSGlIOXdZVWsyeFJBNEdNeHZVK1ZGMm5OWWVkdjhkdURoMk5TdklHOFR4?= =?utf-8?B?L0I4aWZyM0RUUjBWT25HZEhKOW96UFVacnB4blBLdlQyVWdDV0h4Z25RL3Mv?= =?utf-8?B?c1NLcTllNkFpWkVBMXBGZzYzc0hqNmZJTGx6MnZ6Q01Pb3VLWWtaaWN6REk3?= =?utf-8?B?MmdocjQ3Sm42TW5lWFNCdEJHc3FtQ2w3bUw2ekU2cnZBcVNnRjBtbCtXQ2RU?= =?utf-8?B?a0tMWk1XdlpadkdIRE9kNXRTaG4rVWg3VWdPZHpkWG5YcTczbmxIS0ZtS0E5?= =?utf-8?B?M1YwenVUdGZDUk5henVGOHdEM0pyVDBsNlg5bkVRTVNrRmVHQWQ5SVRlUThV?= =?utf-8?B?cWd0N3RTVmJSU1BnWU44bVRSQWIxM2ZmSElEb2Vhc1VGMS81cFlPUVRYTGFy?= =?utf-8?B?MVJqNVFFMkxNbHVpWGI2Nnkyb2ZETC9IQ3Awc2ZuWHYvQjFNeEJTR0hyR3hB?= =?utf-8?B?SmtNS3lmWDg2V2xkVEorRHpKSDA0UDFmU014Q2J1TUtvaWVHdVl1OXRweVg3?= =?utf-8?B?VU5PbmVrdXVwZE9BVFFkWjMxRUNFbVRyTUhBV0h4NDJWblN3dDJyMUtxSmgw?= =?utf-8?B?ZFBEZGZwWVNncm45L1BiUXk3bC94VmNiUUhSY3ZOcThSMzFQcTYxL001MXdr?= =?utf-8?B?MmoxTUcvL1RqN09hUlErZUY4WmFwWjVqK3ZXajFMOG1KKzBQNzZJUHJjaFBU?= =?utf-8?B?NGQrL1RkbHJ6bEhyTHlEN0xiVHNNNXRST0J0WmZNOVJpejBLWnRxQjNBMmFt?= =?utf-8?B?Y0ZQOFU2bHNoNFFoRUh2aWUzcS8rTUJvT0JISHBrZlU4VmxWTnZlQlRkNEo0?= =?utf-8?B?V0lGT1p6dnZJZ3hSMHBxbEhGVFlMV3NEUW9PQ3hOc1RaS0FsdGlsWTg0cTh4?= =?utf-8?B?MElRRnZWNlhGaGd6L1U2bEMvUWVMcmRYc085c0lGU2lRNlc4Q0lHZVBGS1VV?= =?utf-8?B?SU5jTUhkcFl0K2t6d1c2eTBaV2dGMnpvQTFXRlN2aVh5NWRSeCtyRlFoVmk1?= =?utf-8?B?SnhzVXdBQlEra3hvUEJ6R2VkK29xOC9sSWtEM2s1b3Q0RDJ4UmNGWjdxY28r?= =?utf-8?B?YVhYQ25iNUxxeHhQek83UmlWOE9DT09EY1pKaWtJUnplNW9hcmMvdWNoRjVo?= =?utf-8?B?TTlNTERWdytncnl3V0w1cDErRHRtSzdFWEt0TnRaMTVHMTV3SVZkdz09?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: fcb8c184-6045-4007-8a99-08dec01eb269 X-MS-Exchange-CrossTenant-AuthSource: DS0PR12MB8768.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Jun 2026 20:45:20.9298 (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: 6Cqc3sIewpRSQIdRs3hPKqWokZSlWC7qoNwFq/x3WETlWoFV9lrT1F/RQhnTx8aRIDcvsXwo7IrDizlY8aXQ+g== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR12MB5674 FFA_FEATURES in v1.2+ supports a maximum RXTX_MAP size. This maximum size is not checked when page-aligning the RX/TX buffer size, and FFA_RXTX_MAP may fail when passed a size greater than the maximum. Decode the maximum buffer size returned from FFA_FEATURES and limit the buffer size based on this value if it is non-zero (zero indicates no maximum). Include verification that the max returned by the SPMC is larger than the minimum, otherwise use the minimum. While there, also update RXTX_MAP_MIN_BUFSZ() to use FIELD_GET() for consistency. Fixes: 83210251fd70 ("firmware: arm_ffa: Use the correct buffer size during= RXTX_MAP") Assisted-by: Claude:claude-opus-4-8 Signed-off-by: Seth Forshee --- drivers/firmware/arm_ffa/driver.c | 29 +++++++++++++++++++++++++++-- 1 file changed, 27 insertions(+), 2 deletions(-) diff --git a/drivers/firmware/arm_ffa/driver.c b/drivers/firmware/arm_ffa/d= river.c index b9f17fda7243..dc45724a29ba 100644 --- a/drivers/firmware/arm_ffa/driver.c +++ b/drivers/firmware/arm_ffa/driver.c @@ -32,6 +32,7 @@ #include #include #include +#include #include #include #include @@ -55,7 +56,9 @@ (FIELD_PREP(SENDER_ID_MASK, (s)) | FIELD_PREP(RECEIVER_ID_MASK, (r))) =20 #define RXTX_MAP_MIN_BUFSZ_MASK GENMASK(1, 0) -#define RXTX_MAP_MIN_BUFSZ(x) ((x) & RXTX_MAP_MIN_BUFSZ_MASK) +#define RXTX_MAP_MAX_BUFSZ_MASK GENMASK(31, 16) +#define RXTX_MAP_MIN_BUFSZ(x) (FIELD_GET(RXTX_MAP_MIN_BUFSZ_MASK, (x))) +#define RXTX_MAP_MAX_BUFSZ(x) (FIELD_GET(RXTX_MAP_MAX_BUFSZ_MASK, (x))) =20 #define FFA_MAX_NOTIFICATIONS 64 =20 @@ -2086,11 +2089,13 @@ static void ffa_notifications_setup(void) ffa_notifications_cleanup(); } =20 +#define FFA_SUPPORTS_RXTX_MAX_BUFSZ(version) ((version) > FFA_VERSION_1_1) + static int __init ffa_init(void) { int ret; u32 buf_sz; - size_t rxtx_bufsz =3D SZ_4K; + size_t rxtx_bufsz =3D SZ_4K, rxtx_max_bufsz =3D 0; =20 ret =3D ffa_transport_init(&invoke_ffa_fn); if (ret) @@ -2118,9 +2123,29 @@ static int __init ffa_init(void) rxtx_bufsz =3D SZ_16K; else rxtx_bufsz =3D SZ_4K; + + if (FFA_SUPPORTS_RXTX_MAX_BUFSZ(drv_info->version)) { + rxtx_max_bufsz =3D (size_t)RXTX_MAP_MAX_BUFSZ(buf_sz) * SZ_4K; + if (rxtx_max_bufsz !=3D 0 && rxtx_max_bufsz < rxtx_bufsz) { + /* + * Per spec the maximum must be >=3D the minimum, or + * else zero if there is no size limit. If the SPMC + * violates this constraint, use the minimum as the + * effective maximum. + */ + rxtx_max_bufsz =3D rxtx_bufsz; + } + } } =20 + /* + * alloc_pages_exact() allocates full pages. Use the full allocated + * space up to the max supported by the SPMC. + */ rxtx_bufsz =3D PAGE_ALIGN(rxtx_bufsz); + if (rxtx_max_bufsz) + rxtx_bufsz =3D min(rxtx_bufsz, rxtx_max_bufsz); + drv_info->rxtx_bufsz =3D rxtx_bufsz; drv_info->rx_buffer =3D alloc_pages_exact(rxtx_bufsz, GFP_KERNEL); if (!drv_info->rx_buffer) { --=20 2.43.0 From nobody Mon Jun 8 05:26:39 2026 Received: from BYAPR05CU005.outbound.protection.outlook.com (mail-westusazon11010049.outbound.protection.outlook.com [52.101.85.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 01878396D29 for ; Mon, 1 Jun 2026 20:45:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.85.49 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780346730; cv=fail; b=OFXXAhRXcGq07E8HK0TcIzUeMfg0Bt+rZ5FJJXIjBs0Da7SLf5VeXDAqJv4zq1P4mNfNUmzQWLcSlNSZz2ZgA4QMB4l8kDIC52joU1THjfL7A/f7gqH966Y3gNMlf3Psag5sYpkcNbM7JYmB2ncLVBBZl7btNYTo9v2RulLV+Cw= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780346730; c=relaxed/simple; bh=cpr7uYFtGIoIDEb0dcuRUHyp+u/EmqCawWEp0ZeW8ig=; h=From:Date:Subject:Content-Type:Message-Id:References:In-Reply-To: To:Cc:MIME-Version; b=A/1LbckEyz0uf/mIhJoqPmTWcXfclJlRKedjcbIcG4WjgLK69Sowbg5qAeGpP/5MXQog7XQ6oOP91kMFl84d472b8DpHn8wQcrmO/V50F9NxelJfNmv/i20A1e6s/T9Qnm1Bm/3sHu0N9xfzmS3cmih5fufhk2AOTE2EqddZh2E= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=nvidia.com; spf=fail smtp.mailfrom=nvidia.com; dkim=pass (2048-bit key) header.d=Nvidia.com header.i=@Nvidia.com header.b=e71ykhjD; arc=fail smtp.client-ip=52.101.85.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=nvidia.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=nvidia.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=Nvidia.com header.i=@Nvidia.com header.b="e71ykhjD" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=EqeNcwmuHBZNZaKMlinv7MMq2Ahm1oMitL2ijXzDlPqmFB5utfEUZ5EHOskDP/j5iM+PJ609+VgzaunbYZG1UhhltnkrdwTkcgJN/MfGJFsxeJZXsLiB64q84T13bewjB/c3kNi7jzd6hNNEBVIC8wQHx6OKaf1D5yi26b4aPBAN/ATsHX4jGFwBeFY+jx6CjDeFXrsOi74qE1mGh6Axhpj+IPAPVG8ZwtA9sSypm/mbmeut/aRjHavtm5v1qy6DxtUIkXsmQETkQ8NbpLtLE4C+lJlnv3XURfkGkHu3kbmdBojU4dpnjTJ+9Ds/o09u34oul3MaAj5ioj1aSUXnIw== 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=TQE0aY5MhdEY8l2Bw2mamjLMhpmSGJlW4kqMUXxIi/U=; b=SSXXa8Yp2yPEfGElCHxJw9ZTJRkgY3026ef8461eLeElPZYFCx1vFTwuLTtOeXo9A3qNXUh4KwuRbSxuBAXtyQj7GYs3nha8xsVIHx43x6eGAXKSA2foWcLly92o4R/12bKY9VjnXm557Y3QbVTVPT9gfSHww507DSpKmBY86a37+Qps5HoOz4M4igEdFhZROIHmNkptsVWmeAbN+yz/6+9re4PHqx11g/LejhINH10S+4naOFa1tTbDEo1avXldEtMGQe8ibScAekHdbKce/x7DOtMks1pKR5P0bLH6991XHA30cvMjDuYOXZEmuLnXi6xMjPp5aufJe4XeHSh3nQ== 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=TQE0aY5MhdEY8l2Bw2mamjLMhpmSGJlW4kqMUXxIi/U=; b=e71ykhjD03s7tFqZxdZ6pMt1ew5cJAycgtb5aYb7cVn5DoumjT6axv4doHaBWb7wK6W/P8CM9s4ibLkp7TmQScGIzxg9c5d5uAB7On6mkFgBBNkX2SAEJIk7oYKB1H/YZFaSCU5FFkAbcJlaLEIXAkbc/o9g1bpMh6iKCN2XmNR1BNN5PLVN8gxGZObb97JtHmjqVFAbjTerNTYodUVfTp162YbjFQpx9H8IbVRFVfkVdNz9nUjyC2Ya/RbFauktIPSS9MFJ/cbCnoyN0Z3I0yMzBwNuCis66nVQw9M16726LbMwzLfQTiADfvJ4uxXdqrsZLzW3vaslPOwHNIt2Ag== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; Received: from DS0PR12MB8768.namprd12.prod.outlook.com (2603:10b6:8:14f::20) by SJ0PR12MB5674.namprd12.prod.outlook.com (2603:10b6:a03:42c::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.21.71.17; Mon, 1 Jun 2026 20:45:24 +0000 Received: from DS0PR12MB8768.namprd12.prod.outlook.com ([fe80::c16b:c351:2fc7:95e5]) by DS0PR12MB8768.namprd12.prod.outlook.com ([fe80::c16b:c351:2fc7:95e5%7]) with mapi id 15.21.0048.016; Mon, 1 Jun 2026 20:45:22 +0000 From: Seth Forshee Date: Mon, 01 Jun 2026 15:45:12 -0500 Subject: [PATCH 2/2] firmware: arm_ffa: Fall back to minimum buffer size if RXTX_MAP fails Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260601-b4-ffa-rxtx-map-fixes-v1-2-c071b12ae05c@nvidia.com> References: <20260601-b4-ffa-rxtx-map-fixes-v1-0-c071b12ae05c@nvidia.com> In-Reply-To: <20260601-b4-ffa-rxtx-map-fixes-v1-0-c071b12ae05c@nvidia.com> To: Sudeep Holla , Sebastian Ene Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Seth Forshee X-Mailer: b4 0.15.2 X-ClientProxiedBy: DS1PR06CA0014.namprd06.prod.outlook.com (2603:10b6:8:458::9) To DS0PR12MB8768.namprd12.prod.outlook.com (2603:10b6:8:14f::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS0PR12MB8768:EE_|SJ0PR12MB5674:EE_ X-MS-Office365-Filtering-Correlation-Id: 9f3a299b-6bcb-46af-a5dd-08dec01eb372 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|1800799024|366016|56012099006|5023799004|3023799007|18002099003|22082099003|11063799006|6133799003; X-Microsoft-Antispam-Message-Info: klSJI3lFHGweyeFQyTQ6DJCHp7iesmY64cDXrKKjWly2dE68VK4THHTqKe9qV1zB7QE7v7ZLNRl0hE9DZ+HU5s8S9CKnS/ExzwdxNQhkXvDksiElN7oZ37Jd8yNLSHXoo9Zn2y9xqUHYKcRrg4AOAZPixkOLniFUCguEIhvBud2Cw7CuHM4nfrn1zTTQsHY+1jpe1EghObOj9xy2JAORCJ06/RJnJ8F/forcPWmXE0XIKmImk+ZCrL1ux+2xFAiHLx/6v++EyfEsSIp3DbZL89gVy4MO1VL8Al9GQapQvMwq42Vw++ZIoJBBbBzcWXZq7+KM18wC6dbQbwRQ64YxplzX5EXZAxYWSOxKCtB2CuBJL2MGnmp4mubNtoF1ev2NKFDlgj+oVPcyH7kZOJ2ao1gVXvu+tGHfkHzT26pVyWXVsC4Rp4nzSoqzAjpyBU75RwRDws+uP4XIHMiGV4p+Z70n6W/X0z2BH8iVg269COjORvs2Ia3IbGIS3+z6G47uieisbu00dPO/f3S0uJz/E6ztZahhPT1biA/hSxANdXB7N3A7HIFD7bkVXkF2zKpcFJWaDAhGIyyzDqZgxuyYbu4MbWkyIMHNLcrWOpDeuXL+Doysh4Am5hXXC6vErCGHgVzJBW5feogUkpe6EVd6QWl38sBg1lRDRp/kVefuJWBSZ2Scrwg8INkIIJ3+WBdz X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DS0PR12MB8768.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(1800799024)(366016)(56012099006)(5023799004)(3023799007)(18002099003)(22082099003)(11063799006)(6133799003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Z3dhdm1wUFVmSElMMjZpU3pZdjZ6bWEra1M2ckhxOW9qang1aEFXLythM2V5?= =?utf-8?B?VHhGZWo1bzRIQ21YTXlQbGtqWlZrZXg2WnJHeFNqUEVjaEdEYzB5MnduK3dr?= =?utf-8?B?OHVaV2JXc2lCNW5KL0MwTHJkM2RSVG1oaXhscUt6eGRLNmVKelJYZDFtRitB?= =?utf-8?B?L1B1M29TZ0RjQWxWN0JaSXNEdTBvSm5DYXVqeWxKNTVEdTlmaHdUYzVCWUh0?= =?utf-8?B?WXpGbUplcG92RWJxTmtWSGRnbUJUMitqVERqMWZHRVZoVFhITERkTHlERWtD?= =?utf-8?B?Q1VXL01haldNUVFhUlJZWEZpQlFRV2J0Zml1QlpjN1A3MXFWQllaRysvOTJF?= =?utf-8?B?S1R4dkpaNVFOK3JlMUQxNDlQVG52S0VqeFRMank2WVh6ODZoVlZtcWdENXlB?= =?utf-8?B?N1cza0RaektpU2FmRUFFajdDSDVqWWxVNzNiRTd0QWcvUDNZcHp4Yk5sN0VS?= =?utf-8?B?Ry9VSFc0Ylpwc3VkK1FCVUhrMkUvckFGZFJ2dGQrRFYwR05UalBJdnFJdUtS?= =?utf-8?B?cGYxYUVIbUwwRytxZk9JZ3BiYlhVdERqTDlwM015emRaRkViMW9CcFovdmRY?= =?utf-8?B?K2w2MTJPRHJZU1ppTitrZWN5ZTZPeUZ3SzJ0R3hZa1dHT1pOQzA1SjNNME05?= =?utf-8?B?dWRmd0lRdldCa1RQK3BZQ2ZGWGpOSmRNT0trcEJESHd4VTVhMS9WT3FQY1dH?= =?utf-8?B?THczOG9kbFpNVjI4ZVBjSlU4NCttR3gvR1FlTWQ0aG1BWE5Yb1ZIdm43bVFL?= =?utf-8?B?UzJneHVCVEk4MUlCRDIyNG9jL3htTy9ybncxemFYaGRhRDFZWEhTbCsyN1Ex?= =?utf-8?B?bE0zdjJzTWo2YVhiT2szbzAzdW5LZUZ6Q1E1RWVuWVVHckJ0UXNqMDhVV3lZ?= =?utf-8?B?aEVQc1dtc09rZlBrY25xVEhBckRFYTdLRklNeFdubjdpa0FJOFZaSjlPVURF?= =?utf-8?B?U1A2elFDSXMyZEQ2WkZwTXg1c2ErbGdTYXV1QXlhSkRYY2VZbjZwYWFHNFBx?= =?utf-8?B?RGUrTk1xcThNelM0cmNqL3p3andVa0twZHhPeGhPdHI2dWRzNzNSUG1iOG5T?= =?utf-8?B?ODhqbXhEZDl5cGY0U3BRaE9iNklmRFpHNG9ycGNkWU5vVWd0Lzdza2tpWWZ4?= =?utf-8?B?TzgwcDJVVFJXNnp1dms5cHQxMTIvd3VwZ3IxVjkzYS9NVTIyTVZaVW9GQWVW?= =?utf-8?B?bGdOeFgxaG5acTRXTUhqS3ExRGp2a0wvSm5jMEJKVXgySXNveTNQbGxUMW9Y?= =?utf-8?B?VFc3Mk55VG0ybmdpVHJKY214Qjkyb3YyZ3UwU0FOV3orL0lJWlFZbzA1L3lL?= =?utf-8?B?N1dCRGpoYkUyZHNjbjEvV0dteG01b3BWTU1tUE9ZcEZDSEZCaEg5YVhyaXJn?= =?utf-8?B?RTlCS1k0UDN3RDVva2VGWlZORzZ3NVN2V3hHajBJbWlBY0gvSE1IcUhqeEpj?= =?utf-8?B?MjRMSzZ6dEpKYllQZkN0ZjQ2Rys5dWlTaEg3TGN1dGdBRmQ1NHoxRHhXbUNW?= =?utf-8?B?MUtVQk1mak9EenlyQUpuOVUrN3JtQ0JDMjlEc29TekNTOXhnRytla1ZtMkI2?= =?utf-8?B?b1dmRmNrQVBEMDZNam5DME1XQUhrTFhwU2w4Ri9hUmp2ZXZSb1lsM21kUWRC?= =?utf-8?B?WHFmMjN6bmZTcG0wcmYwRHpOS2MwYjVodGdteitDb1ZUSk9WcHgreHcxcWhU?= =?utf-8?B?QlJYU3FZb1d0ejc1Z2tDMEhjNVU5cllKMnpXN2V6akIyZUw1eXNXWmFSODJP?= =?utf-8?B?ek1heFF2VDJtcnQveHNNTkNuOFErOXVSd3JBMnB1N1FpZ3JTTGliMmJsdndZ?= =?utf-8?B?RktPcVp3Y3FLNXRpbmVkNzZ0eXdtY2hhTFM3UTBOMVBtdUZkcFVqVXlxdmsz?= =?utf-8?B?cGNaZkI0dStSVTVGVFRZbkp3SGhvWm5nYkc5Rm1kRDFjNzNVNEdNQ1dMbGJ3?= =?utf-8?B?Y0M0ekZjc1daWFp4WGMydWdKQ1ZuYm5PemxKWWQrdWtpTk5xdklIME5sTFhu?= =?utf-8?B?YXhWTFdwang0SEFLbVFlQ1dIazkyNURDdm5MOExpZFd4aE9VZmJYd3JkTjdz?= =?utf-8?B?SGdNNE0yek5sQlQzbG5wL090QkhFQVl0WjlRc0UzT0cybWFvVEMyVjFVek9W?= =?utf-8?B?SGgvSTNqemFHVHd4R1pjekxLWjhETG4wNDFDalNaRHR6eXljUjRHaDVnYWdk?= =?utf-8?B?YTFMMGNLZVNidW1pOHI0V3h2R2YySll5NGwzdmQrcHhWL2UvWXppRGZIUjIr?= =?utf-8?B?YWpaY2diN0o0K3REUUVlR3pWVjdRcHN0ZmNDTmU1MmxHN0t3dUU2WTJublJy?= =?utf-8?B?MWdETHRMdDZhdGw5eWU4SHh2YWhRTTd4QmpOcjJDTjJlTk15NW9qUT09?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9f3a299b-6bcb-46af-a5dd-08dec01eb372 X-MS-Exchange-CrossTenant-AuthSource: DS0PR12MB8768.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Jun 2026 20:45:22.6782 (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: vQXM4ktiflwEoC/lBKltkdndPSdNSbL2WeX1giZm9AOMDeaD/9k/JLm1O+JHNVl8+O5T7+Jf30IAZwgxM3CFSA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR12MB5674 FFA_FEATURES only supports a maximum size value for RXTX_MAP in v1.2+. Older implementations may still reject larger sizes, which can happen when the minimum size is rounded up to the page size when PAGE_SIZE > 4K. Fall back to the minimum size when RXTX_MAP fails with INVALID_PARAMETERS to fix the regression on these platforms. This also has the side effect of falling back for v1.2+ with a maximum of 0 (i.e. no maximum), where the rounded-up value isn't expected to be rejected, but there's no harm in retrying with a smaller size and it might even be of benefit for a non-compliant SPMC. Note that this may result in passing a size smaller than the allocated size to free_pages_exact(). This does not result in any leaked memory, since the size extends into all allocated pages, and free_pages_exact() correctly handles sizes which are not page aligned. Fixes: 83210251fd70 ("firmware: arm_ffa: Use the correct buffer size during= RXTX_MAP") Assisted-by: Claude:claude-opus-4-8 Signed-off-by: Seth Forshee --- drivers/firmware/arm_ffa/driver.c | 28 +++++++++++++++++++++------- 1 file changed, 21 insertions(+), 7 deletions(-) diff --git a/drivers/firmware/arm_ffa/driver.c b/drivers/firmware/arm_ffa/d= river.c index dc45724a29ba..60ad58e229ce 100644 --- a/drivers/firmware/arm_ffa/driver.c +++ b/drivers/firmware/arm_ffa/driver.c @@ -2095,7 +2095,7 @@ static int __init ffa_init(void) { int ret; u32 buf_sz; - size_t rxtx_bufsz =3D SZ_4K, rxtx_max_bufsz =3D 0; + size_t rxtx_min_bufsz =3D SZ_4K, rxtx_max_bufsz =3D 0, rxtx_bufsz; =20 ret =3D ffa_transport_init(&invoke_ffa_fn); if (ret) @@ -2118,22 +2118,22 @@ static int __init ffa_init(void) ret =3D ffa_features(FFA_FN_NATIVE(RXTX_MAP), 0, &buf_sz, NULL); if (!ret) { if (RXTX_MAP_MIN_BUFSZ(buf_sz) =3D=3D 1) - rxtx_bufsz =3D SZ_64K; + rxtx_min_bufsz =3D SZ_64K; else if (RXTX_MAP_MIN_BUFSZ(buf_sz) =3D=3D 2) - rxtx_bufsz =3D SZ_16K; + rxtx_min_bufsz =3D SZ_16K; else - rxtx_bufsz =3D SZ_4K; + rxtx_min_bufsz =3D SZ_4K; =20 if (FFA_SUPPORTS_RXTX_MAX_BUFSZ(drv_info->version)) { rxtx_max_bufsz =3D (size_t)RXTX_MAP_MAX_BUFSZ(buf_sz) * SZ_4K; - if (rxtx_max_bufsz !=3D 0 && rxtx_max_bufsz < rxtx_bufsz) { + if (rxtx_max_bufsz !=3D 0 && rxtx_max_bufsz < rxtx_min_bufsz) { /* * Per spec the maximum must be >=3D the minimum, or * else zero if there is no size limit. If the SPMC * violates this constraint, use the minimum as the * effective maximum. */ - rxtx_max_bufsz =3D rxtx_bufsz; + rxtx_max_bufsz =3D rxtx_min_bufsz; } } } @@ -2142,7 +2142,7 @@ static int __init ffa_init(void) * alloc_pages_exact() allocates full pages. Use the full allocated * space up to the max supported by the SPMC. */ - rxtx_bufsz =3D PAGE_ALIGN(rxtx_bufsz); + rxtx_bufsz =3D PAGE_ALIGN(rxtx_min_bufsz); if (rxtx_max_bufsz) rxtx_bufsz =3D min(rxtx_bufsz, rxtx_max_bufsz); =20 @@ -2162,6 +2162,20 @@ static int __init ffa_init(void) ret =3D ffa_rxtx_map(virt_to_phys(drv_info->tx_buffer), virt_to_phys(drv_info->rx_buffer), rxtx_bufsz / FFA_PAGE_SIZE); + if (ret =3D=3D -EINVAL && rxtx_max_bufsz =3D=3D 0 && rxtx_min_bufsz < rxt= x_bufsz) { + /* + * FFA_FEATURES only supports maximum buffer size in v1.2+, + * and some implementations without it may fail when the + * buffer size is rounded up to a larger page size. If + * RXTX_MAP fails due to invalid parameters, try again with + * the minimum buffer size. + */ + rxtx_bufsz =3D rxtx_min_bufsz; + drv_info->rxtx_bufsz =3D rxtx_bufsz; + ret =3D ffa_rxtx_map(virt_to_phys(drv_info->tx_buffer), + virt_to_phys(drv_info->rx_buffer), + rxtx_bufsz / FFA_PAGE_SIZE); + } if (ret) { pr_err("failed to register FFA RxTx buffers\n"); goto free_pages; --=20 2.43.0