-
Notifications
You must be signed in to change notification settings - Fork 142
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
[BLS] Expose signing and verification api that takes as input an already mapped-to-point msg #591
Comments
It's not a simple change, so let's first analyze the potential performance improvement (e.g., using flamegraph of batch_verify with 10/100/1000 signatures). |
@benr-ml map-to-point is ~70 microsec (pretty much half the signing cost or ~6.5% faster aggregated sig verification - assuming you were a signer and you had it in LRU). I guess adding the LRU is easy, and the complexity is on utilizing the BLST api to input a map2point directly during verification, right? |
Thanks, let's also check this with batch_verify since that's what we're hoping to optimize (maybe it's better...). |
I thought about breaking the interfaces anyway to utilise pre computation on the pairing (see #507). We should do some experiments first though. |
This will enable caching when the same msg is used for signing and verification (or when we fall back to single sig verification in consensus). See this tweet: https://twitter.com/kostascrypto/status/1656809806198640640?s=20
The text was updated successfully, but these errors were encountered: