Discussion :

[_kal_]

Bonjour,

Je souhaite synchroniser deux processus avec le signal SIGUSR1 et la fonction sigsuspend. Par ailleurs, je rencontre un comportement étrange. Afin d'illustrer mon problème, j'ai créer un petit programme de test dont voici le code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
 
/*
** File     : test_sigsuspend.c
** Author   : kal
** Created  : Fri Oct 12 19:46:12 2007
*/
 
#include <stdio.h>
#include <stdlib.h>
#include <signal.h>
#include <sys/types.h>
#include <unistd.h>
 
void
handler (int signal)
{
  printf("coucou, je suis %d, reception du signal %d\n", getpid(), signal);
}
 
int
main (int argc, char **argv)
{
  sigset_t mask;
 
  /* bloque SIGUSR1 */
  sigemptyset (&mask);
  sigaddset (&mask, SIGUSR1);
  sigprocmask ( SIG_BLOCK, &mask, NULL);
 
  /* envoi de SIGUSR1 */
  kill(getpid(), SIGUSR1);
 
  /* handler SIGUSR1 */
  struct sigaction sa;
  sa.sa_handler = handler;
  sigaction(SIGUSR1,&sa,NULL);
 
  /* attente de SIGUSR1 */
  sigfillset (&mask);
  sigdelset (&mask, SIGUSR1);
  sigsuspend (&mask);
 
  printf("Fin\n");
 
  return EXIT_SUCCESS;
}

Le code est plutot simple : je masque le signal SIGUSR1, je m'envoi SIGUSR1. Il est donc placé dans la liste des signaux pendants. Je me met en attente de celui-ci grace à sigsuspend.

Puisqu'un handler est définit pour SIGUSR1, sigsuspend est censé retourner en restaurant le masque des signaux. Or, voici le résultat de l'exécution :

kal@kal-laptop:~/POSIX$ ./test_sigsuspend
coucou, je suis 6878, reception du signal 10
Erreur de segmentation (core dumped)

Par ailleurs, si j'ajoute une variable non utilisée de type sigset_t, alors le comportement est correct :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
 
/*
** File     : test_sigsuspend.c
** Author   : kal
** Created  : Fri Oct 12 19:46:12 2007
*/
 
#include <stdio.h>
#include <stdlib.h>
#include <signal.h>
#include <sys/types.h>
#include <unistd.h>
 
void
handler (int signal)
{
  printf("coucou, je suis %d, reception du signal %d\n", getpid(), signal);
}
 
int
main (int argc, char **argv)
{
  sigset_t mask, toto;
 
  /* bloque SIGUSR1 */
  sigemptyset (&mask);
  sigaddset (&mask, SIGUSR1);
  sigprocmask ( SIG_BLOCK, &mask, NULL);
 
  /* envoi de SIGUSR1 */
  kill(getpid(), SIGUSR1);
 
  /* handler SIGUSR1 */
  struct sigaction sa;
  sa.sa_handler = handler;
  sigaction(SIGUSR1,&sa,NULL);
 
  /* attente de SIGUSR1 */
  sigfillset (&mask);
  sigdelset (&mask, SIGUSR1);
  sigsuspend (&mask);
 
  printf("Fin\n");
 
  return EXIT_SUCCESS;
}

kal@kal-laptop:~/POSIX$ ./test_sigsuspend
coucou, je suis 6914, reception du signal 10
Fin

Quelqu'un peut il m'expliquer ce comportement étrange ?

[_kal_]

Ah oui, j'ai oublié de préciser : l'environnement utilisé est Linux avec le compilateur GCC 4.1.3.

Kal

[hegros]

Il faut peut être appeler kill pour l'envoi du signal APRES avoir mis en place le handler qui est fait par l'appel de sigaction.

[_kal_]

Non, ceci ne change rien. J'ai tout de même testé pur m'en assurer et j'obtiens toujours une erreur de segmentation.

[Fred83]

Bonjour,

Avant d'appeler sigaction, tu dois initialiser ta structure sa.
Rajoute sa.sa_flags=0;

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
  /* handler SIGUSR1 */
  struct sigaction sa;
  sa.sa_handler = handler;
  printf("avant initialisation sa.sa_flags=%d\n",sa.sa_flags);  /* Pour voir */
  sa.sa_flags=0; /* Ligne a rajouter */
  sigaction(SIGUSR1,&sa,NULL);
 
  /* attente de SIGUSR1 */

Bonne continuation

[_kal_]

Merci beaucoup, Fred83!

En effet, cela fonctionne, je n'ai pu besoin de déclarer une variable non utilisée de type sigset_t. Par contre, je ne comprend pas pourquoi il faut initialiser sa_flags, et pourquoi il influence sur le comportement de sigsuspend.

Merci encore,
Kal

[Fred83]

Ta variable inutilisée n'a rien a voir avec le comportement de sigaction.
En déclarant une variable supplémentaire, tu tapais dans une autre case mémoire et donc passais un paramètre différent à la fonction appelée.

Comme expliqué dans le lien ci dessous, sa_flags affecte le comportement du signal donc c'est quand même mieux de l'initialiser.

http://opengroup.org/onlinepubs/0079...sigaction.html

Par exemple, Si tu initialise sa_flags à SA_SIGINFO, ou laisse une valeur initialisée qui contiendra de temps en temps les bits correspondant a cette valeur, le système appelle un handler avec trois paramètres. Si ton handler n'en dépile qu'une, ça va se vautrer à tous les coups.

Pour t'en convaincre, essaye un handler comme ci dessous avec sa_flags=SA_SIGINFO puis sa_flags=0

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
void handler(int signal,siginfo_t *Info,void *Inutile)
{
  printf("coucou, je suis %d, reception du signal %d\n", getpid(), signal);
  printf("Signal envoye depuis pid=%d\n",Info->si_pid);
}

En conclusion, tu peut faire plein de choses, faut juste savoir quoi.

Bonnes expérimentations

[_kal_]

Bonsoir,

En effet, j'ai compris l'idée. Merci beaucoup.

Kal