Skip to content

decoder: Ensure safe handling of illegal instructions by blocking all side effects#2384

Open
Anubhav-30 wants to merge 1 commit intolowRISC:masterfrom
Anubhav-30:fix/illegal-instruction-safe-handling
Open

decoder: Ensure safe handling of illegal instructions by blocking all side effects#2384
Anubhav-30 wants to merge 1 commit intolowRISC:masterfrom
Anubhav-30:fix/illegal-instruction-safe-handling

Conversation

@Anubhav-30
Copy link
Copy Markdown

This change strengthens the handling of illegal instructions in the decoder
by ensuring that no unintended side effects are triggered when an illegal
instruction is detected.

In addition to disabling write and control signals, this update explicitly:

  • disables register read enables (rf_ren_a_o, rf_ren_b_o)
  • blocks instruction cache invalidation (icache_inval_o)
  • forces CSR operations to a safe read-only mode

This prevents any unintended data access, control flow changes, or CSR
side effects from occurring due to partially decoded illegal instructions.

The change is non-intrusive and aligns with safe CPU design practices,
ensuring that illegal instructions are fully contained within the decoder
stage without affecting downstream pipeline behavior.

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