Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@hngenc
Copy link
Member

@hngenc hngenc commented May 22, 2023

This is an incremental release:

  • Support for rectangular convolutions
  • New Timeloop configurations included in repo
  • Area improvements for runtime-configurable dataflows
  • Faster build and elaboration times

hngenc and others added 30 commits March 19, 2022 10:14
Offloads depthwise convolution scheduling onto the loop-unroller FSM. This improves MobileNet performance by 2-3x.

Also adds more printfs to `mobilenet.c` to show the performance of each individual layer.
This enables correct functionality of gemmini when sbus width != 16 bytes.
When testing ucb-bar/chipyard#1239, the new FIRRTL compiler errors when modules are named module. This avoids this issue.
Some of the updates made for the MLSys tutorial are also useful for general users. This PR merges in those changes.
PE was not synthesizing properly on the FPGA since it was synthesizing multiple mac units in a single PE.
I added a new mac unit class and instantiate a single mac unit per PE to reduce area over head.

This addresses #262

Co-authored-by: Hasan Genc <[email protected]>
Co-authored-by: Eric Dong <[email protected]>
Remove parenthesized forms of asUInt(), asBool(), asSInt(), orR(), andR(), zext()

Co-authored-by: Abraham Gonzalez <[email protected]>
Co-authored-by: Hasan Genc <[email protected]>
Co-authored-by: joey0320 <[email protected]>
Copy link
Contributor

@jerryz123 jerryz123 left a comment

Choose a reason for hiding this comment

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

Is libgemmini being updated to keep the spike-model up-to-date?

@hngenc
Copy link
Member Author

hngenc commented May 22, 2023

@jerryz123 libgemmini should already be up-to-date with Gemmini's dev branch (and if it isn't, then the CI should expose that). If we change the ISA again in the future, we'll update libgemmini again

@jerryz123
Copy link
Contributor

@jerryz123 libgemmini should already be up-to-date with Gemmini's dev branch (and if it isn't, then the CI should expose that). If we change the ISA again in the future, we'll update libgemmini again

Thanks

@jerryz123
Copy link
Contributor

Can you do a merge commit here? It makes the chipyard side a bit easier.

@hngenc hngenc merged commit be2e9f2 into master May 22, 2023
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.

10 participants