commit e531f4cb5766fbf27e7a8af8e19ccf667b53852b
author John Lo <loj@cisco.com> Tue Aug 22 09:16:50 2017 -0400
committer Dave Wallace <dwallacelf@gmail.com> Tue Aug 22 14:30:18 2017 +0000
tree 9d565ad0ca0d8696922c6bed07b76ce023497328
parent 87318463aa3abfe22984f78c1b31db8a355fae76

Increase default MAC learn limit and check it in learn-update path

1. Increase default MAC learn limit from 1M to 8M entries.
2. Check MAC learn limit in MAC learning update path.
3. Allow disable of want_l2_macs_events to set MAC learn limit
4. Other minor cleanups

Change-Id: I62438440937b5fa455e16f4a2e4d910277753395
Signed-off-by: John Lo <loj@cisco.com>

src/vnet/l2/l2.api

diff --git a/src/vnet/l2/l2.api b/src/vnet/l2/l2.api
index e508bfb..9f97ebb 100644
--- a/src/vnet/l2/l2.api
+++ b/src/vnet/l2/l2.api
@@ -134,7 +134,6 @@
 };
 
 /** \brief Register to recive L2 MAC events for leanred and aged MAC
-    Will also change MAC learn limit to L2LEARN_INFORM_LIMIT
     @param client_index - opaque cookie to identify the sender
     @param context - sender context, to match reply w/ request
     @param learn_limit - MAC learn limit, 0 => default to 1000

src/vnet/l2/l2_api.c

diff --git a/src/vnet/l2/l2_api.c b/src/vnet/l2/l2_api.c
index c81cbad..7e79d6f 100644
--- a/src/vnet/l2/l2_api.c
+++ b/src/vnet/l2/l2_api.c
@@ -284,7 +284,10 @@
     {
       lm->client_pid = 0;
       lm->client_index = 0;
-      lm->global_learn_limit = L2LEARN_DEFAULT_LIMIT;
+      if (learn_limit)
+	lm->global_learn_limit = learn_limit;
+      else
+	lm->global_learn_limit = L2LEARN_DEFAULT_LIMIT;
     }
 
 exit:

src/vnet/l2/l2_fib.c

diff --git a/src/vnet/l2/l2_fib.c b/src/vnet/l2/l2_fib.c
index 8aa0ac2..9f4c823 100644
--- a/src/vnet/l2/l2_fib.c
+++ b/src/vnet/l2/l2_fib.c
@@ -1021,7 +1021,7 @@
 
 	      if (PREDICT_FALSE (evt_idx >= fm->max_macs_in_event))
 		{
-		  /* evet message full, sent it and start a new one */
+		  /* event message full, send it and start a new one */
 		  if (q && (q->cursize < q->maxsize))
 		    {
 		      mp->n_macs = htonl (evt_idx);

src/vnet/l2/l2_learn.c

diff --git a/src/vnet/l2/l2_learn.c b/src/vnet/l2/l2_learn.c
index 47c036b..623c2de 100644
--- a/src/vnet/l2/l2_learn.c
+++ b/src/vnet/l2/l2_learn.c
@@ -131,6 +131,8 @@
 
       if (PREDICT_TRUE (!update))
 	return;
+      else if (msm->global_learn_count > msm->global_learn_limit)
+	return;			/* Above learn limit - do not update */
     }
   else if (result0->raw == ~0)
     {

src/vnet/l2/l2_learn.h

diff --git a/src/vnet/l2/l2_learn.h b/src/vnet/l2/l2_learn.h
index 000ab59..d6f41d4 100644
--- a/src/vnet/l2/l2_learn.h
+++ b/src/vnet/l2/l2_learn.h
@@ -46,7 +46,7 @@
   vnet_main_t *vnet_main;
 } l2learn_main_t;
 
-#define L2LEARN_DEFAULT_LIMIT (L2FIB_NUM_BUCKETS * 16)
+#define L2LEARN_DEFAULT_LIMIT (L2FIB_NUM_BUCKETS * 128)
 
 l2learn_main_t l2learn_main;