[LTP] [PATCH v5] Refactor aiocp using new LTP API
Andrea Cervesato
andrea.cervesato@suse.com
Wed Aug 10 09:00:29 CEST 2022
Hi!
Today I send the new patch. Thanks for the review
ndrea
On 8/10/22 08:57, Petr Vorel wrote:
> Hi Andrea, Cyril,
>
>> Hi!
>> The code looks good now, but still needs a runtime treatement similar to
>> the one done in 3ab0b3cb24198a0a833e4450b8bc9cee8b70baf6.
> OK, new version is needed. When you're in it, could you please consider things
> Cyril asked in v1? [1]
>
> * rename functions
> I guess that it would be cleaner to rename the functions to something as
> put_iocb() and get_iocb() since they do not allcate anything but they
> have to stay.
>
> * I suppose later you implement ioctl() BLKSZGET
> > -static int dev_block_size_by_path(const char *path)
>
> Just FYI this is actually the only correct way how to get the O_DIRECT
> block size, we have to get the device and then call the BLKSZGET
> ioctl(). This code has to be put into the test library and reused in all
> O_DIRECT tests in a followup patch.
>
> Kind regards,
> Petr
>
> [1] https://patchwork.ozlabs.org/comment/2917667/
More information about the ltp
mailing list