Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

⚠️ rework CPU debug spec ISA configuration; ✨ enhance trigger module #465

Merged
merged 12 commits into from
Dec 24, 2022

Conversation

stnolting
Copy link
Owner

@stnolting stnolting commented Dec 23, 2022

⚠️ This PR removes the CPU's CPU_EXTENSION_RISCV_DEBUG generic, which is replaced by two new generics to allow a finer configuration of the available debug spec. ISA extensions:

  • CPU_EXTENSION_RISCV_Sdext: set true to enable the RISC-V Sdext ISA extension (external debug support, required for the on-chip debugger)
  • CPU_EXTENSION_RISCV_Sdtrig: set true to enable the RISC-V Sdtrig ISA extension (trigger module)

✨ This PR also enhances the capabilities of the RISC-V trigger module (Sdtrig ISA extension), which can now also be used independently of the on-chip debugger. The trigger module can raise a machine-mode breakpoint exception when execution reaches a programmable address. A simple example program will be added to sw/example/demo_trigger_module.

@stnolting stnolting added enhancement New feature or request risc-v compliance Modification to comply with official RISC-V specs. HW hardware-related labels Dec 23, 2022
@stnolting stnolting self-assigned this Dec 23, 2022
* add explicit generics for "Sdext" and "Sdtrig" ISA extension
* make trigger module available for m-mode software
* provide more configuration options for the trigger module
When in debug-mode all PMP rules are ignored making the debugger have maximum access rights.
@stnolting stnolting marked this pull request as ready for review December 23, 2022 19:46
@stnolting stnolting merged commit 5a0d9dc into main Dec 24, 2022
@stnolting stnolting deleted the trigger_extension branch December 24, 2022 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request HW hardware-related risc-v compliance Modification to comply with official RISC-V specs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant