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.

Sign up for GitHub

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

Jump to bottom

[MVU/VVU] Support for double-pumped DSPs #976

Closed
wants to merge 126 commits into from
Closed

[MVU/VVU] Support for double-pumped DSPs #976

wants to merge 126 commits into from

Conversation

mmrahorovic
Copy link
Contributor

@mmrahorovic mmrahorovic commented Feb 8, 2024 •
edited
Loading

This PR is a clean-up of PR #929 where the Thresholding related changes have been excluded.
Note: this PR depends on

Adds support for double-pumped DSPs

@mmrahorovic
First changes to custom_op for RTL-based MVAU
be1503a
@mmrahorovic
Merge remote-tracking branch 'upstream/dev' into feature/dsp_packing
8265985
@mmrahorovic
[rtl custom op]: initial implementation of mvu_8sx9
afab9cd
@mmrahorovic
[rtl custom op]: testbench for mvu_8sx9
a94fc3b
@mmrahorovic
[rtl custom op]: initial implementation of flow control component for…
98f9acc 
… mvu_8sx9
@mmrahorovic
[rtl custom op]: implementation of replay buffer for mvu
96925a9
@mmrahorovic
[rtl custom op]: testbench for mvu_8sx9_axi (including axi_wrapper & …
a3d1156 
…compute kernel)
@mmrahorovic
[rtl custom op]: initial implementation of verilog wrapper for mvu_8s…
2aea664 
…x9_axi
@mmrahorovic
Merge remote-tracking branch 'upstream/dev' into feature/dsp_packing
c92e4e3
@mmrahorovic
[rtl mvu]: fix tab indentation
8b57849
@mmrahorovic
[rtl custom op]: fix to indentation
5e61f42
@mmrahorovic
[rtl custom-op]: minor changes for compiler integration
cbee193
@mmrahorovic
[rtl custom op]: moved testbenches to separate directory
ba5e77b
@mmrahorovic
[rtl custom op]: fixed output width to ACCU_WIDTH
69310b4
@mmrahorovic
[rtl custom op]: renamed file and added generic to switch between com…
cfcff00 
…pute kernels
@mmrahorovic
[rtl custom op]: renamed file and added generic to switch between com…
72b5196 
…pute kernels
@mmrahorovic
[rtl mvu]: added behavioral model DSP58
c068bb6
@mmrahorovic
[rtl mvu]: extended flow control wrapper with additional compute core…
18f94e7 
… and other minor changes
@mmrahorovic
[rtl mvu]: fix to done_len flag when SIMD dimension fully unrolled an…
6d4a0a7 
…d PyVerilator-related syntax change
@mmrahorovic
[rtl mvu tb]: updated testbench
90c547d
@mmrahorovic
[builder]: added specialize_to_rtl step and changed standalone thresh…
0c37f1f 
…old layers to be by default true
@mmrahorovic
[builder]: added specialize_to_rtl step
5ccb016
@mmrahorovic
[custom op]: added custom op MatrixVectorActivation_rtl
f099f4b
@mmrahorovic
[custom op]: added additional attribute to enable conversion to RTL (…
9a3b0fd 
…custom-op)
@mmrahorovic
[custom op]: modified ip-stitching and code generation
38aa930
@mmrahorovic
[tests]: initial version of unit test for RTL custom op and specializ…
4e44934 
…e_to_rtl transformation for MVU
@mmrahorovic
[rtl mvu]: specialized compute core for 4-bit weights and activations…
cc361d9 
… for DSP48/DSP58
@mmrahorovic
[rtl mvu]: specialized compute core for > 4-bit weights and activatio…
8eefb53 
…ns for DSP48
@mmrahorovic
[fpgadataflow transform]: initial specialize_to_rtl_layers-transform …
e7109e7 
…for MVU
@mmrahorovic
Merge remote-tracking branch 'upstream/dev' into feature/dsp_packing
d107b4d
mmrahorovic and others added 26 commits November 29, 2023 14:02
@mmrahorovic
[rtl mvu op]: minor fix to chain length estimation and enabled behavi…
40d652c 
…oral mode for rtl sim
@mmrahorovic
[rtlsim]: use pyverilator util functions
6e98bac
@mmrahorovic
[mvu vvu axi]: sign extend output tdata (byte-aligned)
5dd74ad
@mmrahorovic
[mvu core]: dsp48 convert unpacked array to packed array to work arou…
b20410b 
…nd limitation on max array indices in Verilator
@mmrahorovic
[mvu axi]: update list of deduced parameters
1c2cc0c
@mmrahorovic
[mvu custom-op]: remove lut-based implementation and update compute c…
eeb3cea 
…ore selection
@mmrahorovic
[mvu axi]: remove LUT-based compute core
0813d14
@mmrahorovic
[hls custom-op]: enable reset in sim
4892d66
@mmrahorovic
[test mvu rtl]: updated test flow (DSP58 only)
44f6e0f
@mmrahorovic
[mvu vvu axi]: reworked flow control and backpressure handling by tpr…
9b2cceb 
…eusser
@preusser
Adding DSP48E1 support for 8-bit compute. Todo: finer core differenti…
ee9f027 
…ation to select DSP48E2 explicitly again.
@preusser
Adding DSP48E1 support for 4-bit compute. Todo: finer core differenti…
3ab8296 
…ation to select DSP48E2 explicitly again.
@mmrahorovic
[mvu vvu axi]: minor bugfixes to enable VVU
bcd72ad
@mmrahorovic
[mvu tb]: created separate vvu testbench and renamed mvu_vvu_axi tb
b116733
@mmrahorovic
[mvu vvu axi]: minor fix -- define mvauin_weight_t
e1f8db1
@mmrahorovic
[folding]: first attempt to extend folding transformation to parallel…
88da696 
…ize multi-packed DSPs in MVU/VVU more efficiently
@mmrahorovic
Merge remote-tracking branch 'upstream/dev' into feature/dsp_packing_vvu
a43c731
@mmrahorovic
[mvu axi]: update list of deduced parameters
1814ea0
@mmrahorovic
[mvu vvu axi]: reworked flow control and backpressure handling by tpr…
f939c3e 
…eusser
@mmrahorovic
[mvu/vvu axi]: picked out modifications from another branch to enable…
ef12de1 
… VVU
@mmrahorovic
[mvu test]: cleaned up test
3d49ab5
@preusser @mmrahorovic
Revised control interface attributes.
105ae6f
@mmrahorovic
[rtl mvu]: extension to allow selecting PE values that are not multip…
936ef69 
…les of 4
@preusser @mmrahorovic
Starting on pumped DSP compute.
9bf7e33
@mmrahorovic
pulled latest changes related to double-pumping
80a5510
@mmrahorovic
minor fix to param
289749b
@mmrahorovic mmrahorovic mentioned this pull request Feb 9, 2024
Closed
@mmrahorovic mmrahorovic mentioned this pull request Mar 4, 2024
Merged
19 tasks
@mmrahorovic mmrahorovic closed this Mar 4, 2024
@mmrahorovic mmrahorovic mentioned this pull request Mar 14, 2024
Merged
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mmrahorovic @maltanar @preusser