Commit 1b16bf42 authored by Jason Wang's avatar Jason Wang Committed by David S. Miller

macvtap: avoid hash calculating for single queue

We decide the rxq through calculating its hash which is not necessary
if we only have one rx queue. So this patch skip this and just return
queue 0. Test shows 22% improving on guest rx pps.

Before: 1201504 pkts/s
After:  1472731 pkts/s
Signed-off-by: default avatarJason Wang <jasowang@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent e980a076
...@@ -299,6 +299,9 @@ static struct macvtap_queue *macvtap_get_queue(struct net_device *dev, ...@@ -299,6 +299,9 @@ static struct macvtap_queue *macvtap_get_queue(struct net_device *dev,
if (!numvtaps) if (!numvtaps)
goto out; goto out;
if (numvtaps == 1)
goto single;
/* Check if we can use flow to select a queue */ /* Check if we can use flow to select a queue */
rxq = skb_get_hash(skb); rxq = skb_get_hash(skb);
if (rxq) { if (rxq) {
...@@ -316,6 +319,7 @@ static struct macvtap_queue *macvtap_get_queue(struct net_device *dev, ...@@ -316,6 +319,7 @@ static struct macvtap_queue *macvtap_get_queue(struct net_device *dev,
goto out; goto out;
} }
single:
tap = rcu_dereference(vlan->taps[0]); tap = rcu_dereference(vlan->taps[0]);
out: out:
return tap; return tap;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment