Skip to content

turbo-tasks: add info_span for blocking waits during task restore#92878

Merged
sokra merged 1 commit into
canaryfrom
sokra/blocking-span
Apr 16, 2026
Merged

turbo-tasks: add info_span for blocking waits during task restore#92878
sokra merged 1 commit into
canaryfrom
sokra/blocking-span

Conversation

@sokra
Copy link
Copy Markdown
Member

@sokra sokra commented Apr 16, 2026

What?

Adds a tracing::info_span!("blocking") around listener.wait() calls in ExecuteContextImpl::task() when a task is still being restored from backing storage.

Why?

When a task is locked for restoration by another thread, the current thread spins in a loop and calls listener.wait() until notified. CPU time accounting treats this waiting time as active CPU usage, which skews profiling results.

By wrapping the wait in a "blocking" span, the tracing infrastructure can correctly exclude this idle blocking time from CPU time — ensuring CPU metrics reflect actual computation rather than time spent waiting for another thread to finish restoring a task.

How?

Import tracing::info_span and enter a "blocking" span immediately before listener.wait() in the restore-wait loop in turbopack/crates/turbo-tasks-backend/src/backend/operation/mod.rs. The span is dropped when the wait returns (or when the loop continues to re-check the condition).

@nextjs-bot nextjs-bot added created-by: Turbopack team PRs by the Turbopack team. Turbopack Related to Turbopack with Next.js. labels Apr 16, 2026
@nextjs-bot
Copy link
Copy Markdown
Contributor

Stats from current PR

✅ No significant changes detected

📊 All Metrics
📖 Metrics Glossary

Dev Server Metrics:

  • Listen = TCP port starts accepting connections
  • First Request = HTTP server returns successful response
  • Cold = Fresh build (no cache)
  • Warm = With cached build artifacts

Build Metrics:

  • Fresh = Clean build (no .next directory)
  • Cached = With existing .next directory

Change Thresholds:

  • Time: Changes < 50ms AND < 10%, OR < 2% are insignificant
  • Size: Changes < 1KB AND < 1% are insignificant
  • All other changes are flagged to catch regressions

⚡ Dev Server

Metric Canary PR Change Trend
Cold (Listen) 455ms 455ms ██▁▁▁
Cold (Ready in log) 442ms 442ms ▃▂▅▄█
Cold (First Request) 833ms 834ms ██▁▁▁
Warm (Listen) 456ms 455ms ▅█▅▅▅
Warm (Ready in log) 441ms 443ms █▆▁▁▇
Warm (First Request) 344ms 344ms ▆▂▁▁█

⚡ Production Builds

Metric Canary PR Change Trend
Fresh Build 4.005s 3.962s ▁▂▇▄▅
Cached Build 3.990s 3.995s ▃▂▆▅▅
📦 Production Builds (Webpack) (Legacy)

📦 Production Builds (Webpack)

Metric Canary PR Change Trend
node_modules Size 494 MB 494 MB ▁▁▁██
📦 Bundle Sizes

Bundle Sizes

⚡ Turbopack

Client

Main Bundles
Canary PR Change
06mj1cjffbema.js gzip 153 B N/A -
07rxhp_1_g4mu.js gzip 13.1 kB N/A -
08avva-dy02e7.js gzip 10.4 kB N/A -
0cz1d0mv5g_q7.js gzip 39.4 kB 39.4 kB
0fli3_wppnim5.js gzip 12.9 kB N/A -
0k09jwjeb-tki.js gzip 13.8 kB N/A -
0kb7_ep3r1z0_.js gzip 10.1 kB N/A -
0kw8xgqdrilf6.js gzip 8.56 kB N/A -
0ojkk2e654xsc.js gzip 8.59 kB N/A -
0rq5rs-1005rh.js gzip 153 B N/A -
0s0uyifjfof0q.js gzip 158 B N/A -
0u39fhh3ddthr.js gzip 65.5 kB N/A -
0wxpyd8r-vipl.js gzip 1.47 kB N/A -
0xy2fhla48_rd.js gzip 9.24 kB N/A -
0za7yrc-tln56.js gzip 154 B N/A -
1-z_xm7pzpbeh.js gzip 49.2 kB N/A -
10wqsvi2mgfmi.js gzip 9.82 kB N/A -
125gb_4ewjnzs.js gzip 152 B N/A -
16lhqjoqbznyg.js gzip 220 B 220 B
16vepdkipri3r.js gzip 8.51 kB N/A -
17n96uu6y1pxq.js gzip 8.6 kB N/A -
18y4_8-9or0mn.js gzip 8.51 kB N/A -
1elt1qium-r2m.css gzip 115 B 115 B
1gq145j3kps-h.js gzip 8.62 kB N/A -
1nsh-mbn0e-se.js gzip 8.56 kB N/A -
1tf4y7qct3ewa.js gzip 70.8 kB N/A -
1tsrrp1tdngti.js gzip 13.3 kB N/A -
2__-e_ym8n788.js gzip 450 B N/A -
22o6xd9_ywdu6.js gzip 233 B N/A -
24mphfbzursun.js gzip 155 B N/A -
2a3xjv2p0uj2p.js gzip 154 B N/A -
2b3b2sv8dxvo9.js gzip 153 B N/A -
2bxsha87pu536.js gzip 151 B N/A -
2k1cds6lrifbh.js gzip 153 B N/A -
2kvj8yrfznmwx.js gzip 5.69 kB N/A -
2of-5y0sqhkxz.js gzip 156 B N/A -
2qv7m7xjnokgr.js gzip 8.58 kB N/A -
2qxc87fru49ft.js gzip 154 B N/A -
342ijzvrpe53h.js gzip 2.29 kB N/A -
3zsi1-ps1ouw8.js gzip 167 B N/A -
44un3--wmqiyh.js gzip 7.61 kB N/A -
turbopack-0c..-7wn.js gzip 4.19 kB N/A -
turbopack-0d..rmm7.js gzip 4.2 kB N/A -
turbopack-0m..6pai.js gzip 4.19 kB N/A -
turbopack-0y..4bjl.js gzip 4.19 kB N/A -
turbopack-16..q_3_.js gzip 4.18 kB N/A -
turbopack-17.._8hv.js gzip 4.19 kB N/A -
turbopack-1f.._vwi.js gzip 4.17 kB N/A -
turbopack-1k..bq3_.js gzip 4.18 kB N/A -
turbopack-1l..qsiw.js gzip 4.19 kB N/A -
turbopack-1n..2a5r.js gzip 4.19 kB N/A -
turbopack-24..5fo1.js gzip 4.18 kB N/A -
turbopack-28..8hh5.js gzip 4.18 kB N/A -
turbopack-3n..v4f9.js gzip 4.19 kB N/A -
turbopack-44..6j-l.js gzip 4.19 kB N/A -
03_szppy5nakp.js gzip N/A 49.2 kB -
0arkbdqpxc37i.js gzip N/A 8.6 kB -
0bz-xifewa17d.js gzip N/A 8.63 kB -
0tvekitj587fh.js gzip N/A 8.51 kB -
0yvk6-wi8e9wh.js gzip N/A 13.3 kB -
0z83a1om5rvtt.js gzip N/A 7.61 kB -
1-jqyfc89tixo.js gzip N/A 1.46 kB -
13b8pizp51zmp.js gzip N/A 168 B -
14t1kneseb8th.js gzip N/A 2.3 kB -
15ay9q51-1-2k.js gzip N/A 157 B -
15sb1-dsqfk_j.js gzip N/A 8.59 kB -
1ab2xruymo-oj.js gzip N/A 449 B -
1je1ker05fo3n.js gzip N/A 153 B -
1tu25qtsmfhar.js gzip N/A 9.82 kB -
1u94mm8sw6744.js gzip N/A 156 B -
1vein_gnv3mwr.js gzip N/A 8.56 kB -
1wzrm0xjjbzn5.js gzip N/A 10.1 kB -
1z3g0uaqtv9_3.js gzip N/A 8.56 kB -
25a1yz7zua29z.js gzip N/A 13.8 kB -
25q7qqln3x7oh.js gzip N/A 70.8 kB -
2b5_wa7s0v9sw.js gzip N/A 155 B -
2bi5hx402juv-.js gzip N/A 8.58 kB -
2hy56297fog9u.js gzip N/A 8.52 kB -
2rla12r8eqgd0.js gzip N/A 155 B -
2u_rpxq3tzytl.js gzip N/A 233 B -
2vyc50etj_81v.js gzip N/A 155 B -
368lim5wq0o0r.js gzip N/A 12.9 kB -
37dvo72x4ua9j.js gzip N/A 153 B -
3drqjohogojbw.js gzip N/A 5.69 kB -
3f6a2ii1-iby5.js gzip N/A 160 B -
3g8l1m2-o-ewi.js gzip N/A 13.1 kB -
3i7exvik0n5yc.js gzip N/A 153 B -
3jmkxsnxg0nrh.js gzip N/A 10.4 kB -
3ocqruj6n3lte.js gzip N/A 156 B -
3ofo5u8c77470.js gzip N/A 158 B -
3pd1x1agjirzx.js gzip N/A 65.5 kB -
3wpp8nvyoj121.js gzip N/A 9.24 kB -
440ab-z364m9w.js gzip N/A 156 B -
turbopack-00..hd-5.js gzip N/A 4.19 kB -
turbopack-01..ln3e.js gzip N/A 4.19 kB -
turbopack-0s..wxiz.js gzip N/A 4.19 kB -
turbopack-0v..7-yv.js gzip N/A 4.2 kB -
turbopack-11..0ou8.js gzip N/A 4.19 kB -
turbopack-1g..76at.js gzip N/A 4.19 kB -
turbopack-1j..ix-k.js gzip N/A 4.19 kB -
turbopack-2b..uv1p.js gzip N/A 4.19 kB -
turbopack-2m..fmyo.js gzip N/A 4.18 kB -
turbopack-2t..8zcl.js gzip N/A 4.19 kB -
turbopack-2y..9ar1.js gzip N/A 4.19 kB -
turbopack-3b..5oz0.js gzip N/A 4.18 kB -
turbopack-3d..lx_r.js gzip N/A 4.17 kB -
turbopack-3e..7qu1.js gzip N/A 4.19 kB -
Total 465 kB 465 kB ⚠️ +71 B

Server

Middleware
Canary PR Change
middleware-b..fest.js gzip 716 B 717 B
Total 716 B 717 B ⚠️ +1 B
Build Details
Build Manifests
Canary PR Change
_buildManifest.js gzip 433 B 433 B
Total 433 B 433 B

🔄 Shared (bundler-independent)

Runtimes
Canary PR Change
app-page-exp...dev.js gzip 346 kB 346 kB
app-page-exp..prod.js gzip 192 kB 192 kB
app-page-tur...dev.js gzip 346 kB 346 kB
app-page-tur..prod.js gzip 192 kB 192 kB
app-page-tur...dev.js gzip 342 kB 342 kB
app-page-tur..prod.js gzip 190 kB 190 kB
app-page.run...dev.js gzip 343 kB 343 kB
app-page.run..prod.js gzip 190 kB 190 kB
app-route-ex...dev.js gzip 77 kB 77 kB
app-route-ex..prod.js gzip 52.5 kB 52.5 kB
app-route-tu...dev.js gzip 77.1 kB 77.1 kB
app-route-tu..prod.js gzip 52.6 kB 52.6 kB
app-route-tu...dev.js gzip 76.7 kB 76.7 kB
app-route-tu..prod.js gzip 52.3 kB 52.3 kB
app-route.ru...dev.js gzip 76.6 kB 76.6 kB
app-route.ru..prod.js gzip 52.3 kB 52.3 kB
dist_client_...dev.js gzip 324 B 324 B
dist_client_...dev.js gzip 326 B 326 B
dist_client_...dev.js gzip 318 B 318 B
dist_client_...dev.js gzip 317 B 317 B
pages-api-tu...dev.js gzip 43.9 kB 43.9 kB
pages-api-tu..prod.js gzip 33.5 kB 33.5 kB
pages-api.ru...dev.js gzip 43.9 kB 43.9 kB
pages-api.ru..prod.js gzip 33.5 kB 33.5 kB
pages-turbo....dev.js gzip 53.3 kB 53.3 kB
pages-turbo...prod.js gzip 39.1 kB 39.1 kB
pages.runtim...dev.js gzip 53.3 kB 53.3 kB
pages.runtim..prod.js gzip 39.1 kB 39.1 kB
server.runti..prod.js gzip 62.9 kB 62.9 kB
Total 3.06 MB 3.06 MB ✅ -1 B
📎 Tarball URL
https://vercel-packages.vercel.app/next/commits/22b00202dbe875c8bc164aab7e55746d54227f21/next

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Apr 16, 2026

Merging this PR will improve performance by 3.34%

⚡ 1 improved benchmark
✅ 16 untouched benchmarks
⏩ 3 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation packages-bundle.js[full] 984.5 ms 952.7 ms +3.34%

Comparing sokra/blocking-span (e247c39) with canary (f497222)

Open in CodSpeed

Footnotes

  1. 3 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@sokra sokra requested a review from lukesandberg April 16, 2026 14:51
@sokra sokra marked this pull request as ready for review April 16, 2026 15:35
@sokra sokra requested a review from mischnic April 16, 2026 15:35
@sokra sokra merged commit 4dd1fcb into canary Apr 16, 2026
188 of 191 checks passed
@sokra sokra deleted the sokra/blocking-span branch April 16, 2026 15:43
@github-actions github-actions Bot added the locked label May 1, 2026
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators May 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

created-by: Turbopack team PRs by the Turbopack team. locked Turbopack Related to Turbopack with Next.js.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants