[LTP] [PATCH 4/8] syscalls/sysinfo03: Add time namespace test

Richard Palethorpe rpalethorpe@suse.de
Mon Mar 9 14:26:48 CET 2020


Hello,

Cyril Hrubis <chrubis@suse.cz> writes:

> Hi!
>> > +static struct tst_test test = {
>> > +	.tcnt = ARRAY_SIZE(offsets),
>> > +	.test = verify_sysinfo,
>> > +	.needs_root = 1,
>> > +	.forks_child = 1,
>> > +	.needs_kconfigs = (const char *[]) {
>> > +		"CONFIG_TIME_NS=y"
>> > +	}
>> > +};
>>
>> Will you add a git ref when the fix is in mainline/for-next?
>>
>> Should be safe to add it as soon as it is in the for-next tree.
>
> I'm not sure how usefull that will be because the fix should get in
> during the RC phase the functionality was introduced, so technically the
> missing support of sysinfo() in time namespaces should not reach any
> officially released kernel.

What is the downside to including it?

Also, I can imagine some people will try backporting this and may miss
something not in the original patch set.

--
Thank you,
Richard.


More information about the ltp mailing list