t/io_uring: fix size confusion in allocate_mem()
authorJens Axboe <axboe@kernel.dk>
Tue, 5 Nov 2024 14:57:41 +0000 (07:57 -0700)
committerJens Axboe <axboe@kernel.dk>
Tue, 5 Nov 2024 14:57:41 +0000 (07:57 -0700)
It should always be using the size passed in as to what amount of memory
should be allocated, not the individual IO size. This didn't matter
previously, as each buffer was allocated independently, but it matters
now where a single region will cover all IO buffers.

Signed-off-by: Jens Axboe <axboe@kernel.dk>
t/io_uring.c

index dec9552b7c99e1aa92ada4a1f213ac7115a66d04..eea5fe0a9fdd184adf09078caabc4ccfb9f1d210 100644 (file)
@@ -1006,7 +1006,7 @@ static void *allocate_mem(struct submitter *s, int size)
                return numa_alloc_onnode(size, s->numa_node);
 #endif
 
-       if (posix_memalign(&buf, t_io_uring_page_size, bs)) {
+       if (posix_memalign(&buf, t_io_uring_page_size, size)) {
                printf("failed alloc\n");
                return NULL;
        }