linux.git/net/openvswitch, branch v3.14-rc2

net: replace macros net_random and net_srandom with direct calls to prandom

2014-01-14T23:15:25+00:00

This patch removes the net_random and net_srandom macros and replaces
them with direct calls to the prandom ones. As new commits only seem to
use prandom_u32 there is no use to keep them around.
This change makes it easier to grep for users of prandom_u32.

Signed-off-by: Aruna-Hewapathirane 
Suggested-by: Hannes Frederic Sowa 
Acked-by: Hannes Frederic Sowa 
Signed-off-by: David S. Miller

openvswitch: Use kmem_cache_free() instead of kfree()

2014-01-09T19:26:39+00:00

memory allocated by kmem_cache_alloc() should be freed using
kmem_cache_free(), not kfree().

Fixes: e298e5057006 ('openvswitch: Per cpu flow stats.')
Signed-off-by: Wei Yongjun 
Acked-by: Jesse Gross 
Signed-off-by: David S. Miller

Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch

2014-01-07T00:48:38+00:00

Jesse Gross says:

====================
[GIT net-next] Open vSwitch

Open vSwitch changes for net-next/3.14. Highlights are:
 * Performance improvements in the mechanism to get packets to userspace
   using memory mapped netlink and skb zero copy where appropriate.
 * Per-cpu flow stats in situations where flows are likely to be shared
   across CPUs. Standard flow stats are used in other situations to save
   memory and allocation time.
 * A handful of code cleanups and rationalization.
====================

Signed-off-by: David S. Miller

ovs: make functions local

2014-01-06T23:54:39+00:00

Several functions and datastructures could be local
Found with 'make namespacecheck'

Signed-off-by: Stephen Hemminger 
Signed-off-by: Jesse Gross

openvswitch: Compute checksum in skb_gso_segment() if needed

2014-01-06T23:53:24+00:00

The copy & csum optimization is no longer present with zerocopy
enabled. Compute the checksum in skb_gso_segment() directly by
dropping the HW CSUM capability from the features passed in.

Signed-off-by: Thomas Graf 
Signed-off-by: Jesse Gross

openvswitch: Use skb_zerocopy() for upcall

2014-01-06T23:53:17+00:00

Use of skb_zerocopy() can avoid the expensive call to memcpy()
when copying the packet data into the Netlink skb. Completes
checksum through skb_checksum_help() if not already done in
GSO segmentation.

Zerocopy is only performed if user space supported unaligned
Netlink messages. memory mapped netlink i/o is preferred over
zerocopy if it is set up.

Cost of upcall is significantly reduced from:
+   7.48%       vhost-8471  [k] memcpy
+   5.57%     ovs-vswitchd  [k] memcpy
+   2.81%       vhost-8471  [k] csum_partial_copy_generic

to:
+   5.72%     ovs-vswitchd  [k] memcpy
+   3.32%       vhost-5153  [k] memcpy
+   0.68%       vhost-5153  [k] skb_zerocopy

(megaflows disabled)

Signed-off-by: Thomas Graf 
Signed-off-by: Jesse Gross

openvswitch: Pass datapath into userspace queue functions

2014-01-06T23:53:07+00:00

Allows removing the net and dp_ifindex argument and simplify the
code.

Signed-off-by: Thomas Graf 
Signed-off-by: Jesse Gross

openvswitch: Drop user features if old user space attempted to create datapath

2014-01-06T23:53:00+00:00

Drop user features if an outdated user space instance that does not
understand the concept of user_features attempted to create a new
datapath.

Signed-off-by: Thomas Graf 
Signed-off-by: Jesse Gross

openvswitch: Allow user space to announce ability to accept unaligned Netlink messages

2014-01-06T23:52:53+00:00

Signed-off-by: Thomas Graf 
Reviewed-by: Daniel Borkmann 
Signed-off-by: Jesse Gross

openvswitch: remove duplicated include from flow_table.c

2014-01-06T23:52:35+00:00

Remove duplicated include.

Signed-off-by: Wei Yongjun 
Signed-off-by: Jesse Gross