feat(fpga): support PCIe XDMA based difftest #453
Merged
+840
−5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Where does the source code come from? Who are the authors? |
xiaokamikami
force-pushed
the
pcie-difftest
branch
from
bdd099c to
01109df
Compare
lewislzh
added a commit
that referenced
this pull request
Sep 2, 2024
xiaokamikami
force-pushed
the
pcie-difftest
branch
10 times, most recently
from
38f0039 to
9b1bf1f
Compare
|
您好，想问一下，目前difftest环境和encore能结合起来吗？我看到encore的流程里在生成snapshot，可以进一步在仿真环境里运行，这里是否可以用difftest来跑？另外，目前fpga上的difftest环境运行时速度跟encore比哪个更快呢？ |
|
这套框架怎么用起来？ 需要在FPGA流程里使用吗？ |
你好，这套系统就是将类似encore的结构做到difftest环境，这套系统目前能传输大约7GB/S的数据，具体运行频率 需要视difftest模式情况而计算，因为不同模式下的difftest产生的数据量不一样 |
你好，暂时还没有完全适配好，后续会出文档的 |
xiaokamikami
force-pushed
the
pcie-difftest
branch
3 times, most recently
from
3ba5887 to
2c3e0a2
Compare
xiaokamikami
force-pushed
the
pcie-difftest
branch
from
c3d5c14 to
64f8928
Compare
xiaokamikami
force-pushed
the
pcie-difftest
branch
7 times, most recently
from
c674bc8 to
756742c
Compare
…g functions according to CPU
…t to repair the processing of data packets and pass the fpga verification
poemonsense
reviewed
Jan 9, 2025
fpga.mk
Outdated
| $(FPGA_TARGET): $(FPGA_CXXFILES) | $(FPGA_TARGET_DIR) | ||
| $(CXX) $(FPGA_CXXFLAGS) $(FPGA_CXXFILES) -o $@ $(FPGA_LDFLAGS) | ||
|
|
||
| $(FPGA_TARGET_DIR): |
There was a problem hiding this comment.
This is useless now. Please remove it
src/test/csrc/common/ram.h
Outdated
| @@ -112,6 +112,9 @@ class SimMemory { | |||
| uint64_t get_size() { | |||
| return memory_size; | |||
| } | |||
| uint64_t get_load_img_size() { | |||
| return get_img_size(); | |||
| } | |||
There was a problem hiding this comment.
This can be removed now using the public get_img_size method
xiaokamikami
force-pushed
the
pcie-difftest
branch
from
6d8ece3 to
be80b04
Compare
poemonsense
approved these changes
Jan 9, 2025
|
OK, I will review it tomorrow. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When using fpga to carry out difftest, difftest will read out the architecture related information from the dma interface as a host program, and check the register on the PC side.
Before use, the computer needs to install the driver: https://github.com/Xilinx/dma_ip_drivers
By default, single-channel DMA is used to run. When multi-channel DMA is enabled, multi-thread is used to receive data packets. After reorganization and sorting, other processes are the same as single-channel，max size DMA_CHANNELS=4.
usage
make fpga-build FPGA=1 DMA_CHANNELS=1, run:sudo ./fpga-host --diff nemu.so -i workload.bin.The idea of this function comes from the revision of encore. Thanks to the support of Teacher Shi Kan and Xu Shuoxiang for providing the XDMA initialization code (f7f0f8b), and Zhang Shoulin for helping debug the FPGA
Co-authored-by: fengkehan [email protected]
Co-authored-by: xushuoxiang [email protected]
Co-authored-by: zhangshoulin [email protected]