diff options
author | Jarno Rajahalme <jrajahalme@nicira.com> | 2013-12-30 15:58:58 -0800 |
---|---|---|
committer | Jarno Rajahalme <jrajahalme@nicira.com> | 2013-12-30 16:52:43 -0800 |
commit | 758c456df570a1af1d9e913d50a3478785663e66 (patch) | |
tree | f81649ae9f49eccb826d954a4fd662e0dde76a8e /lib/dpif-provider.h | |
parent | 09f9da0bcaf0f6fd4221b28ecedc3d4db6235fe5 (diff) | |
download | openvswitch-758c456df570a1af1d9e913d50a3478785663e66.tar.gz |
dpif: Use explicit packet metadata.
This helps reduce confusion about when a flow is a flow and when it is
just metadata.
Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'lib/dpif-provider.h')
-rw-r--r-- | lib/dpif-provider.h | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/lib/dpif-provider.h b/lib/dpif-provider.h index 90a02b436..e16a4263f 100644 --- a/lib/dpif-provider.h +++ b/lib/dpif-provider.h @@ -311,12 +311,10 @@ struct dpif_class { int (*flow_dump_done)(const struct dpif *dpif, void *state); /* Performs the 'execute->actions_len' bytes of actions in - * 'execute->actions' on the Ethernet frame specified in 'execute->packet' - * taken from the flow specified in the 'execute->key_len' bytes of - * 'execute->key'. ('execute->key' is mostly redundant with - * 'execute->packet', but it contains some metadata that cannot be - * recovered from 'execute->packet', such as tunnel and in_port.) */ - int (*execute)(struct dpif *dpif, const struct dpif_execute *execute); + * 'execute->actions' on the Ethernet frame in 'execute->packet' + * and on the packet metadata in 'execute->md'. + * May modify both packet and metadata. */ + int (*execute)(struct dpif *dpif, struct dpif_execute *execute); /* Executes each of the 'n_ops' operations in 'ops' on 'dpif', in the order * in which they are specified, placing each operation's results in the |