Skip to content

Conversation

@jeffng-or
Copy link
Contributor

Tighten up nangate45 jpeg - decreased clock period and increased utilization.

Metric Old New Type
cts__timing__setup__ws -0.06 -0.139 Failing
cts__timing__setup__tns -0.24 -35.6 Failing
cts__timing__hold__ws -0.06 -0.05 Tighten
cts__timing__hold__tns -0.24 -0.2 Tighten
globalroute__timing__setup__ws -0.06 -0.17 Failing
globalroute__timing__setup__tns -0.24 -43.2 Failing
globalroute__timing__hold__ws -0.06 -0.05 Tighten
globalroute__timing__hold__tns -0.24 -0.2 Tighten
detailedroute__route__wirelength 671143 634316 Tighten
detailedroute__timing__setup__tns -0.24 -7.87 Failing
detailedroute__timing__hold__ws -0.06 -0.05 Tighten
detailedroute__timing__hold__tns -0.24 -0.2 Tighten
finish__timing__setup__ws -0.06 -0.159 Failing
finish__timing__setup__tns -0.24 -41.1 Failing
finish__timing__hold__ws -0.06 -0.05 Tighten
finish__timing__hold__tns -0.24 -0.2 Tighten

Signed-off-by: Jeff Ng <jeffng@precisioninno.com>
@jeffng-or jeffng-or requested a review from maliberty November 10, 2025 22:36
@jeffng-or
Copy link
Contributor Author

The other half of #3641

export ABC_AREA = 1

export CORE_UTILIZATION ?= 45
export CORE_UTILIZATION ?= 80
Copy link
Member

Choose a reason for hiding this comment

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

nice!

},
"flow__warnings__count:PDN-1041": {
"value": 270,
"globalroute__flow__warnings__count:DRT-0120": {
Copy link
Member

Choose a reason for hiding this comment

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

[WARNING DRT-0120] Large net net11159 has 102 pins which may impact routing performance. Consider optimization.

seems a bit odd and worth a quick check (not new in the change, just noticed it in the diffs).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

[WARNING DRT-0120] Large net net11159 has 102 pins which may impact routing performance. Consider optimization.

seems a bit odd and worth a quick check (not new in the change, just noticed it in the diffs).

It looks like net11159 is being driven by the instance place11160, which is a "Timing Repair Buffer" and it does drive 102 pins:

image

The net is created in the 3_3_place_gp step, probably during repair. The synthesized net that gets rebuffered is 025959 and the connection looks like:

  • Inst 079408 drives net 025959
  • net 025959 drives inst place11158 (and 21 other inst terms)
  • inst place11158 drives net net11157
  • net net11157 drive inst place11160
  • inst place11160 drives net11159 (and 101 other inst terms)

The previous version of nangate45 jpeg doesn't have this rebuffering, prob since the clock period was lower.

Should I set max fanout or something to reduce the number of connections that this buffer is driving?

Copy link
Member

Choose a reason for hiding this comment

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

@precisionmoon this seems odd, do you want to look at it?

Copy link
Contributor

Choose a reason for hiding this comment

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

@povik, does GP use repair_design -pre_placement? If so, why do we see a net with 102 pins?

Copy link
Contributor

Choose a reason for hiding this comment

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

does GP use repair_design -pre_placement?

It does placement buffering which is different. It can produce a net with 102 pins if it's meeting all constraints.

Copy link
Contributor

Choose a reason for hiding this comment

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

I've looked at the checkpoint and it looks sensible.

To add to what Jeff wrote, the buffer sequence is

gate _079408_ -> (28 side loads) + buffer -> (21 side loads) + buffer -> 101 fanouts

Copy link
Contributor

Choose a reason for hiding this comment

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

In that case, it probably makes sense to add set_max_fanout of 10.

@maliberty maliberty enabled auto-merge November 11, 2025 00:32
@maliberty
Copy link
Member

Interesting little clumps in the layout - not sure if they are from gpl or dpl
image

@maliberty maliberty merged commit 6edb3e7 into The-OpenROAD-Project:master Nov 11, 2025
8 checks passed
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