From owner-freebsd-rc@FreeBSD.ORG Tue Nov 27 11:16:20 2012 Return-Path: Delivered-To: freebsd-rc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id ACEC5BD; Tue, 27 Nov 2012 11:16:20 +0000 (UTC) (envelope-from mjguzik@gmail.com) Received: from mail-wi0-f178.google.com (mail-wi0-f178.google.com [209.85.212.178]) by mx1.freebsd.org (Postfix) with ESMTP id D56008FC08; Tue, 27 Nov 2012 11:16:19 +0000 (UTC) Received: by mail-wi0-f178.google.com with SMTP id hm6so3583729wib.13 for ; Tue, 27 Nov 2012 03:16:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; bh=YbpamqEXrbvYGt6lH3SV8cFkssZM0tAomrb26bIlh7A=; b=Ay1B3GNBP5vflA/T174mG7lZ3Q9LtpvaFpe+J9fSABjGy5IZzYc4k9lAJwijhLmjjM in5/wVbZG2FGsNM/1KFMlA0E03c1bh5Aim7oO1Eubexj28ihC/OlpFTTB4q2C4wqt0bQ cnTX29nAJYbPnmLiUYUUiwwXG/qb3Nn0ujPYMBDOFyOVKWGQKSYZH+0LawcEJKR9Qlxf 3HIZmSoVMSnbGEmuC30PE0NFVJAoUKO392tkpwZUdLxE0fqfUzHihuMfUlAfyBty55vF tadBr5nfm2VKcLV6JDzgHEO5mEMc83pL6jLRDrqeFoIOnKRR0JDNCs7ItFKrRz+TLSNg wRfw== Received: by 10.180.83.132 with SMTP id q4mr13495750wiy.9.1354014973104; Tue, 27 Nov 2012 03:16:13 -0800 (PST) Received: from dft-labs.eu (n1x0n-1-pt.tunnel.tserv5.lon1.ipv6.he.net. [2001:470:1f08:1f7::2]) by mx.google.com with ESMTPS id eu8sm2075640wib.1.2012.11.27.03.16.11 (version=TLSv1/SSLv3 cipher=OTHER); Tue, 27 Nov 2012 03:16:12 -0800 (PST) Date: Tue, 27 Nov 2012 12:16:06 +0100 From: Mateusz Guzik To: Chris Rees Subject: Re: mountlate being too mount-happy Message-ID: <20121127111606.GB15606@dft-labs.eu> References: <20121118.074325.564844639489846824.hrs@allbsd.org> <20121118002245.GB15055@dft-labs.eu> <20121118.150935.240651183336258002.hrs@allbsd.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.20 (2009-06-14) Cc: "freebsd-rc@freebsd.org" , Mateusz Guzik X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 27 Nov 2012 11:16:20 -0000 On Sun, Nov 18, 2012 at 02:28:28PM +0000, Chris Rees wrote: > On 18 November 2012 06:09, Hiroki Sato wrote: > > Mateusz Guzik wrote > > in <20121118002245.GB15055@dft-labs.eu>: > > > > mj> On Sun, Nov 18, 2012 at 07:43:25AM +0900, Hiroki Sato wrote: > > mj> > Chris Rees wrote > > mj> > in : > > mj> > > > mj> > ut> On 2 November 2012 14:21, Eitan Adler wrote: > > mj> > ut> > On 2 November 2012 09:56, Chris Rees wrote: > > mj> > ut> >> I'll take a look. > > mj> > ut> > > > mj> > ut> > untested: > > mj> > ut> > > mj> > ut> Based on Eitan's patch, I've tested this one, and documented it in mount(8) too: > > mj> > ut> > > mj> > ut> http://www.bayofrum.net/~crees/patches/mountonlylate.diff > > mj> > ut> > > mj> > ut> Does anyone have any suggestions/objections/urge to approve it? > > mj> > > > mj> > Is the original problem due to backgrounding of NFS mount only? If > > mj> > so, implementing prevention of duplicate invocation into mount(8) > > mj> > would be more reasonable, I think. > > mj> > > > mj> > > mj> We have 2 distinct scripts that try to mount same set of filesystems. > > mj> I think this is the real bug here and proposed patches makes it go away in > > mj> an IMHO acceptable way. > > > > I just wanted to make sure if the case is limited to background NFS > > mount or not. > > > > rc.d/mountlate just tries to mount the filesystems that are not > > mounted yet at that time in addition to the "late" ones, not always > > to mount the same set twice. If it is a bug, it is better to simply > > fix -l to exclude not-yet-mounted ones without "late" keyword than > > adding another option. > > I don't think it's a bug as such-- -l option is clearly labelled in > the manpage (emphasis mine): > > When used in conjunction with the -a option, *also* mount those > file systems which are marked as ``late''. > > I think that for POLA and to avoid changing behaviour of an option > that's been there a long time we need the -L option. > > I disagree with Mateusz here-- split operations in rc makes two > scripts necessary; mount and mountlate are two separate operations, > done at different times. > So it turns out I expressed myself incorrectly. I'm fine with two separate scripts doing mounts. I'm not fine with the fact that both scripts may try to mount very same filesystem (which is the case now with background nfs). crees's patch fixes this problem in a way that I think is ok. -- Mateusz Guzik