- Oct 03, 2018
-
-
Tim Newsome authored
-
Tim Newsome authored
* Make haltreq and resumereq proper write-only. They already mostly were, but then I clarified some stuff and broke keeping haltreq high during reset, which is the only mechanism to catch a hart out of reset if optional reselthaltreq isn't implemented. Also: * resume ack is cleared by the DM when resumereq is written 1. * haltreq can be high together with hartreset. This is the minimal version of #378. It has all the same functionality, but lacks some of #378's clarifications, which may be making this harder to review. * Leave haltreq high to catch a hart out of all resets. Not just unexpected ones. * Rephrase halt resume explanation. * DM contains->tracks bits of state.
-
Tim Newsome authored
* Minimal implementations can't access all registers * Provide more detail on minimal implementation.
-
Tim Newsome authored
For inches, apparently the thing to do is spell them out, so I did that.
-
Tim Newsome authored
-
Tim Newsome authored
This resolves a contradiction in the spec. Fixes #365.
-
Tim Newsome authored
* Explicitly state what's required for compliance. * Clarify "all registers."
-
- Oct 02, 2018
-
-
Tim Newsome authored
-
Tim Newsome authored
-
Tim Newsome authored
* Clarify W1. The definition used to be vague. I think this is the one that makes the most sense, and shouldn't cause any problems. (Possibly we don't want this for ackhavereset. I sent e-mail about that.) The bits that are defined as W1 are: DM: ackhavreset, setresethaltreq, clrresethaltreq JTAG: dmihardreset, dmireset * Revert my "clarification" of W1.
-
Tim Newsome authored
-
Tim Newsome authored
* Add dcsr.cause for being halted out of reset. * Halt out of reset may be reported as just halted. * Punctuation. * Clarify halt out of reset due to resethaltreq.
-
- Sep 29, 2018
-
-
Tim Newsome authored
* Clean up language, formatting, consistency. * Remove unused R/W0, R/W1. * Define haltresetreq.
-
Tim Newsome authored
* Add short name to register tables. * Little language and formatting cleanups. * Device Tree -> configuration string The privileged spec talks about a configuration string, which may be a Device Tree.
-
- Sep 28, 2018
-
-
Tim Newsome authored
Now it's consistent with dtmcs.dmistat. This reverts 7f0f09a0. It is no longer true that a debugger can assume dmi.data contains a valid value unless a read was explicitly executed, so there's no sense in accommodating that case.
-
Tim Newsome authored
-
Tim Newsome authored
* Distinguish draft and release builds. Rename the output PDFs so it's less likely to cause confusion. The release build is not labelled as DRAFT, does not include the preface warning, and does not include the change log. * Add serial.tex, which got lost somewhere. (Only used by future.tex.) * Fix build. Also add riscv-debug-release.tex which I forgot earlier.
-
- Sep 26, 2018
-
-
Tim Newsome authored
* Stepping over wfi does not enter wait state. * Halt breaks wfi out of stalled state. * Consistently use stall instead of wait.
-
Tim Newsome authored
Explicitly state which arguments abstract commands do or do not change.
-
Tim Newsome authored
-
- Sep 25, 2018
-
-
Tim Newsome authored
Fixes #346.
-
- Sep 19, 2018
-
-
Tim Newsome authored
* Harts can be in exactly 1 of 4 states. Fixes #351. * Grammar fixes. * While held in reset, harts are unavailable. Fixes #353. * Remove "impl specific [reset] indicators" They don't supersede the unavailable status, and their mention is confusing. (They were added in ce1b3593.) Fixes #353. * Unavailable harts can't execute abstract commands. Being in reset doesn't mean you have to be unavailable. #355 * Clarify reset states. * Abstract behavior when hart is/becomes unavailable Just a rewording that (hopefully) makes more sense.
-
- Sep 18, 2018
-
-
Tim Newsome authored
* Mostly match "official" style for credits. * Fix Megan's e-mail address.
-
Tim Newsome authored
Fixes #358.
-
- Sep 15, 2018
-
-
Tim Newsome authored
* Abstract commands might work on a hung hart. From a discussion on today's call. * Rephrase.
-
Tim Newsome authored
Can't change harts during operations, and the current hart becoming unavailable may terminate the abstract command with error. (#322) * Can't change harts during operations. See #315. * Only restrict hartsel changes during abstract cmds reset/halt could take a very long time if a hart is asleep, and we don't want to prevent debugging other harts in that case. * Set cmderr for hartsel change during abstract cmd. * Clarify reset and hasel during abstract command. * Incorporate review feedback. * Make debuggers handle harts becoming unavailable. Instead of having the DM correctly handle a hart that becomes unavailable, make debuggers responsible for that. This seems like a good tradeoff because this should be rare. * Clarifications requested in review. * Clarify per review.
-
- Sep 12, 2018
-
-
Tim Newsome authored
-
- Sep 08, 2018
-
-
Tim Newsome authored
-
Tim Newsome authored
Fixes #301.
-
- Sep 07, 2018
-
-
Florian Zaruba authored
-
Tim Newsome authored
-
Tim Newsome authored
* Document breakpoint exception + enter debug mode * Multiple triggers firing with different timing. * Be a more lenient when trigger actions conflict This came out of the July 11 meeting, where consensus was that requiring both actions to be taken might significantly complicate hardware to handle what is an uncommon corner case.
-
- Sep 06, 2018
-
-
Tim Newsome authored
* Define MXLEN. * Introduce DXLEN. This lets us be more precise about register widths, to deal with the fact that MXLEN can be changed by software. * Mention debugger's responsibility when XLEN!=DXLEN * DXLEN is the widest supported XLEN.
-
- Sep 05, 2018
-
-
Tim Newsome authored
This simplifies implementations and validation by explicitly preventing the debugger from doing things that aren't very useful anyway.
-
- Sep 04, 2018
-
-
Tim Newsome authored
Fixes #320.
-
Tim Newsome authored
-
- Sep 01, 2018
-
-
Tim Newsome authored
-
- Aug 28, 2018
-
-
Tommy Thorn authored
Note, the e.g. must be followed by a backslash to tell LaTeX that it isn't the end of a sentence. $0x400$ tells LaTeX to read "0 x 400" as math, not what you want
-
Tim Newsome authored
This is an alternate to #326.
-
- Aug 22, 2018
-
-
Tim Newsome authored
-