[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