Discussion :

[soad029]

Bonsoir\Bonjour tout le monde !!

Voilà, je cherche à effectuer une liste de processus. Au lancement du programme, celui-ci exécute les 4 premiers processus, puis dés que l'un d'entre eux se termine un autre prend sa place.

Pour cela j'ai fais une fonction creatProcess :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
int createProcess (char* cmd, int indice) {
 
	if (fork() == 0) {
		execlp (cmd, cmd, NULL);
		perror ("Erreur dans l\'exécution de la tache %s\n",cmd);
		exit(1);
	}
	wait (NULL);
}

elle est appelé par

Code :

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

ind = createProcess (tabCommande[z], z);

cmd étant la commande à éxécuter et indice, l'indice de la commande qu'on vient d'éxécuter.

Grosso modo, je vois pas très bien comment lancer d'abord les 4 premiers processus, puis lancer un autre dés que un des 4 premiers est terminé et ainsi de suite...

Un grand merci à ceux qui pourront m'aider et bonne fin de week end.

[Médinoc]

Pour commencer, j'espère que tu es au courant qu'il y a déjà une erreur dans ton code, puisque tu ne retournes rien...

Pour ton pool de processsus, je te conseille une boucle (potentiellement infinie) et un compteur.
Tu incrémentes le compteur à chaque processus créé.
Dès que le compteur atteint 4, tu fais un wait() pour attendre qu'un des fils se termine, puis tu reprends l'exécution de la boucle...

[hiko-seijuro]

pour moi le wait est mal placé.

[Médinoc]

waitpid() ne sera pas trop utilisable pour une attente multiple.
Mais en effet, il faut retirer le wait du createProcess(). Désolé de ne pas l'avoir vu plutôt...

[soad029]

Salut salut et merci beaucoup pour vos réponses !!

Suite à ta réponse Médinoc, voilà ce que j'ai fait :

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
for (z;z<10;z++){
		if (z == 4) {
			wait();
		}else{
			createProcess (tabCommande[z]);
		}
	}
 
 
close(descripteur); 
 
}
 
void createProcess (char* cmd) {
 
	if (fork() == 0) {
		execlp (cmd, cmd, NULL);
		//perror ("Erreur dans l\'exécution de la tache %s\n",cmd);
		//exit(1);
	}
 
}

J'ai juste bouclé 10 fois car mon tableau contient seulement 10 commandes à effectué.

Par contre j'ai dû être fortement maladroit quelque part car mon programme plante, je suis débutant en C et je dois avouer que je ne vois pas trop d'où cela peut provenir.

Merci encore et très bon dimanche !

[Médinoc]

Tu dois utiliser un compteur indépendant pour les processus actifs (un autre compteur que Z)

[soad029]

re,

Comment ça ? je vois pas trop où mettre mon autre compteur, de plus, crois-tu qu'il faille que ma fonction retourne quelque chose ?

Encore merci

[Médinoc]

Pour moi, il faudrait que ta fonction retourne une valeur pour dire si elle a réussi à créer le fils ou non... (si fork a réussi ou non).

D'ailleurs, tu peux remettre le perror() et le exit(), eux étaient bien placés...

[soad029]

Hola lala, je commence à despérer, j'arrive toujours pas, comment t'aurais codé ça Médinoc, je pense avoir fait ce que tu m'as dit mais mon prog reste toujours bloqué :

Code :

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

1
2
3
4
5
6
7
8
9
 
	for (z;z<10;z++){
 
		if (ind == 4) {
			wait();
		}else{
			ind = createProcess (tabCommande[z],ind);
		}
	}

Code :

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

1
2
3
4
5
6
7
8
9
int createProcess (char* cmd, int cpt) {
 
	if (fork() == 0) {
		execlp (cmd, cmd, NULL);
		perror ("Erreur dans l\'exécution de la tache %s\n",cmd);
		exit(1);
	}
	return (cpt);
}

[Médinoc]

Il faut vraiment que tu mettes de l'ordre dans tes pensées...

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
47
48
49
50
/* Retourne 0 pour OK, -1 pour échec. */
int createProcess(char const * cmd)
{
	int pidFils = fork();
	if(pidFils == -1)
		return -1; /* erreur. */
	if(pidFils == 0)
	{
		/* Fils. */
		execlp(cmd, cmd, NULL);
		perror("exec");
		exit(1);
	}
	/* Père. */
	return 0;
}
 
int main(void)
{
	char const * const tabCommande[10] = {...};
	int z;
	int nbProc = 0;
 
	for(z=0 ; z<10 ; z++)
	{	
		if(nbProc == 4)
		{
			wait(NULL);
			nbProc--;
		}
 
		if( createProcess(tabCommande[z]) < 0 )
		{
			/* Erreur. */
		}
		else
		{
			/* OK, processus créé. */
			nbProc++;
		}
	}/* For. */
 
	/* Et maintenant, on atttend la fin des processus restants. */
	while( nbProc > 0 )
	{
		wait(NULL);
		nbProc--;
	}
	return 0;
}

Note: Je n'ai pas testé, je n'ai même pas essayé de compiler. De toute façon, je n'ai pas le contenu de tabCommande.

[soad029]

Formidable, compilation sans problème et processus en // ... merci infiniment en tout cas !