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

Vivado reports 'set_property' expects at least one object when doing HwAccel/Design/05-buttom_up_rtl_kernel #214

Closed
xooxit opened this issue Mar 16, 2022 · 4 comments
Assignees
Labels
Hw Accel Hardware Acceleration related issues

Comments

@xooxit
Copy link

xooxit commented Mar 16, 2022

While building the Hardware_Acceleration/Design_Tutorials/05-bottom_up_rtl_kernel/krnl_aes project, I got some errors at the stage of FPGA logic optimization. My vitis version is 2021.2 and Ubuntu version is 18.04.

I found some advice related to the Error below, and they said created HDL wrapper, but hmm.. anyway, is there any guy who gets a similar error like this?

****** vpl v2021.2 (64-bit)
  **** SW Build 3363252 on 2021-10-14-04:41:01
    ** Copyright 1986-2020 Xilinx, Inc. All Rights Reserved.

INFO: [VPL 60-839] Read in kernel information from file '/home/lab/yong/document/Vitis-Tutorials/Hardware_Acceleration/Design_Tutorials/05-bottom_up_rtl_kernel/krnl_aes/_x/link/int/kernel_info.dat'.
INFO: [VPL 74-78] Compiler Version string: 2021.2
INFO: [VPL 60-423]   Target device: xilinx_u250_xdma_201830_2
INFO: [VPL 60-1032] Extracting hardware platform to /home/lab/yong/document/Vitis-Tutorials/Hardware_Acceleration/Design_Tutorials/05-bottom_up_rtl_kernel/krnl_aes/_x/link/vivado/vpl/.local/hw_platform
[15:25:20] Run vpl: Step create_project: Started
Creating Vivado project.
[15:25:30] Run vpl: Step create_project: Completed
[15:25:30] Run vpl: Step create_bd: Started
[15:26:46] Run vpl: Step create_bd: RUNNING...
[15:27:02] Run vpl: Step create_bd: Completed
[15:27:02] Run vpl: Step update_bd: Started
[15:27:03] Run vpl: Step update_bd: Completed
[15:27:03] Run vpl: Step generate_target: Started
[15:28:19] Run vpl: Step generate_target: RUNNING...
[15:29:35] Run vpl: Step generate_target: RUNNING...
[15:30:17] Run vpl: Step generate_target: Completed
[15:30:17] Run vpl: Step config_hw_runs: Started
[15:30:21] Run vpl: Step config_hw_runs: Completed
[15:30:21] Run vpl: Step synth: Started
[15:30:52] Block-level synthesis in progress, 0 of 8 jobs complete, 1 job running.
[15:31:22] Block-level synthesis in progress, 0 of 8 jobs complete, 1 job running.
[15:31:53] Block-level synthesis in progress, 0 of 8 jobs complete, 1 job running.
[15:32:23] Block-level synthesis in progress, 0 of 8 jobs complete, 1 job running.
[15:32:44] Run vpl: Step synth: Completed
[15:32:44] Run vpl: Step impl: Started
[15:45:21] Finished 2nd of 6 tasks (FPGA linking synthesized kernels to platform). Elapsed time: 00h 20m 42s 

[15:45:21] Starting logic optimization..
[15:46:51] Phase 1 Generate And Synthesize MIG Cores
[15:51:24] Phase 2 Generate And Synthesize Debug Cores
[15:53:25] Phase 3 Retarget
[15:53:55] Phase 4 Constant propagation
[15:53:55] Phase 5 Sweep
[15:54:56] Phase 6 BUFG optimization
[15:55:26] Phase 7 Shift Register Optimization
[15:55:26] Phase 8 Post Processing Netlist
[15:56:39] Run vpl: Step impl: Failed
[15:56:39] Run vpl: FINISHED. Run Status: impl ERROR

===>The following messages were generated while processing /home/lab/yong/document/Vitis-Tutorials/Hardware_Acceleration/Design_Tutorials/05-bottom_up_rtl_kernel/krnl_aes/_x/link/vivado/vpl/prj/prj.runs/impl_1 :
ERROR: [VPL 101-2] ERROR: [Common 17-55] 'set_property' expects at least one object.
Resolution: If [get_<value>] was used to populate the object, check to make sure this command returns at least one valid object.
ERROR: [VPL 101-3] sourcing script /home/lab/yong/document/Vitis-Tutorials/Hardware_Acceleration/Design_Tutorials/05-bottom_up_rtl_kernel/krnl_aes/_x/link/vivado/vpl/scripts/impl_1/_full_place_pre.tcl failed
ERROR: [VPL 60-773] In '/home/lab/yong/document/Vitis-Tutorials/Hardware_Acceleration/Design_Tutorials/05-bottom_up_rtl_kernel/krnl_aes/_x/link/vivado/vpl/runme.log', caught Tcl error:  problem implementing dynamic region, impl_1: place_design ERROR, please look at the run log file '/home/lab/yong/document/Vitis-Tutorials/Hardware_Acceleration/Design_Tutorials/05-bottom_up_rtl_kernel/krnl_aes/_x/link/vivado/vpl/prj/prj.runs/impl_1/runme.log' for more information
WARNING: [VPL 60-732] Link warning: No monitor points found for BD automation.
ERROR: [VPL 60-704] Integration error, problem implementing dynamic region, impl_1: place_design ERROR, please look at the run log file '/home/lab/yong/document/Vitis-Tutorials/Hardware_Acceleration/Design_Tutorials/05-bottom_up_rtl_kernel/krnl_aes/_x/link/vivado/vpl/prj/prj.runs/impl_1/runme.log' for more information
ERROR: [VPL 60-1328] Vpl run 'vpl' failed
ERROR: [VPL 60-806] Failed to finish platform linker
INFO: [v++ 60-1442] [15:56:40] Run run_link: Step vpl: Failed
Time (s): cpu = 00:00:40 ; elapsed = 00:32:04 . Memory (MB): peak = 2071.566 ; gain = 0.000 ; free physical = 83290 ; free virtual = 99154
ERROR: [v++ 60-661] v++ link run 'run_link' failed
ERROR: [v++ 60-626] Kernel link failed to complete
ERROR: [v++ 60-703] Failed to finish linking
INFO: [v++ 60-1653] Closing dispatch client.
Makefile:81: recipe for target 'build_hw' failed
make: *** [build_hw] Error 1
@imrickysu
Copy link
Contributor

Hi @Rampagee , could you please take a look and check whether you can reproduce this issue?
It seems some properties is empty in the tcl script impl_1/_full_place_pre.tcl.

@imrickysu imrickysu changed the title Hardware_Acceleration Vivado reports 'set_property' expects at least one object when doing HwAccel/Design/05-buttom_up_rtl_kernel Mar 21, 2022
@imrickysu imrickysu added the Hw Accel Hardware Acceleration related issues label Mar 22, 2022
@Rampagee
Copy link

Hi, @xooxit , sorry, I cannot re-produce the errors with same configuration. Below is my relevant log segment. Would you please help to confirm whether the Makefile and krnl_aes_test.xdc files are modified according to U250 case?

[15:50:26] Block-level synthesis in progress, 81 of 82 jobs complete, 0 jobs running.
[15:50:56] Block-level synthesis in progress, 81 of 82 jobs complete, 1 job running.
[15:51:26] Block-level synthesis in progress, 82 of 82 jobs complete, 0 jobs running.
[15:51:57] Top-level synthesis in progress.
[15:52:27] Top-level synthesis in progress.
[15:52:54] Run vpl: Step synth: Completed
[15:52:54] Run vpl: Step impl: Started
[16:00:29] Finished 2nd of 6 tasks (FPGA linking synthesized kernels to platform). Elapsed time: 00h 23m 30s

[16:00:29] Starting logic optimization..
[16:00:59] Phase 1 Generate And Synthesize MIG Cores
[16:05:03] Phase 2 Generate And Synthesize Debug Cores
[16:06:34] Phase 3 Retarget
[16:06:34] Phase 4 Constant propagation
[16:06:34] Phase 5 Sweep
[16:07:04] Phase 6 BUFG optimization
[16:07:34] Phase 7 Shift Register Optimization
[16:07:34] Phase 8 Post Processing Netlist
[16:09:36] Finished 3rd of 6 tasks (FPGA logic optimization). Elapsed time: 00h 09m 06s

[16:09:36] Starting logic placement..
[16:10:06] Phase 1 Placer Initialization
[16:10:06] Phase 1.1 Placer Initialization Netlist Sorting
[16:13:39] Phase 1.2 IO Placement/ Clock Placement/ Build Placer Device

@xooxit
Copy link
Author

xooxit commented Mar 22, 2022

Thanks for the reply.
I think I did change Makefile file but did not change krnl_aes_test.xdc file. Meanwhile, I upgrade my platform from xilinx_u250_xdma_201830_2 to xilinx_u250_gen3x16_xdma_3_1_202020_1, maintaining vivado version.
Can I change krnl_aes_test.xdc file to confirm the new platform? Simply uncomment all the line in kernel_aes_test.xdc file?

imrickysu pushed a commit that referenced this issue May 6, 2022
* update from xclbin to xsa

* update Makefile for syntax of copyright

* update README to remove ES

* update graph.cpp

* update for 2022.1

Co-authored-by: brucey <[email protected]>
imrickysu pushed a commit that referenced this issue Nov 3, 2022
* update from xclbin to xsa

* update Makefile for syntax of copyright

* update README to remove ES

* update graph.cpp

* update for 2022.1

Co-authored-by: brucey <[email protected]>
CRTejaswi pushed a commit to CRTejaswi/amd-vitis that referenced this issue Oct 3, 2023
a92c1a5 Merge pull request Xilinx#223 from changg/fix_jinja2
5cf8ff9 revert using virtual env
38ffa7b fix jinja2
ea9ba35 Merge pull request Xilinx#222 from jingt/next
602a025 Fix CR-1108264
d846095 for debug
adc52b5 clange-format
8614e37 clang-format
acff7d6 update
e7bafb9 update
a891529 update
d111f0f update
1e18c43 Merge branch 'FaaSApps-next' into next
c8438f5 resolve conflict
183071b rm pyenv
6c8521d Merge pull request Xilinx#220 from changg/fix_jinja2
aa70063 fix jinja download
c32885a Merge pull request Xilinx#214 from jingt/next
af5468c cherry_pick_fix_cr_1103991
fcc5753 Merge pull request Xilinx#219 from tuol/cherry_pick_fix_cr_1103991
79ab75e disable datamover case in gui
8f4a1b5 add template
2368087 add loadMasterToDdrWithCounter and storeStreamToMasterWithCounter
335f0a5 modify xrt.ini
4c7b783 add load_master_to_stream_with_counter
0f4aa5b rm
ad2b279 update
f15f2ee udpate
096e0f1 udpate
36d5ec9 update
896d803 change 2021.1_stable_latest to 2021.2_stable_latest
51137ae update
4328fd9 update
b183433 update
a76e71a update
fbe622f update
506d793 update
ce3ecc2 udpate
c5d9aa5 udpate
1a83513 update
b3e0332 update
fb2db00 update
0079227 upate
18c6c6a update
11de555 add case
27e7459 add counter
3a366d8 Merge pull request Xilinx#3 from FaaSApps/next
14d1889 Merge pull request #2 from FaaSApps/next
39e16a2 Merge pull request #1 from FaaSApps/next
98aaf87 update
0770234 update
5df03fe upate
6fb3473 update
549749e updarte
30d7584 update
7167819 update
0279873 update
3cd8eb7 update
4c9dde0 update
7c9939f add script
087d326 add dsp
REVERT: 7e07a18 Merge pull request Xilinx#218 from tuol/fix_cr_1103991
REVERT: 3e13860 disable datamover case in gui
REVERT: 6be62b4 create master branch from next branch

git-subtree-dir: utils
git-subtree-split: a92c1a515908da23cc24c873932c23bc74fdfc34

Co-authored-by: sdausr <[email protected]>
CRTejaswi pushed a commit to CRTejaswi/amd-vitis that referenced this issue Oct 3, 2023
3d03602 Merge pull request Xilinx#214 from yuanqian/fix_hls_makefile
c4920c9 update
20f9517 fix hls Makefile
0346b17 Merge pull request Xilinx#213 from RepoOps/rep_8
6f19070 Replace DEVICE with PLATFORM

Co-authored-by: sdausr <[email protected]>
@AnusheelXilinx
Copy link
Collaborator

With the recent updates in the tutorial, we do not see this issue internally.
Please feel free to test the 2023.2 version and let us know if you still see an issue.

Thanks
Anusheel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hw Accel Hardware Acceleration related issues
Projects
None yet
Development

No branches or pull requests

4 participants