mir/ovs
2
0
Fork 0
mirror of https://github.com/openvswitch/ovs synced 2025-08-31 14:25:26 +00:00
Code Issues Releases Wiki Activity

worker: Prevent worker from being responsible for pidfile deletion.

Browse Source 
Currently we are creating the worker process after creation of the pidfile.
This means that the responsibility of deleting the pidfile after process
termination rests with the worker process.

When we restart openvswitch using the startup scripts, we SIGTERM the main
process and once it is cleaned up, we start ovs-vswitchd again. This results
in a race condition. The new ovs-vswitchd will create a pidfile because it is
unlocked. But, if the old worker process exits after the start of new
ovs-vswitchd, it will simply delete the pidfile underneath the new ovs-vswitchd.
This will eventually result in multiple ovs-vswitchd daemons.

This patch gives the responsibility of deleting the pidfile to the main
process.

Bug #16669.
Signed-off-by: Gurucharan Shetty <gshetty@nicira.com>
This commit is contained in:
Gurucharan Shetty
2013-04-28 19:25:55 -07:00
parent 9f27568d9f
commit 7ffd3f6972
3 changed files with 31 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
lib/daemon.c
View File

@@ -163,6 +163,26 @@ daemon_save_fd(int fd)
save_fds[fd] = true;
}
/* Unregisters pidfile from being unlinked when the program terminates via
* exit() or a fatal signal. */
void
remove_pidfile_from_unlink(void)
{
if (pidfile) {
fatal_signal_remove_file_to_unlink(pidfile);
}
}
/* Registers pidfile to be unlinked when the program terminates via exit() or a
* fatal signal. */
void
add_pidfile_to_unlink(void)
{
if (pidfile) {
fatal_signal_add_file_to_unlink(pidfile);
}
}
/* If a pidfile has been configured, creates it and stores the running
* process's pid in it. Ensures that the pidfile will be deleted when the
* process exits. */
@@ -240,8 +260,6 @@ make_pidfile(void)
pidfile_dev = s.st_dev;
pidfile_ino = s.st_ino;
free(tmpfile);
free(pidfile);
pidfile = NULL;
}
/* If configured with set_pidfile() or set_detach(), creates the pid file and
@@ -529,6 +547,11 @@ daemonize_start(void)
void
daemonize_complete(void)
{
if (pidfile) {
free(pidfile);
pidfile = NULL;
}
if (!detached) {
detached = true;