On 1/16/26 11:55 AM, Danilo Krummrich wrote: > On Wed Jan 14, 2026 at 8:29 PM CET, Timur Tabi wrote: >> + regs::NV_PFALCON_FALCON_ENGINE::update(bar, &E::ID, |v| >> v.set_reset(true)); >> + >> + // TIMEOUT: falcon engine should not take more than 10us to reset. >> + fsleep(Delta::from_micros(10)); >> + >> + regs::NV_PFALCON_FALCON_ENGINE::update(bar, &E::ID, |v| >> v.set_reset(false)); > > This code is just moved, but repeated in the subsequent patch adding the > Turing > HAL. > > If we always expect the falcon engine to reset within 10us regardless of the > GPU > architecture, we could add a regs::NV_PFALCON_FALCON_ENGINE::reset() function
On newer GPU architectures, we don't even *do* a falcon reset, fwiw. For example, on Blackwell, the GPU's FSP reboots the GSP by itself. And we don't do an FSP reset. So we are all out of the falcon reset business there. :) thanks, -- John Hubbard > doing the above. > > If it the exact timeout differs, we could just pass it as argument to > regs::NV_PFALCON_FALCON_ENGINE::reset().
