--- rpl/src/interruptions.c	2015/11/26 11:44:42	1.164
+++ rpl/src/interruptions.c	2016/03/15 16:31:15	1.168
@@ -1,7 +1,7 @@
 /*
 ================================================================================
-  RPL/2 (R) version 4.1.24
-  Copyright (C) 1989-2015 Dr. BERTRAND Joël
+  RPL/2 (R) version 4.1.25
+  Copyright (C) 1989-2016 Dr. BERTRAND Joël
 
   This file is part of RPL/2.
 
@@ -70,6 +70,7 @@ thread_surveillance_signaux(void *argume
 	// nécessaires dans la queue des signaux et incrémentent le sémaphore.
 	// Le sémaphore est décrémenté lorsque le signal est effectivement traité.
 
+	int										ios;
 	int										nombre_signaux_envoyes;
 
 	struct_processus						*s_etat_processus;
@@ -109,13 +110,18 @@ thread_surveillance_signaux(void *argume
 			sem_post(semaphore_signalisation);
 
 			nombre_signaux_envoyes = 0;
-			sched_yield();
 
 			// Dans un premier temps, on verrouille la queue des signaux
 			// affectée au processus courant pour vérifier s'il y a quelque
 			// chose à traiter.
 
-			sem_wait(semaphore_queue_signaux);
+			while((ios = sem_wait(semaphore_queue_signaux)) != 0)
+			{
+				if (errno != EINTR)
+				{
+					(*s_etat_processus).erreur_systeme = d_es_processus;
+				}
+			}
 
 			if ((*s_queue_signaux).pointeur_lecture !=
 					(*s_queue_signaux).pointeur_ecriture)
@@ -128,6 +134,7 @@ thread_surveillance_signaux(void *argume
 
 				nombre_signaux_envoyes++;
 				kill(getpid(), SIGALRM);
+				sched_yield();
 			}
 
 			sem_post(semaphore_queue_signaux);
@@ -154,12 +161,12 @@ thread_surveillance_signaux(void *argume
 					if ((*(*((struct_thread *) (*l_element_courant).donnee))
 							.s_etat_processus).pointeur_signal_ecriture !=
 							(*(*((struct_thread *) (*l_element_courant)
-							.donnee)).s_etat_processus)
-							.pointeur_signal_lecture)
+							.donnee)).s_etat_processus).pointeur_signal_lecture)
 					{
 						nombre_signaux_envoyes++;
 						pthread_kill((*((struct_thread *)
 								(*l_element_courant).donnee)).tid, SIGALRM);
+						sched_yield();
 					}
 
 					pthread_mutex_unlock(&((*(*((struct_thread *)
@@ -1515,8 +1522,6 @@ lancement_thread_signaux(struct_processu
 {
 	pthread_attr_t					attributs;
 
-	void							*argument;
-
 	if (pipe((*s_etat_processus).pipe_signaux) != 0)
 	{
 		(*s_etat_processus).erreur_systeme = d_es_processus;
@@ -1537,10 +1542,8 @@ lancement_thread_signaux(struct_processu
 		return(d_erreur);
 	}
 
-	argument = (*s_etat_processus).pipe_signaux;
-
 	if (pthread_create(&((*s_etat_processus).thread_signaux), &attributs,
-			thread_signaux, argument) != 0)
+			thread_signaux, s_etat_processus) != 0)
 	{
 		(*s_etat_processus).erreur_systeme = d_es_processus;
 		return(d_erreur);
@@ -1559,7 +1562,8 @@ arret_thread_signaux(struct_processus *s
 
 	do
 	{
-		n = write((*s_etat_processus).pipe_signaux[1], &signal, sizeof(signal));
+		n = write_atomic(s_etat_processus, (*s_etat_processus).pipe_signaux[1],
+				&signal, sizeof(signal));
 
 		if (n < 0)
 		{
@@ -1584,9 +1588,12 @@ thread_signaux(void *argument)
 
 	struct pollfd			fds;
 
+	struct_processus		*s_etat_processus;
+
 	unsigned char			signal;
 
-	pipe = (int *) argument;
+	s_etat_processus = (struct_processus *) argument;
+	pipe = (*s_etat_processus).pipe_signaux;
 	fds.fd = pipe[0];
 	fds.events = POLLIN;
 	fds.revents = 0;
@@ -1601,12 +1608,10 @@ thread_signaux(void *argument)
 			pthread_exit(NULL);
 		}
 
-#	pragma GCC diagnostic push
-#	pragma GCC diagnostic ignored "-Wunused-result"
-
-		read(fds.fd, &signal, 1);
-
-#	pragma GCC diagnostic pop
+		if (read_atomic(s_etat_processus, fds.fd, &signal, 1) != 1)
+		{
+			pthread_exit(NULL);
+		}
 
 		if (signal != (0xFF & rpl_sigmax))
 		{
@@ -1619,6 +1624,24 @@ thread_signaux(void *argument)
 	pthread_exit(NULL);
 }
 
+
+static inline void
+_write(int fd, const void *buf, size_t count)
+{
+	ssize_t			ios;
+
+	while((ios = write(fd, buf, count)) == -1)
+	{
+		if (errno != EINTR)
+		{
+			break;
+		}
+	}
+
+	return;
+}
+
+
 // Récupération des signaux
 // - SIGINT  (arrêt au clavier)
 // - SIGTERM (signal d'arrêt en provenance du système)
@@ -1630,24 +1653,21 @@ interruption1(int signal)
 
 	test_signal(signal);
 
-#	pragma GCC diagnostic push
-#	pragma GCC diagnostic ignored "-Wunused-result"
-
 	switch(signal)
 	{
 		case SIGINT:
 			signal_tronque = (unsigned char) (rpl_sigint & 0xFF);
-			write(pipe_signaux, &signal_tronque, sizeof(signal_tronque));
+			_write(pipe_signaux, &signal_tronque, sizeof(signal_tronque));
 			break;
 
 		case SIGTERM:
 			signal_tronque = (unsigned char) (rpl_sigterm & 0xFF);
-			write(pipe_signaux, &signal_tronque, sizeof(signal_tronque));
+			_write(pipe_signaux, &signal_tronque, sizeof(signal_tronque));
 			break;
 
 		case SIGUSR1:
 			signal_tronque = (unsigned char) (rpl_sigalrm & 0xFF);
-			write(pipe_signaux, &signal_tronque, sizeof(signal_tronque));
+			_write(pipe_signaux, &signal_tronque, sizeof(signal_tronque));
 			break;
 
 		default:
@@ -1655,8 +1675,6 @@ interruption1(int signal)
 			break;
 	}
 
-#	pragma GCC diagnostic pop
-
 	return;
 }
 
@@ -1678,13 +1696,7 @@ interruption2(int signal)
 	test_signal(signal);
 
 	signal_tronque = (unsigned char) (rpl_sigtstp & 0xFF);
-
-#	pragma GCC diagnostic push
-#	pragma GCC diagnostic ignored "-Wunused-result"
-
-	write(pipe_signaux, &signal_tronque, sizeof(signal_tronque));
-
-#	pragma GCC diagnostic pop
+	_write(pipe_signaux, &signal_tronque, sizeof(signal_tronque));
 
 	return;
 }
@@ -1736,13 +1748,7 @@ interruption4(int signal)
 	test_signal(signal);
 
 	signal_tronque = (unsigned char) (rpl_sighup & 0xFF);
-
-#	pragma GCC diagnostic push
-#	pragma GCC diagnostic ignored "-Wunused-result"
-
-	write(pipe_signaux, &signal_tronque, sizeof(signal_tronque));
-
-#	pragma GCC diagnostic pop
+	_write(pipe_signaux, &signal_tronque, sizeof(signal_tronque));
 
 	return;
 }
@@ -1765,7 +1771,7 @@ interruption5(int signal)
 	if (pid_processus_pere == getpid())
 	{
 		signal_tronque = (unsigned char) (rpl_sigalrm & 0xFF);
-		write(pipe_signaux, &signal_tronque, sizeof(signal_tronque));
+		_write(pipe_signaux, &signal_tronque, sizeof(signal_tronque));
 	}
 
 	write(STDERR_FILENO, message, strlen(message));
@@ -2658,6 +2664,7 @@ envoi_signal_processus(pid_t pid, enum s
 		if (msync(s_queue_signaux, sizeof(s_queue_signaux),
 				MS_ASYNC | MS_INVALIDATE) != 0)
 		{
+			sem_post(semaphore_queue_signaux);
 			return(1);
 		}
 #		endif
@@ -3002,7 +3009,8 @@ creation_queue_signaux(struct_processus
 
 		(*s_queue_signaux).requete_arret = d_faux;
 
-		if (msync(s_queue_signaux, sizeof(struct_queue_signaux), MS_SYNC))
+		if (msync(s_queue_signaux, sizeof(struct_queue_signaux),
+				MS_ASYNC | MS_INVALIDATE) != 0)
 		{
 			(*s_etat_processus).erreur_systeme = d_es_processus;
 			return;
@@ -3168,6 +3176,16 @@ creation_queue_signaux(struct_processus
 	}
 
 	(*s_queue_signaux).controle = getpid();
+
+#	ifndef IPCS_SYSV
+	if (msync(s_queue_signaux, sizeof(s_queue_signaux),
+			MS_ASYNC | MS_INVALIDATE) != 0)
+	{
+		(*s_etat_processus).erreur_systeme = d_es_processus;
+		return;
+	}
+#			endif
+
 	return;
 }
 
@@ -3203,6 +3221,7 @@ liberation_queue_signaux(struct_processu
 
 	if (getpid() == (*s_queue_signaux).controle)
 	{
+		arret_thread_signaux(s_etat_processus);
 		pthread_join((*s_queue_signaux).thread_signaux, NULL);
 	}
 
@@ -3254,7 +3273,6 @@ destruction_queue_signaux(struct_process
 #	endif
 
 	sem_wait(semaphore_arret_signalisation);
-
 	(*s_queue_signaux).requete_arret = d_vrai;
 
 #	ifndef IPCS_SYSV
@@ -3266,7 +3284,7 @@ destruction_queue_signaux(struct_process
 	// Incrémenter le sémaphore pour être sûr de le débloquer.
 
 	sem_post(semaphore_signalisation);
-	pthread_join((*s_queue_signaux).thread_signaux, NULL);
+	arret_thread_signaux(s_etat_processus);
 
 #	ifdef IPCS_SYSV // SystemV
 #		ifndef OS2