-
Notifications
You must be signed in to change notification settings - Fork 148
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
Implement barrett reduction for ed25519 exponent field #43
Comments
Or maybe we are mistaken to assume base |
Oh, hrm, interesting. I don't think I have anything that relies on |
The guess I have with |
Ugh, why are there so many variations on Barrett Reduction. Not only does this version split apart the shifts, it also does reduction modulo |
In this version, we split up the integer division so that we are less likely to overflow in intermediate computations. This is still not the version in HAC 14.42; that version also does early reduction modulo b^(k+1). This is work towards #43
@andres-erbsen Is it acceptable to replace
with (3) ? I think this would make the proofs much easier, and it seems like, by choosing |
From http://cacr.uwaterloo.ca/hac/about/chap14.pdf This should take care of most of #43, at least the specification on Z part of it.
In this version, we split up the integer division so that we are less likely to overflow in intermediate computations. This is still not the version in HAC 14.42; that version also does early reduction modulo b^(k+1). This is work towards #43
From http://cacr.uwaterloo.ca/hac/about/chap14.pdf This should take care of most of #43, at least the specification on Z part of it.
What needs to be done to complete this, on top of #69? |
Closed by #77. |
modulus
m = 2^252 + 27742317777372353535851937790883648493
, lengthk = 256
, (precomputedmu = 1852673427797059126777135760139006525645217721299241702126143248052143860224795
,) inputa < 2^512
. As in https://github.com/floodyberry/supercop/blob/master/crypto_sign/ed25519/ref/sc25519.c#L41.The text was updated successfully, but these errors were encountered: