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

Estimated ofmap bandwidth is quite high #107

Open
Oliverwang11 opened this issue Jul 12, 2024 · 2 comments
Open

Estimated ofmap bandwidth is quite high #107

Oliverwang11 opened this issue Jul 12, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@Oliverwang11
Copy link

Hello,

First thanks a lot for your contribution and useful simulator.

I have a question about the estimated ofmap bandwidth, it seems quite high for me.
I used the tool to simulate a simple fully connected layer which has 1 1 1 1 256 72 1 hyperparameters. In an output stationary model, I set the output buffer to 4 KB. For a double buffer, 2KB is the active buffer size. And a 2KB buffer is enough for my 72 B output right? So we don't need very high bandwidth to shift the things output.

However, when I run the code, I get an estimated 72 bytes per cycle bandwidth, and I check the code, it seems like in the estimated model, the bandwidth for ofmap is directly assigned to the column length of the PE array, which is quite high.

Did I misunderstand something? Since when I run in estimation model, I get quite a high estimated bandwidth (72) for zero stalls, but when I set the bw to 10, there seems like still no stall for the fc layer which I run.

Best wishes!
Thanks in advanced!

@ritikraj7
Copy link
Member

Hi @Oliverwang11, I could not understand your hyperparameters. Can you please provide the exact config and topology files? I can try to reproduce the ofmap bandwidths that you are getting.

@ritikraj7
Copy link
Member

@Oliverwang11
We only estimate ifmap and filter bandwidths through read_buffer_estimate_bw.py file. We currently don't have write_buffer_estimate_bw.py, therefore the numbers that you are getting is invalid. We are working on adding write_buffer_estimate_bw.py file.

@ritikraj7 ritikraj7 added the bug Something isn't working label Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants