Danilo Krummrich
2025-Apr-30 10:32 UTC
[PATCH 11/16] gpu: nova-core: add falcon register definitions and base code
On Wed, Apr 30, 2025 at 06:58:44AM +0000, Joel Fernandes wrote:> > On Apr 22, 2025, at 10:45?AM, Danilo Krummrich <dakr at kernel.org> wrote: > > > [1] https://play.rust-lang.org/?version=stable&mode=debug&edition=2024&gist=bf7035a07e79a4047fb6834eac03a9f2 > > I am still researching this idea from a rust point of view, but quick question - will this even work if the chip type (GAxxx) is determined at runtime? That does need runtime polymorphism.I exetended the example in [2] to address this with `enum HalImpl<E: Engine>` and a second architecture that is picked randomly. It needs match for every access, but that's probably still better than the dynamic dispatch. [2] https://play.rust-lang.org/?version=stable&mode=debug&edition=2024&gist=99ce0f12542488f78e35356c99a1e23f