Discussion :

[muad'dib]

Bonjour à tous,

Voila j'ai une application de client-serveur basé sur des fork() : à chaque connexion d'un client, un processus fils est créé jusqu'à la déconnexion de celui-ci.
Lorsque je connecte un client par seconde, le processus zombie est bien détruit, mais lorsque je simule une grosse charge du serveur (genre 100 connexions par seconde) des processus zombies restent présents dans la liste des processus.

Je sais qu'une telle charge est rare mais cela peut arriver. Voici une partie de code pour illustrer ce que je fais.
Chez le client:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
do {
    usleep(1000);
    sockfd = connect_server("127.0.0.1", 9611);
 
    if (sockfd > 0) {
        send_data(sockfd, "test\n");
 
        close_connection(sockfd);
    }
}
while (1);

Sur le serveur:

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
	/* Un cliente se ha conectado */
		pid = fork();
 
		if(pid < 0)
		{
			result = E_CANT_FORK;
		}
		else if(pid == 0)
		{
			result = create_new_client_process(sockfd);
 
			memset(&msgdisbuf, 0, sizeof(msgdisbuf));
			msgdisbuf.type = MSGDISTYPE;
		    memcpy(&msgdisbuf.sockfd, &sockfd, sizeof(sockfd));
 
 			if (msgsnd(msgdisid, &msgdisbuf, sizeof(msgdisbuf.sockfd), IPC_NOWAIT) < 0) 
			{
		        perror("msgsnd");
		    }
 
			exit(0); /* Finalizar proceso hijo */
		}

Au lancement du programme j'appelle la méthode sig_chld(); définie comme suit:

Code :

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

1
2
3
4
5
void sig_chld() {
	signal(SIGCHLD, SIG_IGN);
	wait(NULL);
	signal(SIGCHLD, sig_chld);
}

[nicolas.sitbon]

Pour répondre à cette question, il faut considérer plusieurs points :
Le comportement de la fonction signal est différent d'un système à l'autre, par exemple sous linux, signal réinstalle automatiquement le gestionnaire sauf si la constante _XOPEN_SOURCE est définie; de plus si un signal arrive entre l'entré dans le gestionnaire et signal(SIGCHLD,SIG_IGN) , le signal ne sera pas traité et le process ne sera pas adopté par init (et c'est ce qui se passe quand le serveur est surchargé).
J'ajoute que tu ne testes pas le retour de signal, qui peut échouer et que de plus, la norme SUS3 déconseille signal(SIGCHLD,SIG_IGN) .
Si tu veux être vraiment propre et portable voilà ce que je conseille :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
struct sigaction action;
action.sa_handler=SIG_IGN;
sigemptyset(&(action.sa_mask));
action.sa_flags=SA_RESTART|SA_NCLDSTOP;
if(sigaction(SIGCHLD,&action,NULL)!=0)
{
  perror("erreur");
  exit(EXIT_FAILURE);
}

Cordialement.

[muad'dib]

Super ça fonctionne impeccable comme ça, merci beaucoup l'ami !