ovs-brcompatd: Don't bother to unlock the config file on fatal signal.
authorBen Pfaff <blp@nicira.com>
Tue, 2 Jun 2009 21:09:03 +0000 (14:09 -0700)
committerBen Pfaff <blp@nicira.com>
Tue, 2 Jun 2009 21:09:03 +0000 (14:09 -0700)
There's no need to do this any longer, because it will happen
automatically when the fd to it (if any) is closed by the kernel.

vswitchd/ovs-brcompatd.c

index bdd131529657a51daae16c7667779e668ad21bd4..67e7118a3982260c63c68f2a5f93660374ec0884 100644 (file)
@@ -68,7 +68,6 @@ enum bmc_action {
     BMC_DEL_PORT
 };
 
-static void release_lock(void *aux UNUSED);
 static void parse_options(int argc, char *argv[]);
 static void usage(void) NO_RETURN;
 
@@ -609,8 +608,6 @@ main(int argc, char *argv[])
     die_if_already_running();
     daemonize();
 
-    fatal_signal_add_hook(release_lock, NULL, true);
-
     retval = unixctl_server_create(NULL, &unixctl);
     if (retval) {
         ovs_fatal(retval, "could not listen for vlog connections");
@@ -767,11 +764,3 @@ usage(void)
     printf("\nThe default reload command is:\n%s\n", reload_command);
     exit(EXIT_SUCCESS);
 }
-
-/* Callback to cleanup when process is dying. */
-static void
-release_lock(void *aux UNUSED)
-{
-    cfg_unlock();
-}
-