On Thu, Jul 11, 2013 at 10:00:21AM -0400, Ed Maste wrote:
> On 10 July 2013 19:03, Ben Pfaff <b...@nicira.com> wrote:
> > The worker process implementation isn't thread-safe and, once OVS
> > itself is threaded, it doesn't make much sense to have a worker
> > process anyway.
> >
> > Signed-off-by: Ben Pfaff <b...@nicira.com>
> 
> Acked-by: Ed Maste <ema...@freebsd.org>

Applied to master, thanks.

I had to move this hunk to the later "worker: Delete library." patch,
because in isolation it just caused assertion failures.  It made more
logical sense in the latter place anyhow:

commit 14017b032237bc166b3d4af256d2c06b6f79817a
Author: Ben Pfaff <b...@nicira.com>
Date:   Thu Jul 11 17:10:16 2013 -0700

    daemon

diff --git a/lib/daemon.c b/lib/daemon.c
index 3c1e5c3..edeeca2 100644
--- a/lib/daemon.c
+++ b/lib/daemon.c
@@ -529,6 +529,8 @@ daemonize_start(void)
         /* Running in daemon process. */
     }
 
+    forbid_forking("running in daemon process");
+
     if (pidfile) {
         make_pidfile();
     }
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to