nodejs / nodejs/node

`wpt/test-wasm-jsapi` sometimes crashes on AIX

Open
#62,647 12 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

aix flaky-test
Dominant language
JavaScript
Stars
122k
Forks
37.3k
Avg merge
4d 2h
Merged PRs (30d)
283

Description

Test

wpt/test-wasm-jsapi

Platform

AIX

Console output
21:53:20 ok 5280 wpt/test-user-timing # TODO : Fix flaky test
21:53:20   ---
21:53:20   duration_ms: 917.03200
21:53:20   ...
21:53:21 not ok 5281 wpt/test-wasm-jsapi
21:53:21   ---
21:53:21   duration_ms: 1557.51600
21:53:21   severity: crashed
21:53:21   exitcode: -5
21:53:21   stack: |-
21:53:21     [SKIPPED] esm-integration/global-exports-live-bindings.tentative.any.js: Live bindings unsupported pending V8 WebAssemblyModuleRecord
...
21:53:22     [PASS] basic 4
21:53:22     (node:29032854) ExperimentalWarning: Importing WebAssembly module instances is an experimental feature and might change at any time
21:53:22     (Use `node --trace-warnings ...` to show where the warning was created)
21:53:22     (node:29032854) ExperimentalWarning: vm.USE_MAIN_CONTEXT_DEFAULT_LOADER is an experimental feature and might change at any time
21:53:22     (Use `node --trace-warnings ...` to show where the warning was created)
21:53:22     (node:29032854) ExperimentalWarning: Importing WebAssembly module instances is an experimental feature and might change at any time
21:53:22     (node:29032854) ExperimentalWarning: vm.USE_MAIN_CONTEXT_DEFAULT_LOADER is an experimental feature and might change at any time
21:53:22     (Use `node --trace-warnings ...` to show where the warning was created)
21:53:22     (node:29032854) ExperimentalWarning: Importing WebAssembly module instances is an experimental feature and might change at any time
21:53:22     (node:29032854) ExperimentalWarning: vm.USE_MAIN_CONTEXT_DEFAULT_LOADER is an experimental feature and might change at any time
21:53:22     (Use `node --trace-warnings ...` to show where the warning was created)
21:53:22     (node:29032854) ExperimentalWarning: Importing WebAssembly module instances is an experimental feature and might change at any time
21:53:22     (node:29032854) ExperimentalWarning: vm.USE_MAIN_CONTEXT_DEFAULT_LOADER is an experimental feature and might change at any time
21:53:22     (Use `node --trace-warnings ...` to show where the warning was created)
21:53:22     (node:29032854) ExperimentalWarning: Importing WebAssembly module instances is an experimental feature and might change at any time
21:53:22     (node:29032854) ExperimentalWarning: vm.USE_MAIN_CONTEXT_DEFAULT_LOADER is an experimental feature and might change at any time
21:53:22     (Use `node --trace-warnings ...` to show where the warning was created)
21:53:22     (node:29032854) ExperimentalWarning: vm.USE_MAIN_CONTEXT_DEFAULT_LOADER is an experimental feature and might change at any time
21:53:22     (Use `node --trace-warnings ...` to show where the warning was created)
21:53:22     (node:29032854) ExperimentalWarning: vm.USE_MAIN_CONTEXT_DEFAULT_LOADER is an experimental feature and might change at any time
21:53:22     (Use `node --trace-warnings ...` to show where the warning was created)
21:53:22     (node:29032854) ExperimentalWarning: vm.USE_MAIN_CONTEXT_DEFAULT_LOADER is an experimental feature and might change at any time
21:53:22     (Use `node --trace-warnings ...` to show where the warning was created)
21:53:22     (node:29032854) ExperimentalWarning: Importing WebAssembly module instances is an experimental feature and might change at any time
21:53:22     (node:29032854) ExperimentalWarning: vm.USE_MAIN_CONTEXT_DEFAULT_LOADER is an experimental feature and might change at any time
21:53:22     (Use `node --trace-warnings ...` to show where the warning was created)
21:53:22     (node:29032854) ExperimentalWarning: Importing WebAssembly module instances is an experimental feature and might change at any time
21:53:22     (node:29032854) ExperimentalWarning: vm.USE_MAIN_CONTEXT_DEFAULT_LOADER is an experimental feature and might change at any time
21:53:22     (Use `node --trace-warnings ...` to show where the warning was created)
21:53:22     (node:29032854) ExperimentalWarning: Importing WebAssembly module instances is an experimental feature and might change at any time
21:53:22     
21:53:22     
21:53:22     #
21:53:22     # Fatal error in , line 0
21:53:22     # Check failed: ptr == address.
21:53:22     #
21:53:22     #
21:53:22     #
21:53:22     #FailureMessage Object: 1135e9ad0
21:53:22     ----- Native stack trace -----
21:53:22     
21:53:22   ...
...

21:53:52 Failed tests:
21:53:52 out/Release/node --experimental-wasm-modules /home/iojs/build/workspace/node-test-commit-aix/nodes/aix72-power9/test/wpt/test-wasm-jsapi.mjs
21:53:52 gmake[1]: *** [Makefile:619: test-ci] Error 1
Build links
Additional information

We suspect it is the same check/comment https://github.com/nodejs/node/blob/5b6091ce31068e1c86f863273cdd397ecb47e3b7/deps/v8/src/base/platform/platform-aix.cc#L194-L199 as referenced in https://github.com/nodejs/node/pull/62470.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by running out/Release/node --experimental-wasm-modules .../test/wpt/test-wasm-jsapi.mjs on AIX and inspect deps/v8/src/base/platform/platform-aix.cc around lines 194-199. Compare the failing console output and linked CI runs to determine why the ptr == address check fails intermittently. Done means the test no longer crashes on the affected AIX jobs.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, nodejs, wasm
Domain
operating-systems, testing-qa
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.