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

Improve global router for asap7/ethmac_lvt where it fails with a single DRC error #6589

Open
oharboe opened this issue Jan 24, 2025 · 4 comments
Assignees
Labels
grt Global Routing ppl IO Pin Placer

Comments

@oharboe
Copy link
Collaborator

oharboe commented Jan 24, 2025

Description

In The-OpenROAD-Project/OpenROAD-flow-scripts#2694 asap7/ethmac_lvt fails global route with a single DRC error after 100 iterations.

To reproduce untar and run https://drive.google.com/file/d/1XCfVGa0DHAz5MO_pJwR_VE8qOhZ1oqoO/view?usp=sharing

No luck in whittling down the problem with deltaDebug.

Image

Suggested Solution

Improve global router.

Additional Context

No response

@eder-matheus eder-matheus self-assigned this Jan 24, 2025
@eder-matheus eder-matheus added the grt Global Routing label Jan 24, 2025
@maliberty
Copy link
Member

The IO pin placement is totally different than master which has few pins on the e/w edges.

@maliberty maliberty added the ppl IO Pin Placer label Jan 28, 2025
@oharboe
Copy link
Collaborator Author

oharboe commented Jan 28, 2025

The IO pin placement is totally different than master which has few pins on the e/w edges.

This is unsurprising to me: a completely different optimal solution can be found with tiny differences in initial conditions.

So what does this mean practically?

  • Pin ppl chooses a pin placements that is never going to work in global routing and it should have done better, so fix ppl?
  • There's nothing wrong here with ppl, nor grt, just lock down pin placement in ORFS test cases in git to avoid random changes breaking master?
  • Wait for initial conditions to change again slightly, then merge synth: nudge .rtlil towards cleaner and more canonical OpenROAD-flow-scripts#2694 ?

I ran out of imagination w.r.t. suggesting alternatives :-)

@maliberty
Copy link
Member

I think it is worth a bit of digging to see if this is just the flap of the butterfly's wings or something more interesting.

@oharboe
Copy link
Collaborator Author

oharboe commented Jan 28, 2025

I think it is worth a bit of digging to see if this is just the flap of the butterfly's wings or something more interesting.

I suppose the test case is somewhat interesting independent of The-OpenROAD-Project/OpenROAD-flow-scripts#2694

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
grt Global Routing ppl IO Pin Placer
Projects
None yet
Development

No branches or pull requests

3 participants