From owner-freebsd-fs@FreeBSD.ORG Fri Mar 25 20:57:00 2011 Return-Path: Delivered-To: freebsd-fs@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 24326106566C for ; Fri, 25 Mar 2011 20:57:00 +0000 (UTC) (envelope-from kostikbel@gmail.com) Received: from mail.zoral.com.ua (mx0.zoral.com.ua [91.193.166.200]) by mx1.freebsd.org (Postfix) with ESMTP id 89F3A8FC15 for ; Fri, 25 Mar 2011 20:56:59 +0000 (UTC) Received: from deviant.kiev.zoral.com.ua (root@deviant.kiev.zoral.com.ua [10.1.1.148]) by mail.zoral.com.ua (8.14.2/8.14.2) with ESMTP id p2PKutdj087207 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Fri, 25 Mar 2011 22:56:55 +0200 (EET) (envelope-from kostikbel@gmail.com) Received: from deviant.kiev.zoral.com.ua (kostik@localhost [127.0.0.1]) by deviant.kiev.zoral.com.ua (8.14.4/8.14.4) with ESMTP id p2PKusZC044052; Fri, 25 Mar 2011 22:56:54 +0200 (EET) (envelope-from kostikbel@gmail.com) Received: (from kostik@localhost) by deviant.kiev.zoral.com.ua (8.14.4/8.14.4/Submit) id p2PKus3C044051; Fri, 25 Mar 2011 22:56:54 +0200 (EET) (envelope-from kostikbel@gmail.com) X-Authentication-Warning: deviant.kiev.zoral.com.ua: kostik set sender to kostikbel@gmail.com using -f Date: Fri, 25 Mar 2011 22:56:54 +0200 From: Kostik Belousov To: "Pedro F. Giffuni" Message-ID: <20110325205654.GP78089@deviant.kiev.zoral.com.ua> References: <20110325165314.GO78089@deviant.kiev.zoral.com.ua> <758552.89055.qm@web113514.mail.gq1.yahoo.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="fciAcgbhTOkUP7Ni" Content-Disposition: inline In-Reply-To: <758552.89055.qm@web113514.mail.gq1.yahoo.com> User-Agent: Mutt/1.4.2.3i X-Virus-Scanned: clamav-milter 0.95.2 at skuns.kiev.zoral.com.ua X-Virus-Status: Clean X-Spam-Status: No, score=-3.4 required=5.0 tests=ALL_TRUSTED,AWL,BAYES_00, DNS_FROM_OPENWHOIS autolearn=no version=3.2.5 X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on skuns.kiev.zoral.com.ua Cc: freebsd-fs@freebsd.org Subject: Re: kern/152079: [msdosfs] [patch] Small cleanups from the other NetBSD/OpenBSD X-BeenThere: freebsd-fs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Filesystems List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 25 Mar 2011 20:57:00 -0000 --fciAcgbhTOkUP7Ni Content-Type: text/plain; charset=koi8-r Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Mar 25, 2011 at 11:13:39AM -0700, Pedro F. Giffuni wrote: > Hello; > --- On Fri, 3/25/11, Kostik Belousov wrote: > ... > > >=9A Fixes NetBSD PR #44661 > > Can you extract the test referenced in the commit message > > ? > Here is the NetBSD link: >=20 > http://cvsweb.netbsd.org/bsdweb.cgi/src/tests/fs/vfs/t_vnops.c.diff?r1=3D= 1.21&r2=3D1.22 >=20 > but I have no idea if it will apply to our testing framework. No, I do not want it in our testing framework. I want to see a standalone test that demonstrates the issue. I think the leak is real, but want to have a way to reproduce it before committing. The diff you pointed out to t_vnops.c does not make much sense to me. >=20 > >=20 > > I think that s/EROFS/EINVAL/ change could and should be > > committed first, and then the (potential) fix for the > > vnode leakage as a separate commit. > > >=20 > It takes some time to get patches committed, so I usually > prefer to submit bigger patches, if I can, in order to save > reviewer's time. EINVAL it trivial, and also it seems to be NOP, because VFS blocks attempts to delete or rename the mount point root directory. For delete, it is explicit check, for rename, the cause is the fact that lookup returns the covered vnode, and you either get a loop or cross-device link error. >=20 > FWIW, I prefer so much bugzilla since permits better patch > handling and obsoleting the diffs that have been applied > already.=20 >=20 >=20 > =20 --fciAcgbhTOkUP7Ni Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (FreeBSD) iEYEARECAAYFAk2NAZYACgkQC3+MBN1Mb4gecACdFAc9BUkPxpKKfovTecNgDh58 4KsAn1uDjuf/FpJYW2e7zkAD2gTL+s66 =pvy/ -----END PGP SIGNATURE----- --fciAcgbhTOkUP7Ni--