Re: CVS commit: src/usr.bin/make

2009-03-26 Thread David Holland
On Tue, Mar 24, 2009 at 11:41:11AM -0400, Perry E. Metzger wrote: Consider building NetBSD 0.9 today. As it was, MAKE_VERSION would seem quite modern, but the make would really be quite old. Using the last rcsid time would at least fix that... Not really, because branches

Re: CVS commit: src/sys/nfs

2009-05-05 Thread David Holland
On Tue, May 05, 2009 at 12:48:31PM +, Christoph Egger wrote: Module Name: src Committed By:cegger Date:Tue May 5 12:48:31 UTC 2009 Modified Files: src/sys/nfs: nfs_bootdhcp.c Log Message: buildfix: use %d for BOOTP_SIZE_(MIN,MAX). Makes i386

Re: CVS commit: src/sys/nfs

2009-05-06 Thread David Holland
On Wed, May 06, 2009 at 07:35:32AM +0200, Christoph Egger wrote: This is wrong; the values are of type size_t and should be printed with %zu. Fixed. gcc warning confused me saying the expected type was 'unsigned int'. Well yes, size_t is unsigned int on i386. Gotta read the source...

Re: CVS commit: src/share/man/man4

2009-05-09 Thread David Holland
On Sat, May 09, 2009 at 07:51:41AM -0700, Paul Goyette wrote: Document that drm(4) needs agp(4). Bump date. (joerg said so, and compilation fails without.) I really hate to contradict joerg, but... I think this is incorrect. You can compile drm with DRM_NO_AGP and it will compile just

Re: CVS commit: src/etc/rc.d

2009-05-14 Thread David Holland
On Thu, May 14, 2009 at 03:33:36PM +, Roy Marples wrote: Module Name: src Committed By:roy Date:Thu May 14 15:33:36 UTC 2009 Modified Files: src/etc/rc.d: network Log Message: Only start dhcpcd per interface if not running the full dhcpcd

Re: CVS commit: src/crypto/external/bsd/netpgp/dist

2009-05-26 Thread David Holland
On Mon, May 11, 2009 at 10:32:30AM -0400, Perry E. Metzger wrote: The only thing I will directly advocate for (besides scrapping the current UI) is something like the ssh-agent functionality. It is painful having to type in your passphrase for every email message you read, every one you

Re: CVS commit: src/sys

2009-06-14 Thread David Holland
On Sun, Jun 14, 2009 at 11:48:49PM +0200, Frank Kardel wrote: Or do we follow the strategy that new stuff uses uint* ? In that case the change would be minor. This is the best approach. Big sweeps are usually a bad idea... -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/usr.sbin/wake

2009-06-27 Thread David Holland
On Sat, Jun 27, 2009 at 08:00:03AM +0900, Izumi Tsutsui wrote: Now you see an example that discussion after commit is annoying ;-p Wake turbulence! -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src

2009-08-08 Thread David Holland
On Sat, Aug 08, 2009 at 09:06:53AM +, YAMAMOTO Takashi wrote: Do you think it would be better to split vnode(9) into 27 (!) man-pages? It is the amount of functions in that page. yes. I'd say there ought to be a summary page with a 1-2 line description of each function, as well as

Re: CVS commit: src/sys

2009-08-10 Thread David Holland
On Mon, Aug 10, 2009 at 03:44:38PM +, David Holland wrote: On Sun, Aug 09, 2009 at 10:49:01PM +, Adam Hamsik wrote: Log Message: Add enum uio_seg argument to do_sys_mknod and do_sys_mkdir so these functions can be called from kernel, too. This is the wrong approach

Re: CVS commit: src/usr.bin/error

2009-08-13 Thread David Holland
On Thu, Aug 13, 2009 at 03:46:39PM +, Christos Zoulas wrote: XXX: does this program actually do anything useful these days? Yes, it works just fine :-) I use it all the time. Better you than me :-) Maybe it should lose support for compilers that disappeared twenty years ago, though.

Re: CVS commit: src/usr.bin/error

2009-08-13 Thread David Holland
On Thu, Aug 13, 2009 at 08:38:18PM -0600, M. Warner Losh wrote: : | Maybe it should lose support for compilers that disappeared twenty : | years ago, though. : : The error reporting syntax has not changed much! Besides, how much space would it really save? In the brain of the next

Re: CVS commit: src/sys/common/bus_dma

2009-08-21 Thread David Holland
On Fri, Aug 21, 2009 at 03:33:17AM +, Jason R Thorpe wrote: Added Files: src/sys/common/bus_dma: bus_dmamem_common.c bus_dmamem_common.h Log Message: Add some common bus_dmamem routines to be shared by multiple platforms. Do these really need to be in their own private

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

2009-08-25 Thread David Holland
On Wed, Aug 26, 2009 at 03:39:16AM +, Elad Efrat wrote: Log Message: Build NiLFS(2). (2)? :-p -- David A. Holland dholl...@netbsd.org

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

2009-08-26 Thread David Holland
On Wed, Aug 26, 2009 at 12:01:39AM -0400, Elad Efrat wrote: Log Message: Build NiLFS(2). (2)? :-p That's how it's written in the configuration file... That seems... odd. -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/sys/dist/ipf/netinet

2009-08-26 Thread David Holland
On Wed, Aug 19, 2009 at 01:08:00PM +, Christoph Egger wrote: Modified Files: src/sys/dist/ipf/netinet: ip_compat.h Log Message: build fix: caddr_t - void * To generate a diff of this commit: cvs rdiff -u -r1.25 -r1.26 src/sys/dist/ipf/netinet/ip_compat.h Why should this

Re: CVS commit: src

2009-08-26 Thread David Holland
On Wed, Aug 26, 2009 at 12:23:39AM -0400, Elad Efrat wrote: Unrelated to the topic of the thread, but kmem(9) says: SECURITY CONSIDERATION As the memory allocated by kmem_alloc() is uninitialized, it can contain security-sensitive data left by its previous user. It is the

Re: CVS commit: src/sys/net

2009-08-26 Thread David Holland
On Sat, Aug 15, 2009 at 11:17:56PM +0200, Manuel Bouyer wrote: On Sat, Aug 15, 2009 at 09:10:07PM +, David Holland wrote: On Fri, Aug 14, 2009 at 11:13:19PM +0200, Manuel Bouyer wrote: I was thinking about that, too. I could add a 'name' node, too: net.interfaces.3

Re: CVS commit: src/sys/common/bus_dma

2009-08-26 Thread David Holland
On Fri, Aug 21, 2009 at 09:30:06AM -0700, Jason Thorpe wrote: Add some common bus_dmamem routines to be shared by multiple platforms. Do these really need to be in their own private directory? (whose name, I might add, is confusing in the presence of src/common...) This is intended to be

Re: CVS commit: src/sys/rump

2009-09-08 Thread David Holland
On Tue, Sep 08, 2009 at 04:18:01PM +0300, Antti Kantee wrote: No, __NetBSD__ is right. For all purposes, code in the rump kernel *is* NetBSD. E.g. if you have #ifdef __NetBSD__ in a kernel driver which was imported from $OtherOS, you must have the rump version think it is running on

Re: CVS commit: src/sys/rump

2009-09-08 Thread David Holland
On Tue, Sep 08, 2009 at 10:02:33AM -0600, M. Warner Losh wrote: __NetBSD__ is the *COMPILER* environment. Depending on it is *BAD*. You need to use a different symbol. This is a bug in the NetBSD code now. __NetBSD__ isn't, and never has bene, the KERNEL. No need to shout... Anyway,

Re: CVS commit: src/sys/ufs/ufs

2009-09-23 Thread David Holland
On Tue, Sep 22, 2009 at 10:42:59PM +0300, Antti Kantee wrote: Blah, I didn't even want to think about this migrane-inducer now. Maybe people who have recently worked on vnode reclaiming could instead be the ones to comment? It's becoming clear that this is something I'm going to need to

Re: CVS commit: src/lib/libc/stdio

2009-10-26 Thread David Holland
On Sun, Oct 25, 2009 at 05:09:34PM +, David Laight wrote: Lint is differentially far too picky... Remove some warnings that only appear on i386 (not on amd64) and that for some reason best known to others are deemed fatal for i386. Making this code 'pass lint' does absolutely nothing

Re: CVS commit: src/sys/conf

2009-11-15 Thread David Holland
On Sun, Nov 15, 2009 at 01:39:00PM +, David Laight wrote: Avoids problems with awk processing floating point numbers when LC_NUMERIC give a decimal point of ','. Isn't it a bug in awk for the input language to be broken by locale settings? -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/share/mk

2009-11-29 Thread David Holland
On Sun, Nov 29, 2009 at 06:43:10PM +, David Laight wrote: Modified Files: src/share/mk: bsd.subdir.mk Log Message: Remove an unneeded test (.if defined(V)) in .for v in ${V} ... .endfor. Tested by running build.sh distribution. Some versions of make error ${V} when V

Re: CVS commit: src

2009-12-06 Thread David Holland
On Sat, Dec 05, 2009 at 04:29:15PM +, Antti Kantee wrote: Log Message: Remove support for NetBSD/playstation2. Does this mean we can get rid of cross/mipsEEel-netbsd in pkgsrc? -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/sys/kern

2009-12-15 Thread David Holland
On Tue, Dec 15, 2009 at 06:35:18PM +, David Laight wrote: Modified Files: src/sys/kern: sys_pipe.c Log Message: Don't ERESTART write() calls for now. I suspect some programs don't allow for the partial transfer. More likely, the pipe is stuttering and repeating itself. Once

Re: CVS commit: src/games/canfield/cfscores

2010-01-01 Thread David Holland
On Fri, Jan 01, 2010 at 03:33:26PM +0100, Marc Balmer wrote: if (argc 2) { - printf(Usage: cfscores -a | cfscores [user]\n); - exit(1); + errx(1, Usage: cfscores -a | cfscores [user]); imo, lower case usage would be more consistent with other

Re: CVS commit: src/usr.sbin/wake

2010-01-11 Thread David Holland
On Sun, Jan 10, 2010 at 11:23:19PM +, Christos Zoulas wrote: Please explain to me how you propose we fix lint. Realistically, lint should be made to know a list of standard functions with usually-uninteresting return values. That would solve 95+% of the practical problem. In the long run

Re: CVS commit: src/usr.sbin/wake

2010-01-11 Thread David Holland
On Mon, Jan 11, 2010 at 06:31:29PM -0500, Christos Zoulas wrote: | In the long run (as I've said before) we should replace our lint with | something more modern... Once we get the llvm tools, I am fine with that. There's also splint, or sparse. -- David A. Holland dholl...@netbsd.org

Re: tn3270 removal (was: CVS commit: src)

2010-01-16 Thread David Holland
On Sat, Jan 16, 2010 at 11:57:13PM +0100, Bernd Ernesti wrote: Where was this discussed before? on tech-userlevel I found that thread which didn't make it obvious that this would happen without a WORKING replacement in pkgsrc. I have added it to pkgsrc. Please let me know if it

Re: tn3270 removal (was: CVS commit: src)

2010-01-17 Thread David Holland
On Sun, Jan 17, 2010 at 08:37:21AM +0100, Alistair Crooks wrote: I have added it to pkgsrc. Please let me know if it works, as I don't have an IBM mainframe in my back pocket for testing. :-/ Until then, there's always pkgsrc/emulators/hercules :-) which, by a staggering coincidence, I

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

2010-01-24 Thread David Holland
On Sun, Jan 24, 2010 at 12:17:05PM +, Frank Wille wrote: No include directive supported in a majors.* file. We have to copy the port-amiga majors. er, why not? It looks like it ought to work... although possibly I'm missing something. -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/sys/kern

2010-01-24 Thread David Holland
On Sun, Dec 20, 2009 at 07:06:45PM +, Antti Kantee wrote: Date:Sun Dec 20 19:06:45 UTC 2009 Modified Files: src/sys/kern: tty_pty.c Log Message: Add a pointing to an explanation of why we have #ifdef pmax stuff in here. Well, that's sort of an explanation,

Re: CVS commit: src/sys/kern

2010-01-24 Thread David Holland
On Sun, Jan 24, 2010 at 09:34:13PM +0200, Antti Kantee wrote: Well, that's sort of an explanation, although I might clarify it some if it were me... but I can't see what purpose these things serve since they're identical to the standard ones. If there's compat_ultrix logic that

Re: CVS commit: src/usr.sbin/powerd

2010-01-27 Thread David Holland
On Tue, Jan 26, 2010 at 11:56:30PM +0100, Martin Husemann wrote: However, the power_type from POWER_IOC_GET_TYPE can currently only be acpi and landisk (...). No apm is being set anywhere in the tree via sysmon_power_settype(). I've been told APM is dead and won't be fixed (it does

Re: CVS commit: src

2010-01-27 Thread David Holland
On Tue, Jan 26, 2010 at 12:25:00PM +, David Brownlee wrote: ... or in a book about the kernel. ?Section 9 describes how to use kernel facilities, not how they work internally. ?By documenting this item you give the impression of availability and stability, neither of which is

Re: CVS commit: src

2010-01-30 Thread David Holland
On Fri, Jan 29, 2010 at 12:24:56AM +, YAMAMOTO Takashi wrote: let's stop being too generic and go back to the change in question. do you think this man page is an appropriate place for these functions? Given that they exist in several MD implementations and (I think) are more

Re: CVS commit: [matt-nb5-mips64] src/sys/arch/mips

2010-01-30 Thread David Holland
On Sat, Jan 30, 2010 at 11:49:31PM +, Matt Thomas wrote: Change MIPS_CURLWP from s7 to t8. In a MALTA64 kernel, s6 is used 9155 times which means the compiler could really use s7 is was free to do so. The least used temporary was t8 (288 times). Once the kernel was switched to use

Re: CVS commit: src

2010-02-03 Thread David Holland
On Wed, Feb 03, 2010 at 03:34:49PM +, Roy Marples wrote: --- src/include/ndbm.h:1.14 Tue Sep 13 01:44:32 2005 +++ src/include/ndbm.h Wed Feb 3 15:34:40 2010 @@ -39,6 +39,7 @@ #include sys/cdefs.h #include sys/featuretest.h +#include stdint.h #include db.h #if

Re: CVS commit: src

2010-02-07 Thread David Holland
On Sat, Feb 06, 2010 at 11:45:27PM +, Havard Eidnes wrote: When using -lcurses, you also need -lterminfo. This fixes the build for sun2, and also builds with LDSTATIC=-static, since archive libraries don't record inter-library dependencies. Right, but this is not really a desirable

Re: CVS commit: src

2010-02-07 Thread David Holland
On Tue, Feb 02, 2010 at 03:30:55AM +, YAMAMOTO Takashi wrote: some MD implementations might have this for their internal use? it sounds useless and confusing to me. It has two legitimate audiences: (1) people bringing up new ports, who may not already know the ins and outs of

Re: CVS commit: [uebayasi-xip] src/sys/ufs/ffs

2010-02-11 Thread David Holland
On Thu, Feb 11, 2010 at 05:22:38AM +, Masao Uebayashi wrote: Modified Files: src/sys/ufs/ffs [uebayasi-xip]: ffs_vfsops.c Log Message: XIP hook for ffs. Shouldn't that be in fs-independent code? -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/distrib/utils/sysinst

2010-02-28 Thread David Holland
On Wed, Feb 17, 2010 at 04:09:52PM +, Martin Husemann wrote: Modified Files: src/distrib/utils/sysinst: disks.c Log Message: eeh says we should not enable fsck_lfs on lfs partitions Even assuming this is true in the medium to long term rather than a temporary problem... this

Re: CVS commit: src/distrib/sets

2010-03-03 Thread David Holland
On Wed, Mar 03, 2010 at 04:13:42PM +, Matthias Scheler wrote: src/distrib/sets: mkvars.mk sets.subr src/distrib/sets/lists/modules: mi Log Message: dtrace,zfs means MKDTRACE=yes *and* MKZFS=yes which is not what we want. Invent a flag solaris which is the or of those two

Re: CVS commit: src/share/man/man3

2010-03-03 Thread David Holland
On Wed, Mar 03, 2010 at 12:58:33PM -0500, Greg A. Woods wrote: I believe that __UNCONST() in particular is _never_ absolutely necessary -- it may sometimes save a very few cycles and a few bytes of storage, but that's the best it can do. No such luck, at least not until C grows a stronger

Re: CVS commit: src/share/man/man3

2010-03-06 Thread David Holland
On Thu, Mar 04, 2010 at 05:26:02PM -0500, Greg A. Woods wrote: No such luck, at least not until C grows a stronger type system. See for example strchr(3). I don't think a stronger type system would really change this issue fundamentally unless it was one which was so radical as to

Re: CVS commit: [netbsd-5] src/sys/dev/ic

2010-03-06 Thread David Holland
On Sun, Mar 07, 2010 at 12:13:38AM +0100, Bernd Ernesti wrote: Log Message: Pull up the following revisions(s) (requested by bouyer in ticket #1317): sys/dev/ic/ahcisata_core.c: revision 1.23 Move ahci_channel_start() after BSY has cleared (do not touch SError while BSY

Re: CVS commit: src/distrib/sets

2010-03-06 Thread David Holland
On Fri, Mar 05, 2010 at 12:05:20AM +0900, Masao Uebayashi wrote: The difference between pkgsrc base is that we don't have control of sources in pkgsrc, which is the reason why we have build options. In base (kernel + userland), we have control fix code, and can make everything modular.

Re: CVS commit: src/distrib/sets

2010-03-07 Thread David Holland
On Mon, Mar 08, 2010 at 12:21:51AM +0900, Masao Uebayashi wrote: In theory. In practice, who's going to make the pam and krb stuff in e.g. su into plugins? Even if someone wants to take the time to try, it's a big can of worms... Considering such an anti-modular atmosphere, no one

Re: CVS commit: src/sbin/atactl

2010-03-13 Thread David Holland
On Sat, Mar 13, 2010 at 07:49:50PM +, matthew green wrote: Log Message: add SMART code 187: Reported uncorrect. from smartmontools. Is that supposed to be read as uncorrected or incorrect? (And can we edit it or does it need to stay verbatim?) -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/sys

2010-03-14 Thread David Holland
On Sun, Mar 14, 2010 at 05:11:55PM +1300, Darran Hunt wrote: I wanted to differentiate it from the sys/ctf.h header that's part of the external/cddl/osnet tree. Sorry to rock the convention. Perhaps kctf.h? sys/debug_ctf.h? ctf is essentially a debug info format, right? -- David A.

Re: CVS commit: src/sys

2010-03-23 Thread David Holland
On Sat, Mar 20, 2010 at 11:31:31PM +, Chuck Silvers wrote: fix copy{in,out}{,str}() to return the error returned by uvm_fault(). fixes PR 41813. Do you know if/how this will affect PR 11904? What happens now on EIO in a memory-mapped file? -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/sys

2010-03-24 Thread David Holland
On Wed, Mar 24, 2010 at 06:41:38AM -0700, Chuck Silvers wrote: fix copy{in,out}{,str}() to return the error returned by uvm_fault(). fixes PR 41813. Do you know if/how this will affect PR 11904? What happens now on EIO in a memory-mapped file? this change doesn't affect

Re: CVS commit: src

2010-03-25 Thread David Holland
On Thu, Mar 25, 2010 at 04:42:10PM +, Andrew Doran wrote: Well wouldn't it then be better to document that they are not of any use to the general kernel? From a quick look around, I see also quite a few instances of stathz and profhz being used/defined. What Joerg said. They

Re: CVS commit: src/sys

2010-03-28 Thread David Holland
On Thu, Mar 25, 2010 at 02:20:42PM +0900, Masao Uebayashi wrote: I suppose I ought to wade through genfs_getpages in more detail sometime... ubc_alloc() ubc_uiomove() uiomove() [...] thank you. -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/sys/rump

2010-04-15 Thread David Holland
On Wed, Apr 14, 2010 at 07:04:27PM +, Christos Zoulas wrote: Use struct kauth_cred * instead of kauth_cred_t in all exported interfaces. Allows to remove hairbrained _t typedef dance. Not that I have a problem with it, but the whole point of kauth_cred_t was to provide opacity on

Re: CVS commit: src/lib/libc/string

2010-04-18 Thread David Holland
On Mon, Apr 19, 2010 at 03:02:05AM +, Christos Zoulas wrote: In that case use an explicit cast to the unsigned type. What you did is penalize the code for no good reason. Any compiler these days knows how to do these micro-optimizations. Furthermore, as swab() is fairly useless, its

Re: CVS commit: src/usr.bin/make

2010-04-22 Thread David Holland
On Thu, Apr 22, 2010 at 02:11:31PM -0700, Simon J. Gerraty wrote: Dh already commited the exact same change I just tried to. sorry, didn't look here first. :-/ -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/external/cddl/osnet

2010-04-23 Thread David Holland
On Sat, Apr 24, 2010 at 02:09:57AM +0100, Mindaugas Rasiukevicius wrote: | Also, while I see the benefits with *var (technically, it is against KNF, | by the way), my preference is to use type. That was not an explicit preference, it was just a random choice, so I have corrected it.

Re: CVS commit: src/lib/libc/sys

2010-04-29 Thread David Holland
On Thu, Apr 29, 2010 at 07:14:35AM +, Jukka Ruohonen wrote: Note that utimes(2) no longer enjoys the blessing of POSIX. What's the currently-blessed alternative? -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/lib/libc/sys

2010-04-29 Thread David Holland
On Thu, Apr 29, 2010 at 12:05:02PM +0300, Jukka Ruohonen wrote: What's the currently-blessed alternative? 2004: APPLICATION USAGE For applications portability, the utime() function should be used to set file access and modification times instead of utimes(). oh goody, yet

Re: CVS commit: src/sys

2010-05-01 Thread David Holland
On Fri, Apr 30, 2010 at 10:03:14AM +, Antti Kantee wrote: Add translation from vtype to dirent type. Is there any reason that the vtypes need to continue to exist in the long term? -- David A. Holland dholl...@netbsd.org

Re: CVS commit: [matt-nb5-mips64] src/libexec/getty

2010-05-01 Thread David Holland
On Fri, Apr 30, 2010 at 04:24:25PM +, Matt Thomas wrote: Modified Files: src/libexec/getty [matt-nb5-mips64]: main.c Log Message: Syslog if login_tty fails any reason not to apply this to head? -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/sys

2010-05-02 Thread David Holland
On Sun, May 02, 2010 at 01:37:23PM +0300, Antti Kantee wrote: Add translation from vtype to dirent type. Is there any reason that the vtypes need to continue to exist in the long term? I view them as an optimization and I'd say no, but it's gonna be one hell of a battlefield

Re: CVS commit: src/lib/libc/stdio

2010-05-06 Thread David Holland
On Wed, May 05, 2010 at 08:05:20AM +0200, Alan Barrett wrote: On Wed, 05 May 2010, Jukka Ruohonen wrote: Few examples of pages that are badly and *systematically* behind: intro(3) intro(4) pci(4) usb(4) mk.conf(5) sysctl(7) Perhaps we need a bot to

Re: CVS commit: src/lib/libc/stdio

2010-05-06 Thread David Holland
On Thu, May 06, 2010 at 08:14:08AM +, Jukka Ruohonen wrote: Correct the discussion about return values: fileno() may fail and return -1. Note that in such cases the NetBSD implementation does not set errno to EBADF, hence diverging from the standard in this small detail. How is that not

Re: CVS commit: src/share/man/man9

2010-05-06 Thread David Holland
On Tue, May 04, 2010 at 05:16:58AM -0700, Paul Goyette wrote: +The functions were originally introduced to handle +.Tn PCI +bus master devices, which assumed little endian byte order in +.Tn DMA +transfers, even on big endian systems. There are a few PCI bus master devices that are big

Re: CVS commit: src/distrib/notes/common

2010-05-07 Thread David Holland
On Mon, May 03, 2010 at 03:14:24PM +, Izumi Tsutsui wrote: Update IP addresses (both IPv4 and IPv6) of current ftp.NetBSD.org. XXX: is it still worth to note those numbers in INSTALL docs nowadays? It's not uncommon for DNS to not be working in the installer for one reason or another.

Re: CVS commit: src/distrib/notes/common

2010-05-09 Thread David Holland
On Fri, May 07, 2010 at 03:59:21PM +0200, Joerg Sonnenberger wrote: Update IP addresses (both IPv4 and IPv6) of current ftp.NetBSD.org. XXX: is it still worth to note those numbers in INSTALL docs nowadays? It's not uncommon for DNS to not be working in the installer for one

Re: CVS commit: src/distrib/notes/common

2010-05-13 Thread David Holland
On Sun, May 09, 2010 at 02:44:49PM +0200, Joerg Sonnenberger wrote: On Sun, May 09, 2010 at 07:26:53AM +, David Holland wrote: If DNS won't go, any IP addresses you need that you failed to write down before you started the install are a show-stopper; you need to abort the install

Re: CVS commit: src/lib/libc/stdio

2010-05-13 Thread David Holland
On Thu, May 06, 2010 at 03:17:38PM +0300, Jukka Ruohonen wrote: Perhaps we need a bot to watch all commits and send reminders like You added a new file, but you haven't updated the sets, see URL for instructions, You added a new sysctl node but you haven't updated sysctl(7), etc.

Re: CVS commit: src/lib/libc/stdio

2010-05-13 Thread David Holland
On Fri, May 14, 2010 at 05:10:24AM +, David Holland wrote: Anyway, can you (at your convenience) send me a list of the things like this that need consistency checking? I will then have a shot at writing some scripts and we can see what things look like. er wait, there's a list upthread

Re: CVS commit: src/lib/libc/stdio

2010-05-13 Thread David Holland
On Thu, May 06, 2010 at 08:51:29AM +, David Holland wrote: On Thu, May 06, 2010 at 08:14:08AM +, Jukka Ruohonen wrote: Correct the discussion about return values: fileno() may fail and return -1. Note that in such cases the NetBSD implementation does not set errno to EBADF

Re: CVS commit: src/sys/kern

2010-05-13 Thread David Holland
On Wed, May 12, 2010 at 03:40:38AM +, Mindaugas Rasiukevicius wrote: execve1: move few PNBUF_PUT() outside the lock paths. Thank you. -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/share/man/man9

2010-05-13 Thread David Holland
On Thu, May 06, 2010 at 08:27:05PM +0900, Izumi Tsutsui wrote: -bus master devices which assumed little endian byte order in +bus master devices that do little endian .Tn DMA -transfers, even on big endian systems. +transfers on big endian systems. What's little endian

Re: CVS commit: src/lib/libc/time

2010-05-15 Thread David Holland
On Sat, May 15, 2010 at 01:54:52PM +, Paul Goyette wrote: XXX Someone else can decide whether we should refer to U.S.A. or to XXX the United States in the parenthetical. The way it is now matches the other use later on, so let's leave it. -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/lib/libc/stdio

2010-05-15 Thread David Holland
On Fri, May 14, 2010 at 08:29:34AM +0300, Jukka Ruohonen wrote: Yes. The trouble is that because libraries and headers don't match up, you need to parse headers to extract the lists, and that seems likely to end up being delicate and a constant source of annoyance. How about the

Re: CVS commit: src/share/man/man9

2010-05-15 Thread David Holland
On Fri, May 14, 2010 at 08:51:46PM +0900, Izumi Tsutsui wrote: A transfer of little-endian data? I guess that's not entirely clear, especially for anyone who doesn't automatically interpret DMA as a form of memcpy. Anyway, I just fixed it up again, please take a look. to

Re: CVS commit: src/games/factor

2010-05-15 Thread David Holland
On Sat, May 15, 2010 at 09:22:39PM +, Joerg Sonnenberger wrote: Modified Files: src/games/factor: factor.6 factor.c Log Message: Follow the Fundamental Theory of Algebra. Disallow factorising of numbers less than 2 as it is not - naturally unique (negative numbers) -

Re: CVS commit: src/games/factor

2010-05-15 Thread David Holland
On Sat, May 15, 2010 at 11:46:16PM +0200, Joerg Sonnenberger wrote: Discussed with the kristaps and wiz Did you bother to read tech-userlevel before doing this? Yes and it was quite interesting. so you blithely ignored the conclusion? -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/games/factor

2010-05-16 Thread David Holland
On Sun, May 16, 2010 at 04:30:45PM +0200, Martin Husemann wrote: [..] We should make our factor(6) compaible to the coreutils one, just in case POSIX picks it up in the next revision ;-} Just in case it's not clear from upthread (tech-userlevel): that compatibility means: -

Re: CVS commit: src

2010-05-16 Thread David Holland
On Sun, May 16, 2010 at 03:33:30PM +, Jukka Ruohonen wrote: As was done with the man-page for sys/types.h, move dir(5) to dirent(3). Add a table describing the stat - dirent types. Note IFTODT() and DTTOIF(). Once upon a time, that page documented the on-disk format of FFS directories,

Re: CVS commit: src/games/factor

2010-05-16 Thread David Holland
On Sun, May 16, 2010 at 10:49:22PM +0200, Martin Husemann wrote: On Sun, May 16, 2010 at 07:30:54PM +, David Holland wrote: of which at least the 0 case is clearly incorrect... That obviously depends on the meaning you give the output, which is not well defined in the man page

Re: CVS commit: src/usr.bin/make

2010-05-17 Thread David Holland
On Mon, May 17, 2010 at 01:01:17PM -0400, Christos Zoulas wrote: Log Message: put back canonicalization, but avoid doing it for files that don't contain / From dholland ...in the sense that I did none of the work? Thanks though :-) -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/lib/libpthread

2010-05-19 Thread David Holland
On Tue, May 18, 2010 at 11:04:12AM +0300, Jukka Ruohonen wrote: In my opinion we could have more introductory sections that connect the pages in a more logical manner.[1] Just listing the manual pages in the intro-sections is not enough. For example [...] Another option could be a

Re: CVS commit: src/sys/rump/librump/rumpkern

2010-05-20 Thread David Holland
On Tue, May 18, 2010 at 04:29:36PM +, Antti Kantee wrote: It's pretty obvious that in terms of scalability simple workload partitioning and replication into multiple kernels wins hands down over complicated locking or locklessing algorithms which depend on globally atomic state. ...in

Re: CVS commit: src/sys/ufs/ufs

2010-05-25 Thread David Holland
On Tue, May 25, 2010 at 11:02:07AM +, Antti Kantee wrote: Add a comment describing an observed boom-crash-burn problem in the code. Fixing it will require a full tank of gas, half a pack of cigarettes, sunglasses, darkness, and most importantly: someone else. Thanks for the merge

Re: CVS commit: src/sys/arch

2010-05-27 Thread David Holland
On Thu, May 27, 2010 at 09:36:42AM +0200, Alan Barrett wrote: On Thu, 27 May 2010, David A. Holland wrote: -cleandir distclean: cleanlibdir +cleandir distclean: .WAIT cleanlibdir I would have expected .WAIT to be a no-op in this context. How does do nothing, wait until you have

Re: CVS commit: src/external/mit/xorg/lib

2010-05-28 Thread David Holland
On Fri, May 28, 2010 at 06:22:05AM +, matthew green wrote: Modified Files: src/external/mit/xorg/lib/libGL: Makefile src/external/mit/xorg/lib/libGLU: Makefile src/external/mit/xorg/lib/libglut: Makefile Log Message: we're now at mesa 7.7.1. XXX: really

Re: CVS commit: src/sys/arch

2010-06-03 Thread David Holland
On Fri, May 28, 2010 at 08:10:39AM +0200, Alan Barrett wrote: I think it's a bug that .WAIT affects things declared in different lines, but OK, thanks for the explanation. It's a separator, not a modifier. It's not a bug, just... counterintuitive. The .WAIT documentation

Re: CVS commit: src/usr.bin/sort

2010-06-05 Thread David Holland
On Sun, Jun 06, 2010 at 12:00:33AM +, Thomas Klausner wrote: Modified Files: src/usr.bin/sort: init.c Log Message: Fix typo in comment. Woops! -- David A. Holland dholl...@netbsd.org

Re: CVS commit: src/lib/libc/locale

2010-06-05 Thread David Holland
On Sat, Jun 05, 2010 at 11:53:32PM +0200, Alistair Crooks wrote: Module Name: src Committed By: tnozaki Date: Wed Jun 2 16:04:52 UTC 2010 Modified Files: src/lib/libc/locale: bsdctype.c Log Message: uint8_t - unsigned char, int16_t - short.

Re: CVS commit: src/etc

2010-06-05 Thread David Holland
On Fri, Jun 04, 2010 at 02:42:55PM -0400, Christos Zoulas wrote: Modified Files: src/etc: rc rc.subr Log Message: print human readable exit code. + elif [ $1 -eq 127 ] + then + echo stopped with signal $(expr $1 / 256) that can't be right... -- David

Re: CVS commit: src/tests/util/sh

2010-06-05 Thread David Holland
On Thu, Jun 03, 2010 at 06:53:36PM +, Christos Zoulas wrote: # according to the standard, only failing *simple* commands # cause an exit under -e. () is not a simple command. - dcheck '(set -e; (set +e; false; echo OK; false); echo OK)' 'OK OK' - echeck '(set -e; (set +e;

Re: CVS commit: src/lib/libc/locale

2010-06-05 Thread David Holland
On Sat, Jun 05, 2010 at 11:28:31PM -0600, M. Warner Losh wrote: : I looked at it and it appears to me that it's a change to use : non-sized types (that are guaranteed to be large enough) in the : in-memory structures. If we ever do a port to a 36-bit machine or : whatever it's probably

Re: CVS commit: src/etc

2010-06-06 Thread David Holland
On Sun, Jun 06, 2010 at 08:37:57AM -0400, Christos Zoulas wrote: Modified Files: src/etc: rc.subr Log Message: fix conditional, from dholland. That makes a lot more sense :-) (but now I'm wondering if sh should provide some kind of WIFEXITED/WEXITSTATUS logic so we don't have to

Re: CVS commit: src/tests/util/sh

2010-06-06 Thread David Holland
On Sun, Jun 06, 2010 at 08:08:57AM -0400, Christos Zoulas wrote: | If it's more important that the nightly test runs come up clean than | that it actually check for correct behavior so sh gets fixed, the test | script should at least document what the correct behavior is. I have | done

Re: CVS commit: src/tests/util/sh

2010-06-06 Thread David Holland
On Sun, Jun 06, 2010 at 07:11:05PM +, David Holland wrote: On Sun, Jun 06, 2010 at 08:08:57AM -0400, Christos Zoulas wrote: | If it's more important that the nightly test runs come up clean than | that it actually check for correct behavior so sh gets fixed, the test | script

Re: CVS commit: src/tests/util/sh

2010-06-07 Thread David Holland
On Mon, Jun 07, 2010 at 07:30:27AM -0400, Christos Zoulas wrote: | Ok, right now ksh diverges from sh on 32 of 75 cases, including | several of the ones you changed. bash diverges on 3. | | Anyway, the test script now at least documents what the deal is, so | I'm ok with leaving it

  1   2   3   4   5   >