From: Ben Pfaff <blp@nicira.com>
Date: Tue, 2 Jun 2009 21:09:03 +0000 (-0700)
Subject: ovs-brcompatd: Don't bother to unlock the config file on fatal signal.
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2efb2e477e7a37d8c3b3fb1b7ac9c71967ab13eb;p=openvswitch

ovs-brcompatd: Don't bother to unlock the config file on fatal signal.

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.
---

diff --git a/vswitchd/ovs-brcompatd.c b/vswitchd/ovs-brcompatd.c
index bdd13152..67e7118a 100644
--- a/vswitchd/ovs-brcompatd.c
+++ b/vswitchd/ovs-brcompatd.c
@@ -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();
-}
-