Discussion :

[miloutch]

Salut à tous !
voilà j'ai un programme qui créé des processus fils, mais malheureusement, le waitpid ne marche que de temps en temps, ca dépend des jours.
j'ai utilisé psignal, et mon fils est terminé par Unknown signal 0
je vous montre le code et son resultat :

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
 
void attend(int n, int *etat, int *fpid)
{
	int i;
	int status;
	for(i=0;i<n;i++)
	{
		waitpid(fpid[i],&status,1);//attend pour tous les fils
		psignal(status,"termination par ");
		printf("errno = '%X'\n",errno);
		if(WIFEXITED(status))
			printf("terminé normalement\n");
		if(WIFSIGNALED(status))
		{
			printf("signal non recepte\n");
			printf("No signal = %d\n",WTERMSIG(status));
		}
 
	}
}

Code :

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

1
2
3
4
5
6
 
[miloutch@ubuntu:~/Bureau/shell2]$firefox
termination*par*:*Unknown*signal*0
errno*=*'0'
terminé*normalement
[miloutch@ubuntu:~/Bureau/shell2]$

et bien entendu, le programme lancé ne s'est pas terminé !

Comme je vou l'ai dit, j'ai remarqué que le waitpid attend mon fils que de temps en temps.
un jour ca marche, un jour ca marche pas. Je n'ai aucune idée d'ou ca peut venir.
en tout cas même si ca dépend des jours, ca marche, donc je suppose que l'erreur ne vient pas du code que j'ai fait pour le waitpid.
si quelqu'un a déjà eu ce genre de problèmes, je suis preneur pour une solution.
merci

[matafan]

Tu n'utilises pas waitpid() correctement. Et psignal() non plus d'ailleurs.

Le dernier argument de waitpid() est un champs de bits qui indique des options. Tu n'es pas sensé passer un nombre sous forme littérale, mais plutôt utiliser les macros définies dans wait.h : WNOHANG, WUNTRACED, WCONTINUED... Si tu passes 1, alors c'est comme si tu avais passé WNOHANG, qui demande justement à waitpid() de ne pas attendre... Ce qui n'est pas ce que tu veux. Dans ton cas il faut probablement faire waitpid(fpid[i], &status, 0).

Le premier argument de psignal() est un numéro de signal. Toi tu lui passes le status retourné par waitpid(), qui n'est pas une numéro de signal. Il est donc normal d'avoir "Unknown signal 0". Le numéro de signal est WTERMSIG(status), à n'utiliser que si WIFSIGNALED(status) est vrai.

[miloutch]

j'ai modifié mon programme comme tu l'as dit, et du coup le processus a l'air de se couper normalement, sauf que c'est pas normal
du coup il ne m'affiche pas les erreurs signal WTERMSIG.
le 0 à la place de waitpid ne change rien malheureusement
merci quand même pour la réponse

[matafan]

Si tu étais plus clair ça donnerait peut-être envie de te répondre... Parce que "le processus a l'air de se couper normalement, sauf que c'est pas normal", moi, je ne comprend pas.

Le plus simple c'est que tu donnes ton code actuel.

[miloutch]

Non mais franchement ca vient pas du code, c'est pas possible, puisque le wait ne peut pas ne pas marcher , puis tout d'un coup fonctionner. ca doit venir de l'environnement. bref je me debrouillerai. je mets le code à tous hasard :

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
 
do
{
	if(c[j].pipe!=0)//si il y a pipe
	{
		if(pipe(fpipe) == -1)//on le crée
		{
			perror("erreur lors de la création du pipe");
			exit(0);
		}
		c[j].sortie=fpipe[1] ;//redirige la sortie vers le pipe
	}
	if ((fpid[j] = fork()) < 0)//crée le processus fils
	{
		perror("erreur lors de la création d'un fils");
		exit(0);
	}
	//processus fils
	if(fpid[j]==0)
	{
		redirection(c+j);//vérifie les redirections et les mets en place
		if(c[j].pipe!=0)//si il y avait pipe
			close(fpipe[0]);//on ferme la sortie
		execute(c[j]);//execute la commande
	}
	if(c[j].pipe!=0)//si il y avait pipe
	{  
		close(fpipe[1]);//on ferme l'entrée
		c[j+1].entree=fpipe[0];//redirige l'entrée de la commande suivante vers le pipe  
	}
	c[j+1].sortie=-1;//on coupe la sortie de la prochaine commande
	j++;//commande suivante
}while(c[j-1].pipe);//on parcours toutes les commandes (un | crée une nouvelle commande)
 
//si en mode background, on attend pas , sinon on attend
if(bg!=1)
{
	attend(j,NULL,fpid);
}

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
 
//attend le(s) processus fils en cours, n =  nombre de pipe
void attend(int n, int *etat, int *fpid)
{
	int i;
	int status;
	for(i=0;i<n;i++)
	{
		waitpid(fpid[i],&status,0);//attend pour tous les fils
		psignal(WTERMSIG(status),"termination par ");
		printf("errno = '%X'\n",errno);
		if(WIFEXITED(status))
			printf("terminé normalement\n");
		if(WIFSIGNALED(status))
		{
			printf("signal non recepte\n");
			printf("No signal = %d\n",WTERMSIG(status));
		}
 
	}
}

[jabbounet]

Non mais franchement ca vient pas du code, c'est pas possible, puisque le wait ne peut pas ne pas marcher , puis tout d'un coup fonctionner. ca doit venir de l'environnement. bref je me debrouillerai. je mets le code à tous hasard :

Ces fonctions existent depuis les débuts d'unix, elles sont aussi utilisé par tout un tas d'applications/services de ton os, si elles ne marchaient pas ton os serait probablement instable.
Quand elles sont utilisées correctement elles ne posent pas de problèmes.



Dans "attend" tu ne vérifie pas que tu es dans le processus père quand tu appelle wait, ce qui fait que les fils aussi appellent wait.
Et comme fork retourne 0 quand tu es le processus fils, l'appel de waipid dans ce cas attendra la fin d'un fils du fils ....

[jabbounet]

personnellement je partirai sur une solution dans ce style dans le cas d'un père générant plusieurs fils qu'il doit ensuite attendre.

Ce code est repris et adapté d'un vieux post de ce forum, plus des exemples de man page trouvé sur le net. je n'ai pas de compilateur sous la main, mais le principe est là.

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
 
int i;
int isFils = 0;
pid_t tabPid[NB_PID];
for (i=0; i < NB_PID; i++)
{
    switch(tabPid[i]=fork())
    {
          case -1: /* Erreur de fork - Cas à traiter */
                perror("fork"); exit(EXIT_FAILURE);
          case 0: // Fils
                fils(i); // appel de la fonction fils
                isFils = 1;  
                break;
          default: 
                printf("fils %d crée avec le pid %d\n",i,tabPid[i]);
                break;
    }
}
 
if (isFils == 0) { /* nous sommes dans le père, avec mode background */
   int status;
   for(i = 0; i < NB_PID ; i ++) {
	 pid_t w = waitpid(tabPid[i], &status, 0);
         if (w == -1) { perror("waitpid"); exit(EXIT_FAILURE); }
         if (WIFEXITED(status)) {
                printf("exited, status=%d\n", WEXITSTATUS(status));
         } else if (WIFSIGNALED(status)) {
                printf("killed by signal %d\n", WTERMSIG(status));
         } else if (WIFSTOPPED(status)) {
                printf("stopped by signal %d\n", WSTOPSIG(status));
         } 
   }
}

je n'ai pas repris ton code avec les pipes pour ne pas alourdir la partie fork/waitpid

[miloutch]

merci beaucoup pour le code, je regarde ca demain

[miloutch]

gros boulet que je suis !
bon bah j'ai trouvé la source du problème lol.
en fait, lorsque une instance de firefox est déjà ouverte lorsque je tape la commande, ca lance une nouvelle fenetre firefox et le wait n'attend pas. si il n'y en a pas, il lance firefox et attend que je le coupe pour continuer.
je crois que c'est normal, non ?

[Médinoc]

oui.