1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-12-22 07:14:47 -05:00
denoland-deno/runtime
Nathan Whitaker 8282c38fe0
fix(cli): increase size of blocking task threadpool on windows (#26465)
Fixes #26179.

The original error reported in that issue is fixed on canary, but in
local testing on my windows machine, `next build` would just hang
forever.

After some digging, what happens is that at some point in next build,
readFile promises (from `fs/promises` ) just never resolve, and so next
hangs.

It turns out the issue is saturating tokio's blocking task thread pool.
We previously limited the number of blocking threads to 32, and at some
point those threads are all in use and there's no thread available for
the file reads.

What's taking up all of those threads? The answer turns out to be
`tokio::process`. On windows, child process stdio uses the blocking
threadpool: https://github.com/tokio-rs/tokio/pull/4824. When you poll
the child's stdio on windows, it spawns a blocking task per poll, and
calls `std::io::Read::read` in the blocking context. That call can block
until data is available.
Putting it all together, what happens is that Next.js spawns `2 * the
number of CPU cores` deno child subprocesses to do work. We implement
`child_process` with `tokio::process`. When the child processes' stdio
get polled, blocking tasks get spawned, and those blocking tasks might
block until data is available. So if you have 16 cores (as I do), there
are going to be potentially >32 blocking task threadpool threads taken
just by the child processes. That leaves no room for other tasks to make
progress

---

To fix this, for now, increase the size of the blocking threadpool on
windows. 4 * the number of CPU cores should be enough to leave room for
other tasks to make progress.

Longer term, this can be fixed more properly when we handroll our own
subprocess code (needed for detached processes and additional pipes on
windows).
2024-10-22 12:52:18 -07:00
..
examples/extension refactor: bury descriptor parsing in PermissionsContainer (#25936) 2024-09-30 09:19:24 -04:00
js fix(unstable/worker): ensure import permissions are passed (#26101) 2024-10-10 14:01:42 +01:00
ops refactor(runtime/ops): use concrete error types (#26409) 2024-10-22 01:41:08 -07:00
permissions chore: forward v2.0.2 release commit to main (#26376) 2024-10-18 03:12:49 +02:00
Cargo.toml refactor(runtime/ops): use concrete error types (#26409) 2024-10-22 01:41:08 -07:00
clippy.toml feat(compile): unstable npm and node specifier support (#19005) 2023-05-10 20:06:59 -04:00
code_cache.rs fix: use hash of in-memory bytes only for code cache (#23966) 2024-05-24 10:15:46 -04:00
errors.rs refactor(runtime/ops): use concrete error types (#26409) 2024-10-22 01:41:08 -07:00
fmt_errors.rs fix: improve suggestions and hints when using CommonJS modules (#26287) 2024-10-15 23:25:24 +00:00
fs_util.rs refactor: use deno_path_util (#25918) 2024-09-28 07:55:01 -04:00
inspector_server.rs fix(runtime): send ws ping frames from inspector server (#26352) 2024-10-17 15:19:43 +00:00
js.rs refactor: remove snapshotting from deno_runtime (#21794) 2024-01-10 16:30:50 +01:00
lib.rs fix: better error for Deno.UnsafeWindowSurface, correct HttpClient name, cleanup unused code (#25833) 2024-09-24 07:04:52 -07:00
permissions.rs refactor: bury descriptor parsing in PermissionsContainer (#25936) 2024-09-30 09:19:24 -04:00
README.md fix (doc): Typo in runtime/README.md (#20020) 2023-12-13 17:24:32 +00:00
shared.rs BREAKING(buffer): remove Deno.Buffer (#25441) 2024-09-06 18:28:05 +10:00
snapshot.rs refactor: improve node permission checks (#26028) 2024-10-04 20:55:41 +01:00
tokio_util.rs fix(cli): increase size of blocking task threadpool on windows (#26465) 2024-10-22 12:52:18 -07:00
web_worker.rs refactor(runtime/ops): use concrete error types (#26409) 2024-10-22 01:41:08 -07:00
worker.rs refactor: bury descriptor parsing in PermissionsContainer (#25936) 2024-09-30 09:19:24 -04:00
worker_bootstrap.rs BREAKING: Remove --unstable flag (#25522) 2024-09-09 23:44:29 +02:00

deno_runtime crate

crates docs

This is a slim version of the Deno CLI which removes typescript integration and various tooling (like lint and doc). Basically only JavaScript execution with Deno's operating system bindings (ops).

Stability

This crate is built using battle-tested modules that were originally in the deno crate, however the API of this crate is subject to rapid and breaking changes.

MainWorker

The main API of this crate is MainWorker. MainWorker is a structure encapsulating deno_core::JsRuntime with a set of ops used to implement Deno namespace.

When creating a MainWorker implementors must call MainWorker::bootstrap to prepare JS runtime for use.

MainWorker is highly configurable and allows to customize many of the runtime's properties:

  • module loading implementation
  • error formatting
  • support for source maps
  • support for V8 inspector and Chrome Devtools debugger
  • HTTP client user agent, CA certificate
  • random number generator seed

Worker Web API

deno_runtime comes with support for Worker Web API. The Worker API is implemented using WebWorker structure.

When creating a new instance of MainWorker implementors must provide a callback function that is used when creating a new instance of Worker.

All WebWorker instances are descendents of MainWorker which is responsible for setting up communication with child worker. Each WebWorker spawns a new OS thread that is dedicated solely to that worker.