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.

Sign up for GitHub

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

Jump to bottom

[TASK] Fix riscv-tests with virtual memory on tandem #2605

Closed
1 task done
valentinThomazic opened this issue Nov 15, 2024 · 2 comments
Closed
1 task done

[TASK] Fix riscv-tests with virtual memory on tandem #2605

valentinThomazic opened this issue Nov 15, 2024 · 2 comments
Labels
CV64A6 Part: 64bits configuration notCV32A65X It is not an CV32A65X issue Status:Stale Issue or PR is stale and hasn't received any updates.

Comments

@valentinThomazic
Copy link
Contributor

Is there an existing CVA6 task for this?

  • I have searched the existing task issues

Task Description

Fix the riscv-tests with virtual memory enabled on cv64a6_imafdc_sv39 with tandem enabled.

Required Changes

Make these tests pass :

  • rv64uf-v-fcmp
  • rv64uf-v-fdiv
  • rv64uf-v-fmin
  • rv64ud-v-move

Current Status

The tests are failing because they are more than 5 mismatches, mostly CSR-related on csrrw instructions

Risks

No response

Prerequisites

No response

KPI (KEY Performance Indicators)

Number of tests passing on riscv-tests-v testlist on cv64a6_imafdc_sv39 configuration and testharness testbench

Description of Done

All tests passing

Associated PRs

No response
@valentinThomazic valentinThomazic added the CV64A6 Part: 64bits configuration label Nov 15, 2024
@JeanRochCoulon JeanRochCoulon added the notCV32A65X It is not an CV32A65X issue label Nov 18, 2024
@valentinThomazic valentinThomazic mentioned this issue Nov 18, 2024
Merged
JeanRochCoulon pushed a commit that referenced this issue Nov 18, 2024
@valentinThomazic
Disable tandem on riscv-tests-v tests (#2609)
e7f27c1 
* Disable tandem on riscv-tests-v testlist
* More relevant error message on report tandem script
Related issue: #2605
@github-actions GitHub Actions
Copy link
Contributor

👋 Hi there!

This issue seems inactive. Need more help? Feel free to update us. If there are no updates within the next few days, we'll go ahead and close this issue. 😊

@github-actions github-actions bot added the Status:Stale Issue or PR is stale and hasn't received any updates. label Dec 19, 2024
@JeanRochCoulon JeanRochCoulon removed the Status:Stale Issue or PR is stale and hasn't received any updates. label Dec 19, 2024
@github-actions GitHub Actions
Copy link
Contributor

👋 Hi there!

This issue seems inactive. Need more help? Feel free to update us. If there are no updates within the next few days, we'll go ahead and close this issue. 😊

@github-actions github-actions bot added the Status:Stale Issue or PR is stale and hasn't received any updates. label Jan 19, 2025
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
CV64A6 Part: 64bits configuration notCV32A65X It is not an CV32A65X issue Status:Stale Issue or PR is stale and hasn't received any updates.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JeanRochCoulon @valentinThomazic