On Fri, 2021-08-13 at 08:59 +0200, Jan Kiszka wrote: > On 12.08.21 18:38, Florian Bezdeka wrote: > > According to POSIX message queues have kernel persistence, which means > > they are not cleaned up until the next reboot. > > > > The test has a mq_unlink() as "precondition", so it will clean up > > failed test runs silently but /proc/xenomai/registry/usage will still > > report a "leak" once the test has completed. > > > > Signed-off-by: Florian Bezdeka <[email protected]> > > --- > > testsuite/smokey/posix-select/posix-select.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/testsuite/smokey/posix-select/posix-select.c > > b/testsuite/smokey/posix-select/posix-select.c > > index e9bf938b3..0a59e4968 100644 > > --- a/testsuite/smokey/posix-select/posix-select.c > > +++ b/testsuite/smokey/posix-select/posix-select.c > > @@ -138,6 +138,8 @@ static int run_posix_select(struct smokey_test *t, int > > argc, char *const argv[]) > > usleep(300000); > > smp_rmb(); > > ret = test_status; > > + > > + mq_unlink("/select_test_mq"); > > out: > > pthread_join(tcb, NULL); > > > > > > Why only cleaning up on successful runs, ie. why doing this before the > "out:" label?
You are right, that's wrong. And we should wait for the tcb thread to finish as well. So kind of double fault. Testing v2... Florian > > Jan >
