node/npm needs SSE #90

philipamour · 2018-11-14T13:57:26Z

Running npm or node returns 'Bad system call'

Mnpn · 2018-11-14T14:14:15Z

Node is on the list. https://github.com/tbodt/ish/projects/7

mattapperson · 2018-11-19T03:58:37Z

If someone (me) wanted to push this forward. Is there a recommended place to start debugging what’s needed here?

tbodt · 2018-11-19T04:01:46Z

You can try to run the program with strace logging enabled (on by default in Xcode, meson configure -Dlog=strace if you're running ish standalone), and find out what system call it's attempting to use. My hunch is that you'll find it's trying to call epoll_create, which is the entry point to an entire unimplemented subsystem that will have to be implemented before node can get anywhere at all. If you're feeling up to the task, a decent starting point would be the implementations of select and poll.

mattapperson · 2018-11-19T18:25:10Z

Spent some time looking into this... it’s just a little too much outside my current ability I’m sad to say 😞

philipamour · 2018-11-21T11:35:26Z

Wish I could help anyhow; however, my expertise is limited to software design (iXD, UI, UX, PM, ...)

_{Sent with GitHawk}

Mnpn · 2019-01-02T22:13:38Z

No longer a bad system call:

Device:~# node
Illegal instruction
Device:~# dmesg
FIXME socketcall 16
13 page fault on 0x50 at 0x567336fd
unknown ioctl 5451
unknown ioctl 5451
unknown ioctl 5451
19 illegal instruction at 0x56cc250f: f2 0f 10 02 b2 00 66 0f

kaleocheng · 2019-01-30T03:21:36Z

what's the next for this?

tbodt · 2019-01-30T03:22:00Z

SSE2 instructions. If you want to work on it, feel free.

zerkotin · 2019-02-22T10:29:59Z

is it a priority? it would be amazing to be able to run node so I can build my project and run it

Envo · 2019-03-18T15:05:10Z

Any update on this?

tbodt · 2019-03-18T15:11:29Z

no

pburtchaell · 2019-03-18T15:50:09Z

Lurker here. To all of the folks replying with "Any update? What's next? What's the priority?", please don't. It's disrespectful to the community and the owner of this project to hammer issues with rhetorical questions.

This is an open source project and the issue is clearly up for grabs (see "No one assigned" on the assignees space). If you're interested in seeing node support added, subscribe to the issue and either (a) do it yourself or (b) wait for someone else to do it.

zerkotin · 2019-03-19T00:30:55Z

A. No one means any disrespect, if anything we cared enough to search for the repo and look for bugs...
B. If I or anyone had a clue how to do that we probably would, but we don’t,
C. It’s important to be able to run JavaScript on iPad because it will open a whole ne world, and as you can see from the poll you made on the main page, most people use JavaScript html and so on

Please see it as a plea to push it up the priority ladder instead of a disrespectful comment.

tbodt · 2019-03-19T02:02:15Z

What's the poll I made on the main page?

zerkotin · 2019-03-19T02:05:29Z

Got it mixed with another page sorry...
Meant this one: http://feedback.textasticapp.com/topic/102399-which-programming-or-markup-language-do-you-use-most-often-with-textastic/

Envo · 2019-03-20T19:45:16Z

I'm sorry if i was disrespectful, it wasn't my intention.

Macmee · 2019-03-28T00:47:49Z

to get node to work, don't we need to implement SSE2 which is pretty big?

tbodt · 2019-03-28T02:14:13Z

Yeah

InukVT · 2019-04-19T17:00:11Z

Would it make sense to abuse JavaScriptCore somehow (not that I know how) to add node support for those who wants it?

I see how this can be problematic for those who doesn't care about nodes at all in iOS (such as myself)

tbodt · 2019-04-19T17:01:42Z

No, JavaScriptCore isn't a good substitute for Node. They're completely different programs, with different supported features.

bc1bb · 2019-04-19T17:03:04Z

is it a priority? it would be amazing to be able to run node so I can build my project and run it

btw Rescript (free on Apple's App Store) has a full and working environment in NodeJS

tbodt · 2019-04-19T17:04:47Z

That's pretty cool, I haven't seen it before. I bet it's really slow though.

bc1bb · 2019-04-19T17:05:40Z

i tried it one time on my iPad (6th) it was pretty good, don't know if it's what he wants

InukVT · 2019-04-19T17:06:17Z

Their sample project is super fast!

tbodt · 2020-04-27T05:29:15Z

Sorry didn't see you were a sponsor. But not in this case, my sponsor page is just a way for you to express your appreciation.

marfier · 2020-06-05T05:47:04Z

Could SpiderNode be a workaround to this issue? SpiderNode is basically Node.js decoupled from V8 to SpiderMonkey.

Edit: Just realized that SpiderNode is dead, and also compiling from source is not easy on iSH.

mfence, ucomisd, comisd, movmskpd, andpd, punpckldq, psubq, comiss, andps, andnps, maxsd, cmpsd. Enough for a node repl. #90

itsmepetrov · 2020-06-22T10:16:36Z

@tbodt I've tried to run nodejs using 55816c8 changes.

Now it's able to run nodejs but it can't close the process.

How to reproduce:

apk add nodejs
echo "console.log(123);" > index.js
node index.js

It prints 123 and gets stuck after that.

tbodt · 2020-06-22T16:08:44Z

That's for the same reason as #472 AFAICT

Fixes #472 and node hanging on exit (#90)

Fixes ctrl-c sometimes sending Node's signal handler into an infinite loop. #90

Node seems to use these archaic phrasings of CLOEXEC. This fixes the Node inspector immediately closing any connections it gets. #90

Fixes 3 / 2 returning 0 in node #90

This was causing the random node segfaults. Node does manual ASLR, so it would randomly pick an address for a memory mapping near the brk region, then later something would raise the brk and clobber that mapping, replacing it with zeroes. Shortly afterward something would crash on a null pointer. Took a lot of digging around in rr to find this. The crash is never directly connected to the brk call, of course. I set a watchpoint on the pointer that got clobbered and saw it had been zero since the page was mapped, but it took setting a watchpoint on the pointer to that pointer to find that when it was mapped before that and was valid at that point. At this point I'm ready to say node is fixed! #90

tbodt · 2020-07-06T19:51:00Z

I think this is pretty much done now.

Rexogamer · 2020-07-06T22:59:49Z

Do you get the bounty?

tbodt · 2020-07-06T23:00:07Z

I think so

saagarjha · 2020-07-06T23:04:45Z

what are you going to do with your newfound wealth

mattapperson · 2020-07-07T02:12:11Z

@tbodt if so, I missed there was a bounty. Happy to contribute to the windfall from this

tbodt · 2020-07-07T02:12:55Z

@mattapperson You may do so at https://www.bountysource.com/issues/66159647-node-npm-needs-sse (except maybe not now since the issue is closed, I have no idea how bountysource works)

mattapperson · 2020-07-07T02:27:40Z

Yea it's closed now, won't let me add to the bounty. Before you claim the next one let me know? ill just back that one

mfence, ucomisd, comisd, movmskpd, andpd, punpckldq, psubq, comiss, andps, andnps, maxsd, cmpsd. Enough for a node repl. ish-app#90

Fixes ish-app#472 and node hanging on exit (ish-app#90)

Fixes ctrl-c sometimes sending Node's signal handler into an infinite loop. ish-app#90

Node seems to use these archaic phrasings of CLOEXEC. This fixes the Node inspector immediately closing any connections it gets. ish-app#90

Fixes 3 / 2 returning 0 in node ish-app#90

This was causing the random node segfaults. Node does manual ASLR, so it would randomly pick an address for a memory mapping near the brk region, then later something would raise the brk and clobber that mapping, replacing it with zeroes. Shortly afterward something would crash on a null pointer. Took a lot of digging around in rr to find this. The crash is never directly connected to the brk call, of course. I set a watchpoint on the pointer that got clobbered and saw it had been zero since the page was mapped, but it took setting a watchpoint on the pointer to that pointer to find that when it was mapped before that and was valid at that point. At this point I'm ready to say node is fixed! ish-app#90

tbodt added the compatibility label Nov 14, 2018

This was referenced Dec 1, 2018

epoll is not implemented #142

Closed

Nodejs calls bad system call #144

Closed

tbodt mentioned this issue Mar 10, 2019

Crash when running yarn #333

Closed

VRuzhentsov mentioned this issue Apr 2, 2019

screen throws "Bad system call" on Android Pie termux/termux-packages#3156

Closed

tbodt mentioned this issue Apr 18, 2019

Crash when i try to use composer #208

Closed

tbodt mentioned this issue Feb 24, 2020

Feature: installing server executable #648

Closed

This comment has been minimized.

Sign in to view

tbodt mentioned this issue Jun 2, 2020

Is there a way to run node.js scripts in ISH ? #760

Closed

tbodt added a commit that referenced this issue Jun 9, 2020

Add a crapton of SSE instructions

55816c8

mfence, ucomisd, comisd, movmskpd, andpd, punpckldq, psubq, comiss, andps, andnps, maxsd, cmpsd. Enough for a node repl. #90

tbodt added a commit that referenced this issue Jun 25, 2020

Implement FUTEX_REQUEUE

f7ecc41

Fixes #472 and node hanging on exit (#90)

tbodt added a commit that referenced this issue Jun 25, 2020

Implement SA_RESETHAND_

260931d

Fixes ctrl-c sometimes sending Node's signal handler into an infinite loop. #90

tbodt added a commit that referenced this issue Jun 25, 2020

Implement FIOCLEX/FIONCLEX

aa8810c

Node seems to use these archaic phrasings of CLOEXEC. This fixes the Node inspector immediately closing any connections it gets. #90

tbodt added a commit that referenced this issue Jun 26, 2020

Fix vec_zero_copy when src == dst

65aca71

Fixes 3 / 2 returning 0 in node #90

tbodt closed this as completed Jul 6, 2020

Mnpn pushed a commit to Mnpn/ish that referenced this issue Oct 22, 2020

Add a crapton of SSE instructions

0452ad6

mfence, ucomisd, comisd, movmskpd, andpd, punpckldq, psubq, comiss, andps, andnps, maxsd, cmpsd. Enough for a node repl. ish-app#90

Mnpn pushed a commit to Mnpn/ish that referenced this issue Oct 22, 2020

Implement FUTEX_REQUEUE

34c986a

Fixes ish-app#472 and node hanging on exit (ish-app#90)

Mnpn pushed a commit to Mnpn/ish that referenced this issue Oct 22, 2020

Implement SA_RESETHAND_

58d981e

Fixes ctrl-c sometimes sending Node's signal handler into an infinite loop. ish-app#90

Mnpn pushed a commit to Mnpn/ish that referenced this issue Oct 22, 2020

Implement FIOCLEX/FIONCLEX

51d1d7b

Node seems to use these archaic phrasings of CLOEXEC. This fixes the Node inspector immediately closing any connections it gets. ish-app#90

Mnpn pushed a commit to Mnpn/ish that referenced this issue Oct 22, 2020

Fix vec_zero_copy when src == dst

04cd045

Fixes 3 / 2 returning 0 in node ish-app#90

specious mentioned this issue Nov 30, 2021

Illegal Instruction in NodeJS #917

Closed

Alex4386 mentioned this issue Dec 18, 2021

Feature Request: Add support for arithmetic instructions for packed decimal #1650

Open

specious mentioned this issue Dec 27, 2021

Nodejs not working on latest version (Illegal instruction) #1482

Closed

node/npm needs SSE #90

node/npm needs SSE #90

Comments

philipamour commented Nov 14, 2018

Mnpn commented Nov 14, 2018

mattapperson commented Nov 19, 2018

tbodt commented Nov 19, 2018

mattapperson commented Nov 19, 2018

philipamour commented Nov 21, 2018

Mnpn commented Jan 2, 2019 • edited

kaleocheng commented Jan 30, 2019

tbodt commented Jan 30, 2019

zerkotin commented Feb 22, 2019

Envo commented Mar 18, 2019

tbodt commented Mar 18, 2019

pburtchaell commented Mar 18, 2019 • edited

zerkotin commented Mar 19, 2019

tbodt commented Mar 19, 2019

zerkotin commented Mar 19, 2019 • edited

Envo commented Mar 20, 2019

Macmee commented Mar 28, 2019

tbodt commented Mar 28, 2019

InukVT commented Apr 19, 2019

tbodt commented Apr 19, 2019

bc1bb commented Apr 19, 2019 • edited

tbodt commented Apr 19, 2019

bc1bb commented Apr 19, 2019

InukVT commented Apr 19, 2019

This comment has been minimized.

tbodt commented Apr 27, 2020

marfier commented Jun 5, 2020 • edited

itsmepetrov commented Jun 22, 2020 • edited

tbodt commented Jun 22, 2020

tbodt commented Jul 6, 2020

Rexogamer commented Jul 6, 2020

tbodt commented Jul 6, 2020

saagarjha commented Jul 6, 2020

mattapperson commented Jul 7, 2020

tbodt commented Jul 7, 2020 • edited

mattapperson commented Jul 7, 2020

Mnpn commented Jan 2, 2019 •

edited

pburtchaell commented Mar 18, 2019 •

edited

zerkotin commented Mar 19, 2019 •

edited

bc1bb commented Apr 19, 2019 •

edited

marfier commented Jun 5, 2020 •

edited

itsmepetrov commented Jun 22, 2020 •

edited

tbodt commented Jul 7, 2020 •

edited