我想向读者/作家提供二进制信号量的问题

时间:2019-05-28 12:11:34

标签: c multithreading binary-semaphore

我有5位作家,20位读者。 我想用二进制信号量解决读写器问题。

但是我的代码有一些问题。存在分段错误(核心已转储)。 我认为创建线程时存在问题。 我该如何解决这个问题?是正确的代码来解决读写问题吗? 我使用了教科书的伪代码。

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

sem_t mutex, rw_mutex;

int data = 0;
int readcount = 0;

void *reader(void* i)
{
    int num = *((int*)i);
    sem_wait(&mutex);
    readcount += 1; 
    if(readcount == 1)
            sem_wait(&rw_mutex);
    sem_post(&mutex);

    printf("I'm reader%d, data is %d \n", num, data);
    sem_wait(&mutex);

    readcount -= 1;
    if( readcount == 0)
            sem_post(&rw_mutex);
    sem_post(&mutex);
}

void *writer(void *i)
{
      int num = *((int*)i);
      sem_wait(&rw_mutex);
      data++;
      printf("I'm writer%d, data is %d\n", num, data);
      sem_post(&rw_mutex);
}

void main()
{
      int i;
      pthread_t writer[5], reader[20];
      sem_init(&rw_mutex, 0, 1);
      sem_init(&mutex, 0, 1);

      for(i=0; i<5; i++)
              pthread_create(&writer[i], NULL, writer, &i);
      for(i=0; i<20; i++)
              pthread_create(&reader[i], NULL, reader, &i);
      for(i=0; i<5; i++)
            pthread_join(writer[i], NULL);
      for(i=0; i<20; i++)
              pthread_join(reader[i], NULL);
      printf("End \n");
}

1 个答案:

答案 0 :(得分:2)

您是否检查了编译器的警告?我收到几个警告。一个例子是:

  

警告:从不兼容的指针类型传递“ pthread_create”的参数3 [默认启用]                  pthread_create(&reader [i],NULL,reader,&i);

问题在于,在main中,您有一个名为reader的数组,但是在程序中,您还有一个名为reader的函数。因此,当您实际需要函数时,编译器(即至少是我的编译器)会使用数​​组。程序崩溃了。

修正警告!通过重命名函数readerwriter或通过重命名数组。

那之后,我再也看不到程序崩溃了。