--- rpl/src/interruptions.c 2016/03/09 22:12:06 1.167 +++ rpl/src/interruptions.c 2016/03/15 16:31:15 1.168 @@ -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; @@ -114,7 +115,13 @@ thread_surveillance_signaux(void *argume // 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) @@ -154,8 +161,7 @@ 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 *) @@ -1516,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; @@ -1538,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); @@ -1560,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) { @@ -1585,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; @@ -1602,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)) { @@ -1620,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) @@ -1631,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: @@ -1656,8 +1675,6 @@ interruption1(int signal) break; } -# pragma GCC diagnostic pop - return; } @@ -1679,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; } @@ -1737,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; } @@ -1766,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)); @@ -3216,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); } @@ -3267,7 +3273,6 @@ destruction_queue_signaux(struct_process # endif sem_wait(semaphore_arret_signalisation); - (*s_queue_signaux).requete_arret = d_vrai; # ifndef IPCS_SYSV @@ -3279,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