ALT Linux Sisyphus discussions
 help / color / mirror / Atom feed
From: Michael Pozhidaev <msp@altlinux.ru>
To: ALT Linux Sisyphus discussions <sisyphus@lists.altlinux.org>
Subject: Re: [sisyphus] Q: shell and process groups
Date: Wed, 05 May 2010 03:43:19 +0700
Message-ID: <m3iq73pfdk.fsf@blard.localdomain> (raw)
In-Reply-To: <t2vd0dba48d1005040843n84daa33fy42445390c47f3012@mail.gmail.com> (Anton Protopopov's message of "Tue\, 4 May 2010 19\:43\:55 +0400")

Hello, Anton Protopopov!

>     после смерти синтезатора.
>
> Но сигнал все-таки доходит?

Скорее всего нет. Исследование в strace планировали провести
завтра. Есть такое наблюдение: поскольку процесс не сразу после своего
порождения меняет свою группу, то SIGKILL шлётся в два этапа: сначала
персонально этому процессу на случай, если он ещё не сменил свой pgid, а
потом на всю группу. Если плеер указывается как exec aplay, а не просто
aplay, то проблема сразу же исчезает. То есть персональный SIGKILL
всегда попадает в цель, а вот SIGKILL на группу иногда даёт промахи.
В случае без exec SIGKILL прибивает /bin/sh, а aplay продолжает работать.

> Ну, например, еще есть race: setpgid() в плейере может запускаться раньше,
> чем setpgrp() в синтезаторе. В этом случае он, очевидно, не сможет установить
> лидером синтезатор, а вернет ошибку.

О, действительно! Только, скорее, наоборот. В мане по setpgid говорится,
что pgid должен быть existing process group. Existing, как понимаю,
имеется ввиду, что существует корень группы. 
В этом race setpgid() провалится, а setpgrp() сам себе отработатм, но
плеер в группу не попадёт. Спасибо, приделаю фикс и о результатах
доложу.

В качестве фикса, наверно, выделю плеер в свою собственную группу, к
которой и будем обращаться. Ещё раз спасибо!

-- 
Michael Pozhidaev. Tomsk, Russia. E-mail: msp@altlinux.ru
Russian info page: http://www.marigostra.ru/



  parent reply	other threads:[~2010-05-04 20:43 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-03  4:28 Michael Pozhidaev
2010-05-04 14:39   ` Michael Pozhidaev
2010-05-04 20:43       ` Michael Pozhidaev [this message]
2010-05-06  8:58           ` Michael Pozhidaev

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=m3iq73pfdk.fsf@blard.localdomain \
    --to=msp@altlinux.ru \
    --cc=sisyphus@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

ALT Linux Sisyphus discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/sisyphus/0 sisyphus/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 sisyphus sisyphus/ http://lore.altlinux.org/sisyphus \
		sisyphus@altlinux.ru sisyphus@altlinux.org sisyphus@lists.altlinux.org sisyphus@lists.altlinux.ru sisyphus@lists.altlinux.com sisyphus@linuxteam.iplabs.ru sisyphus@list.linux-os.ru
	public-inbox-index sisyphus

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.sisyphus


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git