Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

gpu: nova-core: provide a clear error report for unsupported GPUs

Pass in a PCI device to Spec::new(), and provide a Display
implementation for boot42, in order to provide a clear, concise report
of what happened: the driver read NV_PMC_BOOT42, and found that the GPU
is not supported.

For very old GPUs (older than Fermi), the driver still returns ENODEV,
but it does so without a driver-specific dmesg report. That is exactly
appropriate, because if such a GPU is installed, it can only be
supported by Nouveau. And if so, the user is not helped by additional
error messages from Nova.

Here's the full dmesg output for a Blackwell (not yet supported) GPU:

NovaCore 0000:01:00.0: Probe Nova Core GPU driver.
NovaCore 0000:01:00.0: Unsupported chipset: boot42 = 0x1b2a1000 (architecture 0x1b, implementation 0x2)
NovaCore 0000:01:00.0: probe with driver NovaCore failed with error -524

Cc: Alexandre Courbot <acourbot@nvidia.com>
Cc: Danilo Krummrich <dakr@kernel.org>
Cc: Timur Tabi <ttabi@nvidia.com>
Cc: Joel Fernandes <joelagnelf@nvidia.com>
Signed-off-by: John Hubbard <jhubbard@nvidia.com>
[acourbot@nvidia.com: fix commit log with ENODEV (not ENOTSUPP) error
code for unsupported GPUs.]
Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
Message-ID: <20251115010923.1192144-5-jhubbard@nvidia.com>

authored by

John Hubbard and committed by
Alexandre Courbot
ce89e3e0 0ecc08e2

+24 -3
+6 -3
drivers/gpu/nova-core/gpu.rs
··· 182 182 } 183 183 184 184 impl Spec { 185 - fn new(bar: &Bar0) -> Result<Spec> { 185 + fn new(dev: &device::Device, bar: &Bar0) -> Result<Spec> { 186 186 // Some brief notes about boot0 and boot42, in chronological order: 187 187 // 188 188 // NV04 through NV50: ··· 207 207 return Err(ENODEV); 208 208 } 209 209 210 - Spec::try_from(regs::NV_PMC_BOOT_42::read(bar)) 210 + let boot42 = regs::NV_PMC_BOOT_42::read(bar); 211 + Spec::try_from(boot42).inspect_err(|_| { 212 + dev_err!(dev, "Unsupported chipset: {}\n", boot42); 213 + }) 211 214 } 212 215 } 213 216 ··· 262 259 bar: &'a Bar0, 263 260 ) -> impl PinInit<Self, Error> + 'a { 264 261 try_pin_init!(Self { 265 - spec: Spec::new(bar).inspect(|spec| { 262 + spec: Spec::new(pdev.as_ref(), bar).inspect(|spec| { 266 263 dev_info!(pdev.as_ref(),"NVIDIA ({})\n", spec); 267 264 })?, 268 265
+18
drivers/gpu/nova-core/regs.rs
··· 67 67 }) 68 68 .and_then(Chipset::try_from) 69 69 } 70 + 71 + /// Returns the raw architecture value from the register. 72 + fn architecture_raw(self) -> u8 { 73 + ((self.0 >> Self::ARCHITECTURE_RANGE.start()) & ((1 << Self::ARCHITECTURE_RANGE.len()) - 1)) 74 + as u8 75 + } 76 + } 77 + 78 + impl kernel::fmt::Display for NV_PMC_BOOT_42 { 79 + fn fmt(&self, f: &mut kernel::fmt::Formatter<'_>) -> kernel::fmt::Result { 80 + write!( 81 + f, 82 + "boot42 = 0x{:08x} (architecture 0x{:x}, implementation 0x{:x})", 83 + self.0, 84 + self.architecture_raw(), 85 + self.implementation() 86 + ) 87 + } 70 88 } 71 89 72 90 // PBUS