[PATCH net v2] openvswitch: fix conntrack netlink event delivery

2016-06-28 Thread Samuel Gauthier
non-commit case as well. Fixes: 7f8a436eaa2c ("openvswitch: Add conntrack action") CC: Joe Stringer CC: Justin Pettit CC: Andy Zhou CC: Thomas Graf Signed-off-by: Samuel Gauthier --- v1 -> v2: don't break OVS_CT_ATTR_COMMIT attribute This patch was tested against the net

[PATCH net] openvswitch: fix conntrack netlink event delivery

2016-06-28 Thread Samuel Gauthier
, i.e not only in the commit case. Fixes: 7f8a436eaa2c ("openvswitch: Add conntrack action") CC: Joe Stringer CC: Justin Pettit CC: Andy Zhou CC: Thomas Graf Signed-off-by: Samuel Gauthier --- This patch was tested against the net tree, checking the notifications with conntrack -E. D

Re: [PATCH net-next 0/2] ovs: refresh a flow via netlink

2016-03-20 Thread Samuel Gauthier
2016-03-17 0:23 GMT+01:00 Jesse Gross : > On Wed, Mar 16, 2016 at 8:07 AM, Samuel Gauthier > wrote: >> This patchset adds a netlink api to refresh an existing flow in >> openvswitch. >> >> When a packet is sent in the openvswitch kernel datapath and no >> flow

[PATCH net-next 2/2] ovs: support to refresh a flow via netlink

2016-03-19 Thread Samuel Gauthier
_USED to a 'set flow' netlink message refreshes the flow used time to the current time. The value in OVS_FLOW_ATTR_USED attribute is not used in this case. Signed-off-by: Samuel Gauthier --- net/openvswitch/datapath.c | 2 ++ net/openvswitch/flow.c | 6 ++ net/openvswitch/flow.h

[PATCH net-next 1/2] ovs: split ovs_flow_stats_update into skb and stats

2016-03-19 Thread Samuel Gauthier
The function to update statistics takes a skbuff as parameter. It would be handy to have the statistics update part in one function, and the skbuff part in another one. The next commit will make use of the new ovs_flow_stats_update function. Signed-off-by: Samuel Gauthier --- net/openvswitch

[PATCH net-next 0/2] ovs: refresh a flow via netlink

2016-03-18 Thread Samuel Gauthier
behavior for all the flows, which is not always what you want. I tested this with a patch for the openvswitch tree of the ovs-dpctl mod-flow command, which adds a --refresh flag. I will submit the patch if this patchset is accepted. Samuel Gauthier (2): ovs: split ovs_flow_stats_update into skb

[PATCH v2 net-next] ovs: allow nl 'flow set' to use ufid without flow key

2016-03-10 Thread Samuel Gauthier
is used to validate the actions provided by the userland. This commit allows to use the ufid without having to provide the flow key, as it is already done in the netlink 'flow get' and 'flow del' path. The flow key remains mandatory when an action is provided. Signed-off-by: Sam

[PATCH net-next] ovs: allow nl 'flow set' to use ufid without flow key

2016-03-09 Thread Samuel Gauthier
allows to use the ufid without having to provide the flow key, as it is already done in the netlink 'flow get' and 'flow del' path. Signed-off-by: Samuel Gauthier --- net/openvswitch/datapath.c | 21 ++--- 1 file changed, 10 insertions(+), 11 deletions(