[LTP] [PATCH v4] Refactor pidns30 test using new LTP API

Cyril Hrubis chrubis@suse.cz
Fri Feb 10 14:16:06 CET 2023


Hi!
> +static void run(void)
>  {
> -	int status;
> -	char buf[5];
> -	pid_t cpid;
> +	const struct tst_clone_args args = { CLONE_NEWPID, SIGCHLD };
>  
> -	setup();
> +	remove_mqueue(mqd);
> +	received = 0;

I wonder if we really need to clear the flag here, as far as I can see
we only modify that variable in the child process and that shouldn't
propagate to parent, unless we pass CLONE_VM of course.


Other than this it looks really good:

Reviewed-by: Cyril Hrubis <chrubis@suse.cz>

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list