<#2003 Incorrect number of repair_antennas iterati...
# openlane-developmentg
GitHub
09/22/2023, 2:31 PM#2003 Incorrect number of repair_antennas iterations - remove repair_antennas call from grt.tcl
Issue created by jacobhaehn
Description
I have been trying to remedy antenna issues in a large design with many macros. I have been having issues similar to #1970, where the process gets killed after running out of memory. I can normally get through about 7-9 total repair_antenna iterations before the crash (looking at global.log), reducing antenna violations from 1431 to about 30, but always gets stuck on the last few (or maybe it's keeping previous iterations in memory and eventually runs out?).
I have been trying to avoid a crash by reducing GRT_ANT_ITERS. However, since the repair_antenna flow was reworked in #1968, repair_antennas.tcl calls repair_antennas 3 times, each with GRT_ANT_ITERS: once in grt.tcl, once within itself, and then again in grt.tcl. For example, if I have GRT_ANT_ITERS=3, then it actually ends up running 9 total iterations (3, 3, 2) and the process is killed because I always run out of memory around the 9th total iteration. If I have GRT_ANT_ITERS=5, it still runs 9 total iterations, but 5 iterations of the first pass, then 3 on the second pass before crashing on the 9th total iteration.
I suggest removing the repair_antennas call from grt.tcl so that repair_antennas is only called once from repair_antennas.tcl itself, making the true number of antenna repair iterations equal to GRT_ANT_ITERS. Additionally, let me know if you have any suggestions or fixes to prevent the crash in the first place.
Expected Behavior
GRT_ANT_ITERS should be the true total number of repair_antenna iterations.
Environment report
Copy code
Kernel: Linux v6.2.0-26-generic
Distribution: ubuntu 22.04
Python: v3.10.12 (OK)
Container Engine: docker v24.0.4 (OK)
OpenLane Git Version: 5215ea7cc01e5311086e995905492f0e8fe1ceb6
pip: INSTALLED
python-venv: INSTALLED
---
PDK Version Verification Status: OK
---
Git Log (Last 3 Commits)
5215ea7c 2023-09-20T15:53:26+03:00 Update `open_pdks` (#1992) - Mohamed Gaber - (HEAD -> master, origin/master, origin/HEAD)
7e5a2e9f 2023-09-10T19:06:55+03:00 fix synthesis `chparam` issue (#1981) - Kareem Farid - (tag: 2023.09.11)
0e8827eb 2023-09-10T19:04:45+03:00 `run_sta`: fix `-blackbox_check` not running when `-multi_corner` is not set (#1980) - Kareem Farid - ()
---
Git Remotes
origin <https://github.com/The-OpenROAD-Project/OpenLane> (fetch)
origin <https://github.com/The-OpenROAD-Project/OpenLane> (push)GitHub
10/01/2023, 10:00 AM