daemon: Allow daemon child process to report success or failure to parent.
authorBen Pfaff <blp@nicira.com>
Thu, 17 Dec 2009 18:56:01 +0000 (10:56 -0800)
committerBen Pfaff <blp@nicira.com>
Fri, 18 Dec 2009 21:37:44 +0000 (13:37 -0800)
commit95440284bdf8ac9a94c3e119d011d76acab577a7
tree503a473672b4bfd0a6dee762480bcca9f10e1d75
parent058fd2a2741deff874db53f0d7cdbf743e1f2a0f
daemon: Allow daemon child process to report success or failure to parent.

There are conflicting pressures in startup of a daemon process:

    * The parent process should exit with an error code if the daemon
      cannot start up successfully.

    * Some startup actions must be performed in the child process, not in
      the parent.  The most obvious of these are file locking, since
      child processes do not inherit locks, and anything that requires
      knowing the child process's PID (e.g. unixctl sockets).

Until now, this conflict has usually been handled by giving up part of the
first property, i.e. in some cases the parent process would exit
successfully and the child immediately afterward exit with a failure code.

This commit introduces a better approach, by allowing daemons to perform
startup work in the child and only then signal the parent that they have
successfully started.  If the child instead exits without signaling
success, the parent passes this exit code along to its own parent.

This commit also modifies the daemons that can usefully take advantage of
this new feature to do so.
lib/daemon.c
lib/daemon.h
ovsdb/ovsdb-server.c
utilities/ovs-controller.c
utilities/ovs-openflowd.c
vswitchd/ovs-brcompatd.c
vswitchd/ovs-vswitchd.c