We are registering an interest in SIGHUP to reopen log files. But there is an 'ovs-appctl vlog/reopen' command that does the same and is used in the logrotate config for the distributions.
So remove the redundant functionality. Signed-off-by: Gurucharan Shetty <gshe...@nicira.com> --- vswitchd/ovs-vswitchd.8.in | 3 --- vswitchd/ovs-vswitchd.c | 7 ------- 2 files changed, 10 deletions(-) diff --git a/vswitchd/ovs-vswitchd.8.in b/vswitchd/ovs-vswitchd.8.in index d2544f7..e9dc483 100644 --- a/vswitchd/ovs-vswitchd.8.in +++ b/vswitchd/ovs-vswitchd.8.in @@ -30,9 +30,6 @@ switching across each bridge described in its configuration files. As the database changes, \fBovs\-vswitchd\fR automatically updates its configuration to match. .PP -Upon receipt of a SIGHUP signal, \fBovs\-vswitchd\fR reopens its log -file, if one was specified on the command line. -.PP \fBovs\-vswitchd\fR switches may be configured with any of the following features: . diff --git a/vswitchd/ovs-vswitchd.c b/vswitchd/ovs-vswitchd.c index 9da2f49..c4e61d1 100644 --- a/vswitchd/ovs-vswitchd.c +++ b/vswitchd/ovs-vswitchd.c @@ -38,7 +38,6 @@ #include "ovsdb-idl.h" #include "poll-loop.h" #include "process.h" -#include "signals.h" #include "simap.h" #include "stream-ssl.h" #include "stream.h" @@ -66,7 +65,6 @@ main(int argc, char *argv[]) { char *unixctl_path = NULL; struct unixctl_server *unixctl; - struct signal *sighup; char *remote; bool exiting; int retval; @@ -76,7 +74,6 @@ main(int argc, char *argv[]) service_start(&argc, &argv); remote = parse_options(argc, argv, &unixctl_path); signal(SIGPIPE, SIG_IGN); - sighup = signal_register(SIGHUP); process_init(); ovsrec_init(); @@ -103,9 +100,6 @@ main(int argc, char *argv[]) exiting = false; while (!exiting) { - if (signal_poll(sighup)) { - vlog_reopen_log_file(); - } memory_run(); if (memory_should_report()) { struct simap usage; @@ -119,7 +113,6 @@ main(int argc, char *argv[]) unixctl_server_run(unixctl); netdev_run(); - signal_wait(sighup); memory_wait(); bridge_wait(); unixctl_server_wait(unixctl); -- 1.7.9.5 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev