jwasinger

Results 129 comments of jwasinger
trafficstars

closing this as it has become stale. I'm not sure where things will land after https://github.com/ethereum/go-ethereum/pull/32263 is completed and refactored.

Benchmarks of this PR on M2 MBP (arm): ``` BenchmarkPrecompiledBn256Add/chfast1-Gas=150-12 792328 1463 ns/op 150.0 gas/op 102.5 mgas/s 64 B/op 1 allocs/op BenchmarkPrecompiledBn256Add/chfast2-Gas=150-12 795624 1482 ns/op 150.0 gas/op 101.2 mgas/s 64...

this was merged in https://github.com/ethereum/go-ethereum/pull/32026

Ah there's one thing: We only fuzz the MSM precompiles using random inputs. I'm going to add two additional fuzzers that fuzz these with inputs that are known to be...

Yeah definitely. Actually, I need to give the added fuzzers a longer run and prob adjust the seed corpus to increase the coverage.

Oh, this is missing a cross fuzzer for g2 multiexp.

https://ci.appveyor.com/project/ethereum/go-ethereum/builds/51986510/job/6ngj0v1ktbyhbc1b ``` --- FAIL: TestSendTx (0.02s) api_backend_test.go:154: Unexpected error, want: in-flight transaction limit reached for delegated accounts, got: gapped-nonce tx from delegated accounts ```

https://ci.appveyor.com/project/ethereum/go-ethereum/builds/52136841/job/e5at09gr4e02vvkb ``` --- FAIL: TestEventFilter (0.00s) scheduler_test.go:111: Missing call to module1.Process scheduler_test.go:117: Call to module1.Process with wrong events (expected [], got [{0x148a370 0xc0000080c0 }]) scheduler_test.go:117: Call to module1.Process with wrong...

`WebSocketLargeRead`: https://github.com/ethereum/go-ethereum/actions/runs/17145047311/job/48640027449 `DeploymentLibraries`: https://github.com/ethereum/go-ethereum/actions/runs/17129689768/job/48590946294