-
Notifications
You must be signed in to change notification settings - Fork 0
Bump oss-cad-suite to latest #150
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
Conversation
nathanaelhuffman
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm. One q on the bsv change that may or may not have been intentional
4808caa to
289d6af
Compare
|
I rebased this to grab the |
|
FWIW timing of the Sidecar mainboard image is not actually failing (looking at https://buildomat.eng.oxide.computer/wg/0/artefact/01HXWF3Z0CJBQJAK8QRNTT4TPW/0Dqe7RtFEik1kKyKN0KJ28nLqvpVFPPJqV7UrhAX3xDyuYx6/01HXWF6ZA7BP58245F7WCMV75A/01HXWHGRH3XJV1GYTBMK4H1BAT/sidecar_mainboard_controller_rev_b.report.txt, which is from the CI run of current I'll take another look at why the iCE40 targets are regressing when I return because peeps in the community working on this claim the abc9 synthesis pass is supposed to be an all around win. I found last year that at these utilization levels the FF enable and set signals for groups of FFs can cause routing pressure. There are some knobs one can use to tune this, but at the time they weren't exposed in the top level |
289d6af to
5f36092
Compare
|
@arjenroodselaar you are correct! My mistake there. And thanks for that PR to yosys, I think giving folks control over any knobs that exist makes a ton of sense. We'll have to bump the toolchain against once that lands! They cut a release daily, so we could probably make a much more regular habit of doing this. I'm going to make a plan with the Hubris team on how to get these new images integrated and tested before hitting merge on this, but at this point things all seem fine. |
We updated the FPGA toolchain in oxidecomputer/quartz#150. This refreshes the images we have as part of our applications with fresh ones from CI. cc: @nathanaelhuffman @arjenroodselaar

Our current toolchain is pinned to a release of the OSS CAD Suite that is ~1.5 years old. A lot of progress has been made in that time, and using the QSFP design as a spot check, we can get quite the benefit from an update. On that design I saw a ~32% reduction in utilization and a 45% increase in Fmax, all while actually completing PnR 7x faster.
I'm opening this PR to generate images in CI. I'll look at all the PnR reports and make sure they make sense, then I'll pull images into Hubris branches and let them soak on
nilesand see if anything quirky happens. I'll be testing:If we aren't seeing any issues after a few days, I'll explore integrating these into Hubris and try to get the new images released in the dogfood/colo context ASAP to get a good long soak before R9.