linux.git/net/openvswitch/datapath.c, branch v3.17-rc5

openvswitch: distinguish between the dropped and consumed skb

2014-09-04T03:50:51+00:00

distinguish between the dropped and consumed skb, not assume the skb
is consumed always

Cc: Thomas Graf 
Cc: Pravin Shelar 
Signed-off-by: Li RongQing 
Acked-by: Pravin B Shelar 
Signed-off-by: David S. Miller

openvswitch: fix a memory leak

2014-09-02T21:01:21+00:00

The user_skb maybe be leaked if the operation on it failed and codes
skipped into the label "out:" without calling genlmsg_unicast.

Cc: Pravin Shelar 
Signed-off-by: Li RongQing 
Acked-by: Pravin B Shelar 
Signed-off-by: David S. Miller

openvswitch: fix duplicate #include headers

2014-08-07T23:02:57+00:00

The #include headers net/genetlink.h and linux/genetlink.h both were
included twice, so delete each of the duplicate.

Signed-off-by: Jean Sacren 
Cc: Pravin Shelar 
Cc: dev@openvswitch.org
Signed-off-by: David S. Miller

openvswitch: Use IS_ERR_OR_NULL

2014-07-29T22:31:56+00:00

This patch introduces the use of the macro IS_ERR_OR_NULL in place of
tests for NULL and IS_ERR.

The following Coccinelle semantic patch was used for making the change:

@@
expression e;
@@

- e == NULL || IS_ERR(e)
+ IS_ERR_OR_NULL(e)
 || ...

Signed-off-by: Himangi Saraogi 
Acked-by: Julia Lawall 
Acked-by: Pravin B Shelar 
Signed-off-by: David S. Miller

openvswitch: Avoid memory corruption in queue_userspace_packet()

2014-07-24T16:37:20+00:00

In queue_userspace_packet(), the ovs_nla_put_flow return value is
not checked. This is fine as long as key_attr_size() returns the
correct value. In case it does not, the current code may corrupt buffer
memory. Add a run time assertion catch this case to avoid silent
failure.

Reported-by: Ben Pfaff 
Signed-off-by: Andy Zhou 
Signed-off-by: Pravin B Shelar

openvswitch: Allow each vport to have an array of 'port_id's.

2014-07-24T08:15:04+00:00

In order to allow handlers directly read upcalls from datapath,
we need to support per-handler netlink socket for each vport in
datapath.  This commit makes this happen.  Also, it is guaranteed
to be backward compatible with previous branch.

Signed-off-by: Alex Wang 
Acked-by: Thomas Graf 
Signed-off-by: Pravin B Shelar

openvswitch: make generic netlink group const

2014-07-17T06:41:13+00:00

Generic netlink tables can be const.

Signed-off-by: Stephen Hemminger 
Signed-off-by: David S. Miller

Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net

2014-07-16T21:09:34+00:00

Signed-off-by: David S. Miller

openvswitch: introduce rtnl ops stub

2014-07-01T21:40:17+00:00

This stub now allows userspace to see IFLA_INFO_KIND for ovs master and
IFLA_INFO_SLAVE_KIND for slave.

Signed-off-by: Jiri Pirko 
Signed-off-by: David S. Miller

openvswitch: Use exact lookup for flow_get and flow_del.

2014-07-01T03:47:15+00:00

Due to the race condition in userspace, there is chance that two
overlapping megaflows could be installed in datapath.  And this
causes userspace unable to delete the less inclusive megaflow flow
even after it timeout, since the flow_del logic will stop at the
first match of masked flow.

This commit fixes the bug by making the kernel flow_del and flow_get
logic check all masks in that case.

Introduced by 03f0d916a (openvswitch: Mega flow implementation).

Signed-off-by: Alex Wang 
Acked-by: Andy Zhou 
Signed-off-by: Pravin B Shelar