Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps: update V8 to 10.1 #42657

Closed
wants to merge 21 commits into from
Closed

deps: update V8 to 10.1 #42657

wants to merge 21 commits into from

Conversation

Copy link
Member

@targos targos commented Apr 8, 2022

If Windows 32 bits CI still fails, I will open another PR to temporarily make that platform experimental, as discussed in this week's TSC meeting.
Edit: failure confirmed: https://ci.nodejs.org/job/node-compile-windows/44578/nodes=win-vs2019-x86/console

@targos targos added the semver-major label Apr 8, 2022
@nodejs-github-bot
Copy link

@nodejs-github-bot nodejs-github-bot commented Apr 8, 2022

Review requested:

@nodejs-github-bot nodejs-github-bot added build needs-ci v8 engine labels Apr 8, 2022
@targos
Copy link
Author

@targos targos commented Apr 8, 2022

@aduh95
Copy link

@aduh95 aduh95 commented Apr 8, 2022

@targos
Copy link
Author

@targos targos commented Apr 9, 2022

Now I don't understand why test-repl-mode fails everywhere with 10.1. It didn't last week...
I'm wrong. It failed with 10.1 too: https://ci.nodejs.org/job/node-test-pull-request/43341/

@targos
Copy link
Author

@targos targos commented Apr 9, 2022

If we look at canary, it failed once on March 25 and was fixed on March 26. Checking the commits that landed on V8 in between...

@targos
Copy link
Author

@targos targos commented Apr 9, 2022

Added a cherry-pick of v8/v8@a2cae21

@targos
Copy link
Author

@targos targos commented Apr 9, 2022

@nodejs/build assuming #42666 lands, how do we handle x86 Windows in CI?

@richardlau
Copy link

@richardlau richardlau commented Apr 10, 2022

@nodejs/build assuming #42666 lands, how do we handle x86 Windows in CI?

Opened nodejs/build#2921 to not run the Windows x86 builds for Node.js 18 for now.

@joyeecheung
Copy link

@joyeecheung joyeecheung commented Apr 12, 2022

FYI https://chromium-review.googlesource.com/c/v8/v8/+/3575468 has landed and it should fix the near heap limit callback issue without a fix in Node.js (I ran a CI with it before and the test passed https://ci.nodejs.org/job/node-test-commit/52817/)

targos and others added 8 commits Apr 12, 2022
Major V8 updates are usually API/ABI incompatible with previous
versions. This commit adapts NODE_MODULE_VERSION for V8 10.1.

Refs: https://github.com/nodejs/CTC/blob/master/meetings/2016-09-28.md
Original commit message:

    [testrunner] delete ancient junit compatible format support

    Testrunner has ancient support for JUnit compatible XML output.

    This CL removes this old feature.

    R=mstarzinger@chromium.org,jgruber@chromium.org,jkummerow@chromium.org
    CC=​machenbach@chromium.org

    Bug: v8:8728
    Change-Id: I7e1beb011dbaec3aa1a27398a5c52abdd778eaf0
    Reviewed-on: https://chromium-review.googlesource.com/c/1430065
    Reviewed-by: Jakob Gruber <jgruber@chromium.org>
    Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
    Commit-Queue: Tamer Tas <tmrts@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#59045}

Refs: v8/v8@bd019bd

PR-URL: nodejs#32116
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
This should be semver-patch since actual invocation is version
conditional.

PR-URL: nodejs#32116
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
PR-URL: nodejs#35415
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>

deps: patch for v8 on windows

PR-URL: nodejs#40010
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

deps: patch v8 for vs2019 in std17

PR-URL: nodejs#40060
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

deps: workaround debug link error on Windows

PR-URL: nodejs#38807
Refs: nodejs#38788
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ash Cripps <acripps@redhat.com>
`handler-outside-simulator.cc` uses inline assembly, which is not
supported by MSVC.

PR-URL: nodejs#40488
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
We are not ready to migrate yet.

Refs: nodejs/node-v8#214

PR-URL: nodejs#40907
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
targos added a commit that referenced this issue Apr 26, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: #42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue Apr 26, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue Apr 26, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue Apr 27, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue Apr 27, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue Apr 28, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue Apr 28, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue Apr 29, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue Apr 29, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue Apr 30, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue Apr 30, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 1, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 1, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 2, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 2, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 3, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 3, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 4, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 4, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 5, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 5, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 6, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 6, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 7, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 7, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 8, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 8, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 9, 2022
Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
Refs: nodejs/node#42375

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
nodejs-github-bot pushed a commit to nodejs/node-v8 that referenced this issue May 9, 2022
Remove call to `memfd_create`.
The function that references it is only used for V8 testing.

PR-URL: nodejs/node#42657
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready build needs-ci semver-major v8 engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants