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

Updates to Zc handling in IF stage #612

Merged

Conversation

silabs-oysteink
Copy link
Contributor

  • No prefetch_ready handshake on tbljmp. Table jumps will either generate a pc_set and kill_if when they reach ID, or they will get killed (which also implies kill_if and a new pc_set).
  • Sequencer will not get valid_i on trigger matches. Matched instruction will be first && last and enter debug before first operation is retired.
  • Deasserting tbljmp for trigger matches (similar to how all write enables are deassert in ID for trigger matches).

Above three changes are needed for E40S PC hardening logic, and are added to E40X for code consistency.

Not SEC clean due to different prefetch_ready which makes the instruction bus differ in SEC.

Signed-off-by: Oystein Knauserud [email protected]

- No prefetch_ready handshake on tbljmp. Table jumps will either generate a pc_set and kill_if when they reach ID, or they will get killed (which also implies kill_if and a new pc_set).
- Sequencer will not get valid_i on trigger matches. Matched instruction will be first && last and enter debug before first operation is retired.
- Deasserting tbljmp for trigger matches (similar to how all write enables are deassert in ID for trigger matches).

Above three changes are needed for E40S PC hardening logic, and are added to E40X for code consistency.

Not SEC clean due to different prefetch_ready which makes the instruction bus differ in SEC.

Signed-off-by: Oystein Knauserud <[email protected]>
@silabs-oysteink silabs-oysteink added the Component:RTL For issues in the RTL (e.g. for files in the rtl directory) label Jul 7, 2022
@silabs-oysteink
Copy link
Contributor Author

Self merging due to vacation season.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant