aboutgitcodebugslistschat
path: root/flow.c
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2024-01-16 11:50:41 +1100
committerStefano Brivio <sbrivio@redhat.com>2024-01-22 23:35:31 +0100
commit4a849e95267c30e63dbe61c4576c059c927b99d9 (patch)
tree5d73a19f7ee2bb10633deabcf7956bddcc10cdf4 /flow.c
parentfb7c00169d49a99ea2d14dae05bf0c8190f0971c (diff)
downloadpasst-4a849e95267c30e63dbe61c4576c059c927b99d9.tar
passt-4a849e95267c30e63dbe61c4576c059c927b99d9.tar.gz
passt-4a849e95267c30e63dbe61c4576c059c927b99d9.tar.bz2
passt-4a849e95267c30e63dbe61c4576c059c927b99d9.tar.lz
passt-4a849e95267c30e63dbe61c4576c059c927b99d9.tar.xz
passt-4a849e95267c30e63dbe61c4576c059c927b99d9.tar.zst
passt-4a849e95267c30e63dbe61c4576c059c927b99d9.zip
flow: Abstract allocation of new flows with helper function
Currently tcp.c open codes the process of allocating a new flow from the flow table: twice, in fact, once for guest to host and once for host to guest connections. This duplication isn't ideal and will get worse as we add more protocols to the flow table. It also makes it harder to experiment with different ways of handling flow table allocation. Instead, introduce a function to allocate a new flow: flow_alloc(). In some cases we currently check if we're able to allocate, but delay the actual allocation. We now handle that slightly differently with a flow_alloc_cancel() function to back out a recent allocation. We have that separate from a flow_free() function, because future changes we have in mind will need to handle this case a little differently. Signed-off-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'flow.c')
-rw-r--r--flow.c26
1 files changed, 26 insertions, 0 deletions
diff --git a/flow.c b/flow.c
index 64de75c..63eefd6 100644
--- a/flow.c
+++ b/flow.c
@@ -51,6 +51,32 @@ void flow_log_(const struct flow_common *f, int pri, const char *fmt, ...)
}
/**
+ * flow_alloc() - Allocate a new flow
+ *
+ * Return: pointer to an unused flow entry, or NULL if the table is full
+ */
+union flow *flow_alloc(void)
+{
+ if (flow_count >= FLOW_MAX)
+ return NULL;
+
+ return &flowtab[flow_count++];
+}
+
+/**
+ * flow_alloc_cancel() - Free a newly allocated flow
+ * @flow: Flow to deallocate
+ *
+ * @flow must be the last flow allocated by flow_alloc()
+ */
+void flow_alloc_cancel(union flow *flow)
+{
+ ASSERT(FLOW_IDX(flow) == flow_count - 1);
+ memset(flow, 0, sizeof(*flow));
+ flow_count--;
+}
+
+/**
* flow_table_compact() - Perform compaction on flow table
* @c: Execution context
* @hole: Pointer to recently closed flow