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

Auxiliary scripts used to perform experiments with multiple models and security. #2

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

pliniosilveira
Copy link

A part of a research from PUCRS.

@msharmavikram msharmavikram requested a review from ielhajj May 2, 2020 16:23
@ielhajj
Copy link
Member

ielhajj commented May 3, 2020

The objective of adding MIN_N_TILE is not clear. Please clarify why this is necessary.

@pliniosilveira
Copy link
Author

MIN_N_TILE is used to guarantee that it will generate instructions for at least N tiles. The tiles not used have only a halt instruction. This is used to make fair comparison between models with different number of tiles, so they can run in the exactly same puma architecture.

@ielhajj
Copy link
Member

ielhajj commented May 3, 2020

How does adding more tiles make the comparison more fair if these tiles are not doing anything?

@pliniosilveira
Copy link
Author

Because idle tiles stills counts static energy and area.

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.

3 participants