[LTP] [PATCH 06/10] copy_file_range01: Add max_runtime

Cyril Hrubis chrubis@suse.cz
Wed Aug 31 11:25:18 CEST 2022


Hi!
> > LGTM (test uses all_filesystems, but 20 for all possible filesystems should be OK).
> Hmm, I've missed the .all_filesystems setting here. In that case the 
> test should take ~2 seconds per filesystems and the patch is not needed. 
> Feel free to discard it then.

I would still vote for adding something as .max_runtime = 5 or something
like this so that it's obvious that the test can run for a few seconds
per run() call especially on slower disks.

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list