Re: [PATCH] Fix runsvdir so it reaps children and avoid zombi processes when killed.

2018-01-03 Thread Laurent Bercot
Without this fix it will create zombies that might cause deadlocks No. On December 23rd, when you sent a message to the list describing your issue, I sent a reply, the first reply you received, explaining what was happening to you. Did you read it? Did ANYONE read it? Or is the thought of

Re: [PATCH] Fix runsvdir so it reaps children and avoid zombi processes when killed.

2018-01-03 Thread Ralf Friedl
Markus Gothe wrote: Can you run strace on a process invoked from the inittab? I surely have issues with doing that. As far as killing the processes it doesnt matter which signal you use; processes will become zombified forever without the patch. Yes, I can run strace on a process invoked from

RE: [**EXTERNAL**] Re: Re: [PATCH] Fix runsvdir so it reaps children and avoid zombi processes when killed.

2018-01-03 Thread Cathey, Jim
I concur. If a process's parent dies, that process will NOT be a zombie, no way no how, unless either the system's init(8) process is broken, or there is some kind of bug in the kernel. You do not need to reap your own children before dying, and you never have, not on any flavor of Unix since

Re: Re: [PATCH] Fix runsvdir so it reaps children and avoid zombi processes when killed.

2018-01-03 Thread Emmanuel Deloget
Hello, On Wed, Jan 3, 2018 at 11:04 PM, Markus Gothe wrote: > Can you run strace on a process invoked from the inittab? I surely have > issues with doing that. > ​This is definitely possible, but thou shall not forget -ff to follow forks (and prepare for an awfull lot

Re: [PATCH] udhcpd: lease conflict

2018-01-03 Thread Grant Edwards
On 2018-01-03, Jiří Prchal wrote: > Could be, may be I'm absolutely wrong. But when should function > "nobody_responds_to_arp" bee called, when ip is leased or not? And > does it make sense to reserve ip for not responding mac? Whether it "makes sense" could be debated,

SV: Re: [PATCH] Fix runsvdir so it reaps children and avoid zombi processes when killed.

2018-01-03 Thread Markus Gothe
Can you run strace on a process invoked from the inittab? I surely have issues with doing that.As far as killing the processes it doesnt matter which signal you use; processes will become zombified forever without the patch.

Re: [PATCH 0/3] infra: make it possible to install without clobbering existing utilities

2018-01-03 Thread Yann E. MORIN
Hello All! On 2017-12-28 23:49 +0100, Yann E. MORIN spake thusly: > Even though there is a hint of code preventing installation of applets > over existing utilities, this is flawed in two ways: > > - first, the non-clobbering conditional code does not account for > installation as script

Re: [PATCH] udhcpd: lease conflict

2018-01-03 Thread Denys Vlasenko
On Wed, Jan 3, 2018 at 10:30 AM, Jiří Prchal wrote: > > > On 31.12.2017 10:32, Denys Vlasenko wrote: >> >> On Fri, Dec 15, 2017 at 1:41 PM, Jiri Prchal >> wrote: >>> >>> If there is lease for MAC which is no longer connected and only one IP in

Re: [PATCH] Fix runsvdir so it reaps children and avoid zombi processes when killed.

2018-01-03 Thread Ralf Friedl
Markus Gothe wrote: Without this fix it will create zombies that might cause deadlocks, especially when respawned from the inittab and it dies because of a signal (e.g. ‘killall -9 runsvdir'). Installing a simple SIGCHLD-handler makes the problem go away. Signed-off-by: Markus Gothe

[PATCH] Fix runsvdir so it reaps children and avoid zombi processes when killed.

2018-01-03 Thread Markus Gothe
Without this fix it will create zombies that might cause deadlocks, especially when respawned from the inittab and it dies because of a signal (e.g. ‘killall -9 runsvdir'). Installing a simple SIGCHLD-handler makes the problem go away. Signed-off-by: Markus Gothe

Re: [PATCH] udhcpd: lease conflict

2018-01-03 Thread Jiří Prchal
On 31.12.2017 10:32, Denys Vlasenko wrote: On Fri, Dec 15, 2017 at 1:41 PM, Jiri Prchal wrote: If there is lease for MAC which is no longer connected and only one IP in pool, it doesn't lease to new one mac until expires the old one. This is how it is intended to