Skip to content

[tb] Add support to trace retired TCDM operations#71

Open
vakrishna-iis wants to merge 25 commits intopulp-platform:mms/xqueue-cleanfrom
vakrishna-iis:cgra-xqueue-fix
Open

[tb] Add support to trace retired TCDM operations#71
vakrishna-iis wants to merge 25 commits intopulp-platform:mms/xqueue-cleanfrom
vakrishna-iis:cgra-xqueue-fix

Conversation

@vakrishna-iis
Copy link

@vakrishna-iis vakrishna-iis commented Dec 2, 2022 •
edited
Loading

Summarize the changes concisely

Changelog

Added

  • Added the ability to Trace the operations retired by the TCDM adapters. When XQueue = 1, Queue operations, AMO and Load/Store operations are traced; when Xqueue = 0, only Load/Store operations are traced. This tracing capability is not supported on Verilator since a hierarchical reference to signals is not allowed in Verilator.

Checklist

  • Automated tests pass
  • Changelog updated
  • Code style guideline is observed

Please check our contributing guidelines before opening a Pull Request.

sermazz and others added 25 commits September 14, 2022 14:53
@sermazz
[config] Parametrize scratchpad banking factor
dfacf4d
@SamuelRiedel
Merge pull request pulp-platform#57 from pulp-platform/cgra-sw-fix
3a13c7b 
Parametrization fixes to systolic queues software implementation
@khovg @sermazz
[hardware] Add support for atomic Xqueue operations to TCDM adapter
738a8cf
@sermazz
[toolchain] Add toolchain support to xqueues extension (in standard a…
49bc8a5 
…tomic extension, illegal)
@khovg @sermazz
[snitch] Add xqueues extension to instruction decoder
d7bad0a
@khovg @sermazz
[hardware] Fix response acquisition
1b6ea2b
@khovg @sermazz
[hardware] Remove qpush data registers by abusing buffer slot
6224f5f
@SamuelRiedel @sermazz
[apps/hardware] Implement xqueue_test app
03d622e
@khovg @sermazz
[apps] Implement systolic matmul_xqueue (1x1 matmul)
20248cb
@khovg @sermazz
[apps] Optimize systolic matmul_xqueue for 2x2 matmul
ac43b0c
@khovg @sermazz
[apps] Improve matmul_xqueue code
51b6eb5 
- add interleaving and fix data dependency
- hotfix to enforce data dependency in dummy pop/push
@khovg @sermazz
[apps] Optimize matmul_xqueue with asm inline
5f2d0bc
@khovg @sermazz
[apps] Use 2 interleaved queues per direction in matmul_xqueue
72441bc
@khovg @sermazz
[apps] Improve matmul_xqueue code
3674ea7 
- generalize systolic_init()
- add additional grid mappings
@sermazz
[apps] Implement systolic xqueue 2d convolution
3ea900b
@khovg @sermazz
[apps] Improve conv_xqueue code
8a3b524 
- change PEs role names
- add different core mapping
- flush queues at the end of execution
@khovg @sermazz
[apps] Improve 2d conv density for conv_xqueue
e29d84c
@khovg @sermazz
[apps] Improve conv_xqueue ecode
0fbe88b 
- fix illegal multi queue pop
- increase performance via fixed cyclical pattern
- add shuffling of MACs to hide accelerator latency
@khovg @sermazz
[apps] Improve regularity of conv_xqueue
244b9a5
@sermazz
[apps] Fix license and format
4517da9
@sermazz
[CHANGELOG] Add Xqueue extension and sw kernels
4691691
@sermazz
[config] Update systolic config
8e210c7
@sermazz
[hardware] 🐛 Add write response to Xqueue TCDM adapter
26f3877
@sermazz
[software] Generalize systolic matmul NUM_CORES
2d8fa4e
[tb] Add support to trace retired TCDM operations
0c2dffe
@sermazz sermazz force-pushed the cgra-xqueue-fix branch 2 times, most recently from e7a39d9 to 58d6ecc Compare December 6, 2022 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SamuelRiedel SamuelRiedel Awaiting requested review from SamuelRiedel

@suehtamacv suehtamacv Awaiting requested review from suehtamacv

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vakrishna-iis @sermazz @SamuelRiedel @khovg