[LTP] [PATCH] readahead02: Always allow at least 100ms for cache to fill

Andrea Cervesato andrea.cervesato@suse.com
Wed Feb 18 08:32:33 CET 2026


Hi!

On Tue Feb 17, 2026 at 4:39 PM CET, Martin Doucha wrote:
> On 2/17/26 14:42, Andrea Cervesato wrote:
> > Hi!
> > 
> >>   	if (do_readahead) {
> >>   		do {
> >> +			cached_prev = get_file_cached_bytes(fname, fsize);
> >> +			tst_res(TDEBUG, "Per-file cached: %lu kB",
> >> +				cached_prev / 1024);
> > 
> > This should be %zu. The rest LGTM
> > 
> > Reviewed-by: Andrea Cervesato <andrea.cervesato@suse.com>
>
> Good catch. Should I send a v2 with fix for the format string and the 
> MIN_SLEEP_LIMIT rename, or can you fix both during merge?

Maybe it's better to have a v2, so other people will take a look at it.
Thanks,

-- 
Andrea Cervesato
SUSE QE Automation Engineer Linux
andrea.cervesato@suse.com



More information about the ltp mailing list