diff options
| author | Ilya Maximets <i.maximets@ovn.org> | 2019-12-08 18:09:53 +0100 |
|---|---|---|
| committer | Ilya Maximets <i.maximets@ovn.org> | 2020-01-08 16:02:37 +0100 |
| commit | 7a5e0ee7cc1c8911d3b0085a066facab076647d5 (patch) | |
| tree | c2755c6dfa2a932dcbb0c2137d5dd54855e90fb5 /ofproto | |
| parent | e988b8abeec9d4be94b519c5d4ed4586ff71fde0 (diff) | |
| download | openvswitch-7a5e0ee7cc1c8911d3b0085a066facab076647d5.tar.gz | |
dpif: Turn dpif_flow_hash function into generic odp_flow_key_hash.
Current implementation of dpif_flow_hash() doesn't depend on datapath
interface and only complicates the callers by forcing them to figure
out what is their current 'dpif'. If we'll need different hashing
for different 'dpif's we'll implement an API for dpif-providers
and each dpif implementation will be able to use their local function
directly without calling it via dpif API.
This change will allow us to not store 'dpif' pointer in the userspace
datapath implementation which is broken and will be removed in next
commits.
This patch moves dpif_flow_hash() to odp-util module and replaces
unused odp_flow_key_hash() by it, along with removing of unused 'dpif'
argument.
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
Acked-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'ofproto')
| -rw-r--r-- | ofproto/ofproto-dpif.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index f782f865f..d3cb39207 100644 --- a/ofproto/ofproto-dpif.c +++ b/ofproto/ofproto-dpif.c @@ -929,7 +929,7 @@ check_ufid(struct dpif_backer *backer) ofpbuf_use_stack(&key, &keybuf, sizeof keybuf); odp_flow_key_from_flow(&odp_parms, &key); - dpif_flow_hash(backer->dpif, key.data, key.size, &ufid); + odp_flow_key_hash(key.data, key.size, &ufid); enable_ufid = dpif_probe_feature(backer->dpif, "UFID", &key, NULL, &ufid); |