Ilya Maximets <[email protected]> writes: > 'OVS_CLONE_ATTR_EXEC' is an internal attribute that is used for > performance optimization inside the kernel. It's added by the kernel > while parsing user-provided actions and should not be sent during the > flow dump as it's not part of the uAPI. > > The issue doesn't cause any significant problems to the ovs-vswitchd > process, because reported actions are not really used in the > application lifecycle and only supposed to be shown to a human via > ovs-dpctl flow dump. However, the action list is still incorrect > and causes the following error if the user wants to look at the > datapath flows: > > # ovs-dpctl add-dp system@ovs-system > # ovs-dpctl add-flow "<flow match>" "clone(ct(commit),0)" > # ovs-dpctl dump-flows > <flow match>, packets:0, bytes:0, used:never, > actions:clone(bad length 4, expected -1 for: action0(01 00 00 00), > ct(commit),0) > > With the fix: > > # ovs-dpctl dump-flows > <flow match>, packets:0, bytes:0, used:never, > actions:clone(ct(commit),0) > > Additionally fixed an incorrect attribute name in the comment. > > Fixes: b233504033db ("openvswitch: kernel datapath clone action") > Signed-off-by: Ilya Maximets <[email protected]> > ---
Acked-by: Aaron Conole <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
