diff mbox series

[ovs-dev,v3] dpif-netdev: Set MAX_RECIRC_DEPTH to 6.

Message ID 20170922144943.8316-1-ligs@dtdream.com
State Accepted
Headers show
Series [ovs-dev,v3] dpif-netdev: Set MAX_RECIRC_DEPTH to 6. | expand

Commit Message

Guoshuai Li Sept. 22, 2017, 2:49 p.m. UTC
In an ovn gateway node with DPDK, the RECIRC_DEPTH may be greater than 5.

Scenes:
VM ping self floating IP, or
VM ping Floating IP of VMs with the same network.

It need process UNDNAT SNAT in LRouter egress and
UNSNAT DNAT in LRouter ingress, and
output to geneve tunnel also need recirc.

This has an WARN:
dpif_netdev(pmd36)|WARN|Packet dropped. Max recirculation depth exceeded.

Signed-off-by: Guoshuai Li <ligs@dtdream.com>
---
 lib/dpif-netdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Ben Pfaff Nov. 3, 2017, 9:29 p.m. UTC | #1
On Fri, Sep 22, 2017 at 10:49:43PM +0800, Guoshuai Li wrote:
> In an ovn gateway node with DPDK, the RECIRC_DEPTH may be greater than 5.
> 
> Scenes:
> VM ping self floating IP, or
> VM ping Floating IP of VMs with the same network.
> 
> It need process UNDNAT SNAT in LRouter egress and
> UNSNAT DNAT in LRouter ingress, and
> output to geneve tunnel also need recirc.
> 
> This has an WARN:
> dpif_netdev(pmd36)|WARN|Packet dropped. Max recirculation depth exceeded.
> 
> Signed-off-by: Guoshuai Li <ligs@dtdream.com>

Applied, thanks!
diff mbox series

Patch

diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c
index ca74df8c7..a0f620a5c 100644
--- a/lib/dpif-netdev.c
+++ b/lib/dpif-netdev.c
@@ -82,7 +82,7 @@  VLOG_DEFINE_THIS_MODULE(dpif_netdev);
 
 #define FLOW_DUMP_MAX_BATCH 50
 /* Use per thread recirc_depth to prevent recirculation loop. */
-#define MAX_RECIRC_DEPTH 5
+#define MAX_RECIRC_DEPTH 6
 DEFINE_STATIC_PER_THREAD_DATA(uint32_t, recirc_depth, 0)
 
 /* Configuration parameters. */