Skip to content

[rtl] Expose lockstep and shadow bus interface core signals to ibex_top#2365

Merged
nasahlpa merged 2 commits intolowRISC:masterfrom
nasahlpa:upstream_expose_lockstep_signals
Feb 10, 2026
Merged

[rtl] Expose lockstep and shadow bus interface core signals to ibex_top#2365
nasahlpa merged 2 commits intolowRISC:masterfrom
nasahlpa:upstream_expose_lockstep_signals

Conversation

@nasahlpa
Copy link
Member

@nasahlpa nasahlpa commented Feb 3, 2026

This enables users of Ibex to conduct additional security checks on integration-specific bus protocol conversion modules.

@nasahlpa nasahlpa force-pushed the upstream_expose_lockstep_signals branch from 9ab7f40 to be08541 Compare February 3, 2026 14:50
@nasahlpa nasahlpa force-pushed the upstream_expose_lockstep_signals branch from be08541 to f773efb Compare February 6, 2026 19:32
@vogelpi
Copy link
Contributor

vogelpi commented Feb 10, 2026

@nasahlpa , can you please update this PR? I think there should be a second commit here to optimize the data_wdata_intg handling.

This enables users of Ibex to conduct additional security checks on
integration-specific bus protocol conversion modules.

Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
We can remove the integrity portion of the write data as we anyways
compare the `data_wdata` in the lockstep comparison. This helps us
to save some area.

Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
@nasahlpa nasahlpa force-pushed the upstream_expose_lockstep_signals branch from f773efb to afd13a5 Compare February 10, 2026 12:17
Copy link
Contributor

@vogelpi vogelpi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @nasahlpa this looks great!

I've reviewed the code and there is no difference to the code I've reviewed in depth previously.

@nasahlpa nasahlpa added this pull request to the merge queue Feb 10, 2026
Merged via the queue into lowRISC:master with commit f79da72 Feb 10, 2026
12 checks passed
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.

2 participants