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

Update support for openpiton #1

Open
wants to merge 29 commits into
base: master
Choose a base branch
from

Conversation

Jbalkind
Copy link

@Jbalkind Jbalkind commented May 5, 2022

No description provided.

Jbalkind and others added 29 commits April 4, 2022 22:50
Signed-off-by: Guillaume Chauvon <guillaume.chauvon@thalesgroup.com>
* Add support for "high" counter CSRs in 32-bit mode

In 32bit mode MCYCLEH, MINSTRETH, CYCLEH, TIMEH and INSTRETH are
used to return the most significant 32-bit value of the counters
which are now always 64-bit wide.

Signed-off-by: Steffen Persvold <spersvold@gmail.com>

* Enable writing of MCYCLEH and MINSTRETH CSRs

Signed-off-by: Steffen Persvold <spersvold@gmail.com>
* cva6_synth.tcl: fix set_input_delay and set_output_delay tc_sram paths
* ariane_tb.cpp;.sv: [Fix tc_srams] change path for user memory preload

Signed-off-by: Guillaume Chauvon <guillaume.chauvon@thalesgroup.com>
Co-authored-by: Jean-Roch Coulon <jean-roch.coulon@thalesgroup.com>
Signed-off-by: Guillaume Chauvon<guillaume.chauvon@thalesgroup.com>
Several format cleanings:
- split load_store_unit.sv to create lsu_bypass.sv
- add several "begin" and "end"
…g 3 operands (openhwgroup#925)

* re_name.sv: add condition related to CVXIF to rename 3rd operand
* cv32a60x_pkg.sv: set CVXIFEn to 1
* Create bug.yaml

* Update bug.yaml - Case consistency

* Spelling mistakes
Fixes openhwgroup#906

According to the spec:
> If accessing pte violates a PMA or PMP check, raise an access-fault
> exception corresponding to the original access type.

Found by @Phantom1003 and @ProjectDimlight

Signed-off-by: Moritz Schneider <moritz.schneider@inf.ethz.ch>
* Removing CVA6-SDK from task.yaml

* Associated PRs in task.yaml

This commit provide a new textarea to fill links to PRs used to complete the task.

* Create bug.yaml

* Update bug.yaml - Case consistency

Co-authored-by: JeanRochCoulon <jean-roch.coulon@thalesgroup.com>
unlike other instructions with minor opcode == PRIV,
SFENCE.VMA do not check for rs1 != 0.
Only check for rd !=0 to raise illegal instruction

Signed-off-by: Guillaume Chauvon <guillaume.chauvon@thalesgroup.com>
…enhwgroup#939)

It avoids that result from cvxif and illegal instruction use wb bus at the same time
…enhwgroup#940)

see The RISC-V Instruction Set Manual Volume II: Privileged Architecture
Version 20211203

3.1.1 Hardware Performance Monitor
All counters should be implemented, but a legal implementation is to make
both the counter and its corresponding event selector be read-only 0.

Signed-off-by: André Sintzoff <andre.sintzoff@thalesgroup.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet