lzh/MentOS
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix t_semflg test

Browse Source 
Remove the double semaphore cleanup in the child and the parent.

Set nops to 1 since there is only one operation not two.

Use a blocking semaphore operation in the parent to wait for the
child increasing the semaphore value.
This commit is contained in:
Florian Fischer
2024-02-14 14:25:21 +01:00
parent ee2f95e51d
commit 63b8d9420a
1 changed files with 6 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files
programs/tests/t_semflg.c
+6 -15
View File
@@ -17,10 +17,6 @@ int main(int argc, char *argv[])
union semun arg;
long ret, semid;
op[0].sem_num = 0;
op[0].sem_op = -2;
op[0].sem_flg = IPC_NOWAIT;
// ========================================================================
// Create the first semaphore.
semid = semget(IPC_PRIVATE, 1, IPC_CREAT | IPC_EXCL | S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP);
@@ -78,21 +74,16 @@ int main(int argc, char *argv[])
}
printf("[child] Get semaphore value (id : %d, value : %d == 1)\n", semid, ret);
printf("[child] Exit, now.\n", semid, ret);
// ========================================================================
// Delete the semaphore set.
ret = semctl(semid, 0, IPC_RMID, 0);
if (ret < 0) {
perror("Failed to remove semaphore set");
}
printf("[child] Correctly removed semaphore set.\n");
return 0;
}
op[0].sem_num = 0;
op[0].sem_op = -2;
op[0].sem_flg = 0;
// ========================================================================
// Perform the operations.
if (semop(semid, op, 2) < 0) {
// Perform the operation.
if (semop(semid, op, 1) < 0) {
perror("Failed to perform operation");
return 1;
}