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

Provide only one way to run simulation #1505

Merged
merged 2 commits into from
Oct 6, 2023
Merged

Conversation

JeanRochCoulon
Copy link
Contributor

@JeanRochCoulon JeanRochCoulon commented Oct 6, 2023

README.md how to run a simulation by executing Makefile commands, for instance 'make verilate'. This way does not work because additional variables need to be setup before executing these commands, for instance TARGET=cv64a6_imafdc_sv39. Many GitHub issues request support as #1489. This PR removes the getting start related to these commands to keep the way which is support that is to say using cva6.py command.

@MikeOpenHWGroup @zarubaf can you approve ?

README.md how to run a simulation by executing Makefile commands, for instance 'make verilate'. This way does not work because additional variables need to be setup before executing these commands, for instance TARGET=cv64a6_imafdc_sv39. Many GitHub issues request support #1489. This PR removes the getting start related to these commands to keep the way which is support that is to say using cva6.py command.
@hhhsiang
Copy link

hhhsiang commented Oct 6, 2023

@JeanRochCoulon, is there any documentation or README available that explains how to use cva6.py for running simulations? Specifically, if I want to test a single instruction like rv64ui-v-add, what command should I use, in which directory should I run these commands, and what initial settings or environment paths need to be configured? Your assistance is greatly appreciated!

@JeanRochCoulon
Copy link
Contributor Author

JeanRochCoulon commented Oct 6, 2023

Recommandations are in README.md of CVA6 repository. Have you read it before ?
If you would like to run rv64ui-v-add, modify the smoke-tests.sh script in that sense. cva6.py --help produces the user manual.

README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@zarubaf zarubaf left a comment

Choose a reason for hiding this comment

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

👍

Co-authored-by: Florian Zaruba <[email protected]>
@JeanRochCoulon JeanRochCoulon merged commit bb80b3f into master Oct 6, 2023
12 checks passed
@JeanRochCoulon JeanRochCoulon deleted the JeanRochCoulon-patch-1 branch October 6, 2023 07:45
@hhhsiang
Copy link

hhhsiang commented Oct 6, 2023

Recommandations are in README.md of CVA6 repository. Have you read it before ? If you would like to run rv64ui-v-add, modify the smoke-tests.sh script in that sense. cva6.py --help produces the user manual.

@JeanRochCoulon, I had previously read the README.md, but it wasn't as explicit in explaining how to do it as shown in the screenshot below:
image

However, I have a better understanding now. If I want to test any specific instruction, I can simply add it to the smoke-tests.sh script and then execute it, correct? Like the following example:

image

Additionally, I'm a bit unsure about your statement:

cva6.py --help produces the user manual.

I tried running both cva6.py --help and make cva6.py --help, but nothing happened:

image

@JeanRochCoulon
Copy link
Contributor Author

Indeed cva6.py --help produces the list of options (maybe the words "User manual" was to much in relaitn to this list).

@hhhsiang
Copy link

hhhsiang commented Oct 6, 2023

@JeanRochCoulon, I apologize for bothering you again.

If I wish to obtain a .vcd file by verilator or VCS for debugging purposes, how can I achieve that? The smoke-tests.sh script does not seem to generate it, and I couldn't find any information about this in the README.md.

@JeanRochCoulon
Copy link
Contributor Author

@zchamski Can you help on this point ?

cfuguet pushed a commit to cfuguet/cva6 that referenced this pull request Oct 10, 2023
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.

4 participants