Snapshot.load("").warmup! aborts the process (Check failed: !artifacts)
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 76/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- javascript, ruby
- Área
- backend, embedded-iot
Línea de trabajo
Empieza con snapshot() en ext/mini_racer_extension/mini_racer_v8.cc:946 y compara cómo maneja los blobs vacíos con v8_thread_init en la línea 451. Reproduce el problema con el fragmento de Ruby del issue y comprueba si las pruebas de snapshot existentes cubren el warmup. Se considera terminado cuando un snapshot vacío cargado se calienta sin abortar, o genera MiniRacer::SnapshotError, mientras el comportamiento de Context.new descrito sigue siendo válido.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Calling warmup! on an empty loaded snapshot aborts inside V8. Using the same empty snapshot with Context.new works and evaluates 1+1 to 2.
Reproduce
require "mini_racer"
MiniRacer::Snapshot.load("").warmup!("1")
puts "not reached"
What happens
#
# Fatal error in , line 0
# Check failed: !artifacts.
Exit status 133 (SIGTRAP), followed by a native backtrace through v8_thread_main.
Expected
The warmup should run from V8's built-in snapshot, as Context.new does for an empty blob, or raise MiniRacer::SnapshotError.
Cause
snapshot() passes the blob to v8::SnapshotCreator for every warmup, including when its size is zero (existing_blob = is_warmup ? &blob : nullptr at ext/mini_racer_extension/mini_racer_v8.cc:946). In contrast, v8_thread_init sets params.snapshot_blob only for a non-empty blob (ext/mini_racer_extension/mini_racer_v8.cc:451). Passing nullptr for an empty warmup blob would match the Context.new behavior. This change has not been tested.
Environment
- mini_racer 0.22.1 with libv8-node 24.12.0.1 (V8 13.6.233.17-node.37)
- Ruby 3.4.2 x86_64-linux
- Fedora 44
Reproduced in 3 of 3 runs.
- Lenguaje dominante
- JavaScript
- Estrellas
- 628
- Forks
- 100
- Merge medio
- 1 d 20 h
- PR fusionados (30 d)
- 2
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de rubyjs/mini_racer
-
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
rubyjs/mini_racer#450 ·
-
Snapshot.load accepts a blob from another V8 version, and Context.new then kills the processAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
rubyjs/mini_racer#448 ·
-
bug/crash platform: linux/x86
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
rubyjs/mini_racer#422 · 10 comentarios ·
-
platform: darwin/x86
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
rubyjs/mini_racer#388 · 4 comentarios ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
rubyjs/mini_racer#376 ·
Todos los issues de rubyjs/mini_racer
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
rescript-lang/rescript#8996 ·
Los mantenedores suelen responder en 1 día
-
Add: Ministerstvoidey.ruAbiertocheck:passed logos:add
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
fossasia/visdom-dev#69 ·
Los mantenedores suelen responder en 3 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 74/100
-
建议写成 1949-Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
DeanChensj/jiangshan-map#133 ·
Los mantenedores suelen responder en 1 día