From owner-freebsd-current@FreeBSD.ORG Fri Sep 29 16:24:23 2006 Return-Path: X-Original-To: current@FreeBSD.org Delivered-To: freebsd-current@FreeBSD.ORG Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id ACB2916A40F; Fri, 29 Sep 2006 16:24:23 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from cyrus.watson.org (cyrus.watson.org [209.31.154.42]) by mx1.FreeBSD.org (Postfix) with ESMTP id 60C3D43D73; Fri, 29 Sep 2006 16:24:23 +0000 (GMT) (envelope-from rwatson@FreeBSD.org) Received: from fledge.watson.org (fledge.watson.org [209.31.154.41]) by cyrus.watson.org (Postfix) with ESMTP id 744BB46CAE; Fri, 29 Sep 2006 12:24:22 -0400 (EDT) Date: Fri, 29 Sep 2006 17:24:22 +0100 (BST) From: Robert Watson X-X-Sender: robert@fledge.watson.org To: Pawel Jakub Dawidek In-Reply-To: <20060929155526.GA9194@garage.freebsd.pl> Message-ID: <20060929172321.R74256@fledge.watson.org> References: <20060929141709.E70454@fledge.watson.org> <20060929155526.GA9194@garage.freebsd.pl> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: ru@FreeBSD.org, current@FreeBSD.org Subject: Re: lockf in installworld -- not a good idea X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 29 Sep 2006 16:24:23 -0000 On Fri, 29 Sep 2006, Pawel Jakub Dawidek wrote: > On Fri, Sep 29, 2006 at 02:20:06PM +0100, Robert Watson wrote: >> I've noticed an increasing intolerance in our tools for system install and maintenance to locking not being implemented over the past few years. I no longer get working >> cron on boxes with neither rpc.lockd nor local locking enabled, for >> example. [...] > > If you are refering to my change in which cron(8) started to use pidfile(3), > then I'm sorry, but you're wrong. > > cron(8) from the very beginning was exiting when it had problems with > creating a pidfile, please check function acquire_daemonlock() in: Cron may have been a poor example, I picked it from the top of a list of a half dozen failed services, many of which used not to fail. Sorry about that. > The way I prefer is to ignore errors other than EEXIST - you can check > EXAMPLES section in the pidfile(3) manual page for more info. I just didn't > wanted to change cron(8)'s original behaviour. > > I do agree, that this shouldn't be treated as critical error and I can > change it if you like. I think if we get back EOPNOTSUPP from lock attempts in libpidfile, we should ignore it and hope for the best. In an ideal world, we wouldn't do this, but in an ideal world we also wouldn't need to. :-) Robert N M Watson Computer Laboratory University of Cambridge