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

Work towards getting NodeJS running #161

Closed
wants to merge 3 commits into from
Closed

Conversation

fwsGonzo
Copy link
Collaborator

No description provided.

@fwsGonzo fwsGonzo force-pushed the get_node_working branch 3 times, most recently from 1fcf325 to 1b63ec5 Compare June 18, 2024 18:22
@fire
Copy link
Contributor

fire commented Jul 6, 2024

What is blocking this?

@fwsGonzo
Copy link
Collaborator Author

fwsGonzo commented Jul 6, 2024

futex emulation is very minimal in libriscv, which presents problems when there is complex thread-synchronization schemes

The problem might be small, or big. No way to know without discovering what it is.

@fwsGonzo
Copy link
Collaborator Author

This PR is a piece-wise solution to this PR, where v8 is now fully functional inside the sandbox, with Isolates, JIT-compilation and WebAssembly modules

@fire
Copy link
Contributor

fire commented Jul 10, 2024

Task complete?

@fwsGonzo
Copy link
Collaborator Author

fwsGonzo commented Jul 10, 2024

No, now comes the second part where I have to try to see if NodeJS even makes sense. We'll see. Hopefully it doesn't use threading extensively, or there's a switch to disable it.
https://nodejs.org/api/cli.html

@fwsGonzo
Copy link
Collaborator Author

Added improved Futex facility emulation: #199

It's aborting on something still, but hopefully I got a lot further on the startup process. I can use strace on the emulator to see most of what's happening, but for the next abort, likely remote GDB is needed.

@fwsGonzo
Copy link
Collaborator Author

This PR has been superseded by #202

@fwsGonzo fwsGonzo closed this Jul 11, 2024
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.

2 participants