aboutgitcodebugslistschat
path: root/flow.h
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2024-02-28 22:25:10 +1100
committerStefano Brivio <sbrivio@redhat.com>2024-02-29 09:48:01 +0100
commit0f938c3b9a9dbc854c1d2e33fab5af41b4a660c8 (patch)
tree787aef747d60b8aacb3e1d472aeac8b69994ad8c /flow.h
parentd0550f97cd2f38c74806c10904341584f0c0a2ad (diff)
downloadpasst-0f938c3b9a9dbc854c1d2e33fab5af41b4a660c8.tar
passt-0f938c3b9a9dbc854c1d2e33fab5af41b4a660c8.tar.gz
passt-0f938c3b9a9dbc854c1d2e33fab5af41b4a660c8.tar.bz2
passt-0f938c3b9a9dbc854c1d2e33fab5af41b4a660c8.tar.lz
passt-0f938c3b9a9dbc854c1d2e33fab5af41b4a660c8.tar.xz
passt-0f938c3b9a9dbc854c1d2e33fab5af41b4a660c8.tar.zst
passt-0f938c3b9a9dbc854c1d2e33fab5af41b4a660c8.zip
flow: Clarify flow entry life cycle, introduce uniform logging
Our allocation scheme for flow entries means there are some non-obvious constraints on when what things can be done with an entry. Add a big doc comment explaining the life cycle. In addition, make a FLOW_START() macro to mark one of the important transitions. This encourages correct usage, by making it natural to only access the flow type specific structure after calling it. It also logs that a new flow has been created, which is useful for debugging. We also add logging when a flow's lifecycle ends. This doesn't need a new helper, because it can only happen either from flow_alloc_cancel() or from the flow deferred handler. Signed-off-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'flow.h')
-rw-r--r--flow.h5
1 files changed, 5 insertions, 0 deletions
diff --git a/flow.h b/flow.h
index e9b3ce3..8b66751 100644
--- a/flow.h
+++ b/flow.h
@@ -45,6 +45,11 @@ struct flow_common {
#define FLOW_TABLE_PRESSURE 30 /* % of FLOW_MAX */
#define FLOW_FILE_PRESSURE 30 /* % of c->nofile */
+union flow *flow_start(union flow *flow, enum flow_type type,
+ unsigned iniside);
+#define FLOW_START(flow_, t_, var_, i_) \
+ (&flow_start((flow_), (t_), (i_))->var_)
+
/**
* struct flow_sidx - ID for one side of a specific flow
* @side: Side referenced (0 or 1)