Re: [LTP] [PATCH 1/2] fsplough: Implement runtime awareness

[Petr Vorel <pvorel@suse.cz>]

Hi Martin,

> Switch fsplough test to run for 30 seconds by default, with the option
> to set a fixed number of loops on command line instead. The test will
> always exit when runtime expires and a warning will be printed
> if there were too few iterations due to expired runtime.

+1
Reviewed-by: Petr Vorel <pvorel@suse.cz>

> +	if (i < loop_count / 2) {
> +		tst_res(TWARN, "Runtime expired, exiting early after %d loops",
> +			i);
> +		tst_res(TINFO, "If you are running on slow machine, "
> +			"try exporting LTP_TIMEOUT_MUL > 1");
> +	} else if (i < loop_count) {
> +		tst_res(TINFO, "Runtime expired, exiting early after %d loops",
> +			i);
> +	} else if (!loop_arg && i < 10) {
> +		tst_res(TWARN, "Slow sytem: test performed only %d loops!", i);
s/sytem/system/

I'll fix this before merge.

Kind regards,
Petr

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp