Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BLAS multikernel benchmark example doesn't work on U250 platform #193

Open
fiannone opened this issue Jan 3, 2024 · 1 comment
Open

Comments

@fiannone
Copy link

fiannone commented Jan 3, 2024

I'd like to inform you that the VITIS Library example of GEMM multi kernels fail using 4 and 2 kernels targeting the Alveo U250 board compiling with VITIS 2022.2 and 2023.1 (platform xilinx_u250_gen3x16_xdma_4_1_202210_1) . The same example using target the hw_emu works whilst the target hw fails with the following error:

21:18:39] Run vpl: FINISHED. Run Status: impl ERROR

===>The following messages were generated while Compiling (synthesis checkpoint) kernel/IP: ulp_m01_regslice_3 Log file: /afs/enea.it/fra/user/iannone/.Xilinx/Vitis/2022.2/Vitis_Libraries/BLAS_bench/L3/benchmarks/gemm/memKernel/_x_temp.hw.xilinx_u250_gen3x16_xdma_4_1_202210_1/link/vivado/vpl/prj/prj.runs/ulp_m01_regslice_3_synth_1/runme.log :
ERROR: [VPL 17-356] Failed to install all user apps.

===>The following messages were generated while processing /afs/enea.it/fra/user/iannone/.Xilinx/Vitis/2022.2/Vitis_Libraries/BLAS_bench/L3/benchmarks/gemm/memKernel/_x_temp.hw.xilinx_u250_gen3x16_xdma_4_1_202210_1/link/vivado/vpl/prj/prj.runs/impl_1 :
ERROR: [VPL 30-487] The packing of instances into a set of CLBs defined by a pblock constraint could not be obeyed. There are a total of 25680 CLBs in the pblock, of which 20686 CLBs are available, however, the unplaced instances require 23494 CLBs. The unavailable CLBs are either taken by placed instances or are blocked due to exclude placement constraints. Please analyze your design to determine if the pblock can be resized or the number of LUTs, FFs, and/or control sets can be reduced.

Number of control sets and instances constrained to the pBlock
Control sets: 2885
Luts: 232127 (combined) 265463 (total), available capacity: 205440
Flip flops: 365122, available capacity: 410880
NOTE: each CLB can only accommodate up to 4 unique control sets so FFs cannot be packed to fully fill every CLB

To attempt placement at higher effort levels at the expense of runtime, please use the following tcl command, setting the value of limit to 2000 or more.
set_param place.sliceLegEffortLimit limit

My feeling is that is very serious as an example developed by Xilinx developers doesn't work. It's better remove it from GitHub repository in order to avoid my wasting time. Furthermore I'd like to use the tcl command to setting a new limit of place.sliceLegEffortLimit but I don't know how to open a tclsh consolle in the makefile parameters of the GEMM Bias Vitis library example.

Finally I'd like to inform you that I and my Colleague Paolo are involving in the EUROHPC TEXTROSSA project and we are going to deliver a report on which we'll inform the developers EUROHPC community about the bad support by AMD Xilinx on an example developed by AMD that doesn't work.

@afzalxo
Copy link

afzalxo commented Mar 20, 2024

I would like to agree with the gentleman above.

vt-lib-support pushed a commit that referenced this issue Nov 29, 2024
b0c9240 Merge pull request #195 from FaaSApps/next
56df2a0 Merge pull request #194 from ryanw/next
e0f6dd5 Update README.md
1b91b17 Update README.md
0fbe7c4 Merge pull request #193 from ryanw/next
527c04a  recovery mistakenlly change from 23 to 24
da4f111 Copyright © 2019–2024
9103a09 Update README.md
0a88cb9 Update README.md
e6d9056 Update README.md
b09044c Update README.md
e15c5d9 Update README.md
4109f28 Update README.md
2d50067 fix 22.2 TO 24.2  CR-1221872
72c4df4 Merge pull request #192 from yunleiz/doc2024
a5ab68b [doc] rm doc for CR-1220255
e88f6e3 Merge pull request #191 from yunleiz/doc2024
3284dd9 [doc] backup backgroud pages for CR-1220255
aeb66ff Merge pull request #190 from ryanw/next
d762c8d Merge branch 'FaaSApps:next' into next
4f4e5a3 fixing CR-1209088
7f92b57 Merge pull request #183 from RepoOps/update_vitis_json_0810
042a973 Merge pull request #187 from ryanw/next
f992308 remove third-part web links in sresources.rst
801b1e2 keep on setting larger time for test
3c5a824 fixed description.json issue
b7ec90a max_time_min from 300 to 470 for scanline in L2 and L3
9d3dca4 to fix CR-1216091
b94df90 Merge pull request #186 from RepoOps/update_makefile_0929
b8f2bc9 fix
7246c62 fix
61a3d8f update Makefile
5277cb8 Merge pull request #185 from jingt/CR-1203008
4742e39 remove error of CR-1203008
26a593f Merge pull request #180 from RepoOps/update_20240617-183706
a83081e clean vitis case description json
710bf2d updateJenkinsfile
011a328 change 2024.1_stable_latest to 2024.2_stable_latest

Co-authored-by: sdausr <[email protected]>
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

No branches or pull requests

2 participants