Merge pull request #939 from openroadie/fix_all_violation

Fix all violations (on two designs). Added code to cts.tcl to accept the new envar setting (which is in config.mk for the 2 designs).

Author: openroadie

flow/designs/nangate45/aes/config.mk

@@ -8,3 +8,4 @@ export SDC_FILE      = ./designs/$(PLATFORM)/$(DESIGN_NICKNAME)/constraint.sdc
 export FLOORPLAN_DEF = ./designs/$(PLATFORM)/$(DESIGN_NICKNAME)/aes_ng45_fp.def
 
 export PLACE_DENSITY_LB_ADDON = 0.20
+export TNS_END_PERCENT ?= 100

flow/designs/nangate45/aes/metadata-base-ok.json

@@ -15,59 +15,47 @@
   "cts__design__die__area": 62612.4,
   "cts__design__die__area__post_repair": 62612.4,
   "cts__design__die__area__pre_repair": 62612.4,
-  "cts__design__instance__area": 26972.4,
+  "cts__design__instance__area": 28886.5,
   "cts__design__instance__area__macros": 0,
   "cts__design__instance__area__macros__post_repair": 0,
   "cts__design__instance__area__macros__pre_repair": 0,
   "cts__design__instance__area__post_repair": 25797.5,
   "cts__design__instance__area__pre_repair": 25797.5,
-  "cts__design__instance__area__stdcell": 26972.4,
+  "cts__design__instance__area__stdcell": 28886.5,
   "cts__design__instance__area__stdcell__post_repair": 25797.5,
   "cts__design__instance__area__stdcell__pre_repair": 25797.5,
-  "cts__design__instance__count": 16600,
+  "cts__design__instance__count": 17319,
   "cts__design__instance__count__hold_buffer": 0,
   "cts__design__instance__count__macros": 0,
   "cts__design__instance__count__macros__post_repair": 0,
   "cts__design__instance__count__macros__pre_repair": 0,
   "cts__design__instance__count__post_repair": 15623,
   "cts__design__instance__count__pre_repair": 15623,
-  "cts__design__instance__count__setup_buffer": 977.0,
-  "cts__design__instance__count__stdcell": 16600,
+  "cts__design__instance__count__setup_buffer": 1696.0,
+  "cts__design__instance__count__stdcell": 17319,
   "cts__design__instance__count__stdcell__post_repair": 15623,
   "cts__design__instance__count__stdcell__pre_repair": 15623,
-  "cts__design__instance__displacement__max": 8.17,
-  "cts__design__instance__displacement__mean": 0.2095,
-  "cts__design__instance__displacement__total": 3480.72,
-  "cts__design__instance__utilization": 0.510986,
+  "cts__design__instance__displacement__max": 12.4955,
+  "cts__design__instance__displacement__mean": 0.614,
+  "cts__design__instance__displacement__total": 10634.6,
+  "cts__design__instance__utilization": 0.547249,
   "cts__design__instance__utilization__post_repair": 0.488727,
   "cts__design__instance__utilization__pre_repair": 0.488727,
-  "cts__design__instance__utilization__stdcell": 0.510986,
+  "cts__design__instance__utilization__stdcell": 0.547249,
   "cts__design__instance__utilization__stdcell__post_repair": 0.488727,
   "cts__design__instance__utilization__stdcell__pre_repair": 0.488727,
   "cts__design__io": 388,
   "cts__design__io__post_repair": 388,
   "cts__design__io__pre_repair": 388,
   "cts__design__violations": 0,
-  "cts__power__internal__total": 0.0403635,
-  "cts__power__internal__total__post_repair": 0.0404769,
-  "cts__power__internal__total__pre_repair": 0.0404769,
-  "cts__power__leakage__total": 0.000695907,
-  "cts__power__leakage__total__post_repair": 0.000659168,
-  "cts__power__leakage__total__pre_repair": 0.000659168,
-  "cts__power__switching__total": 0.0394677,
-  "cts__power__switching__total__post_repair": 0.0406985,
-  "cts__power__switching__total__pre_repair": 0.0406985,
-  "cts__power__total": 0.0805272,
-  "cts__power__total__post_repair": 0.0818345,
-  "cts__power__total__pre_repair": 0.0818345,
-  "cts__route__wirelength__estimated": 216392,
+  "cts__route__wirelength__estimated": 233950,
   "cts__timing__drv__hold_violation_count": 0,
   "cts__timing__drv__hold_violation_count__post_repair": 0,
   "cts__timing__drv__hold_violation_count__pre_repair": 0,
-  "cts__timing__drv__max_cap": 0,
+  "cts__timing__drv__max_cap": 1,
   "cts__timing__drv__max_cap__post_repair": 0,
   "cts__timing__drv__max_cap__pre_repair": 0,
-  "cts__timing__drv__max_cap_limit": 0.093866,
+  "cts__timing__drv__max_cap_limit": -0.0248363,
   "cts__timing__drv__max_cap_limit__post_repair": 0.00147422,
   "cts__timing__drv__max_cap_limit__pre_repair": 0.00147422,
   "cts__timing__drv__max_fanout": 0,
@@ -79,19 +67,19 @@
   "cts__timing__drv__max_slew": 0,
   "cts__timing__drv__max_slew__post_repair": 0,
   "cts__timing__drv__max_slew__pre_repair": 0,
-  "cts__timing__drv__max_slew_limit": 0.430074,
+  "cts__timing__drv__max_slew_limit": 0.373937,
   "cts__timing__drv__max_slew_limit__post_repair": 0.322401,
   "cts__timing__drv__max_slew_limit__pre_repair": 0.322401,
-  "cts__timing__drv__setup_violation_count": 177,
+  "cts__timing__drv__setup_violation_count": 137,
   "cts__timing__drv__setup_violation_count__post_repair": 212,
   "cts__timing__drv__setup_violation_count__pre_repair": 212,
-  "cts__timing__setup__tns": -12.7149,
+  "cts__timing__setup__tns": -7.19333,
   "cts__timing__setup__tns__post_repair": -28.1029,
   "cts__timing__setup__tns__pre_repair": -28.1029,
   "cts__timing__setup__ws": -0.102313,
   "cts__timing__setup__ws__post_repair": -0.262803,
   "cts__timing__setup__ws__pre_repair": -0.262803,
-  "detailedplace__cpu__total": 14.16,
+  "detailedplace__cpu__total": 8.77,
   "detailedplace__design__core__area": 52785,
   "detailedplace__design__die__area": 62612.4,
   "detailedplace__design__instance__area": 25754.7,
@@ -107,13 +95,9 @@
   "detailedplace__design__instance__utilization__stdcell": 0.487916,
   "detailedplace__design__io": 388,
   "detailedplace__design__violations": 0,
-  "detailedplace__mem__peak": 151228.0,
-  "detailedplace__power__internal__total": 0.0399526,
-  "detailedplace__power__leakage__total": 0.000657188,
-  "detailedplace__power__switching__total": 0.0392602,
-  "detailedplace__power__total": 0.07987,
+  "detailedplace__mem__peak": 170668.0,
   "detailedplace__route__wirelength__estimated": 213599,
-  "detailedplace__runtime__total": "0:14.25",
+  "detailedplace__runtime__total": "0:08.84",
   "detailedplace__timing__drv__hold_violation_count": 0,
   "detailedplace__timing__drv__max_cap": 0,
   "detailedplace__timing__drv__max_cap_limit": 0.00147422,
@@ -125,56 +109,54 @@
   "detailedplace__timing__setup__tns": -28.0494,
   "detailedplace__timing__setup__ws": -0.259305,
   "detailedroute__route__drc_errors": 0,
-  "detailedroute__route__drc_errors__iter:1": 17808,
-  "detailedroute__route__drc_errors__iter:2": 3963,
-  "detailedroute__route__drc_errors__iter:3": 3326,
-  "detailedroute__route__drc_errors__iter:4": 304,
-  "detailedroute__route__drc_errors__iter:5": 17,
-  "detailedroute__route__drc_errors__iter:6": 0,
-  "detailedroute__route__net": 16984,
+  "detailedroute__route__drc_errors__iter:1": 18129,
+  "detailedroute__route__drc_errors__iter:2": 4263,
+  "detailedroute__route__drc_errors__iter:3": 4060,
+  "detailedroute__route__drc_errors__iter:4": 477,
+  "detailedroute__route__drc_errors__iter:5": 62,
+  "detailedroute__route__drc_errors__iter:6": 4,
+  "detailedroute__route__drc_errors__iter:7": 0,
+  "detailedroute__route__net": 17703,
   "detailedroute__route__net__special": 2,
-  "detailedroute__route__vias": 155238,
+  "detailedroute__route__vias": 163790,
   "detailedroute__route__vias__multicut": 0,
-  "detailedroute__route__vias__singlecut": 155238,
-  "detailedroute__route__wirelength": 289539,
-  "detailedroute__route__wirelength__iter:1": 293590,
-  "detailedroute__route__wirelength__iter:2": 290514,
-  "detailedroute__route__wirelength__iter:3": 289565,
-  "detailedroute__route__wirelength__iter:4": 289573,
-  "detailedroute__route__wirelength__iter:5": 289541,
-  "detailedroute__route__wirelength__iter:6": 289539,
-  "finish__clock__skew__hold": 0.0319808,
-  "finish__clock__skew__setup": 0.0319808,
-  "finish__cpu__total": 31.85,
+  "detailedroute__route__vias__singlecut": 163790,
+  "detailedroute__route__wirelength": 306917,
+  "detailedroute__route__wirelength__iter:1": 310870,
+  "detailedroute__route__wirelength__iter:2": 307795,
+  "detailedroute__route__wirelength__iter:3": 306799,
+  "detailedroute__route__wirelength__iter:4": 306967,
+  "detailedroute__route__wirelength__iter:5": 306935,
+  "detailedroute__route__wirelength__iter:6": 306916,
+  "detailedroute__route__wirelength__iter:7": 306917,
+  "finish__clock__skew__hold": 0.0319234,
+  "finish__clock__skew__setup": 0.0319234,
+  "finish__cpu__total": 23.49,
   "finish__design__core__area": 52785,
   "finish__design__die__area": 62612.4,
-  "finish__design__instance__area": 26972.4,
+  "finish__design__instance__area": 28886.5,
   "finish__design__instance__area__macros": 0,
-  "finish__design__instance__area__stdcell": 26972.4,
-  "finish__design__instance__count": 16600,
+  "finish__design__instance__area__stdcell": 28886.5,
+  "finish__design__instance__count": 17319,
   "finish__design__instance__count__macros": 0,
-  "finish__design__instance__count__stdcell": 16600,
-  "finish__design__instance__utilization": 0.510986,
-  "finish__design__instance__utilization__stdcell": 0.510986,
+  "finish__design__instance__count__stdcell": 17319,
+  "finish__design__instance__utilization": 0.547249,
+  "finish__design__instance__utilization__stdcell": 0.547249,
   "finish__design__io": 388,
-  "finish__mem__peak": 528640.0,
-  "finish__power__internal__total": 0.0418134,
-  "finish__power__leakage__total": 0.000695907,
-  "finish__power__switching__total": 0.0441958,
-  "finish__power__total": 0.0867051,
-  "finish__runtime__total": "0:32.36",
+  "finish__mem__peak": 627588.0,
+  "finish__runtime__total": "0:24.01",
   "finish__timing__drv__hold_violation_count": 0.0,
   "finish__timing__drv__max_cap": 1,
-  "finish__timing__drv__max_cap_limit": -0.0551364,
+  "finish__timing__drv__max_cap_limit": -0.180548,
   "finish__timing__drv__max_fanout": 0,
   "finish__timing__drv__max_fanout_limit": 1e+30,
   "finish__timing__drv__max_slew": 0,
-  "finish__timing__drv__max_slew_limit": 0.358356,
+  "finish__timing__drv__max_slew_limit": 0.299804,
   "finish__timing__drv__setup_violation_count": 1.0,
-  "finish__timing__setup__tns": -17.0294,
-  "finish__timing__setup__ws": -0.143168,
-  "finish__timing__wns_percent_delay": -13.361948,
-  "floorplan__cpu__total": 0.48,
+  "finish__timing__setup__tns": -10.3813,
+  "finish__timing__setup__ws": -0.122951,
+  "finish__timing__wns_percent_delay": -11.533052,
+  "floorplan__cpu__total": 0.34,
   "floorplan__design__core__area": 52785,
   "floorplan__design__die__area": 62612.4,
   "floorplan__design__instance__area": 19392.7,
@@ -186,15 +168,11 @@
   "floorplan__design__instance__utilization": 0.367391,
   "floorplan__design__instance__utilization__stdcell": 0.367391,
   "floorplan__design__io": 388,
-  "floorplan__mem__peak": 91584.0,
-  "floorplan__power__internal__total": 0.0303651,
-  "floorplan__power__leakage__total": 0.000431771,
-  "floorplan__power__switching__total": 0.021802,
-  "floorplan__power__total": 0.0525988,
-  "floorplan__runtime__total": "0:00.52",
+  "floorplan__mem__peak": 114172.0,
+  "floorplan__runtime__total": "0:00.37",
   "floorplan__timing__setup__tns": -112.598,
   "floorplan__timing__setup__ws": -0.520426,
-  "globalplace__cpu__total": 20.38,
+  "globalplace__cpu__total": 11.29,
   "globalplace__design__core__area": 52785,
   "globalplace__design__die__area": 62612.4,
   "globalplace__design__instance__area": 19502.1,
@@ -206,45 +184,37 @@
   "globalplace__design__instance__utilization": 0.369462,
   "globalplace__design__instance__utilization__stdcell": 0.369462,
   "globalplace__design__io": 388,
-  "globalplace__mem__peak": 170756.0,
-  "globalplace__power__internal__total": 0.0312184,
-  "globalplace__power__leakage__total": 0.000431771,
-  "globalplace__power__switching__total": 0.0312657,
-  "globalplace__power__total": 0.0629159,
-  "globalplace__runtime__total": "0:20.49",
+  "globalplace__mem__peak": 190808.0,
+  "globalplace__runtime__total": "0:11.36",
   "globalplace__timing__setup__tns": -207.746,
   "globalplace__timing__setup__ws": -0.845204,
   "globalroute__antenna__violating__nets": 0,
   "globalroute__antenna__violating__pins": 0,
-  "globalroute__clock__skew__hold": 0.0280787,
-  "globalroute__clock__skew__setup": 0.0280787,
+  "globalroute__clock__skew__hold": 0.0291523,
+  "globalroute__clock__skew__setup": 0.0291523,
   "globalroute__design__core__area": 52785,
   "globalroute__design__die__area": 62612.4,
-  "globalroute__design__instance__area": 26972.4,
+  "globalroute__design__instance__area": 28886.5,
   "globalroute__design__instance__area__macros": 0,
-  "globalroute__design__instance__area__stdcell": 26972.4,
-  "globalroute__design__instance__count": 16600,
+  "globalroute__design__instance__area__stdcell": 28886.5,
+  "globalroute__design__instance__count": 17319,
   "globalroute__design__instance__count__macros": 0,
-  "globalroute__design__instance__count__stdcell": 16600,
-  "globalroute__design__instance__utilization": 0.510986,
-  "globalroute__design__instance__utilization__stdcell": 0.510986,
+  "globalroute__design__instance__count__stdcell": 17319,
+  "globalroute__design__instance__utilization": 0.547249,
+  "globalroute__design__instance__utilization__stdcell": 0.547249,
   "globalroute__design__io": 388,
-  "globalroute__power__internal__total": 0.0417686,
-  "globalroute__power__leakage__total": 0.000695907,
-  "globalroute__power__switching__total": 0.0433961,
-  "globalroute__power__total": 0.0858606,
-  "globalroute__timing__clock__slack": -0.125,
+  "globalroute__timing__clock__slack": -0.117,
   "globalroute__timing__drv__hold_violation_count": 0,
-  "globalroute__timing__drv__max_cap": 0,
-  "globalroute__timing__drv__max_cap_limit": 0.0770163,
+  "globalroute__timing__drv__max_cap": 1,
+  "globalroute__timing__drv__max_cap_limit": -0.0941402,
   "globalroute__timing__drv__max_fanout": 0,
   "globalroute__timing__drv__max_fanout_limit": 1e+30,
   "globalroute__timing__drv__max_slew": 0,
-  "globalroute__timing__drv__max_slew_limit": 0.421533,
-  "globalroute__timing__drv__setup_violation_count": 189,
-  "globalroute__timing__setup__tns": -15.1807,
-  "globalroute__timing__setup__ws": -0.124502,
-  "placeopt__cpu__total": 20.38,
+  "globalroute__timing__drv__max_slew_limit": 0.341201,
+  "globalroute__timing__drv__setup_violation_count": 157,
+  "globalroute__timing__setup__tns": -9.12866,
+  "globalroute__timing__setup__ws": -0.116502,
+  "placeopt__cpu__total": 11.29,
   "placeopt__design__core__area": 52785,
   "placeopt__design__core__area__pre_opt": 52785,
   "placeopt__design__die__area": 62612.4,
@@ -267,16 +237,8 @@
   "placeopt__design__instance__utilization__stdcell__pre_opt": 0.369462,
   "placeopt__design__io": 388,
   "placeopt__design__io__pre_opt": 388,
-  "placeopt__mem__peak": 170756.0,
-  "placeopt__power__internal__total": 0.0376491,
-  "placeopt__power__internal__total__pre_opt": 0.0312184,
-  "placeopt__power__leakage__total": 0.000657188,
-  "placeopt__power__leakage__total__pre_opt": 0.000431771,
-  "placeopt__power__switching__total": 0.0360734,
-  "placeopt__power__switching__total__pre_opt": 0.0312657,
-  "placeopt__power__total": 0.0743796,
-  "placeopt__power__total__pre_opt": 0.0629159,
-  "placeopt__runtime__total": "0:20.49",
+  "placeopt__mem__peak": 190808.0,
+  "placeopt__runtime__total": "0:11.36",
   "placeopt__timing__drv__hold_violation_count": 0,
   "placeopt__timing__drv__max_cap": 0,
   "placeopt__timing__drv__max_cap_limit": 0.00402856,
@@ -290,10 +252,10 @@
   "placeopt__timing__setup__ws": -0.263469,
   "placeopt__timing__setup__ws__pre_opt": -0.845204,
   "run__flow__design": "aes",
-  "run__flow__generate_date": "2023-01-20 21:31",
+  "run__flow__generate_date": "2023-03-30 11:52",
   "run__flow__metrics_version": "Metrics_2.1.2",
   "run__flow__openroad_commit": "N/A",
-  "run__flow__openroad_version": "v2.0-6565-g7d2bade83",
+  "run__flow__openroad_version": "d5e88bef0089b8d0525f46ac1e7f74c352b16e26",
   "run__flow__platform": "nangate45",
   "run__flow__platform__capacitance_units": "1fF",
   "run__flow__platform__current_units": "1mA",
@@ -302,14 +264,14 @@
   "run__flow__platform__resistance_units": "1kohm",
   "run__flow__platform__time_units": "1ns",
   "run__flow__platform__voltage_units": "1v",
-  "run__flow__platform_commit": "17d6d54f0bf6f520a9e6a7a6d576f587c6742c79",
-  "run__flow__scripts_commit": "17d6d54f0bf6f520a9e6a7a6d576f587c6742c79",
-  "run__flow__uuid": "9a0acff9-8e24-41a7-ad11-81026f75914f",
+  "run__flow__platform_commit": "f4038168de523f988e1312a5b397a87ea2d70968",
+  "run__flow__scripts_commit": "f4038168de523f988e1312a5b397a87ea2d70968",
+  "run__flow__uuid": "19aa0ff3-3a99-4c3f-8886-2b334e4e895e",
   "run__flow__variant": "base",
-  "synth__cpu__total": 137.29,
+  "synth__cpu__total": 88.2,
   "synth__design__instance__area__stdcell": 21221.746,
   "synth__design__instance__count__stdcell": 16478.0,
-  "synth__mem__peak": 702804.0,
-  "synth__runtime__total": "2:20.16",
-  "total_time": "0:03:48.270000"
+  "synth__mem__peak": 713404.0,
+  "synth__runtime__total": "1:29.77",
+  "total_time": "0:02:25.710000"
 }

flow/designs/nangate45/aes/rules-base.json

@@ -32,19 +32,19 @@
         "compare": ">="
     },
     "cts__design__instance__count__setup_buffer": {
-        "value": 1075,
+        "value": 1866,
         "compare": "<="
     },
     "cts__design__instance__count__hold_buffer": {
         "value": 780,
         "compare": "<="
     },
     "globalroute__timing__clock__slack": {
-        "value": -0.16,
+        "value": -0.15,
         "compare": ">="
     },
     "globalroute__timing__setup__ws": {
-        "value": -0.16,
+        "value": -0.15,
         "compare": ">="
     },
     "detailedroute__route__wirelength": {
@@ -56,7 +56,7 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.18,
+        "value": -0.16,
         "compare": ">="
     },
     "finish__design__instance__area": {
@@ -84,7 +84,7 @@
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {
-        "value": -26.03,
+        "value": -23.83,
         "compare": ">="
     }
 }

flow/designs/nangate45/gcd/config.mk

@@ -10,3 +10,5 @@ export ADDER_MAP_FILE :=
 
 export CORE_UTILIZATION ?= 55
 export PLACE_DENSITY_LB_ADDON = 0.20
+export TNS_END_PERCENT ?= 100
+