3

だから私はいくつかのセマフォで哲学者の食事の問題を解決しようとしています..私がそれをコンパイルすると、哲学者1-5が考えているようになります、哲学者1は空腹です彼はフォーク5を取り、1、哲学者3は空腹ですそしてプログラムはそこで停止します...どこに問題があるのか​​わかりません。

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

#define N 5
#define THINKING 0
#define HUNGRY 1
#define EATING 2
#define LEFT (i+4)%N
#define RIGHT (i+1)%N


sem_t mutex;
sem_t S[N];  
void * philospher(void *num);
void take_fork(int);
void put_fork(int);
void test(int);

int state[N];
int phil_num[N]={0,1,2,3,4};

int main()
{
    int i;
    pthread_t thread_id[N];
    sem_init(&mutex,0,1);
    for(i=0;i<N;i++)
        sem_init(&S[i],0,1);
    for(i=0;i<N;i++)
    {
        pthread_create(&thread_id[i],NULL,philospher,(void *)i);
        printf("Philosopher %d is thinking\n",i+1);
    }
    for(i=0;i<N;i++)
        pthread_join(thread_id[i],NULL);
}


void *philospher(void *num)
 {
    while(1)
    {
        int i;
      i = (int)num;
        sleep(1);
        //take_fork(i);


    sem_wait(&mutex);
    state[i] = HUNGRY;
    printf("Philosopher %d is Hungry\n",i+1);
     //test(i);


    if (state[i] == HUNGRY && state[LEFT] != EATING && state[RIGHT] != EATING)
    {
        sem_wait(&S[LEFT]);
        sem_wait(&S[RIGHT]);
        state[i] = EATING;
        sleep(2);
        printf("Philosopher %d takes fork %d and %d\n",i+1,LEFT+1,i+1);
        printf("Philosopher %d is Eating\n",i+1);
    }

    sem_post(&mutex);
    sleep(1);


        //put_fork(*i);


    sem_wait(&mutex);
    state[i] = THINKING;
    printf("Philosopher %d putting fork %d and %d down\n",i+1,LEFT+1,i+1);
    printf("Philosopher %d is thinking\n",i+1);
    sem_post(&S[LEFT]);
    sem_post(&S[RIGHT]);
    //test(LEFT);
    //test(RIGHT);
    sem_post(&mutex);

    }
 }
4

1 に答える 1

0

これは競合状態だと思います。

state[RIGHT]ワーカー関数で参照していstateますが、その時点で配列が初期化されるようにすべてのスレッドが作成されているという保証はありません。

于 2013-01-06T16:10:01.297 に答える