Hello Tomas,

I'm glad there are users with sysvinit. This gives better testing.
Especially given that I have moved to systemd. So your bug reports will
really help.

On Sat, 2015-12-05 at 21:56 +0100, Tomas Ebenlendr wrote:
> Laptop_mode spawns lm-polling-daemon within the same process group as
> itself
> when AC is plugged out. Laptop_mode is launched in process group of
> systemd-udevd on my system. When AC is plugged in later, laptop_mode
> kills all processes in group where lm-polling-daemon is found, i.e.,
> it kills udevd and itself as well.
> 

Oh! Boy. This bug has bitten again. The problem was that the polling
daemon, if killed without the group switch, would leave out stray
processes.

But this group killing bug was found earlier too, fixed, and then
(sigh!!) re-introduced.

> I use battery level polling to trigger custom actions when battery is
> low, but not yet critical.
> 
> The solution proposed in patch is to spawn lm-polling-daemon in its
> own
> process group, using /usr/bin/setsid wrapper from util-linux package.

This looks like a good solution. This way, the children processes will
still be killed by the group option, while the parents will be detached
(systemd-udevd).


Have you tested it well ? 

Like I mentioned above, I now am on systemd init and since the last
release, if under systemd, I've replaced battery-polling-daemon with a
systemd timer.


-- 
Ritesh Raj Sarraf
RESEARCHUT - http://www.researchut.com
"Necessity is the mother of invention."

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to