Skip to content

reduce core utilizaztion of rapidus2hp/hercules_is_int#3479

Merged
maliberty merged 5 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:update_hercules_is_int
Sep 10, 2025
Merged

reduce core utilizaztion of rapidus2hp/hercules_is_int#3479
maliberty merged 5 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:update_hercules_is_int

Conversation

@openroad-ci

@openroad-ci openroad-ci commented Sep 9, 2025

Copy link
Copy Markdown
Member

designs/rapidus2hp/hercules_is_int/rules-base.json updates:

Metric Old New Type
finish__timing__setup__ws -417.07 -411.86 Tighten
finish__timing__wns_percent_delay -66.06 -65.28 Tighten

Update after changing core util to 52%:
designs/rapidus2hp/hercules_is_int/rules-base.json updates:

Metric Old New Type
finish__timing__setup__ws -411.86 -476.31 Failing

Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
@eder-matheus

Copy link
Copy Markdown
Member

@maliberty This should fix the timeout on the CI.

@jeffng-or FYI.

@jeffng-or

Copy link
Copy Markdown
Contributor

@maliberty This should fix the timeout on the CI.

@jeffng-or FYI.

Hi guys,

The fix could fix the timeout, but it will degrade our area correlation. We need to be above 54% to hold the correlation.

@eder-matheus

Copy link
Copy Markdown
Member

The fix could fix the timeout, but it will degrade our area correlation. We need to be above 54% to hold the correlation.

Oh, I see. I'm trying different values now. It's possible that a larger value could work too.

@eder-matheus

Copy link
Copy Markdown
Member

@jeffng-or @maliberty I've tried multiple values above 54%, and all of them fails with placement issues (during DPL or with high congestion in GRT).

An alternative would be revert the Yosys update, but I think it would affect other projects that are running in parallel. Do you guys have any suggestion?

@maliberty

Copy link
Copy Markdown
Member

Can we back off just 1-2%?

@eder-matheus

Copy link
Copy Markdown
Member

Can we back off just 1-2%?

I'm running with 52% now, it should be ready in the next minutes.

@jeffng-or

Copy link
Copy Markdown
Contributor

I think the Verific version (which is the front end that we are quoting Rapidus) still completes with util == 54%. So, go ahead with setting the util to 52% and then I'll add an ifeq to set the util to 54 for Verific.

@maliberty

Copy link
Copy Markdown
Member

We were in the 3-4 hr runtime in the CI and it would be good to get back to something similar.

@eder-matheus

Copy link
Copy Markdown
Member

We were in the 3-4 hr runtime in the CI and it would be good to get back to something similar.

Using 52% keeps this same runtime, at least on GCP. I'll update the PR.

Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
@eder-matheus

Copy link
Copy Markdown
Member

This should be good to go now!

@maliberty maliberty enabled auto-merge September 9, 2025 21:54
@maliberty maliberty merged commit 2f349ad into The-OpenROAD-Project:master Sep 10, 2025
8 checks passed
@maliberty maliberty deleted the update_hercules_is_int branch September 10, 2025 00:57
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