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.

Sign up for GitHub

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

Jump to bottom

Only implement one solution for native triggers. #1793

Merged
merged 4 commits into from
Sep 10, 2024
Merged

Only implement one solution for native triggers. #1793

merged 4 commits into from
Sep 10, 2024

Commits on Sep 5, 2024

  1. Work if tcontrol doesn't exist.

    @rtwfroody
    rtwfroody committed Sep 5, 2024
    Configuration menu
    Copy the full SHA
    9c5a20f View commit details
    Browse the repository at this point in the history

  2. Make allow_action() take proc instead of state

    @rtwfroody
    rtwfroody committed Sep 5, 2024
    Configuration menu
    Copy the full SHA
    4abd669 View commit details
    Browse the repository at this point in the history

Commits on Sep 9, 2024

  1. triggers: Move allow_action() into common_match() 

    They are always called together, and now we get the previous privilege
behavior in both.
    @rtwfroody
    rtwfroody committed Sep 9, 2024
    Configuration menu
    Copy the full SHA
    451a7dc View commit details
    Browse the repository at this point in the history

  2. Only implement one solution for native triggers. 

    When S-mode is present, use option 1 (disable triggers in M-mode unless
MIE is set) from the Debug Spec. When S-mode is not present, use option
2 (implement mte and mpte bits in tcontrol).

See discussion in riscv-software-src#1777.
    @rtwfroody
    rtwfroody committed Sep 9, 2024
    Configuration menu
    Copy the full SHA
    0703b44 View commit details
    Browse the repository at this point in the history