Add ACT4 Configuration for CV32a65x#1176
Conversation
|
How do we know if the unit is bits or bytes? Does the RTL confirm bits? 16 bytes is an unusually short line.
… On Apr 2, 2026, at 5:06 AM, arshiarifat-10x ***@***.***> wrote:
@arshiarifat-10x commented on this pull request.
In config/cores/cva6/cv32a65x/sail.json <#1176?email_source=notifications&email_token=AR4AA36RUT23VIFTXYRTRO34TZJVVA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIMBVGAZDCOJUHAZ2M4TFMFZW63VKON2WE43DOJUWEZLEUVSXMZLOOSXXA4S7OJSXM2LFO5PWG3DJMNVQ#discussion_r3027652300>:
> + "write_idempotent": true,
+ "misaligned_fault": "AlignmentFault",
+ "reservability": "RsrvNone",
+ "supports_cbo_zero": false
+ },
+ "include_in_device_tree": true
+ }
+ ]
+ },
+ "platform": {
+ "vendorid": 1538,
+ "archid": 3,
+ "impid": 0,
+ "hartid": 0,
+ // Cache block size, specified as a power of 2.
+ "cache_block_size_exp": 4,
In the design document https://docs.openhwgroup.org/projects/cva6-user-manual/04_cv32a65x/design/design.html#parameter-configuration <x-msg://54/url> the line width is 128 bits so 128/8 = 16 bytes.
—
Reply to this email directly, view it on GitHub <#1176?email_source=notifications&email_token=AR4AA35UBFDPX5GE333WVBT4TZJVVA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIMBVGAZDCOJUHAZ2M4TFMFZW63VKON2WE43DOJUWEZLEUVSXMZLOOS6XA4S7OJSXM2LFO5PW433UNFTGSY3BORUW63TTL5RWY2LDNM#discussion_r3027652300>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AR4AA33WFL3CXRNXUX5DYY34TZJVVAVCNFSM6AAAAACXJFNXXOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DANJQGIYTSNBYGM>.
You are receiving this because you are subscribed to this thread.
|
|
@davidharrishmc becasue in the documentation, in case of CacheSize the unit is explicitly mentioned as in bytes, for LineWidth there are no units which we assume to be in bits (otherwise bytes would have mentioned there too). |
|
I would feel more comfortable confirming against the RTL. A 16-byte cache
line is unusually small.
…On Thu, Apr 2, 2026 at 5:37 AM arshiarifat-10x ***@***.***> wrote:
*arshiarifat-10x* left a comment (riscv/riscv-arch-test#1176)
<#1176?email_source=notifications&email_token=AR4AA33FDQIHHD3Y2ZCAXUT4TZNITA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMJXG42TOMJQGYZKM4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJNLQOJPWG33NNVSW45C7N5YGK3S7MNWGSY3L#issuecomment-4177571062>
@davidharrishmc <https://github.com/davidharrishmc> becasue in the
documentation, in case of CacheSize the unit is explicitly mentioned as in
bytes, for LineWidth there are no units which we assume to be in bits
(otherwise bytes would have mentioned there too).
—
Reply to this email directly, view it on GitHub
<#1176?email_source=notifications&email_token=AR4AA33FDQIHHD3Y2ZCAXUT4TZNITA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMJXG42TOMJQGYZKM4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJNLQOJPWG33NNVSW45C7N5YGK3S7MNWGSY3L#issuecomment-4177571062>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AR4AA36RRC3J23HGSMNN6ZT4TZNITAVCNFSM6AAAAACXJFNXXOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DCNZXGU3TCMBWGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
@arshiarifat-10x the most recent version of this PR seems to have checked in quite a few files that I assume were not intended to be included (the UDB submodule, multiple generated tests, etc.). |
…cv#1289) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [renovatebot/pre-commit-hooks](https://redirect.github.com/renovatebot/pre-commit-hooks) | repository | minor | `43.126.0` → `43.127.0` | Note: The `pre-commit` manager in Renovate is not supported by the `pre-commit` maintainers or community. Please do not report any problems there, instead [create a Discussion in the Renovate repository](https://redirect.github.com/renovatebot/renovate/discussions/new) if you have any questions. --- ### Release Notes <details> <summary>renovatebot/pre-commit-hooks (renovatebot/pre-commit-hooks)</summary> ### [`v43.127.0`](https://redirect.github.com/renovatebot/pre-commit-hooks/compare/43.126.0...43.127.0) [Compare Source](https://redirect.github.com/renovatebot/pre-commit-hooks/compare/43.126.0...43.127.0) </details> --- ### Configuration 📅 **Schedule**: (in timezone America/Los_Angeles) - Branch creation - At any time (no schedule defined) - Automerge - At any time (no schedule defined) 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/riscv/riscv-arch-test). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4xMjMuOCIsInVwZGF0ZWRJblZlciI6IjQzLjEyMy44IiwidGFyZ2V0QnJhbmNoIjoiYWN0NCIsImxhYmVscyI6WyJkZXBlbmRlbmNpZXMiXX0=--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Jordan Carlin <jordanmcarlin@gmail.com> Signed-off-by: arshiarifat-10x <arshia.rifat@10xengineers.ai>
@davidharrishmc |
jordancarlin
left a comment
jordancarlin
left a comment
There was a problem hiding this comment.
@arshiarifat-10x Just to confirm, all of the tests are passing on CVA6 now? If so, this looks ready to merge.
|
@jordancarlin |
3 participants
No description provided.