Re: [PATCH v8 01/31] gpu: nova-core: Hopper/Blackwell: basic GPU identification

["Gary Guo" <gary@garyguo.net>]

On Wed Mar 25, 2026 at 3:52 AM GMT, John Hubbard wrote:
> Hopper (GH100) and Blackwell identification, including ELF
> .fwsignature_* items.
>
> Signed-off-by: John Hubbard <jhubbard@nvidia.com>
> ---
>  drivers/gpu/nova-core/falcon/hal.rs   |  3 ++-
>  drivers/gpu/nova-core/fb/hal.rs       |  5 ++---
>  drivers/gpu/nova-core/firmware/gsp.rs |  9 +++++++--
>  drivers/gpu/nova-core/gpu.rs          | 22 ++++++++++++++++++++++
>  4 files changed, 33 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/nova-core/gpu.rs b/drivers/gpu/nova-core/gpu.rs
> index 8579d632e717..3b4ccc3d18b9 100644
> --- a/drivers/gpu/nova-core/gpu.rs
> +++ b/drivers/gpu/nova-core/gpu.rs
> @@ -83,12 +83,22 @@ fn try_from(value: u32) -> Result<Self, Self::Error> {
>      GA104 = 0x174,
>      GA106 = 0x176,
>      GA107 = 0x177,
> +    // Hopper
> +    GH100 = 0x180,
>      // Ada
>      AD102 = 0x192,
>      AD103 = 0x193,
>      AD104 = 0x194,
>      AD106 = 0x196,
>      AD107 = 0x197,
> +    // Blackwell
> +    GB100 = 0x1a0,
> +    GB102 = 0x1a2,
> +    GB202 = 0x1b2,
> +    GB203 = 0x1b3,
> +    GB205 = 0x1b5,
> +    GB206 = 0x1b6,
> +    GB207 = 0x1b7,

Sashiko asks if it'll be problematic that 0x1a and 0x1b are both Blackwell.
https://sashiko.dev/#/patchset/20260325035242.368661-1-jhubbard%40nvidia.com

Can you confirm that it is indeed actually okay and is a false positive?

Best,
Gary

> [snip]
>
> @@ -139,7 +157,9 @@ pub(crate) enum Architecture {
>      #[default]
>      Turing = 0x16,
>      Ampere = 0x17,
> +    Hopper = 0x18,
>      Ada = 0x19,
> +    Blackwell = 0x1b,
>  }
>  
>  impl TryFrom<u8> for Architecture {
> @@ -149,7 +169,9 @@ fn try_from(value: u8) -> Result<Self> {
>          match value {
>              0x16 => Ok(Self::Turing),
>              0x17 => Ok(Self::Ampere),
> +            0x18 => Ok(Self::Hopper),
>              0x19 => Ok(Self::Ada),
> +            0x1b => Ok(Self::Blackwell),
>              _ => Err(ENODEV),
>          }
>      }