Skip to content

asap7/ethmac_lvt: study placement densities and grt#2735

Merged
maliberty merged 3 commits intoThe-OpenROAD-Project:masterfrom
Pinata-Consulting:ethmac-lvt-grt
Jan 30, 2025
Merged

asap7/ethmac_lvt: study placement densities and grt#2735
maliberty merged 3 commits intoThe-OpenROAD-Project:masterfrom
Pinata-Consulting:ethmac-lvt-grt

Conversation

@oharboe
Copy link
Collaborator

@oharboe oharboe commented Jan 29, 2025

bazel run //designs/asap7/ethmac_lvt:plot

Builds for ca. 1 hour if you have lots of CPUs... and then shows the graph below.

Here we can clearly see that 0.60 is at the brink of grt routing collapse and that we should expect asap/ethmac_lvt to fall over with random changes in initial conditions, such as in #2694

image

@oharboe
Copy link
Collaborator Author

oharboe commented Jan 29, 2025

@jeffng-or @maliberty @tspyrou FYI

@oharboe oharboe force-pushed the ethmac-lvt-grt branch 2 times, most recently from 585f5c4 to f643236 Compare January 29, 2025 11:15
@oharboe oharboe requested a review from maliberty January 29, 2025 11:17
@oharboe oharboe changed the title [DO NOT MERGE] asap7/ethmac_lvt: study placement densities and grt asap7/ethmac_lvt: study placement densities and grt Jan 29, 2025
@oharboe oharboe marked this pull request as ready for review January 29, 2025 11:17
@oharboe
Copy link
Collaborator Author

oharboe commented Jan 29, 2025

@povik FYI

@oharboe oharboe force-pushed the ethmac-lvt-grt branch 2 times, most recently from fd129cb to dae95e0 Compare January 30, 2025 08:14
@oharboe
Copy link
Collaborator Author

oharboe commented Jan 30, 2025

@maliberty ready for review

@oharboe oharboe requested a review from maliberty January 30, 2025 08:15
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
util: bazel plot_congestion

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
Copy link
Member

Choose a reason for hiding this comment

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

This seems trivial - do we need a script for this?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

As far as I could find out, yes.

@maliberty maliberty merged commit a2c8bee into The-OpenROAD-Project:master Jan 30, 2025
7 checks passed
@oharboe oharboe deleted the ethmac-lvt-grt branch February 1, 2025 08:10
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.

2 participants