aboutgitcodebugslistschat
path: root/tcp_splice.c
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2024-01-16 11:50:43 +1100
committerStefano Brivio <sbrivio@redhat.com>2024-01-22 23:35:37 +0100
commit8981a720aac4ab22beb3375cd77062a8aed693e6 (patch)
tree16cdc6bec085763de21b705f5c8188990251f50d /tcp_splice.c
parent9c0881d4f6dd651fd2a40896b54d554cb7ba5b2e (diff)
downloadpasst-8981a720aac4ab22beb3375cd77062a8aed693e6.tar
passt-8981a720aac4ab22beb3375cd77062a8aed693e6.tar.gz
passt-8981a720aac4ab22beb3375cd77062a8aed693e6.tar.bz2
passt-8981a720aac4ab22beb3375cd77062a8aed693e6.tar.lz
passt-8981a720aac4ab22beb3375cd77062a8aed693e6.tar.xz
passt-8981a720aac4ab22beb3375cd77062a8aed693e6.tar.zst
passt-8981a720aac4ab22beb3375cd77062a8aed693e6.zip
flow: Avoid moving flow entries to compact table
Currently we always keep the flow table maximally compact: that is all the active entries are contiguous at the start of the table. Doing this sometimes requires moving an entry when one is freed. That's kind of fiddly, and potentially expensive: it requires updating the hash table for the new location, and depending on flow type, it may require EPOLL_CTL_MOD, system calls to update epoll tags with the new location too. Implement a new way of managing the flow table that doesn't ever move entries. It attempts to maintain some compactness by always using the first free slot for a new connection, and mitigates the effect of non compactness by cheaply skipping over contiguous blocks of free entries. See the "theory of operation" comment in flow.c for details. Signed-off-by: David Gibson <david@gibson.dropbear.id.au>b [sbrivio: additional ASSERT(flow_first_free <= FLOW_MAX - 2) to avoid Coverity Scan false positive] Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'tcp_splice.c')
-rw-r--r--tcp_splice.c11
1 files changed, 0 insertions, 11 deletions
diff --git a/tcp_splice.c b/tcp_splice.c
index daef7de..26d3206 100644
--- a/tcp_splice.c
+++ b/tcp_splice.c
@@ -232,17 +232,6 @@ static void conn_event_do(const struct ctx *c, struct tcp_splice_conn *conn,
/**
- * tcp_splice_conn_update() - Update tcp_splice_conn when being moved in the table
- * @c: Execution context
- * @new: New location of tcp_splice_conn
- */
-void tcp_splice_conn_update(const struct ctx *c, struct tcp_splice_conn *new)
-{
- if (tcp_splice_epoll_ctl(c, new))
- conn_flag(c, new, CLOSING);
-}
-
-/**
* tcp_splice_flow_defer() - Deferred per-flow handling (clean up closed)
* @flow: Flow table entry for this connection
*