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

perf: optimize ecdsa circuit #461

Open
1 of 3 tasks
gbotrel opened this issue Feb 8, 2023 · 2 comments · Fixed by #497
Open
1 of 3 tasks

perf: optimize ecdsa circuit #461

gbotrel opened this issue Feb 8, 2023 · 2 comments · Fixed by #497
Assignees

Comments

@gbotrel
Copy link
Collaborator

gbotrel commented Feb 8, 2023

  • Batch verification
  • MSM in-circuit
  • Pre-compute constant points
@gbotrel gbotrel added this to the v0.9.0 milestone Feb 8, 2023
@ivokub
Copy link
Collaborator

ivokub commented Feb 16, 2023

Referencing #372 for comments:

  • implement separate variable and base point scalar mul. Base point scalar mul can take advantage of huge precomputation tables + lookups.
  • variable point scalar multiplication requires building lookup tables per-point. For a single signature this allows to implement windowed scalar multiplication and reduce the cost several times. See https://0xparc.org/blog/zk-ecdsa-2.
  • 0xPARC blog describes an interesting idea for checking only a random linear combination of signatures. This would require applying Fiat-Shamir and a lot of hashing, but may probably be more efficient than point operations.
  • When we combine two lookup tables, then can build window over two scalars at a time for ECDSA.

I have a few more links in the list:

@ivokub ivokub linked a pull request Mar 1, 2023 that will close this issue
1 task
@ivokub
Copy link
Collaborator

ivokub commented Mar 1, 2023

From #497, notes from Halo2 book

@ivokub ivokub removed this from the v0.9.0 milestone Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants