Re: CVS commit: src/sys/uvm

2011-08-10 Thread YAMAMOTO Takashi
[ moving to tech-kern ] hi, > y...@mwd.biglobe.ne.jp (YAMAMOTO Takashi) wrote: >> > >> > Here is the updated patch after your changes: >> > >> > http://www.netbsd.org/~rmind/uvm_anon_freelst2.diff >> > >> > As you noted, uvm_anfree() can temporarily release the amap lock - that >> > can happen

Re: CVS commit: src/sys/kern

2011-08-10 Thread David Holland
On Wed, Aug 10, 2011 at 07:22:06AM +, YAMAMOTO Takashi wrote: > > > why is this necessary? can't we just let VOP_LOOKUP fail? > > > > Not to fix PR 44658. > > it's better to fix the vn_isunder check instead of avoiding running it. > IMO vn_isunder should return acutal error code (eg. E

Re: CVS commit: src

2011-08-10 Thread YAMAMOTO Takashi
hi, > On Wed, Aug 10, 2011 at 08:59:48AM +, YAMAMOTO Takashi wrote: >> is extattr_namespace_access really necessary? >> uid-based priviledge check in userland is often a mistake. > > For now it duplicates the same simple access check as in kernel: > system attributes are restricted to root.

Re: CVS commit: src/sys/arch

2011-08-10 Thread Cherry G. Mathew
On 10 August 2011 13:39, Cherry G. Mathew wrote: > Module Name:    src > Committed By:   cherry > Date:           Wed Aug 10 11:39:46 UTC 2011 > > Modified Files: >        src/sys/arch/amd64/amd64: fpu.c machdep.c >        src/sys/arch/i386/isa: npx.c >        src/sys/arch/x86/x86: x86_machdep.c >

Re: CVS commit: src

2011-08-10 Thread Emmanuel Dreyfus
On Wed, Aug 10, 2011 at 08:59:48AM +, YAMAMOTO Takashi wrote: > is extattr_namespace_access really necessary? > uid-based priviledge check in userland is often a mistake. For now it duplicates the same simple access check as in kernel: system attributes are restricted to root. This is just a

Re: CVS commit: src

2011-08-10 Thread YAMAMOTO Takashi
hi, > Module Name: src > Committed By: manu > Date: Wed Aug 3 04:11:17 UTC 2011 > > Modified Files: > src/bin/cp: cp.c utils.c > src/bin/mv: mv.c > src/distrib/sets/lists/comp: mi > src/lib/libc/gen: Makefile.inc extattr.3 extattr.c > src/lib/libc/sys: exta

Re: CVS commit: src/sys/kern

2011-08-10 Thread YAMAMOTO Takashi
> 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_