mbox series

[v3,0/3] syscalls: add tgkill test-cases

Message ID 1552978181-27748-1-git-send-email-sumit.garg@linaro.org
Headers show
Series syscalls: add tgkill test-cases | expand

Message

Sumit Garg March 19, 2019, 6:49 a.m. UTC 
This patch-set is inherited from Greg which adds test-cases for tgkill
syscall.

Changes in v3:
- Use checkpoint library to keep threads asleep while waiting for
  signals.
- Start threads during test setup() rather than test run().

Changes in v2:
- Free allocated memory for threads.
- Remove redundant call to stop_threads().
- Apply reviewed-by tag.

Greg Hackmann (3):
  syscalls/tgkill01: add new test
  syscalls/tgkill02: add new test
  syscalls/tgkill03: add new test

 runtest/syscalls                            |   4 +
 testcases/kernel/syscalls/tgkill/.gitignore |   3 +
 testcases/kernel/syscalls/tgkill/Makefile   |  10 +++
 testcases/kernel/syscalls/tgkill/tgkill.h   |  22 +++++
 testcases/kernel/syscalls/tgkill/tgkill01.c | 134 ++++++++++++++++++++++++++++
 testcases/kernel/syscalls/tgkill/tgkill02.c |  68 ++++++++++++++
 testcases/kernel/syscalls/tgkill/tgkill03.c | 110 +++++++++++++++++++++++
 7 files changed, 351 insertions(+)
 create mode 100644 testcases/kernel/syscalls/tgkill/.gitignore
 create mode 100644 testcases/kernel/syscalls/tgkill/Makefile
 create mode 100644 testcases/kernel/syscalls/tgkill/tgkill.h
 create mode 100644 testcases/kernel/syscalls/tgkill/tgkill01.c
 create mode 100644 testcases/kernel/syscalls/tgkill/tgkill02.c
 create mode 100644 testcases/kernel/syscalls/tgkill/tgkill03.c

Comments

Cyril Hrubis March 19, 2019, 1:54 p.m. UTC | #1 
Hi!
Pushed with minor changes, thanks.

Also we are still missing the EPERM error case, for that we would have
to start the test as root, fork a child change the UID and GID and
attempt to send a signal to parent process.
Sumit Garg March 20, 2019, 7:43 a.m. UTC | #2 
On Tue, 19 Mar 2019 at 19:25, Cyril Hrubis <chrubis@suse.cz> wrote:
>
> Hi!
> Pushed with minor changes, thanks.

Thanks for these changes and merge.

>
> Also we are still missing the EPERM error case, for that we would have
> to start the test as root, fork a child change the UID and GID and
> attempt to send a signal to parent process.
>

Hmm I see. I will try to add test case for this too.

-Sumit

> --
> Cyril Hrubis
> chrubis@suse.cz