Skip to content

flow: Move pre-place buffering from abc to OR floorplan #3101

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

openroad-ci
Copy link
Collaborator

No description provided.

Signed-off-by: Martin Povišer <povik@cutebit.org>
@openroad-ci openroad-ci force-pushed the floorplan-buffering branch from ba891a1 to 9d56d8e Compare April 23, 2025 17:39
@povik
Copy link
Contributor

povik commented Apr 23, 2025

Secure run at secure-floorplan-buffering

@@ -40,14 +40,14 @@ proc repair_timing_helper { {hold_margin 1} } {
log_cmd repair_timing {*}$additional_args
}

proc repair_design_helper {} {
proc repair_design_helper { {extra_args ""} } {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a heads up, used in global_route.tcl too. Should be fine near as I can see.

$ grep -r repair_design_helper scripts/*
scripts/floorplan.tcl:  repair_design_helper
scripts/floorplan.tcl:  repair_design_helper
scripts/global_route.tcl:    repair_design_helper
scripts/resize.tcl:repair_design_helper
scripts/util.tcl:proc repair_design_helper {} {

@povik
Copy link
Contributor

povik commented Apr 24, 2025

Non-metric failures from public CI:

------------------------------
       Failing designs
------------------------------
asap7 mock-array (base)
  Last log file 5_1_grt.log
  Found 6 errors in the logs.
      [ERROR DRT-0073] No access point for ces_3_3/io_ins_down[10] (Element).
      [ERROR DRT-0073] No access point for ces_3_0/io_ins_down[10] (Element).
      [ERROR DRT-0073] No access point for ces_0_0/io_ins_down[10] (Element).
      [ERROR DRT-0073] No access point for ces_1_2/io_ins_down[10] (Element).
      [ERROR DRT-0073] No access point for ces_0_6/io_ins_left[14] (Element).
      [ERROR DRT-0073] No access point for ces_1_7/io_ins_left[14] (Element).

this has the symptoms of The-OpenROAD-Project/OpenROAD#6991

ihp-sg13g2 i2c-gpio-expander (base)
  Last log file 3_3_place_gp.log
  Found 1 errors in the logs.
      [ERROR GRT-0042] Pin io_address_0_PAD does not have geometries in a valid routing layer.

sky130hd ibex (base)
  Last log file 5_2_route.log
  Found 1 errors in the logs.
      [ERROR GRT-0232] Routing congestion too high. Check the congestion heatmap in the GUI.

sky130hd microwatt (base)
  Last log file 5_1_grt.log
  Found 1 errors in the logs.
      [ERROR GRT-0116] Global routing finished with congestion. Check the congestion regions in the DRC Viewer.

@povik
Copy link
Contributor

povik commented Apr 24, 2025

Curious failure mode from macro placer on gf12/bp_single. @AcKoucher is there anything I should do to triage?

Screenshot 2025-04-24 at 10 51 09

@AcKoucher
Copy link
Contributor

AcKoucher commented Apr 25, 2025

@povik I think this might be related with the approach we're using to deal with PADs in MPL - regarding how we map PADs with the internal objects of MPL. There is a PR that changes the current approach. Could you try using those changes? I just updated it.

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.

4 participants