[LTP] [PATCH] tst_test: Allow to set timeout from test setup()

Cyril Hrubis chrubis@suse.cz
Wed Aug 3 17:34:19 CEST 2016


Hi!
> > I think that it will be safer to have it run with either with a default
> > timeout or with a test->timeout. And that is the whole point of setting
> > the timeout twice in case that we call tst_set_timeout() in the test
> > setup().
> 
> Good point, so setting timeout twice seems unavoidable.
> But I'm still thinking about actions needed to set timeout
> that are now not part of tst_set_timeout(). Would it make sense
> to bring those in?

Looks good to me. I will merge your changes into my patch, add your
Signed-off-by and push it, OK?

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list