Remove unused function nat44_ha_resync() and error IN_PROGRESS. 77/19577/2
authorJon Loeliger <jdl@netgate.com>
Tue, 14 May 2019 20:02:39 +0000 (15:02 -0500)
committerOle Trøan <otroan@employees.org>
Wed, 15 May 2019 06:47:12 +0000 (06:47 +0000)
The unused function nat44_ha_resync() was the only function that
used the error message VNET_API_ERROR_IN_PROGRESS.  The error
was the only error code that was positive, and didn't really
play well with the other error codes.

Change-Id: I7d03c2ee915094b635f6efdca7427f71e4d19f2b
Signed-off-by: Jon Loeliger <jdl@netgate.com>
src/plugins/nat/nat_ha.c
src/vnet/api_errno.h

index a7eaf6e..e81e916 100644 (file)
@@ -821,44 +821,6 @@ nat_ha_get_resync_status (u8 * in_resync, u32 * resync_ack_missed)
   *resync_ack_missed = ha->resync_ack_missed;
 }
 
-int
-nat44_ha_resync (u32 client_index, u32 pid,
-                nat_ha_resync_event_cb_t event_callback)
-{
-  nat_ha_main_t *ha = &nat_ha_main;
-  snat_main_t *sm = &snat_main;
-  snat_session_t *ses;
-  snat_main_per_thread_data_t *tsm;
-
-  if (ha->in_resync)
-    return VNET_API_ERROR_IN_PROGRESS;
-
-  ha->in_resync = 1;
-  ha->resync_ack_count = 0;
-  ha->resync_ack_missed = 0;
-  ha->event_callback = event_callback;
-  ha->client_index = client_index;
-  ha->pid = pid;
-
-  /* *INDENT-OFF* */
-  vec_foreach (tsm, sm->per_thread_data)
-    {
-      pool_foreach (ses, tsm->sessions, ({
-        nat_ha_sadd (&ses->in2out.addr, ses->in2out.port,
-                     &ses->out2in.addr, ses->out2in.port,
-                     &ses->ext_host_addr, ses->ext_host_port,
-                     &ses->ext_host_nat_addr, ses->ext_host_nat_port,
-                     ses->in2out.protocol, ses->in2out.fib_index,
-                     ses->flags, 0, 1);
-      }));
-    }
-  /* *INDENT-ON* */
-
-  nat_ha_flush (1);
-
-  return 0;
-}
-
 typedef struct
 {
   ip4_address_t addr;
index c4cb3c9..e59f3cb 100644 (file)
@@ -55,7 +55,6 @@ _(ADDRESS_LENGTH_MISMATCH, -59, "Address length mismatch")              \
 _(ADDRESS_NOT_FOUND_FOR_INTERFACE, -60, "Address not found for interface") \
 _(ADDRESS_NOT_DELETABLE, -61, "Address not deletable")                  \
 _(IP6_NOT_ENABLED, -62, "ip6 not enabled")                             \
-_(IN_PROGRESS, 10, "Operation in progress")                            \
 _(NO_SUCH_NODE, -63, "No such graph node")                             \
 _(NO_SUCH_NODE2, -64, "No such graph node #2")                         \
 _(NO_SUCH_TABLE, -65, "No such table")                                  \