[LTP] [PATCH v2] Refactor aiocp using new LTP API

Cyril Hrubis chrubis@suse.cz
Tue Aug 2 16:30:28 CEST 2022


Hi!
> I think that check doesn't make much sense and we probably should stuck 
> to "n = aionum - busy",
> since that's the value we always need to allocate since became free in 
> the previous loop iteration.

I guess MIN(aionum-busy, tocopy) or something like this since we want to
avoid schedulling reads past the end of the file.

> >> +	aio_blksize = 64 * 1024;
> >> +	filesize = 1 * 1024 * 1024;
> >> +	aionum = 16;
> >> +	srcflags = O_RDONLY;
> >> +	dstflags = O_WRONLY;
> > Can we just initialize the globals directly instead of here in the test
> > setup()?
> In the last ltp-aiodio tests, we decided to initialize global variables 
> inside setup().

Wasn't it exactly the other way around?

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list