liburingutils: Do not requeue SQE entry during syscall failure
Bug: 430396651
Test: Tested logd on Pixel and monitor lock contention.
Change-Id: Ic3dae9746b0da4ca38c894f924c215f834e112c0
Signed-off-by: Akilesh Kailash <akailash@google.com>
diff --git a/src/IOUringSocketHandler.cpp b/src/IOUringSocketHandler.cpp
index a2efd18..2b6d423 100644
--- a/src/IOUringSocketHandler.cpp
+++ b/src/IOUringSocketHandler.cpp
@@ -213,9 +213,10 @@
// Peek failed (no CQEs ready), block waiting for a single CQE
// Since DEFER_TASK_RUN flag is set for the ring, this
// will trigger the task and initiate the receive of packets
+ //
+ // We shouldn't re-queue SQE entry - see b/430396651
int ret = io_uring_wait_cqe(&mCtx->ring, &cqe);
if (ret) {
- EnqueueMultishotRecvmsg();
continue;
}
}
