Re: [PATCH 3/7] gpu: nova-core: rename module from nova_core to nova-core
From: Danilo Krummrich
Date: Fri May 01 2026 - 08:31:31 EST
On Fri May 1, 2026 at 5:25 AM CEST, Alexandre Courbot wrote:
> Danilo, WDYT?
I think the patch is fine as is.
However, the commit message should say that it also changes nova_core.ko to
nova-core.ko, which is the reason I proposed this in [1] as well.
I.e. such that the module name (that we still need to change; I can send a patch
since I haven't heard back from Chou) matches the .ko file name.
I'm not a fan of generic names like lib.rs, mod.rs, main.rs, etc., as it messes
with fuzzy finder tools.
If people feel strongly about this, we can probably rename it to
nova_core_main.rs, but I think it is not necessary.
- Danilo
[1] https://lore.kernel.org/all/DH56PG8GLW4X.2QMHJ91J9D29N@xxxxxxxxxx/