pulp-platform / chimera

Other
9 stars 1 forks source link

add kul sw test function #36

Open xiaoling-yi opened 2 months ago

xiaoling-yi commented 2 months ago

In this PR, we add the software test for the KUL cluster. More specifically, we add:

Scheremo commented 2 months ago

Hi Xiaoling, Thanks for your contribution! I have a few questions before going further in-depth with reviewing. 1) I see you use the DMA in some circumstances. Would it be possible to instead preload all your data in L1? This reduces turn around time by a lot. 2) I didn't see you setting the stack pointer; does this test work on your end?

Best, Moritz

xiaoling-yi commented 2 months ago

Hi Moritz,

I see you use the DMA in some circumstances. Would it be possible to instead preload all your data in L1? This reduces turn around time by a lot.

What do you mean by preload data? Is it putting all the DMA functions together at the beginning of the test function kul_cluster_sw_test?

I didn't see you setting the stack pointer; does this test work on your end?

I set the stack pointer just now to the end of the KUL cluster TCDM (size = 128KB) address. Is it the right way to set it before the offloadToCluster call? Yes, I tested it on the snax cluster level but not the Chimera level yet.

Thanks!

Best regards, Xiaoling