Performance regression in 67d70b4e192e..c2851ff669c5 (are-we-fast-yet run 00087)
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 44/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- rust, wasm
- 領域
- backend, performance
調査の方向性
Start by reading the linked commit, “Publish handle-table slot pointers in VMComponentContext,” and the regression guidance in docs/regressions.md. Reproduce the callgrind comparison using the supplied sightglass commands and benchmarks/cm-online-stats/cm-online-stats.wasm, then investigate changes between the two Wasmtime revisions. Done means identifying whether the commit caused the execution slowdown and documenting or fixing its cause.
索引モデルが issue の本文から書いたものです。
説明
are-we-fast-yet found 1 regression and 1 speedup in virtual cycles (callgrind) between Wasmtime 67d70b4e192e and c2851ff669c5.
- Baseline: run 00086 (metadata), Wasmtime
67d70b4e192e - This run: run 00087 (metadata), Wasmtime
c2851ff669c5
A change counts when a Mann-Whitney U test finds it significant at the 0.01 level and it clears its phase's minimum relative change:
| phase | geomean | individual |
|---|---|---|
| compilation | 0.3% | 1% |
| instantiation | 2.5% | 20% |
| execution | 0.5% | 2% |
Regressions (1)
| benchmark | phase | Δ virtual cycles | virtual cycles noise | Δ instructions retired | instructions retired noise |
|---|---|---|---|---|---|
cm-online-stats/cm-online-stats.wasm |
execution | +12.41% (+12.40% to +12.42%) | 0.015% -> 0.008% | +12.43% | 0.003% -> 0.001% |
Speedups (1)
| benchmark | phase | Δ virtual cycles | virtual cycles noise | Δ instructions retired | instructions retired noise |
|---|---|---|---|---|---|
tinygo/tinygo-regex.wasm |
execution | -2.72% (-2.75% to -2.72%) | 0.003% -> 0.029% | -0.00% | 0.001% -> 0.005% |
A positive delta is a slowdown, a negative delta is a speedup.
instructions retired counts work done rather than how well it was predicted. A regression whose instructions didn't move is often Valgrind's branch predictors aliasing on a code layout change; see docs/regressions.md.
Each noise column is how far that event's own counts spread.
Other callgrind counters that changed for these benchmarks
| benchmark | phase | counter | Δ | noise |
|---|---|---|---|---|
cm-online-stats/cm-online-stats.wasm |
execution | ll-dcache-read-misses |
-39.39% (-54.55% to -36.36%) | 36.364% -> 10.000% |
cm-online-stats/cm-online-stats.wasm |
execution | l1-dcache-read-misses |
-23.32% (-33.33% to -4.15%) | 30.316% -> 23.562% |
cm-online-stats/cm-online-stats.wasm |
execution | ll-icache-misses |
-16.34% (-24.50% to -14.40%) | 11.634% -> 3.636% |
cm-online-stats/cm-online-stats.wasm |
execution | data-writes |
+17.28% (+17.28% to +17.28%) | 0.002% -> 0.001% |
cm-online-stats/cm-online-stats.wasm |
execution | indirect-branch-misses |
-14.22% (-27.34% to -5.83%) | 22.831% -> 10.526% |
cm-online-stats/cm-online-stats.wasm |
execution | instructions-retired |
+12.43% (+12.43% to +12.43%) | 0.003% -> 0.001% |
cm-online-stats/cm-online-stats.wasm |
execution | conditional-branches |
+9.09% (+9.08% to +9.09%) | 0.005% -> 0.002% |
cm-online-stats/cm-online-stats.wasm |
execution | data-reads |
+6.30% (+6.30% to +6.30%) | 0.002% -> 0.000% |
Commits
Reproduce locally
On x86-64 Linux with Valgrind installed:
git clone https://github.com/bytecodealliance/sightglass
cd sightglass
git checkout 2d453335318d05cf42b6f3263aa68f0021985f10
rustc engines/wasmtime/build.rs -o build-engine
mkdir old new
REVISION=67d70b4e192e88faa597008560811dc7e2b1414b ./build-engine old
REVISION=c2851ff669c588563f2362dfd5d5be7518028132 ./build-engine new
cargo run --release -- benchmark --measure callgrind \
--engine old/libengine.so --engine new/libengine.so \
--engine-flags-common="-Cparallel-compilation=n --target x86_64-unknown-linux-gnu -Ccranelift-has-sse3 -Ccranelift-has-ssse3 -Ccranelift-has-sse41 -Ccranelift-has-sse42 -Ccranelift-has-popcnt -Ccranelift-has-avx -Ccranelift-has-avx2 -Ccranelift-has-fma -Ccranelift-has-bmi1 -Ccranelift-has-bmi2 -Ccranelift-has-lzcnt -Ccranelift-has-cmpxchg16b" \
-- benchmarks/cm-online-stats/cm-online-stats.wasm
- 主要言語
- Rust
- スター
- 18.8k
- フォーク
- 1.9k
- 平均マージ
- 1日 3時間
- マージ済み PR（30日）
- 382
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
bytecodealliance/wasmtime のほかの issue
-
bug wasm-proposal:exceptions wasmtime:debugging
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
bytecodealliance/wasmtime#14102 ·
メンテナーはふだん 1 日以内に返信
-
wasm-proposal:gc
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
bytecodealliance/wasmtime#13808 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 20/100
bytecodealliance/wasmtime#15193 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 46/100
bytecodealliance/wasmtime#15168 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
bytecodealliance/wasmtime#15160 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
bytecodealliance/wasmtime の issue をすべて見る
似ている issue
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
bug order-flow studies ui
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
userFRM/egui-charts#2417 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 2 日以内に返信
-
In a plain-text message, Font, Font Size, Font Color and Text Highlight Color stay live対応中かも @ragelfundionctrumnar が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
UCS-Care-Group/mailcraft#862 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
smoltcp-rs/smoltcp#1243 ·
メンテナーはふだん 1 日以内に返信