* [LTP] [PATCH] open_posix_testsuite/lio_listio/2-1: remove check for individual completion
@ 2022-04-19 10:44 Jan Stancek
2022-04-19 11:36 ` Cyril Hrubis
0 siblings, 1 reply; 3+ messages in thread
From: Jan Stancek @ 2022-04-19 10:44 UTC (permalink / raw)
To: ltp
lio_listio() operations are executed in an unspecified order and
checking if any specific one from list already completed appears
meaningless. Remove it.
Signed-off-by: Jan Stancek <jstancek@redhat.com>
---
.../conformance/interfaces/lio_listio/2-1.c | 28 -------------------
1 file changed, 28 deletions(-)
diff --git a/testcases/open_posix_testsuite/conformance/interfaces/lio_listio/2-1.c b/testcases/open_posix_testsuite/conformance/interfaces/lio_listio/2-1.c
index 72a1113e06a3..efcd5b3fcb39 100644
--- a/testcases/open_posix_testsuite/conformance/interfaces/lio_listio/2-1.c
+++ b/testcases/open_posix_testsuite/conformance/interfaces/lio_listio/2-1.c
@@ -37,16 +37,8 @@
#define NUM_AIOCBS 256
#define BUF_SIZE 1024
-static volatile int received_selected;
static volatile int received_all;
-static void sigrt1_handler(int signum PTS_ATTRIBUTE_UNUSED,
- siginfo_t *info,
- void *context PTS_ATTRIBUTE_UNUSED)
-{
- received_selected = info->si_value.sival_int;
-}
-
static void sigrt2_handler(int signum PTS_ATTRIBUTE_UNUSED,
siginfo_t *info PTS_ATTRIBUTE_UNUSED,
void *context PTS_ATTRIBUTE_UNUSED)
@@ -102,11 +94,6 @@ int main(void)
aiocbs[i]->aio_buf = &bufs[i * BUF_SIZE];
aiocbs[i]->aio_nbytes = BUF_SIZE;
aiocbs[i]->aio_lio_opcode = LIO_WRITE;
-
- /* Use SIRTMIN+1 for individual completions */
- aiocbs[i]->aio_sigevent.sigev_notify = SIGEV_SIGNAL;
- aiocbs[i]->aio_sigevent.sigev_signo = SIGRTMIN + 1;
- aiocbs[i]->aio_sigevent.sigev_value.sival_int = i;
}
/* Use SIGRTMIN+2 for list completion */
@@ -114,12 +101,6 @@ int main(void)
event.sigev_signo = SIGRTMIN + 2;
event.sigev_value.sival_ptr = NULL;
- /* Setup handler for individual operation completion */
- action.sa_sigaction = sigrt1_handler;
- sigemptyset(&action.sa_mask);
- action.sa_flags = SA_SIGINFO | SA_RESTART;
- sigaction(SIGRTMIN + 1, &action, NULL);
-
/* Setup handler for list completion */
action.sa_sigaction = sigrt2_handler;
sigemptyset(&action.sa_mask);
@@ -139,15 +120,6 @@ int main(void)
exit(PTS_FAIL);
}
- if (received_selected == NUM_AIOCBS - 1) {
- printf(TNAME " lio_listio() waited\n");
- for (i = 0; i < NUM_AIOCBS; i++)
- free(aiocbs[i]);
- free(bufs);
- close(fd);
- exit(PTS_FAIL);
- }
-
if (received_all != 0) {
printf(TNAME
" Error lio_listio() waited for list completion\n");
--
2.27.0
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [LTP] [PATCH] open_posix_testsuite/lio_listio/2-1: remove check for individual completion
2022-04-19 10:44 [LTP] [PATCH] open_posix_testsuite/lio_listio/2-1: remove check for individual completion Jan Stancek
@ 2022-04-19 11:36 ` Cyril Hrubis
2022-04-20 6:19 ` Jan Stancek
0 siblings, 1 reply; 3+ messages in thread
From: Cyril Hrubis @ 2022-04-19 11:36 UTC (permalink / raw)
To: Jan Stancek; +Cc: ltp
Hi!
Agree.
Reviewed-by: Cyril Hrubis <chrubis@suse.cz>
--
Cyril Hrubis
chrubis@suse.cz
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [LTP] [PATCH] open_posix_testsuite/lio_listio/2-1: remove check for individual completion
2022-04-19 11:36 ` Cyril Hrubis
@ 2022-04-20 6:19 ` Jan Stancek
0 siblings, 0 replies; 3+ messages in thread
From: Jan Stancek @ 2022-04-20 6:19 UTC (permalink / raw)
To: Cyril Hrubis; +Cc: LTP List
Thanks, pushed.
On Tue, Apr 19, 2022 at 1:34 PM Cyril Hrubis <chrubis@suse.cz> wrote:
>
> Hi!
> Agree.
>
> Reviewed-by: Cyril Hrubis <chrubis@suse.cz>
>
> --
> Cyril Hrubis
> chrubis@suse.cz
>
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-04-20 6:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-19 10:44 [LTP] [PATCH] open_posix_testsuite/lio_listio/2-1: remove check for individual completion Jan Stancek
2022-04-19 11:36 ` Cyril Hrubis
2022-04-20 6:19 ` Jan Stancek
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.