Skip to content

ECDSA benchmark#2967

Merged
mmagician merged 3 commits into
nextfrom
andrew-ecdsa-benchmark
May 22, 2026
Merged

ECDSA benchmark#2967
mmagician merged 3 commits into
nextfrom
andrew-ecdsa-benchmark

Conversation

@Fumuran
Copy link
Copy Markdown
Contributor

@Fumuran Fumuran commented May 21, 2026

This small PR adds a benchmark for the transaction which consumes a P2ID note with ECDSA signing, making it possible to compare it with Falcon signed transactions.

Below are results of the two transactions, both are consuming P2ID note, but using Falcon and ECDSA signature schemes respectively:

Execution times:

Execute transaction/Execute transaction which consumes single P2ID note with Falcon signing
                        time:   [4.5632 ms 4.5978 ms 4.6334 ms]

Execute transaction/Execute transaction which consumes single P2ID note with ECDSA signing
                        time:   [2.3323 ms 2.3432 ms 2.3551 ms]

Execution and prove times:

Execute and prove transaction/Execute and prove transaction which consumes single P2ID note with Falcon signing
                        time:   [1.3733 s 1.3866 s 1.4013 s]
                        change: [+0.1142% +1.1048% +2.4073%] (p = 0.07 > 0.05)
                        No change in performance detected.

Execute and prove transaction/Execute and prove transaction which consumes single P2ID note with ECDSA signing
                        time:   [367.96 ms 372.98 ms 378.15 ms]
                        change: [-0.8070% +1.0269% +2.9173%] (p = 0.27 > 0.05)
                        No change in performance detected.

Execution cycle counts:

    /// Falcon
    "epilogue": {
      "total": 72815,
      "auth_procedure": 71322,
      "after_tx_cycles_obtained": 603
    },
    "trace": {
      "core_rows": 78170,
      "chiplets_rows": 61069,
      "range_rows": 20367,
      "chiplets_shape": {
        "hasher_rows": 58320,
        "bitwise_rows": 392,
        "memory_rows": 2301,
        "kernel_rom_rows": 55,
        "ace_rows": 0
      }
    }

    /// ECDSA
    "epilogue": {
      "total": 4823,
      "auth_procedure": 3330,
      "after_tx_cycles_obtained": 603
    },
    "trace": {
      "core_rows": 10178,
      "chiplets_rows": 18845,
      "range_rows": 1201,
      "chiplets_shape": {
        "hasher_rows": 17728,
        "bitwise_rows": 392,
        "memory_rows": 669,
        "kernel_rom_rows": 55,
        "ace_rows": 0
      }
    }

Closes: #2084

@Fumuran Fumuran added the pr-from-maintainers PRs that come from internal contributors or integration partners. They should be given priority label May 21, 2026
@Fumuran Fumuran marked this pull request as ready for review May 21, 2026 23:20
@mmagician mmagician added this pull request to the merge queue May 22, 2026
Merged via the queue into next with commit 5f92765 May 22, 2026
21 checks passed
@mmagician mmagician deleted the andrew-ecdsa-benchmark branch May 22, 2026 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-from-maintainers PRs that come from internal contributors or integration partners. They should be given priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create benchmark for the ECDSA signing

2 participants