Attention is currently required from: laforge, pespin.
6 comments:
File src/core/osmo_io_internal.h:
Patch Set #2, Line 110: void *read_ring[IOFD_MSGHDR_READ_SQES];
some comments here would be useful to explan what those fields are and how hey are used (not just th […]
Done
File src/core/osmo_io_uring.c:
Patch Set #2, Line 67: g_io_uring_queue
pleaes use a more descriptive name. The variable is not a qeuue, but it is the size of the ring. […]
Done
Patch Set #2, Line 113: OSMO_IO_URING_QUEUE
like the variable name in the code, the environment variable name shold be self-descriptive. […]
Done
Patch Set #2, Line 163: osmo_iofd_uring_get_sqe
the 'osmo_' prefix is reserverd for exported functions. […]
Done
Patch Set #2, Line 172: LOGP(DLIO, LOGL_NOTICE, "io_uring too small to handle all SQEs, increasing size to %d.\n", g_io_uring_queue);
minor: while the current implementation alsways doubles the size, it might still be useful to print […]
Done
is there code to clean up the old ring after completion of the last cqe of the old ring?
No, it doesn't, because I don't track the number of submissions and the number of completions. I could do that. This would make it more complex, especially during cancellation process. I can add this if you want and test with the ttcn3 tests.
To view, visit change 40725. To unsubscribe, or for help writing mail filters, visit settings.