Ошибка сегментации: 11 в программе многопоточности на C с использованием библиотеки pthread - PullRequest
0 голосов
/ 23 мая 2019

Мне нужно сделать программу, использующую библиотеку pthread C, для работы с потоками. Текст задания гласит: «Существует коридор, который имеет одну полосу движения для двух направлений, и есть два типа сотрудников, которые будут пересекать коридор, чтобы достичь противоположной стороны от него. Есть сотрудники типа 1, которые идут слева направо, и сотрудники типа 2, которые идут справа налево. В этом коридоре есть служитель, который очищает этот коридор, когда через него никто не проходит. При уборке коридора сотрудники обеих сторон ожидают его окончания, в противном случае, если коридор занят, работник говорит, что не может убрать и заснуть 2 секунды. Когда я компилирую эту программу, компилятор GCC возвращает 5 Предупреждение, но без ошибок. Когда я запускаю программу, сразу появляется ошибка сегментации: 11.

Это обновленный код программы:

#include <stdio.h>
#include <stdlib.h>
#include <pthread.h>
#include <semaphore.h>
#include <unistd.h>


//global variables 
int c1,c2; //counters for imp1, imp2
pthread_mutex_t m1, m2;
sem_t corridor; 
double clean = 0.5;

//implementation thread attendant
void* attendant(void* arg){
    sleep(1); //rest 1 seconds
    if (c1==0 && c2==0){
        sem_wait(&corridor);
        printf("I'm starting to clean up\n");
        sleep(clean);
        printf ("I finished cleaning\n");
        sem_post(&corridor);
        }else{
           printf ("I can't clean, the corridor is busy\n");
        }
    return NULL;
}


//thread employee type 1
void* emp1(void *arg){
    printf("I'm the number %d of em1\n", c1);
    pthread_mutex_lock(&m1); //beginning critical section
    c1++;                    //it increases to signal the presence of a thread of the same type that wants to enter the corridor
    if (c1 == 1){            //the thread is the only one in the corridor. Can pass
        printf ("I am the first of my group emp1\n");
        sem_wait(&corridor);  //takes possession of the corridor
    }
    pthread_mutex_unlock(&m1); //allows other threads of the same type to pass in the corridor since it was the first in his group. End of critical section

    // invents "passage" function

    pthread_mutex_lock(&m1);  //beginning of the critical section. Once crossed the corridor, the variable c1 is modified. A mutex is used to avoid inconsistency
    c1--;
    if(c1 == 0){
        printf ("I am the last of my group emp1\n");
        sem_post(&corridor);
    } //if c1 == 0, it is the last thread imp1 and releases the corridor
    pthread_mutex_unlock(&m1); //end critical section
    return NULL;
}


//thread employee type 2 
void* emp2(void *arg){
    printf("I'm the number %d of emp2\n", c2);
    pthread_mutex_lock(&m2); //beginning critical section
    c2++;                    //it increases to signal the presence of a thread of the same type that wants to enter the corridor
    if (c2 == 1){            // the thread is the only one in the corridor. Can pass
        printf ("I am the first of my group emp2\n");
        sem_wait(&corridor);  //takes possession of the corridor
    }
    pthread_mutex_unlock(&m2); //allows other threads of the same type to pass in the corridor since it was the first in his group. End of critical sectionritica

    // invents "passage" function

    pthread_mutex_lock(&m2);  //beginning of the critical section. Once crossed the corridor, the variable c1 is modified. A mutex is used to avoid inconsistency
    c2--;
    if(c2 == 0){
        printf ("I am the last of my group emp2\n");
        sem_post(&corridor);
    }//if c1 == 0, it is the last thread imp1 and releases the corridor
    pthread_mutex_unlock(&m2); //end critical section
    return NULL;
}


int main(int argc, char const *argv[]) {
    //pthread_t emp1, emp2, attendant;
    pthread_t idt;
    int r; //var random to create thread emp1 or emp2
    int i; //index 

    //variable initialization
    c1 = c2 = 0;
    pthread_mutex_init(&m1, NULL); 
    pthread_mutex_init(&m2, NULL);
    sem_init(&corridor,0,1); 

    pthread_create(&idt,NULL,attendant,NULL); 
    while(i<40){
        sleep(1);
        r = rand()%2;
        if(r==0){
            printf("Employee creation 1\n");
            pthread_create(&idt,NULL,emp1,NULL);
        }else{
            printf("Employee creation 2\n");
            pthread_create(&idt,NULL,emp2,NULL);
        }
        i++;
    pthread_join(idt, NULL);
    }
    return 0;
}

1 Ответ

1 голос
/ 23 мая 2019

pthread_t emp1, emp2, attendant; скрывает функции с одинаковыми именами, поэтому pthread_create(&idt,NULL,attendant,NULL); передает неинициализированное значение переменной attendant в pthread_create как указатель на функцию вместо функции void* attendant(void* arg).

Удалите переменные, вы их вообще не используете.

Также вы забыли #include <unistd.h>.

Это сделает вашу программу компилируемой: https://ideone.com/3CIb4J

Также ваша функция main не ожидает завершения потоков, вы должны использовать pthread_join для ожидания потоков.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...