From owner-freebsd-jail@FreeBSD.ORG Fri Jul 8 06:06:58 2011 Return-Path: Delivered-To: freebsd-jail@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6815A106564A; Fri, 8 Jul 2011 06:06:58 +0000 (UTC) (envelope-from jamesbrandongooch@gmail.com) Received: from mail-ww0-f50.google.com (mail-ww0-f50.google.com [74.125.82.50]) by mx1.freebsd.org (Postfix) with ESMTP id D17EB8FC0C; Fri, 8 Jul 2011 06:06:57 +0000 (UTC) Received: by wwe6 with SMTP id 6so1555622wwe.31 for ; Thu, 07 Jul 2011 23:06:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=v4Z21c17gFbyz9GDkPg/XMuGtYIoKzkUtUq2V3uc6FI=; b=rV/hKzW0ayqIvBq76Jgp2HiVrzZEBOFNFHVocETjzTr869Tsn/G07qXB0mtBmvlZf3 A0+0jP2Jn16Kb9uT/H9Pit7ZSDLcvf40sdzhD8HnxcGz/26injYx0bZqe1M/bOsnfTA1 3adBjZNzs3zkLiwsTUj9Shv1Sf1aEnmKryTkw= MIME-Version: 1.0 Received: by 10.216.78.206 with SMTP id g56mr1429192wee.37.1310105216504; Thu, 07 Jul 2011 23:06:56 -0700 (PDT) Received: by 10.217.5.197 with HTTP; Thu, 7 Jul 2011 23:06:56 -0700 (PDT) In-Reply-To: <4E15475C.9020500@FreeBSD.org> References: <4E114EA9.4000605@FreeBSD.org> <4E15475C.9020500@FreeBSD.org> Date: Fri, 8 Jul 2011 01:06:56 -0500 Message-ID: From: Brandon Gooch To: Jamie Gritton Content-Type: text/plain; charset=ISO-8859-1 Cc: freebsd-jail@freebsd.org Subject: Re: New jail(8) with configuration files, not yet in head X-BeenThere: freebsd-jail@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion about FreeBSD jail\(8\)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 08 Jul 2011 06:06:58 -0000 On Thu, Jul 7, 2011 at 12:42 AM, Jamie Gritton wrote: > This is code that notes error return codes from processes run under the > jail (e.g. the exec.start script). But in this case, it's reporting an > error from a process that was part of the jail's shutdown. The reason > the command is "(null)" is this wasn't a command started from jail(8) > itself. > > I've fixed that dual-use bit of code to stay silent about existing > processes that die as part of jail shutdown. A small patch for that is > attached. > > - Jamie The patch silenced the errors and overall had a positive impact on jail shutdown time as well. It would be nice to hear more feedback from anyone involved in developing ezjail or qjail -- these ports will need reworking I assume. I'll try it myself and see, of course... -Brandon