Problème avec sigsuspend

Version imprimable

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:

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 :

Citation:

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:

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; }
Citation:

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

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

12/10/2007, 20h39
_kal_

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

Kal
12/10/2007, 22h35
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.
12/10/2007, 22h36
_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.

Bonjour,

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

Code:

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

13/10/2007, 10h56
_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
13/10/2007, 19h33
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:

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
13/10/2007, 22h11
_kal_

Bonsoir,

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

Kal