ldp: avoid calling write in destructors 40/16340/2
authorFlorin Coras <fcoras@cisco.com>
Tue, 4 Dec 2018 21:43:45 +0000 (13:43 -0800)
committerDave Barach <openvpp@barachs.net>
Wed, 5 Dec 2018 15:37:13 +0000 (15:37 +0000)
Change-Id: Ia9c3d7a68a23dc4ab3be06f88fdfb053db422372
Signed-off-by: Florin Coras <fcoras@cisco.com>
src/vcl/ldp.c
src/vcl/vppcom.c

index 453ddeb..c3dc64a 100644 (file)
@@ -258,9 +258,12 @@ ldp_init (void)
   rv = vppcom_app_create (ldp_get_app_name ());
   if (rv != VPPCOM_OK)
     {
-      fprintf (stderr, "\nLDP<%d>: ERROR: ldp_init: vppcom_app_create()"
-              " failed!  rv = %d (%s)\n",
-              getpid (), rv, vppcom_retval_str (rv));
+      ldp->vcl_needs_real_epoll = 0;
+      if (rv == VPPCOM_EEXIST)
+       return 0;
+      LDBG (2, "\nLDP<%d>: ERROR: ldp_init: vppcom_app_create()"
+           " failed!  rv = %d (%s)\n",
+           getpid (), rv, vppcom_retval_str (rv));
       ldp->init = 0;
       return rv;
     }
index 07136e9..dbb0d65 100644 (file)
@@ -862,8 +862,8 @@ vppcom_app_create (char *app_name)
 
   if (vcm->is_init)
     {
-      clib_warning ("vcl<%u> already initialized", getpid ());
-      return -1;
+      VDBG (1, "already initialized");
+      return VPPCOM_EEXIST;
     }
 
   vcm->is_init = 1;