Skip to content

Comments

New custom configs for uW and metrics update for MPL changes#3007

Merged
maliberty merged 2 commits intoThe-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:metrics-and-configs-sky130hd-uW
Mar 31, 2025
Merged

New custom configs for uW and metrics update for MPL changes#3007
maliberty merged 2 commits intoThe-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:metrics-and-configs-sky130hd-uW

Conversation

@openroad-ci
Copy link
Collaborator

Metrics:

sky130hd/uW

Metric Old New Type
globalroute__antenna_diodes_count 1185 2756 Failing
detailedroute__antenna__violating__nets 2 0 Tighten
finish__timing__drv__hold_violation_count 122 101 Tighten

gf12/bp_single

Metric Old New Type
finish__timing__drv__hold_violation_count 100 230 Failing

Custom Configs for uW

  1. Increase blockage halos to eliminate available space for GPL. The idea is to avoid gates to be placed in such away that their connections cross over macros. We do this to make it easier to route as the macros are blocked up until metal4 leaving only metal5 available for wires (the macros also have pins in all four sides which makes it worse).

  2. Increase the placement density in order to make GPL able to run. The available space gets smaller due to the new blockages, so the placer requires a higher density.

    1. Use a custom blockage halo to make channels between macros
       inaccessible for GPL in order to avoid congestion problems.
    2. Use a higher placement density for GPL to run with the new
       blockages.

Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>
sky130hd/uW

| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| globalroute__antenna_diodes_count             |     1185 |     2756 | Failing  |
| detailedroute__antenna__violating__nets       |        2 |        0 | Tighten  |
| finish__timing__drv__hold_violation_count     |      122 |      101 | Tighten  |

gf12/bp_single

| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| finish__timing__drv__hold_violation_count     |      100 |      230 | Failing  |

Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>
@AcKoucher
Copy link
Contributor

@gudeh FYI

@maliberty maliberty enabled auto-merge March 28, 2025 14:55
@AcKoucher
Copy link
Contributor

AcKoucher commented Mar 28, 2025

@maliberty
The pr-merge / pr-head failures are due to the detailedroute__antenna__violating__nets tightening. Once the ORFS/OR PRs are merged I can open a subsequent PR to update the OR submodule.

@maliberty maliberty disabled auto-merge March 31, 2025 15:06
@maliberty maliberty merged commit f91d3af into The-OpenROAD-Project:master Mar 31, 2025
5 of 7 checks passed
@maliberty maliberty deleted the metrics-and-configs-sky130hd-uW branch March 31, 2025 15:06
@AcKoucher AcKoucher mentioned this pull request Mar 31, 2025
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

Successfully merging this pull request may close these issues.

3 participants