From owner-svn-src-head@freebsd.org Sat Jul 8 17:34:55 2017 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 20F4AD9F245; Sat, 8 Jul 2017 17:34:55 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (br1.CN84in.dnsmgr.net [69.59.192.140]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id BFF953412; Sat, 8 Jul 2017 17:34:53 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (localhost [127.0.0.1]) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3) with ESMTP id v68HYpsl068501; Sat, 8 Jul 2017 10:34:51 -0700 (PDT) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: (from freebsd@localhost) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3/Submit) id v68HYpN9068500; Sat, 8 Jul 2017 10:34:51 -0700 (PDT) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <201707081734.v68HYpN9068500@pdx.rh.CN85.dnsmgr.net> Subject: Re: svn commit: r320803 - head/sbin/mount In-Reply-To: <20170708171641.GA1129@brick> To: Edward Tomasz Napierala Date: Sat, 8 Jul 2017 10:34:51 -0700 (PDT) CC: Ian Lepore , rgrimes@freebsd.org, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Reply-To: rgrimes@freebsd.org X-Mailer: ELM [version 2.4ME+ PL121h (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 08 Jul 2017 17:34:55 -0000 [ Charset ISO-8859-1 unsupported, converting... ] > On 0708T1102, Ian Lepore wrote: > > On Sat, 2017-07-08 at 09:50 -0700, Rodney W. Grimes wrote: > > > [ Charset UTF-8 unsupported, converting... ] > > > > > > > > Author: trasz > > > > Date: Sat Jul??8 11:06:27 2017 > > > > New Revision: 320803 > > > > URL: https://svnweb.freebsd.org/changeset/base/320803 > > > > > > > > Log: > > > > ? Fix "mount -uw /" when the filesystem type doesn't match. > > > > ?? > > > > ? This basically makes "mount -uw /" work when the filesystem > > > > ? mounted on / is NFS, but the one configured in fstab(5) is UFS, > > > > ? which can happen when you forget to modify fstab. > > > Please do not silence user errors because they are inconvinient, > > > this is a configuration error and the system should fail to? > > > mount the incorrectly configured root. > > > > > > If we start changing things to silently ignore user configuration > > > errors we are going down a very slippery road. > > It doesn't silence down the error. What it does is it makes it possible > to use "mount -uw /" - previously it would fail in a rather nonsensical > way, by calling "mount_nfs -o upgrade,rw /dev/ada0 /". It DOES silence the error. My configuration TOLD it to execute that rather nonsensical command, your change now causes it to execute something that my configuration did NOT tell it to do. > > IMO, this change fixes the right problem, but maybe does so the wrong > > way. ?Mount -u is by definition an update to an existing mount. ?There > > should be no need to consult /etc/fstab for an existing mount since the > > info is available from the kernel. > > > > Note that I say the foregoing with my user hat on. ?I haven't looked at > > the code to see if there's some reason why my common-sensical way of > > thinking about it is actually impossible to implement for some reason. > > I wouldn't expect it to consult fstab either, to be honest. But it does, > and I suspect changing that would break someone's config. It reads the fstab to get the options that may be specified there that your mount -uw / command does not specify, realize the kernel when mounting / does so in a very minimal way, when you invoke mount -u the parameters in /etc/fstab come in to play. I believe KIB refered to this during your differential when he said something like "what if the options disagree". -- Rod Grimes rgrimes@freebsd.org