Re: CVS commit: src/sys/arch/i386/conf

2011-08-09 Thread Alan Barrett
On Mon, 08 Aug 2011, Jared D. McNeill wrote: Modified Files: src/sys/arch/i386/conf: GENERIC Log Message: remove dtv (available as a module) If you remove anything from GENERIC on the grounds that it is available as a module, then please add the same item to MONOLITHIC, so that

Re: CVS commit: src/sys/arch/i386/conf

2011-08-09 Thread Jukka Ruohonen
On Tue, Aug 09, 2011 at 09:44:09AM +0200, Alan Barrett wrote: Whether or not available as a module is a good reason for removing something from GENERIC is a separate topic which I will not consider in this message. Fortunately, people can vote with their own work; henceforth all drivers I

Re: CVS commit: src/sys/arch/i386/conf

2011-08-09 Thread Manuel Bouyer
On Tue, Aug 09, 2011 at 10:47:18AM +0300, Jukka Ruohonen wrote: On Tue, Aug 09, 2011 at 09:44:09AM +0200, Alan Barrett wrote: Whether or not available as a module is a good reason for removing something from GENERIC is a separate topic which I will not consider in this message.

Re: CVS commit: src/sys/arch/i386/conf

2011-08-09 Thread Manuel Bouyer
On Tue, Aug 09, 2011 at 11:16:11AM +0300, Jukka Ruohonen wrote: On Tue, Aug 09, 2011 at 10:14:08AM +0200, Manuel Bouyer wrote: On Tue, Aug 09, 2011 at 10:47:18AM +0300, Jukka Ruohonen wrote: On Tue, Aug 09, 2011 at 09:44:09AM +0200, Alan Barrett wrote: Whether or not available as a

Re: CVS commit: src/sys/arch/i386/conf

2011-08-09 Thread Jukka Ruohonen
On Tue, Aug 09, 2011 at 10:18:29AM +0200, Manuel Bouyer wrote: Sorry, but this is just plain stupid. I could as well make so my work won't work in a modular kernel, and we'll have incompatible features. Of course it will work in non-modular kernels, but you just have to add it there manually.

Re: CVS commit: src/sys/arch/i386/conf

2011-08-09 Thread Manuel Bouyer
On Tue, Aug 09, 2011 at 11:23:57AM +0300, Jukka Ruohonen wrote: On Tue, Aug 09, 2011 at 10:18:29AM +0200, Manuel Bouyer wrote: Sorry, but this is just plain stupid. I could as well make so my work won't work in a modular kernel, and we'll have incompatible features. Of course it will work

Re: CVS commit: src/distrib/sets

2011-08-09 Thread Alan Barrett
On Tue, 09 Aug 2011, Jared D. McNeill wrote: Module Name:src Committed By: jmcneill Date: Tue Aug 9 15:02:30 UTC 2011 Modified Files: src/distrib/sets: Makefile Log Message: when sorting the metalog for unpriv builds, first run it through sort so mtree doesn't fail

Re: CVS commit: src/sys/arch/i386/conf

2011-08-09 Thread David Laight
On Tue, Aug 09, 2011 at 10:14:08AM +0200, Manuel Bouyer wrote: On Tue, Aug 09, 2011 at 10:47:18AM +0300, Jukka Ruohonen wrote: On Tue, Aug 09, 2011 at 09:44:09AM +0200, Alan Barrett wrote: Whether or not available as a module is a good reason for removing something from GENERIC is a

Re: CVS commit: src/sys/uvm

2011-08-09 Thread Jukka Ruohonen
On Sat, Aug 06, 2011 at 05:25:04PM +, Mindaugas Rasiukevicius wrote: Module Name: src Committed By: rmind Date: Sat Aug 6 17:25:04 UTC 2011 Modified Files: src/sys/uvm: uvm_amap.c uvm_anon.c uvm_anon.h uvm_fault.c uvm_loan.c uvm_map.c Log Message: - Rework

Re: CVS commit: src/sys/kern

2011-08-09 Thread YAMAMOTO Takashi
Module Name: src Committed By: dholland Date: Tue Aug 9 23:46:05 UTC 2011 Modified Files: src/sys/kern: vfs_lookup.c Log Message: Fail namei immediately if searchdir is unlinked / has been rmdir'd. Do this by checking if v_size == 0. Should fix PR 44658 (and PR 32661).

Re: CVS commit: src/sys/kern

2011-08-09 Thread tsugutomo . enami
YAMAMOTO Takashi y...@mwd.biglobe.ne.jp writes: the v_size == 0 check sounds wrong. does it work for eg. nfs? Certainly it doesn't work for layered fs. enami.

Re: CVS commit: src/sys/kern

2011-08-09 Thread David Holland
On Wed, Aug 10, 2011 at 03:10:13AM +, YAMAMOTO Takashi wrote: Log Message: Fail namei immediately if searchdir is unlinked / has been rmdir'd. Do this by checking if v_size == 0. Should fix PR 44658 (and PR 32661). why is this necessary? can't we just let VOP_LOOKUP fail? Not