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

mldsa_ctrl just waits for PC incrementation #43

Closed
cbatlubis opened this issue Nov 22, 2024 · 1 comment
Closed

mldsa_ctrl just waits for PC incrementation #43

cbatlubis opened this issue Nov 22, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@cbatlubis
Copy link

This report is just for documentation purpose as it was already fixed.

In line 1421 in module mldsa_ctrl the generated signatures are checked and depending on this evaluation the program counter is incremented by either 1 (signature invalid) or 2 (signature valid).
When the generated signature is valid, then only the done signal, highlighting the completion of this process, needs to be asserted.
This is done when the program counter reaches the value of the local parameter MLDSA_SIGN_GEN_E.
MLDSA_SIGN_GEN_E is defined in module mldsa_ctrl_pkg line 440 as MLDSA_SIGN_GEN_S + 9.
We are wondering why the base value is incremented by 9 and not by 2.
From what we see in the waveform the design just waits the difference of clock cycles without doing anything.

So it was just a non-functional observation that does not affect the functionality of the design.

@mojtaba-bisheh mojtaba-bisheh added the enhancement New feature or request label Nov 22, 2024
@mojtaba-bisheh
Copy link
Collaborator

fixed by #39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants