Skip to content
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

[ERROR PDN-0351] Ring shape falls outside the die bounds. #6618

Closed
oharboe opened this issue Jan 30, 2025 · 14 comments
Closed

[ERROR PDN-0351] Ring shape falls outside the die bounds. #6618

oharboe opened this issue Jan 30, 2025 · 14 comments

Comments

@oharboe
Copy link
Collaborator

oharboe commented Jan 30, 2025

Describe the bug

untar and run pdn_sdq_17x64_asap7_mocked_2025-01-30_15-10.tar.gz

$ ./run-me-sdq_17x64-asap7-mocked.sh 
OpenROAD v2.0-18694-g401a7bb30 
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.
[ERROR PDN-0351] Ring shape falls outside the die bounds.
Error: BLOCK_grid_strategy.tcl, 23 PDN-0351
openroad> 

Expected Behavior

No error.

Environment

OpenROAD v2.0-18694-g401a7bb30

Tried all the way back to OpenROAD v2.0-16961-g6b3508ac6, same error.

To Reproduce

See above

Relevant log output

Screenshots

No response

Additional Context

No response

@oharboe
Copy link
Collaborator Author

oharboe commented Jan 30, 2025

@jeffng-or FYI fails in The-OpenROAD-Project/megaboom#229

@gadfort
Copy link
Collaborator

gadfort commented Jan 30, 2025

@oharboe this is intentional now (#6445)
You need to fix your power grid or explicitly allow the rings to go outside the die (I would recommend fixing the grid instead of allowing it outside the die).

@oharboe
Copy link
Collaborator Author

oharboe commented Jan 30, 2025

But I am using a PDN from ORFS platforms/asap7 folder and this .tcl file is used in a ton of places and this is the first place I have seen it fail. I am not familiar with any design specific knobs that I can tweak.

Could you have a look and give some more specific instructions for non PDN expert...?

@gadfort
Copy link
Collaborator

gadfort commented Jan 30, 2025

I think you can just make the width smaller, spacing smaller, or core offset smaller to resolve this.
The issue needs to be fixed in ORFS.

@oharboe
Copy link
Collaborator Author

oharboe commented Jan 30, 2025

I think you can just make the width smaller, spacing smaller, or core offset smaller to resolve this.
The issue needs to be fixed in ORFS.

Silly question: which ORFS variables?

@gadfort
Copy link
Collaborator

gadfort commented Jan 30, 2025

I think it's in a file in the platform enablement, no variables involved.

@oharboe
Copy link
Collaborator Author

oharboe commented Jan 30, 2025

@maliberty @jeffng-or I guess you are way ahead of me on this :-)

@maliberty
Copy link
Member

Either the pdn config can be changed or the core-to-die spacing needs to be large enough to accommodate the rings. I think it easiest is to increase the core-to-die spacing through CORE_MARGIN or CORE_AREA depending on how you setup the floorplan.

@oharboe
Copy link
Collaborator Author

oharboe commented Jan 30, 2025

Can the error message give more specific actionable advice: increase by X and Y?

@maliberty
Copy link
Member

Can the error message give more specific actionable advice: increase by X and Y?

@gadfort ? Of course there are multiple ways to fix it so it would just be one possibiilty.

@gadfort
Copy link
Collaborator

gadfort commented Jan 30, 2025

@maliberty pdn could output the amount by which the ring extends outside of the die?

@maliberty
Copy link
Member

Let's have some text to suggest how that value might be used. Some like

PDN rings do not fit inside the die area by <val> um in X and <val2> um in Y.  Either reduce the ring area or increase the core to die spacing to accommodate.  Use -allow_out_of_die if this is intentional.

@oharboe
Copy link
Collaborator Author

oharboe commented Jan 30, 2025

I modified bazel-orfs mocked area to scale the die area, but keep the die to core distance unchanged: The-OpenROAD-Project/bazel-orfs#276

@oharboe
Copy link
Collaborator Author

oharboe commented Jan 30, 2025

Closing as duplicate/known problem.

Improved actionable and specific error messages are great usability improvements, suggestion from @maliberty looks good.

@oharboe oharboe closed this as completed Jan 30, 2025
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

No branches or pull requests

3 participants