[LTP] [PATCH] Disable failure hints before we actually run the test

Cyril Hrubis chrubis@suse.cz
Tue Sep 17 17:50:25 CEST 2024


Hi!
> does it make sense to print failure hints after setup() failure? I can 
> think of a few cases where bug effects can be delayed until cleanup() 
> but it seems a bit early to me to cover setup() as well. I think the 
> flag could be moved to shared memory, enabled immediately before calling 
> run_tests() and disabled immediately after handling child's exit status 
> in fork_testrun().

My idea here was that we should always show the hints once we executed a
test function at least once. The rationale is that if the run_test()
corrupted kernel memory it may manifest at any point e.g. when we
attempt to setup a next loop device for the next filesytem (for
.all_filesystems).

So the idea is not to print them during the test library setup i.e.
do_setup() function but enable them right after that and keeping them
on.

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list