Re: CVS commit: src/sys/uvm

2011-07-26 Thread Mindaugas Rasiukevicius
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 in amap_copy(). Patch closes the race

Re: CVS commit: src/sbin/disklabel

2011-07-26 Thread David Laight
On Tue, Jul 26, 2011 at 09:56:24AM +0200, Alan Barrett wrote: > On Mon, 25 Jul 2011, Christos Zoulas wrote: > >Modified Files: > > src/sbin/disklabel: disklabel.8 > > > >Log Message: > >mention that we don't handle more than 2TB disks/partitions. > > I think the limit's actually (2**32 - 1) se

Re: CVS commit: src/sys/arch/alpha/alpha

2011-07-26 Thread Nicolas Joly
On Wed, Jul 27, 2011 at 02:33:24AM +0900, Izumi Tsutsui wrote: > > Looking how many atoi() copies we have in tree, does it make sense to > > add the following at the end of libkern.h > > Please don't use it for #ifdef _STANDALONE ones. > libc's strtoll() is larger than dumb atoi()s. Fine. No need

Re: CVS commit: src/sys/arch/alpha/alpha

2011-07-26 Thread Joerg Sonnenberger
On Wed, Jul 27, 2011 at 02:33:24AM +0900, Izumi Tsutsui wrote: > > Looking how many atoi() copies we have in tree, does it make sense to > > add the following at the end of libkern.h > > Please don't use it for #ifdef _STANDALONE ones. > libc's strtoll() is larger than dumb atoi()s. ...if strtoll

Re: CVS commit: src/sys/arch/alpha/alpha

2011-07-26 Thread Izumi Tsutsui
> Looking how many atoi() copies we have in tree, does it make sense to > add the following at the end of libkern.h Please don't use it for #ifdef _STANDALONE ones. libc's strtoll() is larger than dumb atoi()s. --- Izumi Tsutsui

Re: CVS commit: src/sys/arch/alpha/alpha

2011-07-26 Thread Nicolas Joly
On Tue, Jul 26, 2011 at 06:45:31PM +0200, Joerg Sonnenberger wrote: > On Tue, Jul 26, 2011 at 06:19:21PM +0200, Nicolas Joly wrote: > > On Tue, Jul 26, 2011 at 05:27:38PM +0200, Joerg Sonnenberger wrote: > > > On Tue, Jul 26, 2011 at 02:59:04PM +, Nicolas Joly wrote: > > > > Module Name:src

Re: CVS commit: src/sys/arch/alpha/alpha

2011-07-26 Thread Joerg Sonnenberger
On Tue, Jul 26, 2011 at 06:19:21PM +0200, Nicolas Joly wrote: > On Tue, Jul 26, 2011 at 05:27:38PM +0200, Joerg Sonnenberger wrote: > > On Tue, Jul 26, 2011 at 02:59:04PM +, Nicolas Joly wrote: > > > Module Name: src > > > Committed By: njoly > > > Date: Tue Jul 26 14:59:03

Re: CVS commit: src/sys/arch/alpha/alpha

2011-07-26 Thread Nicolas Joly
On Tue, Jul 26, 2011 at 05:27:38PM +0200, Joerg Sonnenberger wrote: > On Tue, Jul 26, 2011 at 02:59:04PM +, Nicolas Joly wrote: > > Module Name:src > > Committed By: njoly > > Date: Tue Jul 26 14:59:03 UTC 2011 > > > > Modified Files: > > src/sys/arch/alpha/alph

Re: CVS commit: src/sys/arch/alpha/alpha

2011-07-26 Thread Joerg Sonnenberger
On Tue, Jul 26, 2011 at 02:59:04PM +, Nicolas Joly wrote: > Module Name: src > Committed By: njoly > Date: Tue Jul 26 14:59:03 UTC 2011 > > Modified Files: > src/sys/arch/alpha/alpha: autoconf.c > > Log Message: > Make atoi func static, and constify Shouldn't this use strtol t

Re: CVS commit: src/sbin/disklabel

2011-07-26 Thread Alan Barrett
On Mon, 25 Jul 2011, Christos Zoulas wrote: Modified Files: src/sbin/disklabel: disklabel.8 Log Message: mention that we don't handle more than 2TB disks/partitions. I think the limit's actually (2**32 - 1) sectors. With 512-byte sectors, that's one sector less than 2TB. --apb (Alan