Question

编辑：这种代码的和平非常好（所以把它作为信号量的例子;）。我的程序中的错误位于另一个地方 - 由我的朋友找到。

~~我的功能有问题。有时两个进程进入关键部分。我通过调试花了10个小时后才发现问题。我应该瞄准什么？在这段代码中是否有可能产生萌芽？~~

    // lock semaphore
    static int P(int sem_id)
    {
        struct sembuf sem_b;
        sem_b.sem_num = 0;
        sem_b.sem_op = -1; /* P() */
        sem_b.sem_flg = 0;
        if (semop(sem_id, &sem_b, 1) == -1) {
              // error
                 return(0);
        }
        return(1);
    }

    // unlock semaphore
    static int V(int sem_id)
    {
        struct sembuf sem_b[1];
        sem_b.sem_num = 0;
        sem_b.sem_op = 1; /* V() */
        sem_b.sem_flg = 0;
        if (semop(sem_id, &sem_b, 1) == -1) {
              // error
            return(0);
        }
        return(1);
    }

    static int set_semval(int sem_id)  {
        // set to 1 (opened semaphore)
        if (semctl(sem_id, 0, SETVAL, 1) == -1) {
              // error
            return(0);
        }
        return(1);
    }

    static int get_val(int sem_id)
    {
        union semun sem_union;
        //sem_union.val = 0; ?
        return semctl(sem_id, 0, GETVAL, sem_union);
    }

动作循环：

// semaphores init
int mutex;
if ((mutex=semget(key+2, 1, 0666))>=0) {
    // semaphore exists
    fprintf(stderr,"semaphore exists for key %d\n", key+2);
}

if ((mutex=semget(key+2, 1, 0666 | IPC_CREAT)) == -1) { 
    exit(EXIT_FAILURE);
}
if (!set_semval(mutex)) {
    exit(EXIT_FAILURE);
}
fork()    // some times with good conditionals

// in some children
while(1) {
        P(mutex);   
        assert(get_val(mutex)==0); // always ok
        action();  // sometimes made by two processes at same time - fault
        V(mutex);
}

请随时编辑我的问题。

非常感谢

Answer 1

在你的“动作循环”中，如果你的信号量不存在，你会怎么做？

目前你的semget的第3个参数是0666或PERMISSION_RW常量。您可能想要使用：

shmget（key，1，PERMISSION_RW | IPC_CREAT）;

这样，如果您的信号量不存在，它将创建一个。

Answer 2

所以bug出现在另一个地方......

锁定C问题sys / sem中的信号量

2 个答案: