Re: CVS commit: src/sys/kern

2012-02-20 Thread tsugutomo . enami
Christos Zoulas writes: > Modified Files: > src/sys/kern: kern_exec.c > > Log Message: > fix fae free'ing, from enami. The fa->len in the posix_spawn_fa_free() might no longer be equal to the value used when fa->fae is allocated. So you can't use it as is. That's why I said 'original fa-

Re: CVS commit: src/sys/arch/powerpc/include

2012-02-20 Thread Joerg Sonnenberger
On Tue, Feb 21, 2012 at 02:19:01AM +, Matt Thomas wrote: > Module Name: src > Committed By: matt > Date: Tue Feb 21 02:19:01 UTC 2012 > > Modified Files: > src/sys/arch/powerpc/include: cdefs.h > > Log Message: > Restore back to double alignment. For reference: __ALIGNBYTES is

Re: CVS commit: src/sys/kern

2012-02-20 Thread tsugutomo . enami
Christos Zoulas writes: > To generate a diff of this commit: > cvs rdiff -u -r1.341 -r1.342 src/sys/kern/kern_exec.c + kmem_free(fa->fae, sizeof(*fa->fae)); Two bugs here. fa->fae isn't allocated if (original) fa->len is zero. And the size of allocation is `sizeof(*fa->fae) * fa->len' (ag

Re: CVS commit: src/sys/uvm

2012-02-20 Thread Manuel Bouyer
On Mon, Feb 20, 2012 at 07:49:16PM +, Mindaugas Rasiukevicius wrote: > "Manuel Bouyer" wrote: > > Module Name:src > > Committed By: bouyer > > Date: Mon Feb 20 19:14:24 UTC 2012 > > > > Modified Files: > > src/sys/uvm: uvm_km.c uvm_kmguard.c uvm_map.c > > > >

Re: CVS commit: src/sys/uvm

2012-02-20 Thread Mindaugas Rasiukevicius
"Manuel Bouyer" wrote: > Module Name: src > Committed By: bouyer > Date: Mon Feb 20 19:14:24 UTC 2012 > > Modified Files: > src/sys/uvm: uvm_km.c uvm_kmguard.c uvm_map.c > > Log Message: > When using uvm_km_pgremove_intrsafe() make sure mappings are removed > before returning the