Re: [ovs-dev] [PATCH v2] ofproto-dpif-xlate: do not unwildcard source address if not needed

2020-04-06 Thread William Tu
Hi Hepeng, Thanks for your reply. Can you re-send a separate v3 patch email? I can't apply your patch to master. Also some comment below On Thu, Mar 26, 2020 at 7:21 PM 贺鹏 wrote: > > Hi, I rebase the patch on the latest master branch. > > > From 07f10707699a39e3a6de68012dd6c0453120c543 Mon Sep

Re: [ovs-dev] [PATCH v2] ofproto-dpif-xlate: do not unwildcard source address if not needed

2020-03-26 Thread 贺鹏
Hi, I rebase the patch on the latest master branch. From 07f10707699a39e3a6de68012dd6c0453120c543 Mon Sep 17 00:00:00 2001 From: hepeng Date: Wed, 4 Dec 2019 17:38:22 +0800 Subject: [PATCH] ofproto-dpif-xlate: do not unwildcard source address if not needed if the tunnel is specified as "remote

Re: [ovs-dev] [PATCH v2] ofproto-dpif-xlate: do not unwildcard source address if not needed

2020-03-26 Thread William Tu
On Sun, Mar 22, 2020 at 10:42:16AM +0800, 贺鹏 wrote: > if the tunnel is specified as "remote_ip=flow", we can try to generate > a megaflow > which ignores all source address, such as source mac, source IP address > in the outter header of the packet. This can reduce the number of > megaflows and avo

Re: [ovs-dev] [PATCH v2] ofproto-dpif-xlate: do not unwildcard source address if not needed

2020-03-21 Thread 0-day Robot
Bleep bloop. Greetings 贺鹏, 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. git-am: error: patch fragment without header at line 16: @@ -623,3 +623,47 @@ NXST_FLOW reply: error: could not build

[ovs-dev] [PATCH v2] ofproto-dpif-xlate: do not unwildcard source address if not needed

2020-03-21 Thread 贺鹏
if the tunnel is specified as "remote_ip=flow", we can try to generate a megaflow which ignores all source address, such as source mac, source IP address in the outter header of the packet. This can reduce the number of megaflows and avoid expensive upcall and improve the performance. Signed-off-b