janet-lang / janet

A dynamic language and bytecode vm
https://janet-lang.org
MIT License
3.57k stars 229 forks source link

Linux event loop hanging #1452

Closed anastasiatimit closed 5 months ago

anastasiatimit commented 5 months ago

Can't upgrade janet version due to stream or event error, not sure, no error is thrown, just app can't finish loading files. The app is built on alpine linux.

After some testing, noticed that this is the first commit that made the application unresponsive, I think src/core/ev.c change started to change the behaviour, as building next commit and replacing this file from the working commit solved the issue in next commit.

Do you have any suggestion please what to do?

Thank you

bakpakin commented 5 months ago

Hi @anastasiatimit, what version of Janet are you trying to run? Does trying the latest master fix the problem? There was a bug involving epoll that would sometimes miss connections that is only fixed on the latest master

anastasiatimit commented 5 months ago

@bakpakin, yes, I tried master, but same issue persists

bakpakin commented 5 months ago

In that case we need a repro for the bug. Not enough to go off of here yet.

sogaiu commented 5 months ago

Possibly related might be this issue filed at the spork repository.

bakpakin commented 5 months ago

@anastasiatimit Please check the latest master and see if this fixes your problem

anastasiatimit commented 5 months ago

@bakpakin yes, such a great catch, can't reproduce the issue anymore, thank you!

sogaiu commented 5 months ago

@anastasiatimit Great to hear things are working!

May be the issue can be closed?