[ovs-dev] [PATCH 14/23] datapath: select vport upcall portid directly
Tonghao Zhang
xiangxia.m.yue at gmail.com
Tue Aug 25 04:00:51 UTC 2020
On Fri, Aug 21, 2020 at 6:50 AM Greg Rose <gvrose8192 at gmail.com> wrote:
>
> From: Tonghao Zhang <xiangxia.m.yue at gmail.com>
>
> Upstream commit:
> commit 90ce9f23a886bdef7a4b7a9bd52c7a50a6a81635
> Author: Tonghao Zhang <xiangxia.m.yue at gmail.com>
> Date: Thu Nov 7 00:34:28 2019 +0800
>
> net: openvswitch: select vport upcall portid directly
>
> The commit 69c51582ff786 ("dpif-netlink: don't allocate per
> thread netlink sockets"), in Open vSwitch ovs-vswitchd, has
> changed the number of allocated sockets to just one per port
> by moving the socket array from a per handler structure to
> a per datapath one. In the kernel datapath, a vport will have
> only one socket in most case, if so select it directly in
> fast-path.
>
> Signed-off-by: Tonghao Zhang <xiangxia.m.yue at gmail.com>
> Acked-by: Pravin B Shelar <pshelar at ovn.org>
> Signed-off-by: David S. Miller <davem at davemloft.net>
>
> Cc: Tonghao Zhang <xiangxia.m.yue at gmail.com>
> Signed-off-by: Greg Rose <gvrose8192 at gmail.com>
Reviewed-by: Tonghao Zhang <xiangxia.m.yue at gmail.com>
> ---
> datapath/vport.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/datapath/vport.c b/datapath/vport.c
> index f929282..bd62c56 100644
> --- a/datapath/vport.c
> +++ b/datapath/vport.c
> @@ -507,8 +507,9 @@ u32 ovs_vport_find_upcall_portid(const struct vport *vport, struct sk_buff *skb)
>
> ids = rcu_dereference(vport->upcall_portids);
>
> - if (ids->n_ids == 1 && ids->ids[0] == 0)
> - return 0;
> + /* If there is only one portid, select it in the fast-path. */
> + if (ids->n_ids == 1)
> + return ids->ids[0];
>
> hash = skb_get_hash(skb);
> ids_index = hash - ids->n_ids * reciprocal_divide(hash, ids->rn_ids);
> --
> 1.8.3.1
>
--
Best regards, Tonghao
More information about the dev
mailing list