Date: Fri, 25 Mar 2011 11:13:39 -0700 (PDT) From: "Pedro F. Giffuni" <giffunip@tutopia.com> To: Kostik Belousov <kostikbel@gmail.com> Cc: freebsd-fs@freebsd.org Subject: Re: kern/152079: [msdosfs] [patch] Small cleanups from the other NetBSD/OpenBSD Message-ID: <758552.89055.qm@web113514.mail.gq1.yahoo.com> In-Reply-To: <20110325165314.GO78089@deviant.kiev.zoral.com.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
Hello;=0A--- On Fri, 3/25/11, Kostik Belousov <kostikbel@gmail.com> wrote:= =0A...=0A> >=A0 Fixes NetBSD PR #44661=0A> Can you extract the test referen= ced in the commit message=0A> ?=0AHere is the NetBSD link:=0A=0Ahttp://cvsw= eb.netbsd.org/bsdweb.cgi/src/tests/fs/vfs/t_vnops.c.diff?r1=3D1.21&r2=3D1.2= 2=0A=0Abut I have no idea if it will apply to our testing framework.=0A=0A>= =0A> I think that s/EROFS/EINVAL/ change could and should be=0A> committed= first, and then the (potential) fix for the=0A> vnode leakage as a separat= e commit.=0A>=0A=0AIt takes some time to get patches committed, so I usuall= y=0Aprefer to submit bigger patches, if I can, in order to save=0Areviewer'= s time.=0A=0AFWIW, I prefer so much bugzilla since permits better patch=0Ah= andling and obsoleting the diffs that have been applied=0Aalready. =0A=0A= =0A
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?758552.89055.qm>