<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-family:monospace"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 3, 2021 at 9:03 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Ziyao,<br>
<br>
> Instead of using the libaio wrapper function, the system call is changed to be invoked via syscall(2).<br></blockquote><div><br></div><div><div style="font-family:monospace" class="gmail_default">I probably missed some discussion. Why are we replacing it with syscall?<br></div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Generally LGTM. Major thing for me is to replace <libaio.h> with <linux/aio_abi.h>.<br>
Do you plan to transform other tests which now still use <libaio.h><br>
(testcases/kernel/io/aio/ and testcases/kernel/io/ltp-aiodio/, which are BTW<br>
problematic on mainline kernel)? I guess it'd be good to keep some test using <libaio.h>.<br>
<br>
FYI if you don't prefer put your copyright, I'll update it with LTP copyright.<br>
<br>
You also mix more things in single commit: using kernel API instead of libaio<br>
API (that you mentioned) with both code and comments cleanup. But commits are<br>
quite compact thus LGTM.<br>
<br>
Reviewed-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
For whole patchset.<br>
<br>
Kind regards,<br>
Petr<br>
<br>
> Xie Ziyao (3):<br>
>   syscalls/io_destroy: Convert libaio wrapper function to kernel syscall<br>
>   syscalls/io_setup: Convert libaio wrapper function to kernel syscall<br>
>   syscalls/io_submit: Convert libaio wrapper function to kernel syscall<br>
<br>
>  .../kernel/syscalls/io_destroy/io_destroy01.c |  49 ++------<br>
>  .../kernel/syscalls/io_setup/io_setup01.c     |  94 +++++----------<br>
>  .../kernel/syscalls/io_submit/io_submit01.c   | 110 ++++++++----------<br>
>  3 files changed, 86 insertions(+), 167 deletions(-)<br>
<br>
-- <br>
Mailing list info: <a href="https://lists.linux.it/listinfo/ltp" rel="noreferrer" target="_blank">https://lists.linux.it/listinfo/ltp</a><br>
<br>
</blockquote></div></div>