search

zigdon / twirssi

An irssi script allowing the use of Twitter from within the IRC client.
http://twirssi.com
134 stars 26 forks source link

mitigate child processes with 100% CPU and/or zombies #76

Open mmitch opened 10 years ago

mmitch commented 10 years ago

This should work around the 100% CPU and/or zombie issues (for details see commit log). It does not fix the root cause of the endless loop in the child processes, but it kill's them, so they don't live very long.

This is by no means good code, but a works-for-me with some ugly hacks. Should probably be refined or tested by more people than just me before a merge :-) (is there an experimental or -dev branch?)

yarikoptic commented 10 years ago

FWIW -- very looking forward to see the fix. @mmitch if you could share variant 3 patch I would be glad to try it out as well

mmitch commented 10 years ago

@yarikoptic with commit https://github.com/mmitch/twirssi/commit/7e976ff59524f6b8df6f5eee3fc45b7930f990d6 variant 3 is active It's working fine for me so far: 8 successfully killed runaway processes in the last 36 hours.

sandebert commented 9 years ago

I'm also affected by this bug. I'm trying the variant 3 by @mmitch now to see it that solves it for me.

sandebert commented 9 years ago

Reporting back: 16 runaway processes killed in the last 24 hours.

jikuja commented 6 years ago

Is twirssi now stable without this PR or is there something wrong with this? I have been running mmitch@7e976ff since may 2015 without problems.

Now to get longer tweets I should update script but is it still creating runaway processes?