Re: [ovs-dev] [PATCH v2] datapath: Add hash info to upcall.

2020-05-28 Thread Ilya Maximets
On 5/27/20 8:56 AM, aginwala wrote: > Thanks Han for backporting it for lower kernels. I tested this patch with > some running workloads with old  3.13* kernels (ovs 2.11*) and it works fine. > In addition I am also considering the discussion [1],  and for ecmp workloads > I see all packets are

Re: [ovs-dev] [PATCH v2] datapath: Add hash info to upcall.

2020-05-27 Thread aginwala
Thanks Han for backporting it for lower kernels. I tested this patch with some running workloads with old 3.13* kernels (ovs 2.11*) and it works fine. In addition I am also considering the discussion [1], and for ecmp workloads I see all packets are hitting the different bucket instead of the

Re: [ovs-dev] [PATCH v2] datapath: Add hash info to upcall.

2020-05-26 Thread 0-day Robot
Bleep bloop. Greetings Han Zhou, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: ERROR: Author Tonghao Zhang needs to sign off. WARNING: Unexpected sign-offs from developers who

[ovs-dev] [PATCH v2] datapath: Add hash info to upcall.

2020-05-26 Thread Han Zhou
This patch backports below upstream patches, and add __skb_set_hash to compat for older kernels. commit b5ab1f1be6180a2e975eede18731804b5164a05d Author: Jakub Kicinski Date: Mon Mar 2 21:05:18 2020 -0800 openvswitch: add missing attribute validation for hash Add missing attribute