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

Attempt access on every byte of an unaligned access #23

Open
wants to merge 1 commit into
base: ibex_cosim
Choose a base branch
from

Conversation

GregAC
Copy link

@GregAC GregAC commented Nov 16, 2022

To match Ibex behaviour this updates the spike unaligned load/store behaviour to try accessing all bytes rather than stopping as soon as an error is seen. If one or more errors are seen whilst accessing these bytes the first one is thrown.

To match Ibex behaviour this updates the spike unaligned load/store
behaviour to try accessing all bytes rather than stopping as soon as an
error is seen. If one or more errors are seen whilst accessing these
bytes the first one is thrown.
@GregAC
Copy link
Author

GregAC commented Nov 17, 2022

I think we'll hold off on merging this for now, we also need changes on the co-sim side for it to work and some more investigation is needed there lowRISC/ibex#1948

@Boyifyoudontstop
Copy link

I tried applying these changes myself and ran into the same issue, are there any updates on this that allow spike to continue running after an exception?

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