Skip to content

Add sequencer timeout ereports#2488

Draft
mkeeter wants to merge 5 commits intomasterfrom
mkeeter/seq-ereports
Draft

Add sequencer timeout ereports#2488
mkeeter wants to merge 5 commits intomasterfrom
mkeeter/seq-ereports

Conversation

@mkeeter
Copy link
Copy Markdown
Collaborator

@mkeeter mkeeter commented Apr 23, 2026

This PR adds two ereport flavors for Cosmo sequencer issues:

  • hw.seq.regs is a raw dump of the registers, along with a reason why it was produced (MAPO, timeout, or "suddenly we're not in A0")
  • hw.seq.timeout.* is a family of ereports representing timeouts at each stage of sequencing, along with a diagnosis. It includes an Option<u64> that refers to a hw.seq.regs ENA, so that we can also look at raw register values (if the diagnosis is not helpful).

For failure modes other than timeout, this PR doesn't send any additional ereports (just the hw.seq.regs value).

Consider this a starting point for brainstorming how we want to organize this data (e.g. one alternative is sending the high-level ereport first, then have the hw.seq.regs refer to it).

@mkeeter mkeeter requested review from hawkw and rmustacc April 23, 2026 14:43
@mkeeter mkeeter force-pushed the mkeeter/seq-ereports branch from 0b1af44 to 6b3f76f Compare April 23, 2026 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant