CLEANUP: fd: remove fd_remove() and rename fd_dodelete() to fd_delete()
This essentially undoes what we did in fd.c in 1.8 to support seamless
reload. Since we don't need to remove an fd anymore we can turn
fd_delete() to the simple function it used to be.
diff --git a/include/haproxy/fd.h b/include/haproxy/fd.h
index 7b77567..240842a 100644
--- a/include/haproxy/fd.h
+++ b/include/haproxy/fd.h
@@ -60,11 +60,6 @@
*/
void fd_delete(int fd);
-/* Deletes an FD from the fdsets.
- * The file descriptor is kept open.
- */
-void fd_remove(int fd);
-
/*
* Take over a FD belonging to another thread.
* Returns 0 on success, and -1 on failure.
diff --git a/src/fd.c b/src/fd.c
index bf2383e..86c2d43 100644
--- a/src/fd.c
+++ b/src/fd.c
@@ -297,7 +297,7 @@
/* Deletes an FD from the fdsets.
* The file descriptor is also closed.
*/
-static void fd_dodelete(int fd, int do_close)
+void fd_delete(int fd)
{
int locked = fdtab[fd].running_mask != tid_bit;
@@ -329,10 +329,8 @@
fdinfo[fd].port_range = NULL;
fdtab[fd].owner = NULL;
fdtab[fd].thread_mask = 0;
- if (do_close) {
- close(fd);
- _HA_ATOMIC_SUB(&ha_used_fds, 1);
- }
+ close(fd);
+ _HA_ATOMIC_SUB(&ha_used_fds, 1);
if (locked)
fd_clr_running(fd);
}
@@ -393,22 +391,6 @@
return ret;
}
-/* Deletes an FD from the fdsets.
- * The file descriptor is also closed.
- */
-void fd_delete(int fd)
-{
- fd_dodelete(fd, 1);
-}
-
-/* Deletes an FD from the fdsets.
- * The file descriptor is kept open.
- */
-void fd_remove(int fd)
-{
- fd_dodelete(fd, 0);
-}
-
void updt_fd_polling(const int fd)
{
if ((fdtab[fd].thread_mask & all_threads_mask) == tid_bit) {