[ovs-dev] [PATCH] windows: Remove vlog_init() from service_start function

Anand Kumar kumaranand at vmware.com
Fri Jun 16 17:06:26 UTC 2017


When --syslog-method is passed as a cmd line argument and
ovs daemons are started as a service, --syslog-method argument is not
honoured, as vlog module is initialized by service_start function
before parsing the command line arguments.

Easy way to fix this, is to remove vlog_init() from the service start
function and let the first log message to initialize the vlog module.

Signed-off-by: Anand Kumar <kumaranand at vmware.com>
---
 lib/daemon-windows.c | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/lib/daemon-windows.c b/lib/daemon-windows.c
index 5804284..1ba714d 100644
--- a/lib/daemon-windows.c
+++ b/lib/daemon-windows.c
@@ -138,13 +138,6 @@ service_start(int *argcp, char **argvp[])
         *argcp = sargc;
         *argvp = *sargvp;
 
-        /* XXX: Windows implementation cannot have a unixctl commands in the
-        * traditional sense of unix domain sockets. If an implementation is
-        * done that involves 'unixctl' vlog commands the following call is
-        * needed to make sure that the unixctl commands for vlog get
-        * registered in a daemon, even before the first log message. */
-        vlog_init();
-
         return;
     }
 
-- 
2.9.3.windows.1



More information about the dev mailing list