Date: Mon, 18 Feb 2008 17:40:05 +0000 From: Craig Rodrigues <rodrigc@crodrigues.org> To: Yar Tikhiy <yar@comp.chem.msu.su> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sbin/mount_nfs mount_nfs.c Message-ID: <20080218174005.GA83923@crodrigues.org> In-Reply-To: <20080218142013.GH30694@comp.chem.msu.su> References: <200802181024.m1IAOmoF089632@repoman.freebsd.org> <20080218104846.GA81672@crodrigues.org> <20080218142013.GH30694@comp.chem.msu.su>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Feb 18, 2008 at 05:20:14PM +0300, Yar Tikhiy wrote: > This horrible hack is necessary because Someone(tm) started converting > our NFS client to the new nmount(2) semantics a year ago but he has > never finished the job. Now the NFS client just uses old mount(2) Yes, I acknowledge I started this work and did not finish. I have some pending patches to fully convert mount_nfs to pass nmount() string args to the kernel, but I am in the middle of moving, so do not have time to do a lot of FreeBSD work until the end of next week, when I move into my new house. Additionally, touching the NFS mount code is quite involved, so once I am ready to get back to FreeBSD work, I want to circulate the patches a bit more for feedback and testing before committing. >> change and 1.274 to vfs_mount.c out, until we can analyze >> the full implications of your change, and properly fix the >> necessary infrastructure, instead of committing hack after hack? > > Which else _hacks_ did I commit? The mount_nfs hack is the main one that I saw. Can you revert that plus the vfs_mount.c change until we have time to do this right? Some other people already mentioned possible implications of your change to NFS + bootp, which you didn't test. Thanks. -- Craig Rodrigues rodrigc@crodrigues.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080218174005.GA83923>