diff options
author | Jens Axboe <axboe@kernel.dk> | 2019-01-10 15:11:07 -0700 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2019-01-10 15:11:07 -0700 |
commit | a992ffa48292f1175ea2621de58fb4cde3bb62dd (patch) | |
tree | 36db000c24bba813380e8df4dedf66f86713f88d /test | |
parent | fa863f6b4ad73c6bea2cabbd50803af78895e1d5 (diff) |
io_queue_init: pass in flags, not io_uring_params
We don't need any of the information in there in the caller, and
this makes it harder to abuse as we don't require the caller to
have memset() the struct first.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'test')
-rw-r--r-- | test/io_uring-cp.c | 5 | ||||
-rw-r--r-- | test/io_uring-test.c | 5 |
2 files changed, 2 insertions, 8 deletions
diff --git a/test/io_uring-cp.c b/test/io_uring-cp.c index 3850743..a588cc2 100644 --- a/test/io_uring-cp.c +++ b/test/io_uring-cp.c @@ -25,12 +25,9 @@ struct io_data { static int setup_context(unsigned entries, struct io_uring *ring) { - struct io_uring_params p; int ret; - memset(&p, 0, sizeof(p)); - - ret = io_uring_queue_init(entries, &p, ring); + ret = io_uring_queue_init(entries, ring, 0); if (ret < 0) { fprintf(stderr, "queue_init: %s\n", strerror(-ret)); return -1; diff --git a/test/io_uring-test.c b/test/io_uring-test.c index a13e405..42a8745 100644 --- a/test/io_uring-test.c +++ b/test/io_uring-test.c @@ -15,7 +15,6 @@ int main(int argc, char *argv[]) { - struct io_uring_params p; struct io_uring ring; int i, fd, ret, pending, done; struct io_uring_sqe *sqe; @@ -29,9 +28,7 @@ int main(int argc, char *argv[]) return 1; } - memset(&p, 0, sizeof(p)); - - ret = io_uring_queue_init(QD, &p, &ring); + ret = io_uring_queue_init(QD, &ring, 0); if (ret < 0) { fprintf(stderr, "queue_init: %s\n", strerror(-ret)); return 1; |