Cv32a65x-act4 - #3265
Conversation
cainria
left a comment
There was a problem hiding this comment.
Please add a description to this pull request so that we can understand the rationale of these changes and discuss them.
|
Hello @arshiarifat-10x |
|
@JeanRochCoulon |
|
As specified in CONTRIBUTING.md, can you give me the right to rebase your branch. |
|
@JeanRochCoulon |
cainria
left a comment
There was a problem hiding this comment.
Shouldn't we add a README to document how to use this new testing framework?
Co-authored-by: Côme <come.allart@inria.fr>
Co-authored-by: Côme <come.allart@inria.fr>
Co-authored-by: Côme <come.allart@inria.fr>
Co-authored-by: Côme <come.allart@inria.fr>
Co-authored-by: Côme <come.allart@inria.fr>
Co-authored-by: Côme <come.allart@inria.fr>
Co-authored-by: Côme <come.allart@inria.fr>
Co-authored-by: Côme <come.allart@inria.fr>
|
Currently, test suites are cloned on demand when the corresponding tests are executed. For instance, there is already the script Could you consider to reuse this script? |
|
The ACT4 certification test suite does not work that way @ASintzoff. ACT4 is not a library of tests, it is a test-generator. The process of generating these tests is rather long and complex so for other CORE-V cores we are employing the following strategy:
|
|
👋 Hi there! This pull request seems inactive. Need more help or have updates? Feel free to let us know. If there are no updates within the next few days, we'll go ahead and close this PR. 😊 |
|
Hi @ASintzoff, we should discuss how to move forward on this. |
|
Hello @arshiarifat-10x Why core-v-verif is updated ? This PR should not modify the core-v-verif hash |
|
@AlexChenIC The Verible repo cannot be fetched. Do you have an idea howto fix it ? |
|
❌ failed run, report available here. |
|
Hello @arshiarifat-10x we are ready to merge. But the CI is red due to an installation issue. |
|
I add it here for visibility: as mentioned in #3265 (comment) this PR adds a requirement to |
|
@AlexChenIC What about adding the job in your Github action CI ? This will reduce engineering work to directly push it into the final targeted CI. It would be more efficient. Do you share this view ? |
Hi @JeanRochCoulon, yes, I share this view. I think that ACT4 would be a good fit for the Tier CI as a dedicated Tier 2 job for CV32A65X. I should be able to take on this integration. I’ll start by evaluating the workflow. Based on my initial assessment, the integration looks feasible |
|
Hi! Any news about this PR ? |
|
@cainria README was pending i added that. I hope the CI issue is resolved. |
|
Since @AlexChenIC mentioned taking on the Tier-2 ACT4 CI integration, should I wait for that, or should I add the uv installation support on my side? @JeanRochCoulon |
This PR integrates the RISC-V Architectural Compatibility Test (ACT) v4.0 framework for the CV32A65x configuration of the CVA6 core, enabling architecture compliance test generation and execution within the CVA6 simulation environment.