Question

我是C和Unix的新手，所以这可能是一个愚蠢的问题。昨天我一直试图制作2个通过共享内存相互通信的程序。一个是接收者，另一个是发送者。

下面的接收器设置共享内存以及2个信号量（一个用于读取另一个用于写入）并等待在循环中接收的内容，如果接收的字符串为空则表示循环终止。两个程序都使用3个标志： -m（shared_memory_name） -w（writing_semaphore_name） -r（reading_semaphore_name）

int main(int argc, char** argv)
{

 //sem_in,sem_out,shared_mem_fd,getopt stuff

 //getopt()...

while(1)
{   
    //waiting for sender to change the sem

    if(sem_wait(sem_out)==-1){ //waits for the sender to change the semaphore
            //error
    }
    char* data =  mmap(NULL,4096,PROT_READ|PROT_WRITE,MAP_SHARED,SHM_FD,0);

    if(data == MAP_FAILED){
        //error
    }

    if(strlen(data) == 0){
        //terminate program
    }       


    if(sem_post(sem_in)==-1){
        //error
    }

}

//close sems and shared memory

return 0
}

下面的发送者假设打开共享内存FD并输入一些东西，这是我从argv [optind]得到的附加参数，通常是一个字符串。

int main(int argc, char** argv){

//sems,getopt variables,filedescriptor

char* data;

//getopt()

data = argv[optind];

if(sem_in == SEM_FAILED || sem_out == SEM_FAILED || SHM_FD == -1){
        //error
}

//printf("waiting\n");

if(sem_wait(sem_in)==-1){
        //error
}

data = mmap(NULL,strlen(data),PROT_READ|PROT_WRITE,MAP_SHARED,SHM_FD,0);    

if(data==MAP_FAILED){
    //error
}

if(sem_post(sem_out)==-1){
        //error
}

close(shared memory);

return 0;
}

这是我使用该程序的方式：

./receiver -m /mem1 -w /s1 -r /s2 (creates the memory and sems and waits)
waiting

./sender -m /mem1 -w /s1 -r /s2 hello (send a string)

./receiver -m /mem1 -w /s1 -r /s2
waiting
empty

尽管我发送了一些东西，接收器说字符串是空的。信号量发生了变化，所以就是这样。

就像我说的那样，我在这方面几乎是一个新手，所以我很容易犯错误，我希望我能很好地解释我的问题，任何帮助都会受到赞赏。

P.S。变量名用我的语言，所以请原谅我，如果我忘了翻译一些，我也进行了很多实验，因此以前版本的一些包括仍在那里......

Answer 1

问题是您没有在共享内存中存储任何内容。你在代码中有以下两行......

data = argv[optind];

data = (char*)mmap(NULL,strlen(data),PROT_READ|PROT_WRITE,MAP_SHARED,SHM_FD,(off_t)0);

第二个覆盖data的内容，然后你不会将任何内容复制到它所指向的内存中。

由于它实际上没有做任何有用的事情，你应该删除第一行。要填充共享内存，请在之后添加此行，然后再打开它。

memcpy(data,argv[optind],strlen(argv[optind])+1);

由于您不再将data指向参数，因此您还需要更改共享内存的开放。

data = (char*)mmap(NULL,strlen(argv[optind])+1,PROT_READ|PROT_WRITE,MAP_SHARED,SHM_FD,(off_t)0);

你会注意到我在字符串的长度上添加了1 - 这是为了确保有足够的空间来包含NUL终止字符。

共享内存/ mmap

1 个答案: