[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aXHsqXC-dZhrD2VA@google.com>
Date: Thu, 22 Jan 2026 09:23:53 +0000
From: Alice Ryhl <aliceryhl@...gle.com>
To: Daniel Almeida <daniel.almeida@...labora.com>
Cc: Danilo Krummrich <dakr@...nel.org>, Boris Brezillon <boris.brezillon@...labora.com>,
Janne Grunau <j@...nau.net>, Matthew Brost <matthew.brost@...el.com>,
"Thomas Hellström" <thomas.hellstrom@...ux.intel.com>, Lyude Paul <lyude@...hat.com>,
Asahi Lina <lina+kernel@...hilina.net>, dri-devel@...ts.freedesktop.org,
linux-kernel@...r.kernel.org, rust-for-linux@...r.kernel.org
Subject: Re: [PATCH v3 1/6] rust: drm: add base GPUVM immediate mode abstraction
On Wed, Jan 21, 2026 at 02:04:33PM -0300, Daniel Almeida wrote:
> Hi Alice,
>
> > On 21 Jan 2026, at 08:31, Alice Ryhl <aliceryhl@...gle.com> wrote:
> > +/// The manager for a GPUVM.
> > +pub trait DriverGpuVm: Sized {
> > + /// Parent `Driver` for this object.
> > + type Driver: drm::Driver;
> > +
> > + /// The kind of GEM object stored in this GPUVM.
> > + type Object: IntoGEMObject;
>
> Hmm, can’t we derive that from Driver::AllocOps? More specifically, shouldn’t we enforce it?
Hrm, we may wish to require that Self::Object == Self::Driver::Object.
Not sure what you mean by AllocOps.
Alice
Powered by blists - more mailing lists