----- Original Message -----
> From: "Matus Marhefka" <[email protected]>
> To: [email protected]
> Sent: Thursday, 26 February, 2015 1:57:41 PM
> Subject: [LTP] [PATCH v2] containers: semctl replaced with semop in
> sysvipc/sem_comm.c
>
> semctl call replaced with semop call for setting up semaphores initial
> values because semctl needs non-standard type (not in sys/sem.h).
>
> Signed-off-by: Matus Marhefka <[email protected]>
Looks good to me.
Reviewed-by: Jan Stancek <[email protected]>
Regards,
Jan
> ---
> testcases/kernel/containers/sysvipc/sem_comm.c | 24 ++++++++++++++----------
> 1 file changed, 14 insertions(+), 10 deletions(-)
>
> diff --git a/testcases/kernel/containers/sysvipc/sem_comm.c
> b/testcases/kernel/containers/sysvipc/sem_comm.c
> index b44be6d..fba6767 100644
> --- a/testcases/kernel/containers/sysvipc/sem_comm.c
> +++ b/testcases/kernel/containers/sysvipc/sem_comm.c
> @@ -59,14 +59,17 @@ int chld1_sem(void *arg)
> int id;
> struct sembuf sm;
>
> - id = semget(TESTKEY, 1, IPC_CREAT);
> + id = semget(TESTKEY, 1, IPC_CREAT|S_IRUSR|S_IWUSR|IPC_EXCL);
> if (id == -1) {
> perror("semget");
> return 2;
> }
>
> - if (semctl(id, 0, SETVAL, 1) == -1) {
> - perror("semctl");
> + sm.sem_num = 0;
> + sm.sem_op = 1;
> + sm.sem_flg = IPC_NOWAIT;
> + if (semop(id, &sm, 1) == -1) {
> + perror("semop");
> semctl(id, 0, IPC_RMID);
> return 2;
> }
> @@ -74,9 +77,7 @@ int chld1_sem(void *arg)
> /* tell child2 to continue and wait for it to create the semaphore */
> TST_SAFE_CHECKPOINT_WAKE_AND_WAIT(NULL, 0);
>
> - sm.sem_num = 0;
> sm.sem_op = -1;
> - sm.sem_flg = IPC_NOWAIT;
> if (semop(id, &sm, 1) == -1) {
> perror("semop");
> semctl(id, 0, IPC_RMID);
> @@ -101,14 +102,19 @@ int chld2_sem(void *arg)
> /* wait for child1 to create the semaphore */
> TST_SAFE_CHECKPOINT_WAIT(NULL, 0);
>
> - id = semget(TESTKEY, 1, IPC_CREAT);
> + /* IPC_EXCL is excluded because we do not want semget call to fail
> + * if namespaces logic doesn't work correctly */
> + id = semget(TESTKEY, 1, IPC_CREAT|S_IRUSR|S_IWUSR);
> if (id == -1) {
> perror("semget");
> return 2;
> }
>
> - if (semctl(id, 0, SETVAL, 1) == -1) {
> - perror("semctl");
> + sm.sem_num = 0;
> + sm.sem_op = 1;
> + sm.sem_flg = IPC_NOWAIT;
> + if (semop(id, &sm, 1) == -1) {
> + perror("semop");
> semctl(id, 0, IPC_RMID);
> return 2;
> }
> @@ -116,9 +122,7 @@ int chld2_sem(void *arg)
> /* tell child1 to continue and wait for it to lock the semaphore */
> TST_SAFE_CHECKPOINT_WAKE_AND_WAIT(NULL, 0);
>
> - sm.sem_num = 0;
> sm.sem_op = -1;
> - sm.sem_flg = IPC_NOWAIT;
> if (semop(id, &sm, 1) == -1) {
> if (errno == EAGAIN) {
> rval = 1;
> --
> 1.8.3.1
>
>
> ------------------------------------------------------------------------------
> Dive into the World of Parallel Programming The Go Parallel Website,
> sponsored
> by Intel and developed in partnership with Slashdot Media, is your hub for
> all
> things parallel software development, from weekly thought leadership blogs to
> news, videos, case studies, tutorials and more. Take a look and join the
> conversation now. http://goparallel.sourceforge.net/
> _______________________________________________
> Ltp-list mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/ltp-list
>
------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
