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

[WIP] src: v8 snapshot command #199

Closed
wants to merge 6 commits into from

Conversation

mmarchini
Copy link
Contributor

DO NOT LAND YET

[P.S.: The code is a mess right now, I'm opening a PR to get feedback on how the feature should look like]

This PR introduces a new command to create a V8 snapshot from a core dump. Right now the snapshot is not fully functional, but it works on Chrome DevTools. The goal is to enable the use of Chrome DevTools alongside llnode.

We still need to address #195 and #197 before making it work properly. Right now it can be used with the command v8 snapshot, which will create a core.heapsnapshot in the current working directory. The generated file can be open by Chrome DevTools, but it doesn't show a list of retainers yet.

@indutny
Copy link
Member

indutny commented May 16, 2018

Neat! Just in case, you may find this helpful: https://github.com/indutny/core2dump (if it is still relevant)

@mmarchini
Copy link
Contributor Author

mmarchini commented May 21, 2018

I found indutny/core2dump when working on this, but couldn't make it work :( (probably because so many things have changed in V8 in the last two years)

@joyeecheung
Copy link
Member

It would be cool to be able to expose a toHeapSnapshot() method in #147 I plan to revamp that PR a bit soon to expose structured objects instead of strings.

@mmarchini
Copy link
Contributor Author

Closing since I haven't had the time to move this forward. If anyone wants to work on it, feel free to pick it up! (Although I think this will be much easier to accomplish once our JS API is out of experimental).

@mmarchini mmarchini closed this Nov 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants