From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: passt.top; dmarc=none (p=none dis=none) header.from=gibson.dropbear.id.au Authentication-Results: passt.top; dkim=pass (2048-bit key; secure) header.d=gibson.dropbear.id.au header.i=@gibson.dropbear.id.au header.a=rsa-sha256 header.s=202512 header.b=dtOB8eKo; dkim-atps=neutral Received: from mail.ozlabs.org (gandalf.ozlabs.org [150.107.74.76]) by passt.top (Postfix) with ESMTPS id 2C5FD5A0271 for ; Fri, 09 Jan 2026 00:34:02 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gibson.dropbear.id.au; s=202512; t=1767915239; bh=n5NKrNTAj0gIdwJMZHtBYhUXpTvX/tY6w9I4fOoL+Lg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=dtOB8eKo7NWGAXomgkSyHMVDTplxXDrfY8Xos4q69jlg8JDqzVoe2QWvFRjXXvoXL nONSq1S30HuuqA30Y71vluLBphsUYqSznuMPMEIGhXtb91GxVqypftjB/er+056QwC jrv0+VlnF0wGuV6ov3wuwEmX+P1XapMpgTl5OfBap25SahyRj2xxmzp8p9/9kdL4zo LgICZib2NgDNJGB3EC6lZVvu/QoOu81ykOBTfTJFS3IjhMjJorvLnGo8SBnX91jTVb kXyO0+UoCjoKcYGFBTZzSz0SctQOFsptaeHsEvoieSxl0xMR/8YfeHbeBf+guUbpzu WIz89KtXxHeZw== Received: by gandalf.ozlabs.org (Postfix, from userid 1007) id 4dnLnz5z0kz4wMG; Fri, 09 Jan 2026 10:33:59 +1100 (AEDT) Date: Fri, 9 Jan 2026 10:26:16 +1100 From: David Gibson To: Laurent Vivier Subject: Re: [PATCH v2 1/5] tcp: remove timer update in tcp_epoll_ctl() Message-ID: References: <20260108140119.1204797-1-lvivier@redhat.com> <20260108140119.1204797-2-lvivier@redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="pT96swu1G1sBykYW" Content-Disposition: inline In-Reply-To: <20260108140119.1204797-2-lvivier@redhat.com> Message-ID-Hash: KQCFUBZPDHYF7TAJ65NAHAEP5YXOZWFV X-Message-ID-Hash: KQCFUBZPDHYF7TAJ65NAHAEP5YXOZWFV X-MailFrom: dgibson@gandalf.ozlabs.org X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: passt-dev@passt.top X-Mailman-Version: 3.3.8 Precedence: list List-Id: Development discussion and patches for passt Archived-At: Archived-At: List-Archive: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: --pT96swu1G1sBykYW Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Jan 08, 2026 at 03:01:15PM +0100, Laurent Vivier wrote: > Timer update in tcp_epoll_ctl() has been added by > bb708111833e ("treewide: Packet abstraction with mandatory boundary che= cks") > because epoll_ref stores "conn - tc" that can change in tcp_table_compact= (). >=20 > But since > e2e8219f13b8 ("flow, tcp: Consolidate flow pointer<->index helpers") > we use FLOW_IDX() and since > 8981a720aac4 ("flow: Avoid moving flow entries to compact table") > flow table doesn't use a compaction procedure so FLOW_IDX() never changes. >=20 > Updating the timer event is now a no-op, remove it from tcp_epoll_ctl(). >=20 > Fixes: 8981a720aac4 ("flow: Avoid moving flow entries to compact table") > Signed-off-by: Laurent Vivier Reviewed-by: David Gibson > --- > tcp.c | 12 ------------ > 1 file changed, 12 deletions(-) >=20 > diff --git a/tcp.c b/tcp.c > index e7fa85f346b2..5141cdc7e839 100644 > --- a/tcp.c > +++ b/tcp.c > @@ -552,18 +552,6 @@ static int tcp_epoll_ctl(const struct ctx *c, struct= tcp_tap_conn *conn) > =20 > flow_epollid_set(&conn->f, EPOLLFD_ID_DEFAULT); > =20 > - if (conn->timer !=3D -1) { > - union epoll_ref ref_t =3D { .type =3D EPOLL_TYPE_TCP_TIMER, > - .fd =3D conn->timer, > - .flow =3D FLOW_IDX(conn) }; > - struct epoll_event ev_t =3D { .data.u64 =3D ref_t.u64, > - .events =3D EPOLLIN | EPOLLET }; > - > - if (epoll_ctl(flow_epollfd(&conn->f), EPOLL_CTL_MOD, > - conn->timer, &ev_t)) > - return -errno; > - } > - > return 0; > } > =20 > --=20 > 2.52.0 >=20 --=20 David Gibson (he or they) | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you, not the other way | around. http://www.ozlabs.org/~dgibson --pT96swu1G1sBykYW Content-Type: application/pgp-signature; name=signature.asc -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEEO+dNsU4E3yXUXRK2zQJF27ox2GcFAmlgPRgACgkQzQJF27ox 2GdAMw/+PaSoYYDhmn6y4M+sDG7o41F6uc7pRM6AJ/Z0DPEuEyGNbnjcvaN1CsKb n5Sp1ddqqaDs3KY4E3XllUg62ft4XFcu36uCJJUitEKvpVevxYEidubtYT2ldZxd 1OKqKu+0YmHYU7zoxMVy0wDhCoQskUG5InLfBkwCB10N71Zicf3dh+HqfP9uZkbM qWoBA8ett9gPrU7B7alFC811HvDrcJqbDlmjZS3asC9lIKEq6RdDJ8fmk5JGqO7Y OC7tcG8mr987fmre45C4wMr9wyGHhUXy3T0sZUBgVkDah13Jrymqc2BjgcTozvcG pLLxizhKesuWdOLOXj5bEJ2kYvp8Q4chIisoazh5Rq41Uxan7L4vyuTICK0cWP4l RbK34KR3Nyev6sE9+WdyvWG7RdNjYlF2M1pM+uaARiFquHNra6WGwurks+oP0TPb i+uvUu07teFd62xxkN4/FVmO7d1RCK/rPL2LEO9FHb5vw908NU/gIQEmtIFJvYMT 14NhK+9vMRVAt/HnB1IfdsjEmoyOUDxIwIIqcJUbxCV73SekPUuZ0xUmS/EHn7sk tJS4Js/Y7hUQMEU6pZE5Jz+NOJXOqToAr/NF4crt8LW/fiEcY+cXgU4sJxWb/pUj 7iz2cpVqggI4hjOL2GFAHT+IEcLyHYVp4iMd4TKaAR5u7jrLCiU= =+wDF -----END PGP SIGNATURE----- --pT96swu1G1sBykYW--