Re: [PATCH v7 00/31] gpu: nova-core: firmware: Hopper/Blackwell support
From: John Hubbard
Date: Wed Mar 18 2026 - 16:26:05 EST
On 3/17/26 3:53 PM, John Hubbard wrote:
> This is based on today's drm-rust-next, which has Alex's register!()
> macro series. A git branch is here:
>
> https://github.com/johnhubbard/linux/tree/nova-core-blackwell-v7
Hi Alex, Danilo,
I've fixed up patches 27 and 31 with the "make rustfmt" changes that
the kernel bot reported, and that's standing by here:
https://github.com/johnhubbard/linux/commits/nova-core-blackwell-v8/
I'm hoping that otherwise, we're about ready to merge, on top of
Alex's register!() changes patchset [1], it sounds like.
Let me know if or when I should post anything, such as v8.
[1] https://lore.kernel.org/20260318-b4-nova-register-v1-0-22a358aa4c63@xxxxxxxxxx
thanks,
--
John Hubbard
>
> It's been re-tested on Turing, Ampere and Blackwell:
>
> NovaCore 0000:e1:00.0: GPU name: NVIDIA GeForce GTX 1650
> NovaCore 0000:e1:00.0: GPU name: NVIDIA RTX A4000
> NovaCore 0000:01:00.0: GPU name: NVIDIA RTX PRO 6000 Blackwell Max-Q
> Workstation Edition
>
> Changes in v7:
> * Rebased onto Alexandre Courbot's rust register!() series in
> drm-rust-next, including the related generic I/O accessor and
> IoCapable changes.
>
> * Rebased onto drm-rust-next (v7.0-rc4 based).
>
> * Dropped the v6 patches that are already in drm-rust-next: the
> aux-device fix, the pdev helper macro patch, and the one-item-per-line
> use cleanup.
>
> * Reworked the GPU init pieces per review. DMA mask setup now stays in
> driver probe, with the mask width selected by GPU architecture, and
> the GFW boot policy now lives in a dedicated GPU HAL.
>
> * Reworked firmware image parsing per review around a single ElfFormat
> trait with associated header types. Also added support for both ELF32
> and ELF64 images, with automatic format detection.
>
> * Reworked the MCTP/NVDM protocol code to use bitfield! and typed
> accessors, removing the open-coded bit handling.
>
> * Reworked the FSP messaging part of the series so that the message
> structures are introduced in the first patches that use them, instead
> of as a standalone dead-code-only patch. Also changed fmc_full to use
> KVec<u8> from the start.
>
> * Split the WPR heap overflow handling out into a separate prep patch.
> That patch makes management_overhead() and wpr_heap_size() fallible,
> uses checked arithmetic, and leaves the larger WPR2 heap patch with
> only the Hopper and Blackwell sizing changes.
>
> * Added a code comment documenting the Hopper and Blackwell PCI config
> mirror base change.
>
> Changes in v6:
>
> * Rebased onto drm-rust-next (v7.0-rc1 based).
>
> * Dropped the first two patches from v5 (aux device fix and pdev
> macros), which have since been merged independently.
>
> * const_align_up(): reworked per review from Gary Guo, Miguel Ojeda,
> and Danilo Krummrich: now returns Option<usize> instead of panicking,
> takes an Alignment argument instead of a const generic, and no longer
> needs the inline_const feature addition in scripts/Makefile.build.
>
> * The rust/sizes and SZ_*_U64 patches from v5 are no longer included.
> I plan to post those as a separate series that depends on this one.
>
> Changes in v5:
>
> * Rebased onto linux.git master.
>
> * Split MCTP protocol into its own module and file.
>
> * Many Rust-based improvements: more use of types, especially. Also
> used Result and Option more.
>
> * Lots of cleanup of comments and print output and error handling.
>
> * Added const_align_up() to rust/ and used it in nova-core. This
> required enabling a Rust feature: inline_const, as recommended by
> Miguel Ojeda.
>
> * Refactoring various things, such as Gpu::new() to own Spec creation,
> and several more such things.
>
> * Fixed three Delta::ZERO busy-polls (patches 21, 24, 31) to use
> non-zero sleep intervals (after just realizing that it was a bad
> choice to have zero in there).
>
> * Reduced GH100/GB100 HAL duplication. Made FSP_PKEY_SIZE/FSP_SIG_SIZE
> consistent across patches. Replaced fragile architecture checks with
> chipset.arch(). Renamed LIBOS_BLACKWELL.
>
> * Narrowed the scope of some of the #![expect(dead_code)] cases,
> although that really only matters within the series, not once it is
> fully applied.
>
> John Hubbard (31):
> gpu: nova-core: Hopper/Blackwell: basic GPU identification
> gpu: nova-core: factor .fwsignature* selection into a new
> find_gsp_sigs_section()
> gpu: nova-core: use GPU Architecture to simplify HAL selections
> gpu: nova-core: move GPU init into Gpu::new()
> gpu: nova-core: set DMA mask width based on GPU architecture
> gpu: nova-core: Hopper/Blackwell: skip GFW boot waiting
> gpu: nova-core: move firmware image parsing code to firmware.rs
> gpu: nova-core: factor out an elf_str() function
> gpu: nova-core: don't assume 64-bit firmware images
> gpu: nova-core: add support for 32-bit firmware images
> gpu: nova-core: add auto-detection of 32-bit, 64-bit firmware images
> gpu: nova-core: Hopper/Blackwell: add FMC firmware image, in support
> of FSP
> gpu: nova-core: Hopper/Blackwell: add FSP falcon engine stub
> gpu: nova-core: Hopper/Blackwell: add FSP falcon EMEM operations
> gpu: nova-core: Hopper/Blackwell: add FSP message infrastructure
> rust: ptr: add const_align_up()
> gpu: nova-core: Hopper/Blackwell: calculate reserved FB heap size
> gpu: nova-core: add MCTP/NVDM protocol types for firmware
> communication
> gpu: nova-core: Hopper/Blackwell: add FSP secure boot completion
> waiting
> gpu: nova-core: Hopper/Blackwell: add FMC signature extraction
> gpu: nova-core: Hopper/Blackwell: add FSP send/receive messaging
> gpu: nova-core: Hopper/Blackwell: add FspCotVersion type
> gpu: nova-core: Hopper/Blackwell: larger non-WPR heap
> gpu: nova-core: Hopper/Blackwell: add FSP Chain of Trust boot
> gpu: nova-core: Blackwell: use correct sysmem flush registers
> gpu: nova-core: make WPR heap sizing fallible
> gpu: nova-core: Hopper/Blackwell: larger WPR2 (GSP) heap
> gpu: nova-core: refactor SEC2 booter loading into
> BooterFirmware::run()
> gpu: nova-core: Hopper/Blackwell: add GSP lockdown release polling
> gpu: nova-core: Hopper/Blackwell: new location for PCI config mirror
> gpu: nova-core: Hopper/Blackwell: integrate FSP boot path into boot()
>
> drivers/gpu/nova-core/driver.rs | 28 +-
> drivers/gpu/nova-core/falcon.rs | 1 +
> drivers/gpu/nova-core/falcon/fsp.rs | 220 ++++++++++
> drivers/gpu/nova-core/falcon/hal.rs | 20 +-
> drivers/gpu/nova-core/fb.rs | 26 +-
> drivers/gpu/nova-core/fb/hal.rs | 38 +-
> drivers/gpu/nova-core/fb/hal/ga102.rs | 2 +-
> drivers/gpu/nova-core/fb/hal/gb100.rs | 75 ++++
> drivers/gpu/nova-core/fb/hal/gb202.rs | 62 +++
> drivers/gpu/nova-core/fb/hal/gh100.rs | 38 ++
> drivers/gpu/nova-core/firmware.rs | 204 +++++++++
> drivers/gpu/nova-core/firmware/booter.rs | 35 +-
> drivers/gpu/nova-core/firmware/fsp.rs | 47 ++
> drivers/gpu/nova-core/firmware/gsp.rs | 128 ++----
> drivers/gpu/nova-core/fsp.rs | 527 +++++++++++++++++++++++
> drivers/gpu/nova-core/gpu.rs | 86 +++-
> drivers/gpu/nova-core/gpu/hal.rs | 54 +++
> drivers/gpu/nova-core/gsp/boot.rs | 298 ++++++++++---
> drivers/gpu/nova-core/gsp/commands.rs | 8 +-
> drivers/gpu/nova-core/gsp/fw.rs | 83 +++-
> drivers/gpu/nova-core/gsp/fw/commands.rs | 20 +-
> drivers/gpu/nova-core/mctp.rs | 119 +++++
> drivers/gpu/nova-core/nova_core.rs | 2 +
> drivers/gpu/nova-core/regs.rs | 96 +++++
> rust/kernel/ptr.rs | 24 ++
> 25 files changed, 2001 insertions(+), 240 deletions(-)
> create mode 100644 drivers/gpu/nova-core/falcon/fsp.rs
> create mode 100644 drivers/gpu/nova-core/fb/hal/gb100.rs
> create mode 100644 drivers/gpu/nova-core/fb/hal/gb202.rs
> create mode 100644 drivers/gpu/nova-core/fb/hal/gh100.rs
> create mode 100644 drivers/gpu/nova-core/firmware/fsp.rs
> create mode 100644 drivers/gpu/nova-core/fsp.rs
> create mode 100644 drivers/gpu/nova-core/gpu/hal.rs
> create mode 100644 drivers/gpu/nova-core/mctp.rs
>
>
> base-commit: d19ab42867ae7c68be84ed957d95712b7934773f