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

[BUG] <misalgined access> #267

Open
Jiahua-Gong opened this issue May 21, 2024 · 5 comments
Open

[BUG] <misalgined access> #267

Jiahua-Gong opened this issue May 21, 2024 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@Jiahua-Gong
Copy link

Bug Description

obi2ahbm_adapter.sv.. if byte access 24bits , bridge don't split 2 trans

@MikeOpenHWGroup MikeOpenHWGroup added bug Something isn't working question Further information is requested labels May 21, 2024
@MikeOpenHWGroup
Copy link
Member

Thanks for creating this issue @shanshuixiangyi.

Hi @davideschiavone, I have added both bug and question labels to this issue as I am not sure if this feature is supported by the CVE2. If it is not, then the OBI2AHB bridge does not need to support this. If that is the case, we should clearly document the behavior of the bridge.

@davideschiavone
Copy link

hi @MikeOpenHWGroup and @shanshuixiangyi - the CVE2 supports misaligned on the OBI interface, so it is very well possible that the bridge has this bug -

@MikeOpenHWGroup MikeOpenHWGroup removed the question Further information is requested label May 21, 2024
@MikeOpenHWGroup
Copy link
Member

Thanks @davideschiavone, I have removed the question label and this is now officially a bug! 😉

Hi @shanshuixiangyi, do you have an example test-program to exercise this?

@Jiahua-Gong
Copy link
Author

图片
risc-v load store uint can transmit red select in the pitcture. but bridge can't support this byte condition. I have modified the rtl to support this condition. I am learning how to submit code and then make a fix commit using Git.

@MikeOpenHWGroup
Copy link
Member

I am learning how to submit code and then make a fix commit using Git.

Thank you very much @Jiahua-Gong! I notice that the CONTRIBUTING file for this repo is not complete (I will fix that). In the meantime, please review CONTRIBUTING.md in CORE-V-VERIF, especially the Contributor Agreement section.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants