--- rpl/src/gestion_threads.c 2015/11/26 11:44:32 1.99 +++ rpl/src/gestion_threads.c 2023/08/08 13:33:09 1.120 @@ -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.35 + Copyright (C) 1989-2023 Dr. BERTRAND Joël This file is part of RPL/2. @@ -77,15 +77,15 @@ lancement_thread(void *argument) s_etat_processus = (*s_argument_thread).s_nouvel_etat_processus; # ifndef SEMAPHORES_NOMMES - sem_init(&((*s_etat_processus).semaphore_fork), 0, 0); + if (sem_init(&((*s_etat_processus).semaphore_fork), 0, 0) != 0) # else if (((*s_etat_processus).semaphore_fork = sem_init3(0, getpid(), pthread_self(), SEM_FORK)) == SEM_FAILED) - { - (*s_etat_processus).erreur_systeme = d_es_allocation_memoire; - return(NULL); - } # endif + { + (*s_etat_processus).erreur_systeme = d_es_allocation_memoire; + pthread_exit(NULL); + } (*s_argument_thread).tid = pthread_self(); insertion_thread(s_etat_processus, d_faux); @@ -93,7 +93,7 @@ lancement_thread(void *argument) if (pthread_mutex_lock(&((*s_argument_thread).mutex)) != 0) { (*s_etat_processus).erreur_systeme = d_es_processus; - return(NULL); + pthread_exit(NULL); } (*s_argument_thread).thread_actif = d_vrai; @@ -101,7 +101,7 @@ lancement_thread(void *argument) if (pthread_mutex_unlock(&((*s_argument_thread).mutex)) != 0) { (*s_etat_processus).erreur_systeme = d_es_processus; - return(NULL); + pthread_exit(NULL); } // Envoi d'une donnée pour signaler le démarrage du thread au thread @@ -150,7 +150,22 @@ lancement_thread(void *argument) break; } + if (sem_post(&((*s_etat_processus).semaphore_fork)) != 0) + { + (*s_etat_processus).erreur_systeme = d_es_processus; + pthread_exit(NULL); + } + nanosleep(&attente, NULL); + + while(sem_wait(&((*s_etat_processus).semaphore_fork)) != 0) + { + if (errno != EINTR) + { + (*s_etat_processus).erreur_systeme = d_es_processus; + pthread_exit(NULL); + } + } } if ((*s_etat_processus).debug == d_vrai) @@ -337,22 +352,16 @@ lancement_thread(void *argument) } } - close((*s_argument_thread).pipe_erreurs[1]); - close((*s_argument_thread).pipe_interruptions[1]); - close((*s_argument_thread).pipe_nombre_elements_attente[1]); - close((*s_argument_thread).pipe_objets[1]); - close((*s_argument_thread).pipe_injections[0]); - close((*s_argument_thread).pipe_nombre_injections[0]); - close((*s_argument_thread).pipe_acquittement[0]); - l_element_courant = (*s_etat_processus).s_fichiers; while(l_element_courant != NULL) { l_element_suivant = (*l_element_courant).suivant; - if ((*((struct_descripteur_fichier *) (*l_element_courant) - .donnee)).pid == getpid()) + if (((*((struct_descripteur_fichier *) (*l_element_courant) + .donnee)).pid == getpid()) && + ((*((struct_descripteur_fichier *) (*l_element_courant) + .donnee)).tid == pthread_self())) { fclose((*((struct_descripteur_fichier *) (*l_element_courant).donnee)).descripteur_c); @@ -417,7 +426,7 @@ lancement_thread(void *argument) { envoi_signal_processus((*(*((struct_processus_fils *) (*(*l_element_courant) - .donnee).objet)).thread).pid, rpl_sigurg); + .donnee).objet)).thread).pid, rpl_sigurg, d_faux); } else { @@ -425,13 +434,13 @@ lancement_thread(void *argument) { envoi_signal_processus((*(*((struct_processus_fils *) (*(*l_element_courant) - .donnee).objet)).thread).pid, rpl_sigabort); + .donnee).objet)).thread).pid, rpl_sigabort, d_faux); } else { envoi_signal_processus((*(*((struct_processus_fils *) (*(*l_element_courant) - .donnee).objet)).thread).pid, rpl_sigstop); + .donnee).objet)).thread).pid, rpl_sigstop,d_faux); } } } @@ -482,7 +491,8 @@ lancement_thread(void *argument) (*(*l_element_courant).donnee).objet)).thread) .thread_actif == d_vrai) { - envoi_signal_thread((*(*((struct_processus_fils *) + envoi_signal_thread(NULL, + (*(*((struct_processus_fils *) (*(*l_element_courant).donnee).objet)).thread).tid, rpl_sigurg); } @@ -527,13 +537,15 @@ lancement_thread(void *argument) { if ((*s_etat_processus).arret_depuis_abort == -1) { - envoi_signal_thread((*(*((struct_processus_fils *) + envoi_signal_thread(NULL, + (*(*((struct_processus_fils *) (*(*l_element_courant).donnee).objet)).thread) .tid, rpl_sigabort); } else { - envoi_signal_thread((*(*((struct_processus_fils *) + envoi_signal_thread(NULL, + (*(*((struct_processus_fils *) (*(*l_element_courant).donnee).objet)).thread) .tid, rpl_sigstop); } @@ -594,11 +606,18 @@ lancement_thread(void *argument) (*(*((struct_processus_fils *) (*(*l_element_courant) .donnee).objet)).thread).nombre_objets_dans_pipe--; + if (pthread_mutex_lock(&mutex_sigaction) != 0) + { + (*s_etat_processus).erreur_systeme = d_es_processus; + pthread_exit(NULL); + } + action.sa_handler = SIG_IGN; - action.sa_flags = SA_ONSTACK; + action.sa_flags = 0; if (sigaction(SIGPIPE, &action, ®istre) != 0) { + pthread_mutex_unlock(&mutex_sigaction); pthread_mutex_unlock(&((*s_etat_processus) .mutex_pile_processus)); @@ -618,7 +637,7 @@ lancement_thread(void *argument) } (*s_etat_processus).erreur_systeme = d_es_signal; - exit(EXIT_FAILURE); + pthread_exit(NULL); } while((longueur_ecriture = write_atomic( @@ -651,11 +670,21 @@ lancement_thread(void *argument) if (sigaction(SIGPIPE, ®istre, NULL) != 0) { + pthread_mutex_unlock(&mutex_sigaction); + pthread_mutex_unlock(&((*s_etat_processus) + .mutex_pile_processus)); + + (*s_etat_processus).erreur_systeme = d_es_signal; + pthread_exit(NULL); + } + + if (pthread_mutex_unlock(&mutex_sigaction) != 0) + { pthread_mutex_unlock(&((*s_etat_processus) .mutex_pile_processus)); (*s_etat_processus).erreur_systeme = d_es_signal; - exit(EXIT_FAILURE); + pthread_exit(NULL); } } } @@ -675,7 +704,7 @@ lancement_thread(void *argument) pthread_mutex_unlock(&((*s_etat_processus).mutex_pile_processus)); (*s_etat_processus).erreur_systeme = d_es_processus; - exit(EXIT_FAILURE); + pthread_exit(NULL); } if ((*s_etat_processus).nombre_interruptions_non_affectees != 0) @@ -689,7 +718,7 @@ lancement_thread(void *argument) pthread_mutex_unlock(&((*s_etat_processus).mutex_pile_processus)); (*s_etat_processus).erreur_systeme = d_es_processus; - exit(EXIT_FAILURE); + pthread_exit(NULL); } pthread_mutex_unlock(&((*s_etat_processus).mutex_pile_processus)); @@ -1022,12 +1051,22 @@ lancement_thread(void *argument) pthread_mutex_destroy(&((*s_etat_processus).mutex_signaux)); # ifndef SEMAPHORES_NOMMES + sem_post(&((*s_etat_processus).semaphore_fork)); sem_destroy(&((*s_etat_processus).semaphore_fork)); # else + sem_post((*s_etat_processus).semaphore_fork); sem_destroy3((*s_etat_processus).semaphore_fork, getpid(), pthread_self(), SEM_FORK); # endif + close((*s_argument_thread).pipe_erreurs[1]); + close((*s_argument_thread).pipe_interruptions[1]); + close((*s_argument_thread).pipe_nombre_elements_attente[1]); + close((*s_argument_thread).pipe_objets[1]); + close((*s_argument_thread).pipe_injections[0]); + close((*s_argument_thread).pipe_nombre_injections[0]); + close((*s_argument_thread).pipe_acquittement[0]); + liberation_contexte_cas(s_etat_processus); free((*s_etat_processus).localisation); liberation_allocateur(s_etat_processus); @@ -1040,7 +1079,6 @@ lancement_thread(void *argument) pthread_mutex_unlock(&((*s_argument_thread).mutex)); pthread_exit(NULL); - return(NULL); } // vim: ts=4