On Wed Mar 4, 2026 at 11:46 AM JST, Eliot Courtney wrote: > Remove magic numbers and add a default timeout for callers to use. > > Tested-by: Zhi Wang <[email protected]> > Signed-off-by: Eliot Courtney <[email protected]> > --- > drivers/gpu/nova-core/gsp/cmdq.rs | 3 +++ > drivers/gpu/nova-core/gsp/commands.rs | 5 ++--- > drivers/gpu/nova-core/gsp/sequencer.rs | 2 +- > 3 files changed, 6 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/nova-core/gsp/cmdq.rs > b/drivers/gpu/nova-core/gsp/cmdq.rs > index 4829830b6921..0192c85ddd75 100644 > --- a/drivers/gpu/nova-core/gsp/cmdq.rs > +++ b/drivers/gpu/nova-core/gsp/cmdq.rs > @@ -496,6 +496,9 @@ impl Cmdq { > /// Timeout for waiting for space on the command queue. > const ALLOCATE_TIMEOUT: Delta = Delta::from_secs(1); > > + /// Default timeout for receiving a message from the GSP. > + pub(super) const RECEIVE_TIMEOUT: Delta = Delta::from_secs(10);
Agree with the idea, but let's settle on 5 seconds - GSP replies are supposed to be fast and there should be no good reason for any command to take longer than that (and potentially hang the whole system for the same time).
