Given that all our I/Os are now directed from top to bottom and not the
opposite way around, and the FD cache was removed, it doesn't make sense
anymore to create FDs that are marked not ready since this would prevent
the first accesses unless the caller explicitly does an fd_may_recv()
which is not expected to be its job (which conn_ctrl_init() has to do
by the way). Let's move this into fd_insert() instead, and have a single
atomic operation for both directions via fd_may_both().
int fd = conn->handle.fd;
fd_insert(fd, conn, conn_fd_handler, tid_bit);
- /* mark the fd as ready so as not to needlessly poll at the beginning */
- fd_may_recv(fd);
- fd_may_send(fd);
conn->flags |= CO_FL_CTRL_READY;
}
}
return;
}
+/* Report that FD <fd> may receive and send without polling. Used at FD
+ * initialization.
+ */
+static inline void fd_may_both(const int fd)
+{
+ HA_ATOMIC_OR(&fdtab[fd].state, FD_EV_READY_RW);
+}
+
/* Disable readiness when active. This is useful to interrupt reading when it
* is suspected that the end of data might have been reached (eg: short read).
* This can only be done using level-triggered pollers, so if any edge-triggered
*/
if (locked)
HA_SPIN_UNLOCK(FD_LOCK, &fdtab[fd].lock);
+ /* the two directions are ready until proven otherwise */
+ fd_may_both(fd);
_HA_ATOMIC_ADD(&ha_used_fds, 1);
}