1

一周前我开始理解和使用信号量和共享内存，并实际创建了这个程序，问题是我找不到任何问题我一直在看它几个小时，一切似乎都是正确的。代码编译，我可以创建构建但是当我执行它时没有任何反应......

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/ipc.h> 
#include <sys/shm.h>
#include <stdio.h>
#include <sys/fcntl.h>
#include <semaphore.h>

#define MAXCHILDS 4
#define MAX_SIZE 10
#define MAX_WRITES 4

typedef struct{
    int m[MAX_SIZE][MAX_SIZE];
} matrix;

/*fork variables*/
pid_t child[MAXCHILDS];
/*semphores variables */
sem_t *empty, *full, * mutex;
/*share memory id*/
int shmid;
/*shared memory array pointer */
matrix * sh_mem;
/*pointer to matrix*/
int **p;


void init(){
      /*create pointer to matrix*/
      p = &sh_mem->m;
     /*semaphores unlink and creation */    
     sem_unlink("EMPTY");
     empty=sem_open("EMPTY",O_CREAT|O_EXCL,0700,MAX_WRITES);
     sem_unlink("FULL");
     full=sem_open("FULL",O_CREAT|O_EXCL,0700,0);
     sem_unlink("MUTEX");
     mutex=sem_open("MUTEX",O_CREAT|O_EXCL,0700,1);
    /*initialize shared memory */
    shmid = shmget(IPC_PRIVATE,sizeof(matrix),IPC_CREAT|0777);
    /*map shared memory*/
    sh_mem = (matrix*)shmat(shmid,NULL,0);
    if(sh_mem== (matrix*)(-1)){
        perror("shmat");
    }
}
void writer(int ** m){
    int i,k;
    for(i = 0;i<MAX_SIZE;i++){
        for(k= 0;k<MAX_SIZE;k++){
            m[i][k] = 0;
        }
    }

}
void reader(int **m){
    int i = 0;
    int k = 0;
    for(i = 0;i<MAX_SIZE;i++){
        for(k= 0;k<MAX_SIZE;k++){
            printf(m[i][k]);
        }
        printf("\n");
    }

}

void terminate() {
  sem_close(empty);
  sem_close(full);
  sem_close(mutex);
  sem_unlink("EMPTY");
  sem_unlink("FULL");
  sem_unlink("MUTEX");
  shmctl(shmid, IPC_RMID, NULL);
}

int main(int argc, char **argv)
{
    int i;
    init();

    for(i = 0;i<MAXCHILDS;i++){
        if((child[i]= fork()) < 0) // error occured
        {
            perror("Fork Failed");
            exit(1);
        }
        if((child[i] =fork())==0){
            writer(p);
            exit(0);
        }
    }
    /*father*/  
    reader(p);
    wait(NULL);

    terminate();


    return 0;
}

孩子应该在共享内存中写入矩阵，而父亲应该读取共享内存数组并打印矩阵。你能帮我解决这个问题吗？谢谢您的帮助 ...

4

1 回答 1

2

您的代码有几个问题。

  • 你声明你的p变量如下：

    int **p;

    但是您的数组的实际类型是int[MAXSIZE][MAXSIZE]. 尽管它们都可以被视为二维数组，但它们根本不是同一类型，并且在内存中不共享相同的结构：

    • int **是一个指向整数的指针。当用作二维数组时，它描述了一个 2 级数组。第一级（取消引用指针一次）包含一个int *指针数组，每个指针指向一维整数数组。
    • int[MAXSIZE][MAXSIZE]是一个平面二维数组。MAXSIZE*MAXSIZE它是指向整数数组的单个指针。int[MAXSIZE*MAXSIZE]尽管它提供了使用 2 级下标的便利，但它具有与一维数组相同的内存结构。

    在 C 中传递指向type[][]-style 数组的指针是不直观的，并且很难正确处理。我不建议这样做。您可能会考虑将指针传递给您的matrix类型。尽管如此，还是可以让它与这个声明一起工作：

    typedef int array2d[MAX_SIZE][MAX_SIZE];
array2d *p;

    那么您必须将m参数更改为readerandwriter来array2d *m代替 orint **m并将其用作*m而不是 just m。这变得丑陋。就像我说的，考虑改用你的matrix类型。

    这个问题应该已经被编译器发现了。它应该给你很多类型不匹配的警告。确保您始终编译gcc -Wall并旨在让您的程序编译时没有错误和警告。

  • 在init()中，你设置p得太早了。您应该在分配后将其设置在函数的末尾sh_mem，而不是在函数的开头。

  • 正如上面的评论中所讨论的，你在你的函数中fork()处理了太多的进程。main()您可能打算fork()每次通过for循环只调用一次，而不是两次。

  • 您不必等到编写者完成了共享内存结构的填写后，才继续阅读其内容。

    请注意，即使您wait(NULL)在调用之前将您的移动到reader()，这还不够，因为wait(NULL)只会等待一个子进程完成，而不是所有子进程。

通常，您应该在启用警告的情况下编译您的程序，注意警告，并在想知道您的程序可能有什么问题之前修复它们。如果它仍然做一些意想不到的事情，你应该在调试器下运行它，使用断点，检查变量，并尝试看看发生了什么。使用这两种技术，您可能已经解决了您的问题，而无需在此处发布。

于 2012-11-05T01:59:17.780 回答