Skip to content

Conversation

legendecas
Copy link
Member

@legendecas legendecas commented Jul 25, 2021

This backport contains following commits:

report: generates report on threads with no isolates

PR-URL: #38994
Reviewed-By: Gireesh Punathil gpunathi@in.ibm.com
Reviewed-By: Anna Henningsen anna@addaleax.net
Reviewed-By: James M Snell jasnell@gmail.com

src: fix fatal errors when a current isolate not exist

napi_fatal_error and node watchdog trigger fatal error but rather
running on a thread that hold no current isolate.

PR-URL: #38624
Reviewed-By: Michael Dawson midawson@redhat.com

@github-actions github-actions bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. report Issues and PRs related to process.report. v14.x labels Jul 25, 2021
@legendecas legendecas requested review from cjihrig, gireeshpunathil and himself65 and removed request for cjihrig and gireeshpunathil July 25, 2021 16:42
@legendecas legendecas force-pushed the backport-to-14/38994 branch from bfb6773 to 60b98e8 Compare July 25, 2021 16:47
@legendecas legendecas requested a review from mhdawson July 25, 2021 16:49
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@legendecas legendecas added node-api Issues and PRs related to the Node-API. and removed node-api Issues and PRs related to the Node-API. labels Jul 26, 2021
@targos
Copy link
Member

targos commented Aug 8, 2021

Can you please rebase?

@targos
Copy link
Member

targos commented Aug 29, 2021

ping @legendecas

@legendecas legendecas force-pushed the backport-to-14/38994 branch from 60b98e8 to 550f5d9 Compare August 30, 2021 16:08
@legendecas
Copy link
Member Author

@targos updated! :)

napi_fatal_error and node watchdog trigger fatal error but rather
running on a thread that hold no current isolate.

PR-URL: nodejs#38624
Reviewed-By: Michael Dawson <midawson@redhat.com>
PR-URL: nodejs#38994
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
@targos targos force-pushed the backport-to-14/38994 branch from 550f5d9 to 3cab19c Compare September 1, 2021 17:33
targos pushed a commit that referenced this pull request Sep 3, 2021
napi_fatal_error and node watchdog trigger fatal error but rather
running on a thread that hold no current isolate.

PR-URL: #38624
Backport-PR-URL: #39515
Reviewed-By: Michael Dawson <midawson@redhat.com>
targos pushed a commit that referenced this pull request Sep 3, 2021
PR-URL: #38994
Backport-PR-URL: #39515
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
@targos
Copy link
Member

targos commented Sep 3, 2021

Landed in a74032a...f69c934

@targos targos closed this Sep 3, 2021
@legendecas legendecas deleted the backport-to-14/38994 branch March 25, 2022 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. report Issues and PRs related to process.report.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants