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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 0BC58EFCE53 for ; Thu, 5 Mar 2026 01:37:31 +0000 (UTC) Received: from kara.freedesktop.org (unknown [131.252.210.166]) by gabe.freedesktop.org (Postfix) with ESMTPS id D822710EAE1; Thu, 5 Mar 2026 01:37:30 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=Nvidia.com header.i=@Nvidia.com header.b="MYxYjIWY"; dkim-atps=neutral Received: from kara.freedesktop.org (localhost [127.0.0.1]) by kara.freedesktop.org (Postfix) with ESMTP id A5A9C44E78; Thu, 5 Mar 2026 01:27:09 +0000 (UTC) ARC-Seal: i=2; cv=pass; a=rsa-sha256; d=lists.freedesktop.org; s=20240201; t=1772674029; b=I/syQuXKf+6GhkL5XzsncFOYRPQo9a3e4C09y7WIjcGijJmbJu9hYu9aO4u8X9Ymg6DBz hpdBE9ilXJIs2MKniKujFjAN2Mgj8W1WCABicPKljLKp3zzzrbPmyl1IEhh2LbxENEITc0t ua21TosoB8HfYuvneb+g15O64zb9+Qa0pwQaTWr0vioBfjsxxTdYExfiGMcjFGriwEb5RQ/ DOLQmLsDE9DR2i6n1R2P0cUr1S3qhnsZCazbZSM+xOFuLM/elgjc3lwuS4865M0qiYRzYwS EHAxxnvHkhoGR6mhW/kJdUE0deJMhzbO39O/BkXIhJHp/Q2niPjMD3jTzMPg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=lists.freedesktop.org; s=20240201; t=1772674029; h=from : sender : reply-to : subject : date : message-id : to : cc : mime-version : content-type : content-transfer-encoding : content-id : content-description : resent-date : resent-from : resent-sender : resent-to : resent-cc : resent-message-id : in-reply-to : references : list-id : list-help : list-unsubscribe : list-subscribe : list-post : list-owner : list-archive; bh=bIS/M2jtUr4IQoW/e/A5Qor0AIasFstl8PHPqf4eBFY=; b=sNKdaSySpbk5EbDq2cNfY5PCEWNeB5gZN0T0Rio8eYwmJpiAJLO6gRroDsmBDyO27R8iQ 4MDoAD49PXC29G3Ol8MFt4Ita7jFUc2I3NcmOKLwQOLrzAJJFpUZHb1l/bR4uZBZOAYF9Kz GYFArALAuqW2k3jlsC39UwJskXj0KXIpv2i56CxLwPlHX2rhmwq63xW/6oFIbZOOaIcSiCm exVe9ArGThWX7zhExzUm6AxHKbWbBFfC3wtHR99b36iAvJ/VLE0Uu4CP1+YtaI8Ofwj8KwL VPSX+FkViPNY9mIaHdG3li7jaFBks6+HFi5rnuXoAcwOjTSKuZD7rKUUgwgw== ARC-Authentication-Results: i=2; mail.freedesktop.org; dkim=pass header.d=Nvidia.com; arc=pass; dmarc=pass (Used From Domain Record) header.from=nvidia.com policy.dmarc=reject Authentication-Results: mail.freedesktop.org; dkim=pass header.d=Nvidia.com; arc=pass; dmarc=pass (Used From Domain Record) header.from=nvidia.com policy.dmarc=reject Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by kara.freedesktop.org (Postfix) with ESMTPS id CF8B9402B7 for ; Thu, 5 Mar 2026 01:27:06 +0000 (UTC) Received: from CO1PR03CU002.outbound.protection.outlook.com (mail-westus2azon11010020.outbound.protection.outlook.com [52.101.46.20]) by gabe.freedesktop.org (Postfix) with ESMTPS id D4F3810E1D4; Thu, 5 Mar 2026 01:37:27 +0000 (UTC) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=VefaBfmto5LPeEqcfRa4V5vnbdaXpVOpImgQhafT+CUlLm3PXreiYh66Cb2mZpPXFoGLmADR2p+vEgnBQkIIPqehY4miA4cUWpTgOzszDDwFP1AbcGVXqXXtsK3+bCZ55jCyLMzUBxmwUYESASETrXe8+4pf5U2rAbkw09+VQ2Bqnh1iOVnlGZp6vfAGJ2tn0nWFXGJB0J+IPqdBCArLEdkVlzVI5CZlrIzGhdiW+obseVm9WDQCM6kKyoXJTeXXhPZD9Tu6DPGjuborqmSoeXzxb0AdgUDk/kpcKCYsli2yY2Fgz4UahoNKdl6NqW4Ma+xWbVC29aBFFyNazw9arA== 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=bIS/M2jtUr4IQoW/e/A5Qor0AIasFstl8PHPqf4eBFY=; b=A1+zb4+jS57J8L16Zf8kl79Bxuu4FwHwvM8tqHTPd87vPbQr7+myopOWtvIeFfKeffHklq4DIx5XOYS3j0SWQXeqQm8PtOKn7DhuvdBW82QuacMY+UmE7WMKz7DU2Eo1zC2z8L5Bvvvi2FeHmOktO745WeDSfwqlTBM2XGRdFhmJV+rXcWCcaBELRYqx7X7wDV3waCLvL5S5p34THUUXSQXt/Zn4D7ahZppPIBCJDdACFjR5bUr8zCpDSjqTMJO395cca2f3rfqG04uvIqi0Fe0EcP480v41Wm8MrfgVw7/Yj8fmRdbeJzLu3Nr7ptY2s1ww8oEeDvwm7jVrNtt0/A== 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=bIS/M2jtUr4IQoW/e/A5Qor0AIasFstl8PHPqf4eBFY=; b=MYxYjIWYlh+5SE689dVbUI96qsbhIv0CscMpbrZYs3IuekOEQgnwPYFPMib8XKUid1mW6/bkuf56qZ6oBewmKHQwfevH73K6cSGgTQ/0tGm4AsViZ7axVfszaTkYDLB0/x1/dK0cA1woGFqM2JCEHcQjk1hClhUaSpZIyFqZbu4Fc85Ila9OgdkHdtP41e0mSsy6OkVPIret060yhG/Dwn35r7FgQq+1pO/naWxjUTlXaxaIAnIBjfk0WONX+21qWjnuWbZ8NO8BjFh7hQIRG9DV6aXeM7HkenQp91IHUc0qCWN8AkEui2BViyg+wqpJfhCnMGoGkVJgAsd/CnkhtQ== Received: from CH2PR12MB3990.namprd12.prod.outlook.com (2603:10b6:610:28::18) by CH0PR12MB8550.namprd12.prod.outlook.com (2603:10b6:610:192::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9678.17; Thu, 5 Mar 2026 01:37:22 +0000 Received: from CH2PR12MB3990.namprd12.prod.outlook.com ([fe80::7de1:4fe5:8ead:5989]) by CH2PR12MB3990.namprd12.prod.outlook.com ([fe80::7de1:4fe5:8ead:5989%6]) with mapi id 15.20.9678.016; Thu, 5 Mar 2026 01:37:22 +0000 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Thu, 05 Mar 2026 10:37:19 +0900 Message-Id: Subject: Re: [PATCH v3 3/5] gpu: nova-core: gsp: add reply/no-reply info to `CommandToGsp` From: "Alexandre Courbot" To: "Eliot Courtney" References: <20260304-cmdq-locking-v3-0-a6314b708850@nvidia.com> <20260304-cmdq-locking-v3-3-a6314b708850@nvidia.com> In-Reply-To: X-ClientProxiedBy: TYCP301CA0012.JPNP301.PROD.OUTLOOK.COM (2603:1096:400:386::10) To CH2PR12MB3990.namprd12.prod.outlook.com (2603:10b6:610:28::18) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB3990:EE_|CH0PR12MB8550:EE_ X-MS-Office365-Filtering-Correlation-Id: 4ad7f3a5-b3e2-4a92-1c66-08de7a57bf5d X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|7416014|376014|1800799024|10070799003; X-Microsoft-Antispam-Message-Info: UlhHPoiRWYWgeTvaj1DmYbBKO5v38QY+eadeCyTn8klUJp3rr7L2CvXBNlwyEw0SJC6oEpRZDkk0yVeJsL4Wsx/eCIQoidkzxAisS+szwAYIfBFlzKWD1rd74QGcMl8Oq0C+2SkYDVDxPaYC3SfPMjRpEgAjB78LWkf7i7DDLQXNmQ2+FNjDwsuyN9991LC+VWpllwkivKnl7nNfbxLQjf8rKuXFnrHzA9OqwynZ+lfMEnhM6WToFR+WFPVMQuOW7v5FzJUzxlWVdFS2+xSgyGPfd/G1eQMFtcDTZpP5LVJJ8frv/nCjes/KjHC6ALG3zCPm4b9h38pn4ntSjhorECuiLePjb3P2Uf4mX+Gou08N3eb1yTbgth0o3rJQcSsTK2gty6z8J/fLx48wSujMk0WDjGZcx5OQ98zPJITgNUirbbPaT7TuWuIEYu6e2tYvn2YT+oqJ5A4OJ7XLbKXE6dZJh+B1GaQHgwKADoYEHj03xKz63fRiEOGQ+fWaWVAzlemLleeI/sQw1dyj/iWmt5TKuu2acIAAHYNOlm1DG6jgEWj0XFK6Yzxz66PGgyuimmK5jrMKkoqbd3ltC8w9m1QubpFM01QI/kBwiZyYafOMsSJer92HkNSBx4vDop6hAC2rc217MdYfQCRZmkuMUCJsUP73eLFA9S02BtK8cX+LSzHaKIWpelydE2kg/Q1czjh+zGyawa82D0+LPDI7ECjj/NO33PQBK3C6O46WUG0= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:CH2PR12MB3990.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(7416014)(376014)(1800799024)(10070799003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 2 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?OUFSREJLeEJNWE5OcEtObDNtbm9QT3pQeXl6VjAyRDVGMWVDM2FpdjRieEVG?= =?utf-8?B?RStaaGVtY3ErZThkYWhTVmJCSHRqNlVtcU5jeTFlVy9FbnFJbzZNelVWNmlP?= =?utf-8?B?dW80VlJlRm9zejgrQ1huK3NNZnRqSjMwZ0tyRWo2SDZSK2VDajdtZm9IQzZG?= =?utf-8?B?RGxVeUxyVVlEVHBaTFB5dkxWZGJKUjhPUDEwTUVKZVlKQ091MHJvMk0xeFNR?= =?utf-8?B?MCtDRDB6MXNUOUgwU1BpZmo2dDYyQm9kdTJuV0RhV3pDV3Y4aUgyajVhRjJH?= =?utf-8?B?R3RsMkpXSmMwUmVNV3ROeVM5OWRtOHk0NmhmSDVVV1ZHRytQR05QSC9xaUZO?= =?utf-8?B?NEVKU1UwTjZuQjRBYjR6MDNiUDF3WURPQlA4RHNEUC9jSHpjemVwd1o5WmFq?= =?utf-8?B?MFVITVAyUkdPM2J0WkwxNE93RExFN0FhVWE4bmNIQmlpcUJMdElXcUxVM09H?= =?utf-8?B?KzBNV1Y3c0I0alQ1UEo0ajN0bVUydnZrWWVTV1o4NmRjdjJBNEZoQWhHWEFE?= =?utf-8?B?U2VrYWR1S3E4b3NCNTZHbWt2OG0zc1laWXZCUk1mMUdLTS96dmVzTUdNZGF0?= =?utf-8?B?U3pQN1p3eFNGV2MrWElFWXJCWmpZVkdBaVpIaXRnbysrM002M3BFWWJxNllH?= =?utf-8?B?TlFHcWZIS3dia3BSWUhLbmZWQjUvUHhFUVh1NjI3NVFqWVhvQ1VVUWFibVB0?= =?utf-8?B?VTVKQzg3TFEyUEE1WVBBNnV3OXVJdndEVFJ0UFBBN0JVNldPaHpRT2xvNTdu?= =?utf-8?B?SjV3TkFLcElyWXdKTDgxUFF1NE9HZzkyVlV2c2VGSUtJdzVqS05DWlM1Tkpw?= =?utf-8?B?RVA3WXhka3ExejNSRFRObUFIZzhCb3VrZGFtMUZIWTF0ZEgyaFc1SnRKV28w?= =?utf-8?B?RlNOeG1ETUtNVTZPQjZxa0JYUzZJZi81MFlySHZjNDBUSTNWWWgyVUVUb1Y5?= =?utf-8?B?T05wbmcyQS9NVS9vMU9wWGk1SWF2ZE9UWkJqbUY1OHdCQUFvZ3ZtekVTelEv?= =?utf-8?B?WlprNTJqMW51YTdmSmVZcG9pWS94UGFyLzYzRFFHekJ4RW4yRkEvM09tZUhZ?= =?utf-8?B?VHFVczBNY0N4eTVWRzhXT1NmZENjZEltV0NCWHVVYjRUOFFxZmVVS1llVW0w?= =?utf-8?B?VXJZN0dtclBjNmtaSmpqTThXM1VvVXY0RmFaRE9JS2U4UkhpNGM0NEFhTStv?= =?utf-8?B?aTdSQmJ3UDVFVVAwZEVzNkxmV20ydFg5TVkxNDBZUUVZNUdBV3Ayc2tiNk9m?= =?utf-8?B?RFlqbitJcjUzdkJUUVZOZzd1TlRXT0hsL1FBdUE0ektoUEcvcXJFSUJUSTJT?= =?utf-8?B?a2tDMlc1c3JodUo0S3VMbjdkT1VBcC9kNFkwVytKNVQxamcvOTRoNTJrOU5i?= =?utf-8?B?eXNBTjg2VWVOV1lvcTJMaUs1T0VQMGJremloeW5DYnVlNXIweFR3K1QvclIx?= =?utf-8?B?ODljcFYzY2lKUWs1RDVBR3B0R0NnZlZRYkkvaFNhNk9PNFFBU1lEcUxiSEpm?= =?utf-8?B?KzEyc2tNRzlBbGtldzV4UzF1ZnBncCtkTkJFQzYybGY1emJSK2xzdGdNUjF3?= =?utf-8?B?SVZOWjZndHdIN0xpeGhZNGlPOWFORFBka2lWaEtlS3NUdEp2bmFHSFVQR05V?= =?utf-8?B?emloWDIzaTEwWHJNZlFieHlzNlRCdVB1b3U2bGtJYUlVdUtFY3JoNk5nakYx?= =?utf-8?B?dXZFU3dvMDlKeEo1Tjg3Y2o4U3dqSVRGcytmbmdZMDIxL2hOallpaHJpMnI2?= =?utf-8?B?RncrbzV1YjRFWE42dnNsZng4NHNDS29Db3k3dzBUcjlxZGJYSjhDeDhTcXN0?= =?utf-8?B?ZVRTUU5ZWjhXQzJpVlVFU2VMTGdvcjJWdXlDbkhsZHlLakM5dW5VQzQ5NlZl?= =?utf-8?B?S05WejlNam1VTFc0dUtuSmhBS2swbWJIVVlCM0ZQc2xlSkhNV0VJRHZJNHE2?= =?utf-8?B?MzdxS28yY3JDcTFWdUk5M05Eb3dqMWlGZURIcWFQSDc2UG51Z0FLU1h1M3JS?= =?utf-8?B?d1ZvNnhSMDJvZzdxS2R0M28xWjhsZWZGSnBZRHh6ckNJT2J5cTBhZi8yVG54?= =?utf-8?B?M1UxTGNmVUMwQkt3RUVrb2xaMWF4SnR3VDhvVzFiWUlVbTlLTjhXUXhzUDNH?= =?utf-8?B?UXZ4ZDNjbEFpME1vTDJBN0RCSXZGbkY2YTZVazNveGRzUld6eThkOXBWTWVj?= =?utf-8?B?WWxpNkFqRWRFanVDUGZvcU9vZE9pOEVMRTZrcDdaK0tNTXVUQksxZi8vM2dq?= =?utf-8?B?Wk8ycU1ITTVaWnNuZ0RoSlRPeG51ZHlWMExTTlg3REJjUWIrVFVsQ0RQazJH?= =?utf-8?B?RThNQW5aeTEyUWdtZzZpU0pWRk9GVERUazY4Q1dKcStLTldKcWtkUWV3UDdH?= =?utf-8?Q?UehYO+N/rOtCEpCyX8wOLrwYELlFliacZCnGW2Eav5QfE?= X-MS-Exchange-AntiSpam-MessageData-1: U2CyAKigHl2zDg== X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4ad7f3a5-b3e2-4a92-1c66-08de7a57bf5d X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB3990.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Mar 2026 01:37:22.6435 (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: DwhM8l/umzffHUDDOv/Lqxjh1Jue4MlqNYFZeR3jTsg5guybdd8i+0xxnGBNwgBEoy9DIhokQwmaUGgk+r7/WA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR12MB8550 Message-ID-Hash: 6FESAZJL5GQTTH46QINJW2WFYN36OLHL X-Message-ID-Hash: 6FESAZJL5GQTTH46QINJW2WFYN36OLHL X-MailFrom: acourbot@nvidia.com X-Mailman-Rule-Hits: nonmember-moderation X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation CC: Danilo Krummrich , Alice Ryhl , Simona Vetter , Benno Lossin , Gary Guo , Alistair Popple , Joel Fernandes , nouveau@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, rust-for-linux@vger.kernel.org, Zhi Wang , dri-devel X-Mailman-Version: 3.3.8 Precedence: list List-Id: Nouveau development list Archived-At: Archived-At: List-Archive: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: On Thu Mar 5, 2026 at 10:29 AM JST, Eliot Courtney wrote: > On Wed Mar 4, 2026 at 11:17 PM JST, Alexandre Courbot wrote: >> On Wed Mar 4, 2026 at 11:46 AM JST, Eliot Courtney wrote: >> >>> /// Send the [`GetGspInfo`] command and awaits for its reply. >>> pub(crate) fn get_gsp_info(cmdq: &mut Cmdq, bar: &Bar0) -> Result { >>> - cmdq.send_command(bar, GetGspStaticInfo)?; >>> - >>> - loop { >>> - match cmdq.receive_msg::(Cmdq::RECEIVE_= TIMEOUT) { >>> - Ok(info) =3D> return Ok(info), >>> - Err(ERANGE) =3D> continue, >>> - Err(e) =3D> return Err(e), >>> - } >>> - } >>> + cmdq.send_command(bar, GetGspStaticInfo) >> >> Also noticed something cool while rebasing the unload series on top of >> this one: all these command helpers become one-liners and are not really >> needed anymore! Here we can just make `GetGspStaticInfo` public and call >> `cmdq.send_command` on it directly in `boot`. It removes a layer of >> black magic and makes it easier to understand what is going on. > > I think this is ok, but we will still need some helpers (e.g. for RM > control RPCs) that mimic the structure we have here. But there are a > bunch of simple RPCs like this that can just be called without the > helper. Do you think it's better to do this and have mixed helper vs > direct call, or just go all helpers? Mmm ok, let's keep the current helpers and reconsider once we have RM controls and more visibility. From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from CH5PR02CU005.outbound.protection.outlook.com (mail-northcentralusazon11012019.outbound.protection.outlook.com [40.107.200.19]) (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 E7D091DE3DC; Thu, 5 Mar 2026 01:37:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.200.19 ARC-Seal:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772674649; cv=fail; b=oLJZZUDqNibtDkX3ZGMytkem10jSgE3tpnINPjvquoe7JWqkMtjBqYxkD1/QORdnD7aFt3IcvnuA30aMk1q14wxYQydXkcxl8tri+HhbNIP0Uq/GOGZ3hOx1NePNtTrXwENRLIi75MQoyRp7cE9jn8er1wFRH7fXDu/lsCLPH5M= ARC-Message-Signature:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772674649; c=relaxed/simple; bh=kwS1KaNCRJySSHNvrdWRQdh8sqT7spAyIfwRSdGe3N8=; h=Content-Type:Date:Message-Id:Subject:From:To:Cc:References: In-Reply-To:MIME-Version; b=tQWv94p0w7IIqN9ulloVa/VN3Bn52LubFAxQE8Lqp9SyYwMj+YQfFNqP0D+7mR9yOt2fgYpYiaaWyVMmXpi3cNiucykHQ0okvdkkDmXCsMnOIeM2Jbn7hgUK74MB/yY3t/xlglSvSG3ePIPAqrH4wG318I9caAIApnUEpBKltzc= 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=MYxYjIWY; arc=fail smtp.client-ip=40.107.200.19 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="MYxYjIWY" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=VefaBfmto5LPeEqcfRa4V5vnbdaXpVOpImgQhafT+CUlLm3PXreiYh66Cb2mZpPXFoGLmADR2p+vEgnBQkIIPqehY4miA4cUWpTgOzszDDwFP1AbcGVXqXXtsK3+bCZ55jCyLMzUBxmwUYESASETrXe8+4pf5U2rAbkw09+VQ2Bqnh1iOVnlGZp6vfAGJ2tn0nWFXGJB0J+IPqdBCArLEdkVlzVI5CZlrIzGhdiW+obseVm9WDQCM6kKyoXJTeXXhPZD9Tu6DPGjuborqmSoeXzxb0AdgUDk/kpcKCYsli2yY2Fgz4UahoNKdl6NqW4Ma+xWbVC29aBFFyNazw9arA== 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=bIS/M2jtUr4IQoW/e/A5Qor0AIasFstl8PHPqf4eBFY=; b=A1+zb4+jS57J8L16Zf8kl79Bxuu4FwHwvM8tqHTPd87vPbQr7+myopOWtvIeFfKeffHklq4DIx5XOYS3j0SWQXeqQm8PtOKn7DhuvdBW82QuacMY+UmE7WMKz7DU2Eo1zC2z8L5Bvvvi2FeHmOktO745WeDSfwqlTBM2XGRdFhmJV+rXcWCcaBELRYqx7X7wDV3waCLvL5S5p34THUUXSQXt/Zn4D7ahZppPIBCJDdACFjR5bUr8zCpDSjqTMJO395cca2f3rfqG04uvIqi0Fe0EcP480v41Wm8MrfgVw7/Yj8fmRdbeJzLu3Nr7ptY2s1ww8oEeDvwm7jVrNtt0/A== 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=bIS/M2jtUr4IQoW/e/A5Qor0AIasFstl8PHPqf4eBFY=; b=MYxYjIWYlh+5SE689dVbUI96qsbhIv0CscMpbrZYs3IuekOEQgnwPYFPMib8XKUid1mW6/bkuf56qZ6oBewmKHQwfevH73K6cSGgTQ/0tGm4AsViZ7axVfszaTkYDLB0/x1/dK0cA1woGFqM2JCEHcQjk1hClhUaSpZIyFqZbu4Fc85Ila9OgdkHdtP41e0mSsy6OkVPIret060yhG/Dwn35r7FgQq+1pO/naWxjUTlXaxaIAnIBjfk0WONX+21qWjnuWbZ8NO8BjFh7hQIRG9DV6aXeM7HkenQp91IHUc0qCWN8AkEui2BViyg+wqpJfhCnMGoGkVJgAsd/CnkhtQ== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; Received: from CH2PR12MB3990.namprd12.prod.outlook.com (2603:10b6:610:28::18) by CH0PR12MB8550.namprd12.prod.outlook.com (2603:10b6:610:192::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9678.17; Thu, 5 Mar 2026 01:37:22 +0000 Received: from CH2PR12MB3990.namprd12.prod.outlook.com ([fe80::7de1:4fe5:8ead:5989]) by CH2PR12MB3990.namprd12.prod.outlook.com ([fe80::7de1:4fe5:8ead:5989%6]) with mapi id 15.20.9678.016; Thu, 5 Mar 2026 01:37:22 +0000 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Thu, 05 Mar 2026 10:37:19 +0900 Message-Id: Subject: Re: [PATCH v3 3/5] gpu: nova-core: gsp: add reply/no-reply info to `CommandToGsp` From: "Alexandre Courbot" To: "Eliot Courtney" Cc: "Danilo Krummrich" , "Alice Ryhl" , "Simona Vetter" , "Benno Lossin" , "Gary Guo" , "Alistair Popple" , "Joel Fernandes" , , , , , "Zhi Wang" , "dri-devel" References: <20260304-cmdq-locking-v3-0-a6314b708850@nvidia.com> <20260304-cmdq-locking-v3-3-a6314b708850@nvidia.com> In-Reply-To: X-ClientProxiedBy: TYCP301CA0012.JPNP301.PROD.OUTLOOK.COM (2603:1096:400:386::10) To CH2PR12MB3990.namprd12.prod.outlook.com (2603:10b6:610:28::18) Precedence: bulk X-Mailing-List: rust-for-linux@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB3990:EE_|CH0PR12MB8550:EE_ X-MS-Office365-Filtering-Correlation-Id: 4ad7f3a5-b3e2-4a92-1c66-08de7a57bf5d X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|7416014|376014|1800799024|10070799003; X-Microsoft-Antispam-Message-Info: UlhHPoiRWYWgeTvaj1DmYbBKO5v38QY+eadeCyTn8klUJp3rr7L2CvXBNlwyEw0SJC6oEpRZDkk0yVeJsL4Wsx/eCIQoidkzxAisS+szwAYIfBFlzKWD1rd74QGcMl8Oq0C+2SkYDVDxPaYC3SfPMjRpEgAjB78LWkf7i7DDLQXNmQ2+FNjDwsuyN9991LC+VWpllwkivKnl7nNfbxLQjf8rKuXFnrHzA9OqwynZ+lfMEnhM6WToFR+WFPVMQuOW7v5FzJUzxlWVdFS2+xSgyGPfd/G1eQMFtcDTZpP5LVJJ8frv/nCjes/KjHC6ALG3zCPm4b9h38pn4ntSjhorECuiLePjb3P2Uf4mX+Gou08N3eb1yTbgth0o3rJQcSsTK2gty6z8J/fLx48wSujMk0WDjGZcx5OQ98zPJITgNUirbbPaT7TuWuIEYu6e2tYvn2YT+oqJ5A4OJ7XLbKXE6dZJh+B1GaQHgwKADoYEHj03xKz63fRiEOGQ+fWaWVAzlemLleeI/sQw1dyj/iWmt5TKuu2acIAAHYNOlm1DG6jgEWj0XFK6Yzxz66PGgyuimmK5jrMKkoqbd3ltC8w9m1QubpFM01QI/kBwiZyYafOMsSJer92HkNSBx4vDop6hAC2rc217MdYfQCRZmkuMUCJsUP73eLFA9S02BtK8cX+LSzHaKIWpelydE2kg/Q1czjh+zGyawa82D0+LPDI7ECjj/NO33PQBK3C6O46WUG0= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:CH2PR12MB3990.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(7416014)(376014)(1800799024)(10070799003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 2 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?OUFSREJLeEJNWE5OcEtObDNtbm9QT3pQeXl6VjAyRDVGMWVDM2FpdjRieEVG?= =?utf-8?B?RStaaGVtY3ErZThkYWhTVmJCSHRqNlVtcU5jeTFlVy9FbnFJbzZNelVWNmlP?= =?utf-8?B?dW80VlJlRm9zejgrQ1huK3NNZnRqSjMwZ0tyRWo2SDZSK2VDajdtZm9IQzZG?= =?utf-8?B?RGxVeUxyVVlEVHBaTFB5dkxWZGJKUjhPUDEwTUVKZVlKQ091MHJvMk0xeFNR?= =?utf-8?B?MCtDRDB6MXNUOUgwU1BpZmo2dDYyQm9kdTJuV0RhV3pDV3Y4aUgyajVhRjJH?= =?utf-8?B?R3RsMkpXSmMwUmVNV3ROeVM5OWRtOHk0NmhmSDVVV1ZHRytQR05QSC9xaUZO?= =?utf-8?B?NEVKU1UwTjZuQjRBYjR6MDNiUDF3WURPQlA4RHNEUC9jSHpjemVwd1o5WmFq?= =?utf-8?B?MFVITVAyUkdPM2J0WkwxNE93RExFN0FhVWE4bmNIQmlpcUJMdElXcUxVM09H?= =?utf-8?B?KzBNV1Y3c0I0alQ1UEo0ajN0bVUydnZrWWVTV1o4NmRjdjJBNEZoQWhHWEFE?= =?utf-8?B?U2VrYWR1S3E4b3NCNTZHbWt2OG0zc1laWXZCUk1mMUdLTS96dmVzTUdNZGF0?= =?utf-8?B?U3pQN1p3eFNGV2MrWElFWXJCWmpZVkdBaVpIaXRnbysrM002M3BFWWJxNllH?= =?utf-8?B?TlFHcWZIS3dia3BSWUhLbmZWQjUvUHhFUVh1NjI3NVFqWVhvQ1VVUWFibVB0?= =?utf-8?B?VTVKQzg3TFEyUEE1WVBBNnV3OXVJdndEVFJ0UFBBN0JVNldPaHpRT2xvNTdu?= =?utf-8?B?SjV3TkFLcElyWXdKTDgxUFF1NE9HZzkyVlV2c2VGSUtJdzVqS05DWlM1Tkpw?= =?utf-8?B?RVA3WXhka3ExejNSRFRObUFIZzhCb3VrZGFtMUZIWTF0ZEgyaFc1SnRKV28w?= =?utf-8?B?RlNOeG1ETUtNVTZPQjZxa0JYUzZJZi81MFlySHZjNDBUSTNWWWgyVUVUb1Y5?= =?utf-8?B?T05wbmcyQS9NVS9vMU9wWGk1SWF2ZE9UWkJqbUY1OHdCQUFvZ3ZtekVTelEv?= =?utf-8?B?WlprNTJqMW51YTdmSmVZcG9pWS94UGFyLzYzRFFHekJ4RW4yRkEvM09tZUhZ?= =?utf-8?B?VHFVczBNY0N4eTVWRzhXT1NmZENjZEltV0NCWHVVYjRUOFFxZmVVS1llVW0w?= =?utf-8?B?VXJZN0dtclBjNmtaSmpqTThXM1VvVXY0RmFaRE9JS2U4UkhpNGM0NEFhTStv?= =?utf-8?B?aTdSQmJ3UDVFVVAwZEVzNkxmV20ydFg5TVkxNDBZUUVZNUdBV3Ayc2tiNk9m?= =?utf-8?B?RFlqbitJcjUzdkJUUVZOZzd1TlRXT0hsL1FBdUE0ektoUEcvcXJFSUJUSTJT?= =?utf-8?B?a2tDMlc1c3JodUo0S3VMbjdkT1VBcC9kNFkwVytKNVQxamcvOTRoNTJrOU5i?= =?utf-8?B?eXNBTjg2VWVOV1lvcTJMaUs1T0VQMGJremloeW5DYnVlNXIweFR3K1QvclIx?= =?utf-8?B?ODljcFYzY2lKUWs1RDVBR3B0R0NnZlZRYkkvaFNhNk9PNFFBU1lEcUxiSEpm?= =?utf-8?B?KzEyc2tNRzlBbGtldzV4UzF1ZnBncCtkTkJFQzYybGY1emJSK2xzdGdNUjF3?= =?utf-8?B?SVZOWjZndHdIN0xpeGhZNGlPOWFORFBka2lWaEtlS3NUdEp2bmFHSFVQR05V?= =?utf-8?B?emloWDIzaTEwWHJNZlFieHlzNlRCdVB1b3U2bGtJYUlVdUtFY3JoNk5nakYx?= =?utf-8?B?dXZFU3dvMDlKeEo1Tjg3Y2o4U3dqSVRGcytmbmdZMDIxL2hOallpaHJpMnI2?= =?utf-8?B?RncrbzV1YjRFWE42dnNsZng4NHNDS29Db3k3dzBUcjlxZGJYSjhDeDhTcXN0?= =?utf-8?B?ZVRTUU5ZWjhXQzJpVlVFU2VMTGdvcjJWdXlDbkhsZHlLakM5dW5VQzQ5NlZl?= =?utf-8?B?S05WejlNam1VTFc0dUtuSmhBS2swbWJIVVlCM0ZQc2xlSkhNV0VJRHZJNHE2?= =?utf-8?B?MzdxS28yY3JDcTFWdUk5M05Eb3dqMWlGZURIcWFQSDc2UG51Z0FLU1h1M3JS?= =?utf-8?B?d1ZvNnhSMDJvZzdxS2R0M28xWjhsZWZGSnBZRHh6ckNJT2J5cTBhZi8yVG54?= =?utf-8?B?M1UxTGNmVUMwQkt3RUVrb2xaMWF4SnR3VDhvVzFiWUlVbTlLTjhXUXhzUDNH?= =?utf-8?B?UXZ4ZDNjbEFpME1vTDJBN0RCSXZGbkY2YTZVazNveGRzUld6eThkOXBWTWVj?= =?utf-8?B?WWxpNkFqRWRFanVDUGZvcU9vZE9pOEVMRTZrcDdaK0tNTXVUQksxZi8vM2dq?= =?utf-8?B?Wk8ycU1ITTVaWnNuZ0RoSlRPeG51ZHlWMExTTlg3REJjUWIrVFVsQ0RQazJH?= =?utf-8?B?RThNQW5aeTEyUWdtZzZpU0pWRk9GVERUazY4Q1dKcStLTldKcWtkUWV3UDdH?= =?utf-8?Q?UehYO+N/rOtCEpCyX8wOLrwYELlFliacZCnGW2Eav5QfE?= X-MS-Exchange-AntiSpam-MessageData-1: U2CyAKigHl2zDg== X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4ad7f3a5-b3e2-4a92-1c66-08de7a57bf5d X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB3990.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Mar 2026 01:37:22.6435 (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: DwhM8l/umzffHUDDOv/Lqxjh1Jue4MlqNYFZeR3jTsg5guybdd8i+0xxnGBNwgBEoy9DIhokQwmaUGgk+r7/WA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR12MB8550 On Thu Mar 5, 2026 at 10:29 AM JST, Eliot Courtney wrote: > On Wed Mar 4, 2026 at 11:17 PM JST, Alexandre Courbot wrote: >> On Wed Mar 4, 2026 at 11:46 AM JST, Eliot Courtney wrote: >> >>> /// Send the [`GetGspInfo`] command and awaits for its reply. >>> pub(crate) fn get_gsp_info(cmdq: &mut Cmdq, bar: &Bar0) -> Result { >>> - cmdq.send_command(bar, GetGspStaticInfo)?; >>> - >>> - loop { >>> - match cmdq.receive_msg::(Cmdq::RECEIVE_= TIMEOUT) { >>> - Ok(info) =3D> return Ok(info), >>> - Err(ERANGE) =3D> continue, >>> - Err(e) =3D> return Err(e), >>> - } >>> - } >>> + cmdq.send_command(bar, GetGspStaticInfo) >> >> Also noticed something cool while rebasing the unload series on top of >> this one: all these command helpers become one-liners and are not really >> needed anymore! Here we can just make `GetGspStaticInfo` public and call >> `cmdq.send_command` on it directly in `boot`. It removes a layer of >> black magic and makes it easier to understand what is going on. > > I think this is ok, but we will still need some helpers (e.g. for RM > control RPCs) that mimic the structure we have here. But there are a > bunch of simple RPCs like this that can just be called without the > helper. Do you think it's better to do this and have mixed helper vs > direct call, or just go all helpers? Mmm ok, let's keep the current helpers and reconsider once we have RM controls and more visibility.