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

crypto/elliptic: mention P-521 slowdown in Go 1.18 release notes #51508

Open
pascaldekloe opened this issue Mar 6, 2022 · 3 comments
Open

crypto/elliptic: mention P-521 slowdown in Go 1.18 release notes #51508

pascaldekloe opened this issue Mar 6, 2022 · 3 comments
Labels
Documentation NeedsDecision
Milestone

Comments

@pascaldekloe
Copy link
Contributor

@pascaldekloe pascaldekloe commented Mar 6, 2022

The release notes for Go 1.18 state “P-224 and P-384 are now approximately four times faster.”, which is correct. However, it fails to mention the two times slower for P-521. I would argue that such a significant loss in performance is something people should be aware of.

@ALTree
Copy link
Member

@ALTree ALTree commented Mar 6, 2022

@ALTree ALTree changed the title elliptic: Mention P-521 slowdown in Go 1.18 release notes. crypto/elliptic: Mention P-521 slowdown in Go 1.18 release notes. Mar 6, 2022
@ALTree ALTree changed the title crypto/elliptic: Mention P-521 slowdown in Go 1.18 release notes. crypto/elliptic: mention P-521 slowdown in Go 1.18 release notes Mar 6, 2022
@ALTree ALTree added the NeedsDecision label Mar 6, 2022
@bcmills bcmills added this to the Go1.18 milestone Mar 8, 2022
@dibrinsofor
Copy link

@dibrinsofor dibrinsofor commented Mar 13, 2022

@bcmills can I take this up?

@dmitshur
Copy link
Contributor

@dmitshur dmitshur commented Mar 14, 2022

@dibrinsofor Thanks, it's fine to prepare a CL as long as you're taking into account that this issue is currently in NeedsDecision state.

@heschi heschi removed this from the Go1.18 milestone Mar 15, 2022
@heschi heschi added this to the Go1.19 milestone Mar 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation NeedsDecision
Projects
None yet
Development

No branches or pull requests

6 participants