Re: [LEDE-DEV] The procd daemon does not handle correctly the instances

2016-09-29 Thread John Crispin
o:j...@phrozen.org] > Sent: donderdag 29 september 2016 7:43 > To: Lebleu Pierre ; > lede-dev@lists.infradead.org > Subject: Re: [LEDE-DEV] The procd daemon does not handle correctly the > instances > > this might actually be solvable int he scripts, will need to take a closer

Re: [LEDE-DEV] The procd daemon does not handle correctly the instances

2016-09-29 Thread Lebleu Pierre
t; rather than "set" when we specify the instance. Pierre -Original Message- From: John Crispin [mailto:j...@phrozen.org] Sent: donderdag 29 september 2016 7:43 To: Lebleu Pierre ; lede-dev@lists.infradead.org Subject: Re: [LEDE-DEV] The procd daemon does not handle correctly the

Re: [LEDE-DEV] The procd daemon does not handle correctly the instances

2016-09-28 Thread John Crispin
44 >> --- a/package/system/procd/files/procd.sh >> +++ b/package/system/procd/files/procd.sh >> @@ -72,11 +72,15 @@ _procd_open_service() { >> } >> >> _procd_close_service() { >> + local method="set" >> + [ -n "$1" ] &am

Re: [LEDE-DEV] The procd daemon does not handle correctly the instances

2016-09-28 Thread John Crispin
_trigger > - _procd_ubus_call set > + > + _procd_ubus_call $method > } > > _procd_add_array_data() { > > > With this patch, it seems to work. > > Pierre > -Original Message- > From: John Crispin [mailto:j...@phrozen.org] > Sent: di

Re: [LEDE-DEV] The procd daemon does not handle correctly the instances

2016-09-28 Thread Lebleu Pierre
_procd_ubus_call set + + _procd_ubus_call $method } _procd_add_array_data() { With this patch, it seems to work. Pierre -Original Message----- From: John Crispin [mailto:j...@phrozen.org] Sent: dinsdag 27 september 2016 17:48 To: Lebleu Pierre ; lede-dev@lists.infradead.org Sub

Re: [LEDE-DEV] The procd daemon does not handle correctly the instances

2016-09-27 Thread John Crispin
On 27/09/2016 17:34, Lebleu Pierre wrote: > Hi all, > > I found a bug in the daemon process management. Indeed, when we have several > instances of one daemon such as "dnsmasq" and we want to restart only one > instance, the other instance is killed by procd. It seems procd updates the > seco

[LEDE-DEV] The procd daemon does not handle correctly the instances

2016-09-27 Thread Lebleu Pierre
Hi all, I found a bug in the daemon process management. Indeed, when we have several instances of one daemon such as "dnsmasq" and we want to restart only one instance, the other instance is killed by procd. It seems procd updates the second instance and then, kill it. Please find the patch :