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 715C0C5AD7B for ; Sat, 21 Feb 2026 15:09:49 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id AEDA510E099; Sat, 21 Feb 2026 15:09:48 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=kernel.org header.i=@kernel.org header.b="X6/2wUb3"; dkim-atps=neutral Received: from tor.source.kernel.org (tor.source.kernel.org [172.105.4.254]) by gabe.freedesktop.org (Postfix) with ESMTPS id 2DEFA10E099 for ; Sat, 21 Feb 2026 15:09:47 +0000 (UTC) Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by tor.source.kernel.org (Postfix) with ESMTP id 26D786001D; Sat, 21 Feb 2026 15:09:46 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id C4CAAC4CEF7; Sat, 21 Feb 2026 15:09:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1771686585; bh=jTIT9NqpyoWBxCeo/vIgFgNyhqW6Cp5fG9tUy3nKyeA=; h=Date:Subject:Cc:To:From:References:In-Reply-To:From; b=X6/2wUb3g64KQyNHy+qsGgWX/nzikXjB7/VpDNAAvMR8ct44WmuPx5mLNY1RmsZ8R MgrMWVAErZZ+AWsOfakPP4iII49dIZYN5/tA7Jdq5Zgjm2f8Q/432TOUNHUk8M+KTY b0QDAeXWGhZikedRflCYX+nWh4EoQQbycaKWDNHKKgn/veVn1OyoSCZiYDm2VehdWh 2dY3npv651mv6xSDcbhFTxEty8qvZguEGFDdUC40w0YzHNRc7LtQ9ozZ0kFk2CriBp 40plQQvpAI+KsmR8tYddu0xtVrVeZMHYaXFyow+n/c53V7ZwA0kDJ6S5cSCPjNbuAb zUMoLfw5XF1Hg== Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Sat, 21 Feb 2026 16:09:41 +0100 Message-Id: Subject: Re: [PATCH v4 3/6] rust: gpuvm: add GpuVm::obtain() Cc: "Daniel Almeida" , "Boris Brezillon" , "Janne Grunau" , "Matthew Brost" , =?utf-8?q?Thomas_Hellstr=C3=B6m?= , "Lyude Paul" , "Asahi Lina" , , , To: "Alice Ryhl" From: "Danilo Krummrich" References: <20260130-gpuvm-rust-v4-0-8364d104ff40@google.com> <20260130-gpuvm-rust-v4-3-8364d104ff40@google.com> In-Reply-To: X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On Sat Feb 21, 2026 at 9:46 AM CET, Alice Ryhl wrote: > So, should we get rid of GpuVmBoRegistered in favor of ARef>? I wanted to avoid exposing the reference count, as I suspect drivers might = not need it in Rust, but I don't know for sure. We could also define it as GpuVmBo(ARef>), where GpuVmBoInner is private, but I also don't want you to go back and forth = with this in case it turns out we do need drivers to be able to take a reference count and I also don't think it hurts too much exposing the reference count= , even if not needed. So, either is fine with me, ARef> or GpuVmBo(ARef>).