From owner-p4-projects@FreeBSD.ORG Thu Sep 1 19:41:34 2005 Return-Path: X-Original-To: p4-projects@freebsd.org Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 6027F16A421; Thu, 1 Sep 2005 19:41:34 +0000 (GMT) X-Original-To: perforce@freebsd.org Delivered-To: perforce@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 383B316A41F for ; Thu, 1 Sep 2005 19:41:34 +0000 (GMT) (envelope-from soc-tyler@freebsd.org) Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id DF09B43D46 for ; Thu, 1 Sep 2005 19:41:33 +0000 (GMT) (envelope-from soc-tyler@freebsd.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.13.1/8.13.1) with ESMTP id j81JfXq7051878 for ; Thu, 1 Sep 2005 19:41:33 GMT (envelope-from soc-tyler@freebsd.org) Received: (from perforce@localhost) by repoman.freebsd.org (8.13.1/8.13.1/Submit) id j81JfXXA051875 for perforce@freebsd.org; Thu, 1 Sep 2005 19:41:33 GMT (envelope-from soc-tyler@freebsd.org) Date: Thu, 1 Sep 2005 19:41:33 GMT Message-Id: <200509011941.j81JfXXA051875@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: perforce set sender to soc-tyler@freebsd.org using -f From: soc-tyler To: Perforce Change Reviews Cc: Subject: PERFORCE change 82974 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 01 Sep 2005 19:41:35 -0000 http://perforce.freebsd.org/chv.cgi?CH=82974 Change 82974 by soc-tyler@soc-tyler_launchd on 2005/09/01 19:40:41 Remove debugging statements, and uncomment some other debuggery Affected files ... .. //depot/projects/soc2005/launchd/launchctl/launchctl.c#17 edit .. //depot/projects/soc2005/launchd/launchd.c#19 edit Differences ... ==== //depot/projects/soc2005/launchd/launchctl/launchctl.c#17 (text+ko) ==== @@ -149,7 +149,7 @@ { "umask", umask_cmd, "Change launchd's umask" }, { "help", help_cmd, "This help output" }, {"exit", exit_cmd, "Exit launchctl" }, - {"quit", exit_cmd, "Quit launchctl"}, + {"quit", exit_cmd, "Exit launchctl"}, }; static bool istty = false; ==== //depot/projects/soc2005/launchd/launchd.c#19 (text+ko) ==== @@ -343,22 +343,22 @@ static struct timespec timeout = { 30, 0 }; struct timespec *timeoutp = NULL; - /* if (getpid() == 1) { + if (getpid() == 1) { if (readcfg_pid == 0) init_pre_kevent(); } else { /* in theory, this will make sure we don't exit if we * have (a) any more jobs nd (b) open socket connections * to say, something like, launchctl? ;) - * + */ if (TAILQ_EMPTY(&jobs) && TAILQ_EMPTY(&connections)) { - /* liblaunch will restart launchd if we're needed again * + /* liblaunch will restart us if we're needed again */ timeoutp = &timeout; } else if (shutdown_in_progress && total_children == 0) { exit(EXIT_SUCCESS); } } - */ + switch (kevent(mainkq, NULL, 0, &kev, 1, timeoutp)) { case -1: syslog(LOG_DEBUG, "kevent(): %m"); @@ -1011,9 +1011,6 @@ if (NULL == resp) resp = launch_data_new_errno(ESRCH); } else if (!strcmp(cmd, LAUNCH_KEY_REMOVEJOB)) { - fprintf(stderr, "**debug** j == %p\n", j); - fprintf(stderr, "**debug** jobs.tqh_first == %p\n", jobs.tqh_first); - fprintf(stderr, "**debug** jobs.tqh_last == %p\n", jobs.tqh_last); TAILQ_FOREACH(j, &jobs, tqe) { if (!strcmp(j->label, launch_data_get_string(data))) { job_remove(j); @@ -2545,8 +2542,6 @@ static pthread_mutex_t forklock = PTHREAD_MUTEX_INITIALIZER; pid_t r; size_t i; - - fprintf(stderr, "**debug** starting launch_fork()\n"); pthread_mutex_lock(&forklock);