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

Increase number of jobs in riscof to speedup building. #671

Merged
merged 1 commit into from
Mar 13, 2024

Conversation

Karl-Han
Copy link
Contributor

No description provided.

@rosethompson
Copy link
Contributor

rosethompson commented Mar 13, 2024

Is there a way we can make the number of threads platform dependent, similar to how make does --jobs?

@Karl-Han
Copy link
Contributor Author

I don't think so as I check the help from RISCOF:

$ riscof run --help
    INFO | ****** RISCOF: RISC-V Architectural Test Framework 1.25.3 *******
    INFO | using riscv_isac version : 0.18.0
    INFO | using riscv_config version : 3.14.3
Usage: riscof run [OPTIONS]

  Run the tests on DUT and reference and compare signatures

Options:
  --suite PATH     Path to the custom Suite Directory.  [required]
  --env PATH       Path to the env directory for the suite.  [required]
  --config PATH    The Path to the config file. [Default=./config.ini]
  --work-dir PATH  Path to the work directory.
  --no-browser     Do not open the browser for showing the test report.
  --dbfile PATH    The Path to the database file. NOTE: This argument is
                   mutually exclusive with  arguments: [testfile].
  --testfile PATH  The Path to the testlist file. NOTE: This argument is
                   mutually exclusive with  arguments: [dbfile].
  --no-ref-run     Do not run tests on Reference
  --no-dut-run     Do not run tests on DUT
  --no-clean       Do not clean work directory(if exists).
  --help           Show this message and exit.

And I think it should be done with config.ini since it is the main input for RISCOF.

Maybe I will put all the configuration in one place, for example, .env with python to configure the all the tools, or global environment.

@davidharrishmc
Copy link
Contributor

davidharrishmc commented Mar 13, 2024 via email

@davidharrishmc davidharrishmc merged commit 5e3ff3e into openhwgroup:main Mar 13, 2024
1 check passed
@davidharrishmc
Copy link
Contributor

38 minutes on chips before this PR.
12 minutes after.
Big improvement!

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