Re: cvs commit: src/sys/fs/msdosfs bpb.h

2006-12-19 Thread Bruce Evans
On Tue, 19 Dec 2006, Craig Rodrigues wrote: rodrigc 2006-12-19 01:55:45 UTC FreeBSD src repository Modified files: sys/fs/msdosfs bpb.h Log: Fix get_ulong() macro on AMD64 (or any little-endian 64-bit platform). This bug caused vn_stat() to fail on files larger than 2gb on ms

cvs commit: src/sys/dev/bge if_bge.c

2006-12-20 Thread Bruce Evans
bde 2006-12-20 11:14:46 UTC FreeBSD src repository Modified files: sys/dev/bge if_bge.c Log: Avoid a race and a pessimization in bge_intr(): - moved the synchronizing bus read to after the bus write for the first interrupt ack so that it actually synchronize

cvs commit: src/sys/dev/bge if_bge.c

2006-12-20 Thread Bruce Evans
bde 2006-12-20 12:03:21 UTC FreeBSD src repository Modified files: sys/dev/bge if_bge.c Log: In bge_txeof(), cancel the watchdog timeout if all descriptors have been handled instead of when at least one descriptor was just handled. For bge, it is normal to get a

Re: cvs commit: src/sys/dev/bge if_bge.c

2006-12-21 Thread Bruce Evans
On Wed, 20 Dec 2006, Gleb Smirnoff wrote: On Wed, Dec 20, 2006 at 12:03:21PM +, Bruce Evans wrote: B> bde 2006-12-20 12:03:21 UTC B> B> FreeBSD src repository B> B> Modified files: B> sys/dev/bge if_bge.c B> Log: B> In bge_txeof(), cancel

Re: cvs commit: src/sys/dev/bge if_bge.c

2006-12-22 Thread Bruce Evans
On Fri, 22 Dec 2006, I wrote: bge_start_locked() starts with the bge (sc) lock held and never releases it as far as I can see. This this problem can't happen (the lock prevents both txeof and the watchdog from being reached before start resets the timeout to 5 seconds). I could only find the l

Re: cvs commit: src/sys/dev/bge if_bge.c

2006-12-23 Thread Bruce Evans
On Sat, 23 Dec 2006, Robert Watson wrote: On Sat, 23 Dec 2006, John Polstra wrote: That said, dropping and regrabbing the driver lock in the rxeof routine of any driver is bad. It may be safe to do, but it incurs horrible performance penalties. It essentially allows the time-critical, high

Re: cvs commit: src/sys/dev/bge if_bge.c

2006-12-23 Thread Bruce Evans
On Sun, 24 Dec 2006, Oleg Bulyzhin wrote: On Fri, Dec 22, 2006 at 01:24:45AM +1100, Bruce Evans wrote: On Wed, 20 Dec 2006, Gleb Smirnoff wrote: I have a suspicion that this may cause a problem under high load. Imagine that thread #1 is spinning in bge_start_locked() getting packets out of

Re: cvs commit: src/sys/dev/bge if_bge.c

2006-12-24 Thread Bruce Evans
On Sun, 24 Dec 2006, Scott Long wrote: Bruce Evans wrote: On Sat, 23 Dec 2006, Robert Watson wrote: On Sat, 23 Dec 2006, John Polstra wrote: That said, dropping and regrabbing the driver lock in the rxeof routine of any driver is bad. It may be safe to do, but it incurs horrible

Re: cvs commit: src/sys/dev/bge if_bge.c

2006-12-24 Thread Bruce Evans
On Sun, 24 Dec 2006, Scott Long wrote: Bruce Evans wrote: On Sun, 24 Dec 2006, Oleg Bulyzhin wrote: it's quite unusal) and it is not lock related: 1) bge_start_locked() & bge_encap fills tx ring. 2) during next 5 seconds we do not have packets for transmit (i.e. no bge_sta

Re: cvs commit: src/sys/dev/bge if_bge.c

2006-12-24 Thread Bruce Evans
On Sun, 24 Dec 2006, Robert Watson wrote: From the perspective of optimizing these particular paths, small packet sizes best reveal processing overhead up to about the TCP/socket buffer layer on modern hardware (DMA, etc). The uni/bidirectional axis is interesting because it helps reveal the

cvs commit: src/sys/dev/bge if_bge.c

2006-12-26 Thread Bruce Evans
bde 2006-12-26 18:33:55 UTC FreeBSD src repository Modified files: sys/dev/bge if_bge.c Log: After rev.1.169, the "interrupt" coalescing parameters are not used in bge_intr(). Some of them are used in bge_poll(). Simplify by only initializing these for polling

cvs commit: src/sys/amd64/include atomic.h src/sys/i386/include atomic.h

2006-12-27 Thread Bruce Evans
bde 2006-12-27 20:26:00 UTC FreeBSD src repository Modified files: sys/amd64/includeatomic.h sys/i386/include atomic.h Log: Avoid an instruction in atomic_cmpset_{int_long)() in most cases. These functions are used a lot for mutexes, so this reduces the text

cvs commit: src/sys/amd64/include atomic.h

2006-12-28 Thread Bruce Evans
bde 2006-12-28 08:15:15 UTC FreeBSD src repository Modified files: sys/amd64/includeatomic.h Log: Removed gratuitous cosmetic differences with the i386 version. This mainly involves removing all __CC_SUPPORTS___INLINE__ ifdefs. These ifdefs are even less needed for

cvs commit: src/sys/amd64/include atomic.h src/sys/i386/include atomic.h

2006-12-29 Thread Bruce Evans
bde 2006-12-29 13:36:26 UTC FreeBSD src repository Modified files: sys/amd64/includeatomic.h sys/i386/include atomic.h Log: Try harder to garbage-collect the "LOCORE" (really asm) version of MPLOCKED. The cleaning in rev.1.25 was supposed to have been undone

cvs commit: src/sys/amd64/include atomic.h src/sys/i386/include atomic.h

2006-12-29 Thread Bruce Evans
bde 2006-12-29 14:28:23 UTC FreeBSD src repository Modified files: sys/amd64/includeatomic.h sys/i386/include atomic.h Log: Fixed some style bugs (whitespace only). Revision ChangesPath 1.43 +31 -30src/sys/amd64/include/atomic.h 1.45 +

cvs commit: src/sys/amd64/include atomic.h src/sys/i386/include atomic.h

2006-12-29 Thread Bruce Evans
bde 2006-12-29 15:29:49 UTC FreeBSD src repository Modified files: sys/amd64/includeatomic.h sys/i386/include atomic.h Log: Fixed some style bugs (mainly assorted errors in comments, and inconsistent spelling of `result'). Revision ChangesPath 1.44

cvs commit: src/sys/i386/include atomic.h

2006-12-29 Thread Bruce Evans
bde 2006-12-29 15:48:18 UTC FreeBSD src repository Modified files: sys/i386/include atomic.h Log: Fix oops in previous commit. Revision ChangesPath 1.47 +1 -1 src/sys/i386/include/atomic.h ___ cvs-all@freeb

cvs commit: src/sys/dev/cy cy_isa.c

2007-01-13 Thread Bruce Evans
bde 2007-01-13 11:00:56 UTC FreeBSD src repository Modified files: sys/dev/cy cy_isa.c Log: Fixed a panic in the probe. The memory resource was accessed after releasing it. This seems to have worked until a few days ago, but now the memory is unmapped. Re

Re: cvs commit: src/sys/compat/linprocfs linprocfs.c

2007-01-21 Thread Bruce Evans
On Sun, 21 Jan 2007, M. Warner Losh wrote: In message: <[EMAIL PROTECTED]> Max Laier <[EMAIL PROTECTED]> writes: : On Sunday 21 January 2007 16:47, M. Warner Losh wrote: : > In message: <[EMAIL PROTECTED]> : > : > Alexander Leidinger <[EMAIL PROTECTED]> writes: : > : I was

cvs commit: src/sys/i386/include clock.h src/sys/i386/i386 machdep.c src/sys/i386/isa clock.c src/sys/amd64/amd64 machdep.c src/sys/amd64/include clock.h src/sys/amd64/isa clock.c

2007-01-23 Thread Bruce Evans
bde 2007-01-23 08:01:20 UTC FreeBSD src repository Modified files: sys/i386/include clock.h sys/i386/i386machdep.c sys/i386/isa clock.c sys/amd64/amd64 machdep.c sys/amd64/includeclock.h sys/amd64/isaclock.c Log: C

cvs commit: src/sys/pc98/cbus clock.c src/sys/pc98/pc98 machdep.c

2007-01-23 Thread Bruce Evans
bde 2007-01-23 08:48:26 UTC FreeBSD src repository Modified files: sys/pc98/cbusclock.c sys/pc98/pc98machdep.c Log: Oops, pc98 is independent of i386 for clock.c and machdep.c but not for clock.h, so changing th i386 clock.h broke it. MFi386 (not teste

cvs commit: src/sys/nfs4client nfs4_vfs_subs.c src/sys/nfsclient nfs.h nfs_vfsops.c

2007-01-25 Thread Bruce Evans
bde 2007-01-25 13:07:25 UTC FreeBSD src repository Modified files: sys/nfs4client nfs4_vfs_subs.c sys/nfsclientnfs.h nfs_vfsops.c Log: Unstaticize nfs_iosize() in nfsclient and use it in nfs4client instead of duplicating it except for larger style bugs in

cvs commit: src/sys/nfs4client nfs4_vfsops.c

2007-01-25 Thread Bruce Evans
bde 2007-01-25 14:18:40 UTC FreeBSD src repository Modified files: sys/nfs4client nfs4_vfsops.c Log: Rename some functions and variables (mainly vfsops entry points) from nfs_* to nfs4_* to avoid collisions with nfsclient's names. Even static names should have a u

cvs commit: src/sys/nfs4client nfs4_vnops.c

2007-01-25 Thread Bruce Evans
bde 2007-01-25 14:33:13 UTC FreeBSD src repository Modified files: sys/nfs4client nfs4_vnops.c Log: Rename some functions and variables from nfs_* to nfs4_* to avoid collisions with nfsclient's names. Even static names should have a unique prefix so that they can b

Re: cvs commit: src/sys/ufs/ufs ufs_quota.c

2007-02-01 Thread Bruce Evans
On Thu, 1 Feb 2007, Mike Pritchard wrote: mpp 2007-02-01 01:01:57 UTC FreeBSD src repository Modified files: sys/ufs/ufs ufs_quota.c Log: Disallow negative UIDs when processing quotactl options. Er, uids are unsigned, so they cannot be negative. The function actually

Re: cvs commit: src/sys/amd64/include atomic.h src/sys/i386/include atomic.h

2007-02-03 Thread Bruce Evans
On Sat, 3 Feb 2007, Bjoern A. Zeeb wrote: On Wed, 27 Dec 2006, Bruce Evans wrote: bde 2006-12-27 20:26:00 UTC FreeBSD src repository Modified files: sys/amd64/includeatomic.h sys/i386/include atomic.h Log: ... Revision ChangesPath 1.40 +10 -10src

cvs commit: src/sys/amd64/include pcpu.h src/sys/i386/include pcpu.h

2007-02-06 Thread Bruce Evans
bde 2007-02-06 16:21:09 UTC FreeBSD src repository Modified files: sys/amd64/includepcpu.h sys/i386/include pcpu.h Log: Simplified PCPU_GET() and PCPU_SET(). We must copy through a temporary variable to avoid invalid constraints in dead code. Use an array of

cvs commit: src/sys/amd64/include pcpu.h src/sys/i386/include pcpu.h

2007-02-06 Thread Bruce Evans
bde 2007-02-06 18:04:02 UTC FreeBSD src repository Modified files: sys/amd64/includepcpu.h sys/i386/include pcpu.h Log: Fixed some style bugs. Routine except: - don't use __GNUCLIKE___OFFSETOF, since __offsetof() is a standard FreeBSD implementaion detail

cvs commit: src/sys/dev/bge if_bge.c if_bgereg.h

2007-02-15 Thread Bruce Evans
bde 2007-02-15 10:33:49 UTC FreeBSD src repository Modified files:(Branch: RELENG_4) sys/dev/bge if_bge.c if_bgereg.h Log: MFC (if_bge.c 1.108, etc., less some style bugs: eliminate one PCI read per call to bge_start()). In packet blasting tests using

cvs commit: src/contrib/gcc function.c

2007-02-18 Thread Bruce Evans
bde 2007-02-19 02:47:30 UTC FreeBSD src repository Modified files:(Branch: RELENG_6) contrib/gcc function.c Log: MFC (1.23: fix -mprofiler-epilogue by working around a garbage-collection problem). Revision ChangesPath 1.22.2.1 +3 -1 src/co

cvs commit: src/sys/conf kern.pre.mk

2007-02-24 Thread Bruce Evans
bde 2007-02-25 05:01:05 UTC FreeBSD src repository Modified files:(Branch: RELENG_6) sys/conf kern.pre.mk Log: MFC (1.80: style fixes and 1.81: fix high resolution kernel profiling). I was reminded of the brokenness when attempting to profile sendto()

cvs commit: src/sys/net route.c

2007-02-24 Thread Bruce Evans
bde 2007-02-25 05:36:25 UTC FreeBSD src repository Modified files:(Branch: RELENG_6) sys/net route.c Log: MFC (1.118: only bzero() data in rtalloc1() if the data is actually used). Revision ChangesPath 1.109.2.3 +2 -1 src/sys/net/route.

cvs commit: src/usr.bin/netstat if.c

2007-02-26 Thread Bruce Evans
bde 2007-02-27 04:54:33 UTC FreeBSD src repository Modified files: usr.bin/netstat if.c Log: Use a periodic itimer instead of repeated calls to alarm() in sidewaysintpr(). This increases the accuracy of the per-interval counts when they are interpreted as rates. Re

cvs commit: src/usr.bin/netstat if.c

2007-02-26 Thread Bruce Evans
bde 2007-02-27 05:10:36 UTC FreeBSD src repository Modified files: usr.bin/netstat if.c Log: Fixed some style bugs (whitespace lossage for removal of __P(()), and lots of naming and typing errors involving `interval'). Revision ChangesPath 1.68 +8 -8

cvs commit: src/sys/i386/isa clock.c

2007-03-05 Thread Bruce Evans
bde 2007-03-05 09:10:17 UTC FreeBSD src repository Modified files: sys/i386/isa clock.c Log: Partial fix for a bug in rev.1.231. If suspend/resume clobbers the RTC state, then it may clobber the RTC index register, so the index register must be restored before us

Re: cvs commit: src/usr.sbin/pkg_install/lib url.c

2007-03-06 Thread Bruce Evans
On Tue, 6 Mar 2007, Tim Kientzle wrote: ... GNU tar(1) implied the -p option for root, but BSD tar(1) doesn't do that. Hmmm... This might actually be considered a bsdtar bug. I'll look into it. That behavior of BSD tar(1) surprised me, to be honest. It's a trivial fix; there's already a c

Re: cvs commit: src/usr.sbin/pkg_install/lib url.c

2007-03-08 Thread Bruce Evans
On Wed, 7 Mar 2007, Tim Kientzle wrote: Tim Kientzle wrote: ... GNU tar(1) implied the -p option for root, but BSD tar(1) doesn't do that. Ah. Here we are: contrib/tar/src/extract.c, revision 1.2 (07-Jun-2002) by sobomax +#ifndef __FreeBSD__ same_permissions_option += we_are_root; same

Re: cvs commit: src/lib/libufs type.c

2007-03-16 Thread Bruce Evans
On Fri, 16 Mar 2007, Pawel Jakub Dawidek wrote: On Fri, Mar 16, 2007 at 03:13:29AM +, Pawel Jakub Dawidek wrote: pjd 2007-03-16 03:13:29 UTC FreeBSD src repository Modified files: lib/libufs type.c Log: The ufs_disk_fillout(3) can take special device name (wi

Re: cvs commit: src/sys/kern sched_ule.c

2007-03-17 Thread Bruce Evans
On Sat, 17 Mar 2007, Jeff Roberson wrote: Any language lawyers care to comment on this? Don't all C programmers know this? :) On Sat, 17 Mar 2007, Jeff Roberson wrote: jeff2007-03-17 18:13:33 UTC FreeBSD src repository Modified files: sys/kern sched_ule.c Log:

Re: cvs commit: src/sys/dev/mii brgphy.c

2007-03-19 Thread Bruce Evans
On Mon, 19 Mar 2007, Jung-uk Kim wrote: jkim2007-03-19 23:17:39 UTC FreeBSD src repository Modified files: sys/dev/mii brgphy.c Log: Revert couple of changes from 1.51 and 1.52. Reading link status with BMSR is okay for most of the chipsets but BCM5701 PHY does not se

Re: cvs commit: src/lib/libc/gen closedir.c

2007-12-03 Thread Bruce Evans
On Mon, 3 Dec 2007, [utf-8] Dag-Erling Sm??rgrav wrote: Olivier Houchard <[EMAIL PROTECTED]> writes: Log: Change the casts from (pthread_mutex_t *) to (void *) to keep gcc quiet. Anybody with a cleaner solution feel free to change it. Declare dd_lock as a struct pthread_mutex * instead

Re: cvs commit: src/sys/i386/i386 trap.c src/sys/amd64/amd64 trap.c

2007-12-04 Thread Bruce Evans
Do anyone have objection for this? Of course. Bruce ___ cvs-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/cvs-all To unsubscribe, send any mail to "[EMAIL PROTECTED]"

Re: cvs commit: src/sys/netinet/libalias alias_util.c

2007-12-04 Thread Bruce Evans
On Tue, 4 Dec 2007, Max Laier wrote: On Tuesday 04 December 2007, Alexey Dokuchaev wrote: On Mon, Dec 03, 2007 at 04:57:33PM -0500, John Baldwin wrote: On Monday 03 December 2007 10:24:52 am Dag-Erling Sm??rgrav wrote: John Birrell <[EMAIL PROTECTED]> writes: Log: Fix strict alias warnin

Re: cvs commit: src/sys/sys param.h

2007-12-06 Thread Bruce Evans
On Thu, 6 Dec 2007, Kip Macy wrote: kmacy 2007-12-06 04:00:59 UTC FreeBSD src repository Modified files: sys/sys param.h Log: Respect the fact that the value a may be constant so cast to const uint8_t * Revision ChangesPath 1.318 +2 -2 src/sys/sys/par

Re: cvs commit: src/lib/libc/gen closedir.c

2007-12-06 Thread Bruce Evans
On Thu, 6 Dec 2007, [utf-8] Dag-Erling Sm??rgrav wrote: Bruce Evans <[EMAIL PROTECTED]> writes: dd_lock cannot be declared as struct pthread_mutex *, since pthread_mutex is not permitted in the application namespace of dirent.h. dirent.h is (was) careful about namespaces. E.g., it kee

Re: cvs commit: src/sys/sys param.h

2007-12-06 Thread Bruce Evans
On Thu, 6 Dec 2007, Kip Macy wrote: kmacy 2007-12-06 19:11:15 UTC FreeBSD src repository Modified files: sys/sys param.h Log: Change uint8_t to equivalent base language type and add line break where 80 characters is exceeded in bit macros Requested by: Bruce Evans

Re: cvs commit: src/sys/sys param.h

2007-12-07 Thread Bruce Evans
On Thu, 6 Dec 2007, Christoph Mallon wrote: Kip Macy wrote: On Dec 6, 2007 2:25 AM, Bruce Evans <[EMAIL PROTECTED]> wrote: On Thu, 6 Dec 2007, Kip Macy wrote: kmacy 2007-12-06 04:00:59 UTC FreeBSD src repository Modified files: sys/sys param.h Log: Respe

cvs commit: src/lib/msun Makefile

2007-12-17 Thread Bruce Evans
bde 2007-12-17 13:20:38 UTC FreeBSD src repository Modified files: lib/msun Makefile Log: Don't try to build s_nanl.c before it is committed. Revision ChangesPath 1.83 +1 -1 src/lib/msun/Makefile __

cvs commit: src/lib/msun/amd64 s_logbl.S

2007-12-17 Thread Bruce Evans
bde 2007-12-17 18:12:06 UTC FreeBSD src repository Modified files: lib/msun/amd64 s_logbl.S Log: Translate from the i386 so that this compiles and runs. I hope that this and the i386 version of it will not be needed, but this is currently about 16 cycles or 36% f

cvs commit: src/lib/msun Makefile

2007-12-17 Thread Bruce Evans
bde 2007-12-17 18:21:23 UTC FreeBSD src repository Modified files: lib/msun Makefile Log: Oops, the previous commit was not needed -- the file was committed but not checked out due to my checkout error. Revision ChangesPath 1.84 +1 -1 src/l

Re: cvs commit: src/lib/msun Makefile

2007-12-17 Thread Bruce Evans
On Mon, 17 Dec 2007, David Schultz wrote: On Mon, Dec 17, 2007, Bruce Evans wrote: bde 2007-12-17 13:20:38 UTC FreeBSD src repository Modified files: lib/msun Makefile Log: Don't try to build s_nanl.c before it is committed. I got your email, bu

Re: cvs commit: src/lib/libc/stdtime localtime.c

2007-12-19 Thread Bruce Evans
On Wed, 19 Dec 2007, Warner Losh wrote: imp 2007-12-19 04:30:11 UTC FreeBSD src repository Modified files: lib/libc/stdtime localtime.c Log: Reduce lock contention for simple cases. # this really should be done with pthread_once, but I've debugged this code. Reviewed by:

Re: cvs commit: src/lib/libc/stdtime localtime.c

2007-12-19 Thread Bruce Evans
On Wed, 19 Dec 2007, Bruce Evans wrote: On Wed, 19 Dec 2007, Warner Losh wrote: imp 2007-12-19 04:30:11 UTC FreeBSD src repository Modified files: lib/libc/stdtime localtime.c Log: Reduce lock contention for simple cases. # this really should be done with pthread_once

Re: cvs commit: src/bin/mv mv.c

2007-12-27 Thread Bruce Evans
On Thu, 27 Dec 2007, Diomidis Spinellis wrote: dds 2007-12-27 11:33:42 UTC FreeBSD src repository Modified files: bin/mv mv.c Log: - Roll-back attempts to mimic rename(2) atomicity introduced in 1.47, and follow the letter of the POSIX specification. - Moving

Re: cvs commit: src/sys/netinet sctp_bsd_addr.c

2007-12-31 Thread Bruce Evans
On Mon, 31 Dec 2007, M. Warner Losh wrote: In message: <[EMAIL PROTECTED]> John Baldwin <[EMAIL PROTECTED]> writes: : The more correct fix though is to do a 'sched_prio()' at the start of the : thread's main loop to set the priority and then not adjust it via msleep(). : Kernel thr

Re: cvs commit: src/sys/netinet sctp_bsd_addr.c

2008-01-01 Thread Bruce Evans
On Tue, 1 Jan 2008, M. Warner Losh wrote: In message: <[EMAIL PROTECTED]> Bruce Evans <[EMAIL PROTECTED]> writes: : PZERO is compatibility cruft which should never be used. Just a few ... Do we want to document the other P priorities? Maybe, but not in sleep.9.

Re: cvs commit: src/lib/libarchive archive_endian.h archive_read_support_format_zip.c

2008-01-04 Thread Bruce Evans
On Fri, 4 Jan 2008, Tim Kientzle wrote: Dag-Erling Smorgrav wrote: ... 1.1 +142 -0src/lib/libarchive/archive_endian.h (new) 1.19 +22 -57src/lib/libarchive/archive_read_support_format_zip.c Is this really right? +be32dec(const void *pp) +{ + unsigned char const *

cvs commit: src/sys/kern vfs_vnops.c

2008-01-05 Thread Bruce Evans
bde 2008-01-05 08:54:52 UTC FreeBSD src repository Modified files: sys/kern vfs_vnops.c Log: In sequential_heuristic(): - spell 16384 as 16384 and not as BKVASIZE. 16384 is (not quite) just a magic size that works well in practice. BKVASIZE should be MAXBS

cvs commit: src/sys/amd64/include ieeefp.h

2008-01-09 Thread Bruce Evans
bde 2008-01-09 10:42:47 UTC FreeBSD src repository Modified files: sys/amd64/includeieeefp.h Log: Improve some comments. Revision ChangesPath 1.15 +13 -14src/sys/amd64/include/ieeefp.h ___ cvs-all@freebsd.or

cvs commit: src/sys/amd64/include ieeefp.h

2008-01-09 Thread Bruce Evans
bde 2008-01-09 13:35:31 UTC FreeBSD src repository Modified files: sys/amd64/includeieeefp.h Log: Fix some style bugs (mainly, use explicit shifts when accessing bit-fields even if the shift count happens to be 0, sort declarations, and spell __inline normally).

cvs commit: src/sys/amd64/include ieeefp.h

2008-01-09 Thread Bruce Evans
bde 2008-01-09 15:03:03 UTC FreeBSD src repository Modified files: sys/amd64/includeieeefp.h Log: Simplify the ifdefs: - fix this to compile with C++ by casting ints to enums in a few places and by using the correct parameter type for _fpsetprec(). Remove __cpl

cvs commit: src/sys/amd64/include ieeefp.h

2008-01-11 Thread Bruce Evans
bde 2008-01-11 17:11:32 UTC FreeBSD src repository Modified files: sys/amd64/includeieeefp.h Log: Fix fpset*() to not trap if there is a currently unmasked exception. Unmasked exceptions (which can be fixed up using fpset*() before they trap) are very rare, especially

cvs commit: src/sys/amd64/include ieeefp.h

2008-01-11 Thread Bruce Evans
bde 2008-01-11 14:11:46 UTC FreeBSD src repository Modified files: sys/amd64/includeieeefp.h Log: Fix some style bugs: - fix a previous style fix: shifts should be in the correct direction even if they are null. - restore a comment about namespace pollution from f

cvs commit: src/sys/i386/include ieeefp.h

2008-01-11 Thread Bruce Evans
bde 2008-01-11 17:54:20 UTC FreeBSD src repository Modified files: sys/i386/include ieeefp.h Log: MFamd64 1.15-1.18 (cosmetic changes, mainly to comments). The inline functions haven't been cleaned up here because the amd64 cleanups don't apply directly and the funct

cvs commit: src/sys/i386/include ieeefp.h

2008-01-11 Thread Bruce Evans
bde 2008-01-11 18:27:01 UTC FreeBSD src repository Modified files: sys/i386/include ieeefp.h Log: Separate fpresetsticky() from the other fpset functions so that the others can be replaced cleanly by the amd64 versions. There is no current amd64 version to merge, bu

cvs commit: src/sys/i386/include ieeefp.h

2008-01-11 Thread Bruce Evans
bde 2008-01-11 18:59:35 UTC FreeBSD src repository Modified files: sys/i386/include ieeefp.h Log: MFamd64 (everything possible up to 1.19; mainly the amd64 implementations of fpget*() and fpset*()). The i386 fpget*() were efficient but a bit obfuscated (using macro

Re: cvs commit: src/usr.sbin/iostat iostat.c

2008-01-13 Thread Bruce Evans
On Sun, 13 Jan 2008, [utf-8] Dag-Erling Sm??rgrav wrote: David Schultz <[EMAIL PROTECTED]> writes: I don't think it always does set errno, but you can set errno to 0 before the call, and you'll get either the real error message from err() or "unknown error". If asprintf() fails because malloc

cvs commit: src/sys/amd64/include _types.h float.h

2008-01-17 Thread Bruce Evans
bde 2008-01-17 13:12:47 UTC FreeBSD src repository Modified files: sys/amd64/include_types.h float.h Log: Translate from the i386. All FP constants and operations are evaluated in the range and precision of their type(s) on amd64, but FLT_EVAL_METHOD said that they w

cvs commit: src/lib/libc/amd64 _fpmath.h src/lib/libc/i386 _fpmath.h src/lib/libc/ia64 _fpmath.h

2008-01-17 Thread Bruce Evans
bde 2008-01-17 16:39:07 UTC FreeBSD src repository Modified files: lib/libc/amd64 _fpmath.h lib/libc/i386_fpmath.h lib/libc/ia64_fpmath.h Log: Add an alternative view of the bits in an 80-bit long double (64+16 instead of 32+32+15+1) on all a

cvs commit: src/lib/msun/src math_private.h

2008-01-17 Thread Bruce Evans
bde 2008-01-17 17:02:11 UTC FreeBSD src repository Modified files: lib/msun/src math_private.h Log: Add a macro STRICT_ASSIGN() to help avoid the compiler bug that assignments and casts don't clip extra precision, if any. The implementation is to assign to a temp

cvs commit: src/lib/msun/src s_rint.c s_rintf.c

2008-01-19 Thread Bruce Evans
bde 2008-01-19 16:37:57 UTC FreeBSD src repository Modified files: lib/msun/src s_rint.c s_rintf.c Log: Use STRICT_ASSIGN() instead of assorted direct volatile hacks to work around assignments not working for gcc on i386. Now volatile hacks for rint() and rintf()

cvs commit: src/lib/msun/src s_log1p.c s_log1pf.c

2008-01-19 Thread Bruce Evans
bde 2008-01-19 18:13:21 UTC FreeBSD src repository Modified files: lib/msun/src s_log1p.c s_log1pf.c Log: Use STRICT_ASSIGN() for log1pf() and log1p() instead of a volatile cast hack for log1pf() only. The cast hack broke with gcc-4, resulting in ~1 million error

cvs commit: src/lib/msun/src k_rem_pio2.c k_rem_pio2f.c

2008-01-19 Thread Bruce Evans
bde 2008-01-19 20:02:55 UTC FreeBSD src repository Modified files: lib/msun/src k_rem_pio2.c k_rem_pio2f.c Log: Use STRICT_ASSIGN() for _kernel_rem_pio2f() and _kernel_rem_pio2f() instead of a volatile cast hack for the float version only. The cast hack broke wit

cvs commit: src/lib/msun/src s_exp2.c s_exp2f.c

2008-01-19 Thread Bruce Evans
bde 2008-01-19 21:37:14 UTC FreeBSD src repository Modified files: lib/msun/src s_exp2.c s_exp2f.c Log: Use STRICT_ASSIGN() for exp2f() and exp2() instead of a volatile variable hack for exp2f() only. The volatile variable had a surprisingly large cost for exp2

cvs commit: src/lib/msun/src math_private.h

2008-01-19 Thread Bruce Evans
bde 2008-01-19 22:05:14 UTC FreeBSD src repository Modified files: lib/msun/src math_private.h Log: Do an ordinary assignment in STRICT_ASSIGN() except for floats until there is a problem with non-floats (when i386 defaults to extra precision). This essentially r

cvs commit: src/lib/msun/src k_rem_pio2f.c

2008-01-19 Thread Bruce Evans
bde 2008-01-19 22:50:50 UTC FreeBSD src repository Removed files: lib/msun/src k_rem_pio2f.c Log: Remove the float version of the kernel of arg reduction for pi/2, since it should never have existed and it has not been used for many years (floats are reduced faste

cvs commit: src/lib/msun/src k_rem_pio2.c

2008-01-19 Thread Bruce Evans
bde 2008-01-20 04:09:44 UTC FreeBSD src repository Modified files: lib/msun/src k_rem_pio2.c Log: Oops, when merging from the float version to the double versions, don't forget to translate "float" to "double". ucbtest didn't detect the bug, but exhaustive test

cvs commit: src/lib/msun/src s_log1pf.c

2008-01-21 Thread Bruce Evans
bde 2008-01-21 13:46:21 UTC FreeBSD src repository Modified files: lib/msun/src s_log1pf.c Log: Fix cutoffs. This is just a cleanup and an optimization for unusual cases which are used mainly by regression tests. As usual, the cutoff for tiny args was not corr

cvs commit: src/lib/msun/src k_cos.c

2008-01-25 Thread Bruce Evans
bde 2008-01-25 21:09:21 UTC FreeBSD src repository Modified files: lib/msun/src k_cos.c Log: Fix a harmless type error in 1.9. Revision ChangesPath 1.11 +1 -1 src/lib/msun/src/k_cos.c ___ cvs-all@freebs

Re: cvs commit: src UPDATING src/include fts.h src/lib/libc/gen Makefile.inc Symbol.map fts-compat.c fts-compat.h fts.3 fts.c src/sys/sys param.h

2008-01-28 Thread Bruce Evans
On Mon, 28 Jan 2008, Yar Tikhiy wrote: On Mon, Jan 28, 2008 at 08:28:27AM -1000, Juli Mallett wrote: * Yar Tikhiy <[EMAIL PROTECTED]> [ 2008-01-28 ] [ Re: cvs commit: src UPDATING src/include fts.h src/lib/libc/gen Makefile.inc Symbol.map fts-compat.c fts-compat.h fts.3 fts.c src/sys/s

cvs commit: src/lib/msun/src e_expf.c

2008-02-05 Thread Bruce Evans
bde 2008-02-06 06:35:21 UTC FreeBSD src repository Modified files: lib/msun/src e_expf.c Log: As for the float trig functions and logf, use a minimax polynomial that is specialized for float precision. The new polynomial has degree 5 instead of 11, and a maximum

cvs commit: src/lib/msun/ld128 s_exp2l.c src/lib/msun/ld80 s_exp2l.c src/lib/msun/src e_exp.c e_expf.c s_exp2.c s_exp2f.c

2008-02-06 Thread Bruce Evans
bde 2008-02-07 03:17:05 UTC FreeBSD src repository Modified files: lib/msun/ld128 s_exp2l.c lib/msun/ld80s_exp2l.c lib/msun/src e_exp.c e_expf.c s_exp2.c s_exp2f.c Log: Use a better method of scaling by 2**k. Instead of adding to the exponen

cvs commit: src/lib/msun/src s_expm1.c s_expm1f.c

2008-02-07 Thread Bruce Evans
bde 2008-02-07 09:42:19 UTC FreeBSD src repository Modified files: lib/msun/src s_expm1.c s_expm1f.c Log: Use a better method of scaling by 2**k. Instead of adding to the exponent bits of the reduced result, construct 2**k (hopefully in parallel with the construc

cvs commit: src/lib/msun/src s_logb.c

2008-02-07 Thread Bruce Evans
bde 2008-02-08 01:22:13 UTC FreeBSD src repository Modified files: lib/msun/src s_logb.c Log: Oops, fix the fix in rev.1.10. logb() and logbf() were broken on denormals, and logb() remained broken after 1.10 because the fix for logbf() was incompletely translated

cvs commit: src/lib/msun/src s_truncl.c

2008-02-07 Thread Bruce Evans
bde 2008-02-08 01:45:52 UTC FreeBSD src repository Modified files: lib/msun/src s_truncl.c Log: Fix truncl() when the result should be -0.0L. When the result is +-0.0L, it must have the same sign as the arg in all rounding modes, but it was always +0.0L. Rev

cvs commit: src/lib/msun/src s_expm1.c

2008-02-09 Thread Bruce Evans
bde 2008-02-09 10:36:08 UTC FreeBSD src repository Modified files: lib/msun/src s_expm1.c Log: Fix a comment about coefficients and expand a related one. Revision ChangesPath 1.9 +2 -2 src/lib/msun/src/s_expm1.c _

cvs commit: src/lib/msun/src s_expm1f.c

2008-02-09 Thread Bruce Evans
bde 2008-02-09 12:53:15 UTC FreeBSD src repository Modified files: lib/msun/src s_expm1f.c Log: As usual, use a minimax polynomial that is specialized for float precision. The new polynomial has degree 4 instead of 10, and a maximum error of 2**-30.04 ulps instea

cvs commit: src/lib/msun/src s_exp2f.c

2008-02-10 Thread Bruce Evans
bde 2008-02-11 05:20:02 UTC FreeBSD src repository Modified files: lib/msun/src s_exp2f.c Log: Use double precision for z and thus for the entire calculation of exp2(i/TBLSIZE) * p(z) instead of only for the final multiplication and addition. This fixes the code

cvs commit: src/lib/msun/amd64 Makefile.inc e_remainder.S e_remainderf.S

2008-02-12 Thread Bruce Evans
bde 2008-02-13 06:01:49 UTC FreeBSD src repository Modified files: lib/msun/amd64 Makefile.inc Added files: lib/msun/amd64 e_remainder.S e_remainderf.S Log: Use hardware remainder on amd64 since it is 5 to 10 times faster than software remainder and is al

cvs commit: src/lib/msun/src e_remainder.c e_remainderf.c

2008-02-12 Thread Bruce Evans
bde 2008-02-12 17:11:36 UTC FreeBSD src repository Modified files: lib/msun/src e_remainder.c e_remainderf.c Log: Fix remainder() and remainderf() in round-towards-minus-infinity mode when the result is +-0. IEEE754 requires (in all rounding modes) that if the re

cvs commit: src/lib/msun/src s_exp2f.c

2008-02-13 Thread Bruce Evans
bde 2008-02-13 08:36:13 UTC FreeBSD src repository Modified files: lib/msun/src s_exp2f.c Log: Rearrange the polynomial evaluation for better parallelism. This is faster on all machines tested (old Celeron (P2), A64 (amd64 and i386) and ia64) except on ia64 when

cvs commit: src/lib/msun/src s_exp2.c s_exp2f.c src/lib/msun/ld128 s_exp2l.c src/lib/msun/ld80 s_exp2l.c

2008-02-13 Thread Bruce Evans
bde 2008-02-13 10:44:45 UTC FreeBSD src repository Modified files: lib/msun/src s_exp2f.c s_exp2.c lib/msun/ld128 s_exp2l.c lib/msun/ld80s_exp2l.c Log: Fix exp2*(x) on signaling NaNs by returning x+x as usual. This has the side effect of c

cvs commit: src/lib/msun/src s_ceill.c

2008-02-13 Thread Bruce Evans
bde 2008-02-13 15:22:53 UTC FreeBSD src repository Modified files: lib/msun/src s_ceill.c Log: Fix the C version of ceill(x) for -1 < x <= -0 in all rounding modes. The result should be -0, but was +0. Revision ChangesPath 1.5 +1 -1 src/lib/ms

cvs commit: src/lib/msun/src s_ceill.c s_floorl.c s_truncl.c src/lib/msun Makefile

2008-02-13 Thread Bruce Evans
bde 2008-02-13 16:56:52 UTC FreeBSD src repository Modified files: lib/msun/src s_ceill.c s_floorl.c s_truncl.c lib/msun Makefile Log: On arches where long double is the same as double, alias ceil(), floor() and trunc() to the corresponding long dou

cvs commit: src/lib/msun/src s_ceill.c s_floorl.c s_truncl.c

2008-02-13 Thread Bruce Evans
bde 2008-02-13 17:38:17 UTC FreeBSD src repository Modified files: lib/msun/src s_ceill.c s_floorl.c s_truncl.c Log: s_ceill.c s_floorl.c s_truncl.c Revision ChangesPath 1.7 +2 -3 src/lib/msun/src/s_ceill.c 1.6 +2 -3 src/lib/msu

cvs commit: src/lib/msun/src s_ceill.c s_floorl.c s_truncl.c

2008-02-13 Thread Bruce Evans
bde 2008-02-13 18:16:43 UTC FreeBSD src repository Modified files: lib/msun/src s_ceill.c s_floorl.c s_truncl.c Log: Forced commit to note that the lost log message for the previous commit said that the previous commit was almost a null forced commit too. It just

cvs commit: src/lib/msun/src e_pow.c e_powf.c

2008-02-14 Thread Bruce Evans
bde 2008-02-14 09:42:24 UTC FreeBSD src repository Modified files: lib/msun/src e_pow.c e_powf.c Log: Use the expression (x+0.0)-(y+0.0) instead of x+y when mixing NaN arg(s). This uses 2 tricks to improve consistency so that more serious problems aren't hidden in

cvs commit: src/lib/msun/src e_powf.c

2008-02-14 Thread Bruce Evans
bde 2008-02-14 10:23:51 UTC FreeBSD src repository Modified files: lib/msun/src e_powf.c Log: Fix the hi+lo decomposition for 2/(3ln2). The decomposition needs to be into 12+24 bits of precision for extra-precision multiplication, but was into 13+24 bits. On i38

cvs commit: src/lib/msun/src e_pow.c e_powf.c

2008-02-14 Thread Bruce Evans
bde 2008-02-14 12:56:35 UTC FreeBSD src repository Modified files: lib/msun/src e_pow.c e_powf.c Log: Forced commit to note that the minus sign in the fancy expression (x+0.0)-(y+0.0) for mixing NaNs documented in a previous log message didn't actually get committ

cvs commit: src/lib/msun/src e_hypot.c e_hypotf.c

2008-02-14 Thread Bruce Evans
bde 2008-02-14 13:44:03 UTC FreeBSD src repository Modified files: lib/msun/src e_hypot.c e_hypotf.c Log: Use the expression fabs(x+0.0)+fabs(y+0.0) instad of a+b (where a is |x| or |y| and b is |y| or |x|) when mixing NaN arg(s). hypot*() had its own foot shoo

<    1   2   3   4   5   6   >