search

Xilinx / Vitis-HLS-Introductory-Examples

Other
599 stars 158 forks source link

predecessor and read issue in dataflow #1

Closed aeeshms closed 2 years ago

aeeshms commented 4 years ago

When compiling convolution kernel following warning is reported, how to overcome this ?

Process Loop_HConvH_proc5 has both a predecessor and reads an input from its caller (see the GUI dataflow viewer). This may lead to lower throughput. Consider copying this input via a predecessor process.

In doing any dataflow design this warning is coming although it was not in 2019.1.

Test_delay = 2070607@2.342 ns

define TEST_IMG_ROWS 135

define TEST_IMG_COLS 240

reports timing violations, how to improve it.

ramanan-radhakrishnan commented 2 years ago

What this means is that both the previous process and the current process may get sequentialized because they are reading from the input interface. If you perform both reads in the first function and then pass this through to the second function, then these functions can be executed in an overlapped fashion. This is what is meant by "Consider copying this input via a predecessor process".