<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<p><br>
</p>
<div class="moz-cite-prefix">On 15/05/20 9:38 am, Viresh Kumar wrote:<br>
</div>
<blockquote type="cite" cite="mid:20200515040849.yu27rdihdih5wxc3@vireshk-i7">
<pre class="moz-quote-pre" wrap="">On 14-05-20, 22:58, Vikas Kumar wrote:
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">Added Linux asynchronous I/O API Test for io_uring_setup(),
io_uring_register() and io_uring_enter().
This test will check io_uring api basic operation.
Signed-off-by: Vikas Kumar <a class="moz-txt-link-rfc2396E" href="mailto:vikas.kumar2@arm.com"><vikas.kumar2@arm.com></a>
Reviewed-by: Cyril Hrubis <a class="moz-txt-link-rfc2396E" href="mailto:chrubis@suse.cz"><chrubis@suse.cz></a>
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
Please don't do this unless someone gave you this explicitly.
</pre>
</blockquote>
<p>Sure I will remove it in next patch set its my mistake. </p>
<p>Thank you Viresh, Petr and Cyril for correcting my mistake.<br>
</p>
<blockquote type="cite" cite="mid:20200515040849.yu27rdihdih5wxc3@vireshk-i7">
<pre class="moz-quote-pre" wrap="">
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">diff --git a/testcases/kernel/syscalls/io_uring/io_uring01.c b/testcases/kernel/syscalls/io_uring/io_uring01.c
+int setup_io_uring_test(struct submitter *s)
+{
+ struct io_sq_ring *sring = &s->sq_ring;
+ struct io_cq_ring *cring = &s->cq_ring;
+ struct io_uring_params p;
+ void *ptr;
+
+ memset(&p, 0, sizeof(p));
+ s->ring_fd = io_uring_setup(QUEUE_DEPTH, &p);
+ if (s->ring_fd == -1) {
+ tst_res(TFAIL | TTERRNO, "io_uring setup return error");
+ return 1;
+ }
+
+ //submission queue ring buffer mapping
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
You need to follow kernel coding guidelines everywhere in the code
here and so no such comments. Try to run checkpatch on this file with
--strict option and you will see the problems..
This API is new and you also need to implement something like
fsopen_supported_by_kernel().
I didn't do in depth reviews though..</pre>
</blockquote>
<p>I will implement io_uring_supported_by_kernel() and checkpatch with --strict option and send In next patch.</p>
<p></p>
<blockquote type="cite" style="color: #000000;">
<pre class="moz-quote-pre" wrap="">
struct io_sq_ring, struct io_cq_ring and struct submitter
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">Cyril: These are local structure although derived from kernel structure so it can be local in this file in place of lapi/io_uring.h
</pre>
<blockquote type="cite" cite="mid:20200515040849.yu27rdihdih5wxc3@vireshk-i7"></blockquote>
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose,
or store or copy the information in any medium. Thank you.
</body>
</html>