pulp-platform / FlooNoC

A Fast, Low-Overhead On-chip Network
Apache License 2.0
135 stars 21 forks source link

Issue with DMA Testbench in FlooNoC Examples #34

Closed Abdul-muheet-ghani closed 2 months ago

Abdul-muheet-ghani commented 7 months ago

I am currently working with my team on FlooNoC and running the testbenches provided in the FlooNoC examples. We have noticed that all examples are running correctly except for the DMA examples. It seems that the environmental configuration is correct, but there is an issue with the DMA testbench itself or may be the commit of pulp repo.

fischeti commented 7 months ago

Hi Abdul, Which testbench exactly are you having problems with? the DMA testbenches have DMA models which require job files to tell what transactions to generate and where to. There is a script. The DMA mesh testbench is also part of the internal Gitlab CI, and you can maybe check how it's done there (see here)

Abdul-muheet-ghani commented 7 months ago

When we run any example like make run-sim VSIM_TB_DUT=tb_floo_axi_chimney or tb_floo_router, it runs successfully, and data along with stimulus is generated in the waveform. However, when attempting to run any DMA example, such as make run-sim VSIM_TB_DUT=tb_floo_dma_nw_chimney or any other DMA testbench, it seems to compile correctly. However, upon opening the waveform in QuestaSim, we encounter an error of this sort.

Screenshot from 2024-03-23 04-51-17

Could you please provide some guidance on this matter or suggest a specific command? Since we are new to this project, we are not familiar with how scripts work, and any specific commands apart from make run-sim VSIM_TB_DUT=tb_floo_dma_nw_chimney would be helpful.

fischeti commented 7 months ago

What version of QuestaSim are you using? Because I cannot reproduce your issue on my machine.

Abdul-muheet-ghani commented 7 months ago

This is the version.

QuestaSim-64 qrun 2021.2_1 Utility 2021.05 May 15 2021

fischeti commented 7 months ago

I can reproduce the error with this version, but I don't have a fix for it. I am using a newer version of QuestaSim (2023.4) which works fine for me, so try that one if you have it installed.

Abdul-muheet-ghani commented 7 months ago

Thank you for figuring out the problem. I will try with this version for sure. I don't have this version right now, but I will install it.