BUILD: evports: remove a leftover from the dead_fd cleanup
authorWilly Tarreau <w@1wt.eu>
Tue, 30 Nov 2021 08:32:21 +0000 (09:32 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Thu, 2 Dec 2021 14:10:45 +0000 (15:10 +0100)
Commit b1f29bc62 ("MINOR: activity/fd: remove the dead_fd counter") got
rid of FD_UPDT_DEAD, but evports managed to slip through the cracks and
wasn't cleaned up, thus it doesn't build anymore, as reported in github
issue #1467. We just need to remove the related lines since the situation
is already handled by the remaining conditions.

Thanks to Dominik Hassler for reporting the issue and confirming the fix.

This must be backported to 2.5 only.

(cherry picked from commit 3cc1e3d5ca9b0429f4f1e6618c4fbabac0cfb38b)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>

src/ev_evports.c

index 73e9751..c3be00e 100644 (file)
@@ -250,10 +250,6 @@ static void _do_poll(struct poller *p, int exp, int wake)
                 */
                ret = fd_update_events(fd, n);
 
-               /* If the FD was already dead , skip it */
-               if (ret == FD_UPDT_DEAD)
-                       continue;
-
                /* disable polling on this instance if the FD was migrated */
                if (ret == FD_UPDT_MIGRATED) {
                        if (!HA_ATOMIC_BTS(&fdtab[fd].update_mask, tid))