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

Missing Resistance for vias in tech lef #19

Open
kareefardi opened this issue Sep 4, 2022 · 14 comments
Open

Missing Resistance for vias in tech lef #19

kareefardi opened this issue Sep 4, 2022 · 14 comments
Assignees

Comments

@kareefardi
Copy link
Contributor

Expected Behavior

No Via resistance in tech lef

Actual Behavior

Via resistance in tech lef

Steps to Reproduce the Problem

Specifications

  • Version:
  • Platform:
@proppy
Copy link
Member

proppy commented Sep 7, 2022

@kareefardi
Copy link
Contributor Author

Yes exactly. The resistance information is lost through VIARULE for instance.

@proppy
Copy link
Member

proppy commented Sep 9, 2022

@RTimothyEdwards @tspyrou @donn is that a hard blocker for the PDK to be usable with OpenLane?

@atorkmabrains
Copy link
Collaborator

@mohanad0mohamed Please take a look at this. We will discuss on Sunday.
@proppy We will get back to you soon. I believe this is really important.

@maliberty
Copy link

VIARULE GENERATE statements have an optional RESISTANCE field but it defaults per the manual:
Default: The resistance value in the LAYER (Cut) statement

So either the VIARULE needs a resistance or the cut layer does. It appears that neither has it here which isn't good.
@proppy @tspyrou

As for importance, this will only affect vias in the power grid as the detailed router doesn't use generated vias. So IR drop analysis would be somewhat off but timing would be unaffected.

@tspyrou
Copy link

tspyrou commented Sep 12, 2022

@RTimothyEdwards @proppy In looking in the PDK I don't see any openrcx setup. The LEF rules are very rough estimates only. We have found that normally the rules in LEF are optimistic and in most pdks we have a setRC.tcl
For example these values override the LEF and are much more accurate.
https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/blob/master/flow/platforms/sky130hs/setRC.tcl
I think we will likely need a setRC.tcl file for this PDK as well as the OpenRCX setup for final timing.

@RTimothyEdwards
Copy link
Collaborator

@tspyrou : The rules file for OpenRCX is maintained in open_pdks. It is derived from the capacitance modeling in magic, which currently is about as good as it is in SkyWater (that is, needs a bit of refinement in the modeling plus scripts to run a complete set of curve fits on the existing GF data, and/or data from running FasterCap. All that is in the works). The current rules file for OpenRCX for GF180MCU is a rough estimate, much better than the LEF rules, although like the SkyWater rules file is tending toward pessimistic calculations of delay. If the tendency toward pessimism is in the modeling, I should be able to figure that out and get rid of it.

@proppy
Copy link
Member

proppy commented Sep 13, 2022

@maliberty @tspyrou looking at the tech lef:

It seems that there are some RESISTANCE value for the fixed VIA definitions:

Given that that dimentions seem to correspond to the VIARULE definitions:

And that according to http://coriolis.lip6.fr/doc/lefdef/lefdefref/LEFSyntax.html#918957

Note: A RESISTANCE value attached to an individual via is no longer recommended.

Would it be "correct" to move the RESISTANCE value from the individual VIA to the corresponding VIARULE definition?

@maliberty
Copy link

No it would be better to have a per cut value on the LAYER definition assuming only simple cuts are used (which is likely but I haven't verified).

@proppy
Copy link
Member

proppy commented Sep 14, 2022

@tspyrou is there a setRC.tcl equivalent for OpenLane?

@RTimothyEdwards is RTimothyEdwards/open_pdks#281 the right issue to follow to track the OpenRCX refinements?

@proppy
Copy link
Member

proppy commented Sep 14, 2022

@maliberty oh I see, so this would eventually come up from the gf180mcu open_pdks openlane config: https://github.com/RTimothyEdwards/open_pdks/blob/cc0029b45c68137aa21323912f50d2fc17eeea13/gf180mcu/openlane/config.tcl#L114

@maliberty
Copy link

I believe so

@antonblanchard
Copy link

Note that ORFS has a set of scripts from @jjcherry56 to create RC layer estimates based on real designs. We really need to do this vs using the LEF data as @tspyrou mentions.

It would be useful to integrate this functionality into Openlane (I haven't had the time myself yet). The script to create the data is at:

https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/blob/master/flow/util/write_net_rc_script.tcl

It creates the data in a separate pass, but we could just dump the data during the openlane flow based on a variable. The script to create the RC layer estimates is at:

https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/blob/master/flow/util/correlateRC.py

Which not only creates more accurate estimates for each layer, but creates better RC values used before global routing (set_wire_rc).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

8 participants