Skip to content

[codex] Port Jingtao V2 RVV model#901

Draft
jensen-yan wants to merge 5 commits into
xs-devfrom
codex/tencent-vector-rebase
Draft

[codex] Port Jingtao V2 RVV model#901
jensen-yan wants to merge 5 commits into
xs-devfrom
codex/tencent-vector-rebase

Conversation

@jensen-yan

Copy link
Copy Markdown
Collaborator

Motivation

This draft PR brings in the latest Tencent/Jingtao V2 RVV model as a reference branch for evaluating vector modeling against the current Kunminghu V3 direction.

Changes

  • Port the Jingtao V2 RVV model onto xs-dev.
  • Move the vector ISA generation path to the new banked RVV model.
  • Add VLEN=256, DPLEN=128, VecBuf, banked vector register handling, and related O3 rename/regfile/scheduler support.
  • Add design documentation for both the Jingtao V2 RVV port and the Kunminghu V3 RVV comparison.
  • Enable gem5-vector.yml on this branch and add a no-difftest vector CI step.
  • Fix an O3 pinned-write wakeup issue by marking final pinned writes bypass-ready before waking dependents.

Validation

  • scons build/RISCV/gem5.opt --linker=gold -j64 --rvv-impl=simple passed locally.
  • Local no-difftest RVV samples passed, including vmsltu.vx-2, vmsbc.vxm-1, vlsseg7e64.v-0, and vluxei8.v-0.
  • CI run: https://github.com/OpenXiangShan/GEM5/actions/runs/27185002034
    • Build GEM5 opt: passed
    • run vector test without difftest: passed
    • run vector test: failed

Known Issues

The remaining vector CI failure is in difftest. Local reproduction shows the first vsetvli diverges because this branch models VLEN=256, while the current NEMU ref source/config uses VLEN=128. This branch should either use a matching 256-bit VLEN NEMU ref for Jingtao validation, or be treated as a no-difftest reference branch until the Kunminghu V3 VLEN=128 direction is implemented.

Change-Id: I0fd4745860581f4e4cf59f075eed62d5624e25bf
Change-Id: Iaa28e4564cc8e3c60e589e2e60b9eb5111dd5152
Change-Id: I746687c3aafc8602ad9acefdb8015ee2b89936db
Change-Id: I22861ff52ff3e974d61a2e1d6c4ea51c1637fcce
@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 406c209a-5ac3-4005-bf12-c35b4cd672b1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/tencent-vector-rebase

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

🚀 Coremark Smoke Test Results

Branch IPC Change
Base (xs-dev) 2.3168 -
This PR 2.3168 ➡️ 0.0000 (0.00%)

✅ Difftest smoke test passed!

@jensen-yan

Copy link
Copy Markdown
Collaborator Author

/nfs/home/yanyue/Gem5-jiezi_v2_merge_kmhv3_202606

Change-Id: Iad8d7e0176d24cc2f620ecec5762a16030909e3a
@github-actions

Copy link
Copy Markdown

🚀 Coremark Smoke Test Results

Branch IPC Change
Base (xs-dev) 2.3155 -
This PR 2.3155 ➡️ 0.0000 (0.00%)

✅ Difftest smoke test passed!

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.

1 participant