On 14 Sep 2020, at 14:23, Eelco Chaudron wrote:
On 14 Sep 2020, at 7:54, Tonghao Zhang wrote:
On Fri, Sep 11, 2020 at 5:46 PM Eelco Chaudron <[email protected]>
wrote:
Hi Tonghao, at al.
This patch is breaking basic flow adding/deletion with ovs-dpctl. I
did
not look at the root cause yet, maybe you can take a peek. This is
present since 13.0, so the potential fix needs backporting.
Here is how to replicate:
ip link add name vport0 type veth peer name vport1
modprobe openvswitch
ovs-dpctl add-dp test
ovs-dpctl add-if test vport0
ovs-dpctl add-flow test
"in_port(0),eth(),eth_type(0x800),ipv4(src=100.1.0.1)" 0
ovs-dpctl del-flow test
"in_port(0),eth(),eth_type(0x800),ipv4(src=100.1.0.1)"
The last command will give:
2020-09-11T09:40:06Z|00001|dpif|WARN|system@test: failed to flow_del
(No
such file or directory) ufid:e4457189-3990-4a01-bdcf-1e5f8b208711
in_port(0),eth(src=00:00:00:00:00:00,dst=00:00:00:00:00:00),eth_type(0x0800),ipv4(src=100.1.0.1,dst=0.0.0.0,proto=0,tos=0,ttl=0,frag=no)
ovs-dpctl: deleting flow (No such file or directory)
Perhaps you need to specify a UFID?
Hi Eelco
In odp_flow_key_hash, OVS will generate a random key for hashing
flow.
The commands ovs-appctl dpctl/add-flow and ovs-appctl dpctl/del-flow
use the
same key, but ovs-dpctl use the different key for hashing flow.
I haven't found a good way to fix it.
1. don't generate a random key for hashing flow
Don't know what the idea was behind this, but could be a good option.
2. use ovs-appctl instead of ovs-dpctl
Not an option as ovs-dpctl can be used without vswitchd running, don't
think ovs-appctl can?
But I think it’s calling the same library functions?
3. revert that patch
I see this is part of a larger set, was it a requirement of the set?
Tonghao, any update on this? As the current branches are broken?
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
