Skip to content

open: better logging#3084

Merged
maliberty merged 2 commits intoThe-OpenROAD-Project:masterfrom
Pinata-Consulting:open-logging
Apr 16, 2025
Merged

open: better logging#3084
maliberty merged 2 commits intoThe-OpenROAD-Project:masterfrom
Pinata-Consulting:open-logging

Conversation

@oharboe
Copy link
Collaborator

@oharboe oharboe commented Apr 16, 2025

log_cmd accounts for pretty much all time to open a desig now

All but 10 seconds accounted for by log_cmd:

OpenROAD v2.0-20670-g4939290b5 
Features included (+) or not (-): +GPU +GUI +Python
This program is licensed under the BSD-3 license. See the LICENSE file for details.
Components of this program may be licensed under more restrictive licenses which must be honored.
read_db results/asap7/BoomTile/base/5_1_grt.odb
Took 8 seconds: read_db results/asap7/BoomTile/base/5_1_grt.odb
GUI_TIMING=1 reading timing, takes a little while for large designs...
read_sdc /home/oyvind/OpenROAD-flow-scripts/foo/results/asap7/BoomTile/base/5_1_grt.sdc
Took 10 seconds: read_sdc /home/oyvind/OpenROAD-flow-scripts/foo/results/asap7/BoomTile/base/5_1_grt.sdc
grt::have_routes
Took 48 seconds: grt::have_routes
No global routing results available, skipping estimate_parasitics
Load .//reports/asap7/BoomTile/base/congestion.rpt for details
find_timing_paths
Took 71 seconds: find_timing_paths

real	2m27,613s
user	2m23,664s
sys	0m3,944s

log_cmd accounts for pretty much all time to open a desig now

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

Conflicts with my uplevel change

@oharboe
Copy link
Collaborator Author

oharboe commented Apr 16, 2025

Conflicts with my uplevel change

A quick attempt at merge, let us see if CI agrees.

@oharboe
Copy link
Collaborator Author

oharboe commented Apr 16, 2025

@maliberty unrelated CI error

@maliberty maliberty merged commit b7fa57d into The-OpenROAD-Project:master Apr 16, 2025
5 of 7 checks passed
@oharboe oharboe deleted the open-logging branch April 17, 2025 11:33
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

Comments