Re: svn commit: r225586 - in head/sys: modules/netgraph/ipfw netgraph

2011-10-08 Thread Gleb Smirnoff
Alexander, Andrey, see a couple of comments below please. On Thu, Sep 15, 2011 at 12:28:17PM +, Andrey V. Elsukov wrote: A> Author: ae A> Date: Thu Sep 15 12:28:17 2011 A> New Revision: 225586 A> URL: http://svn.freebsd.org/changeset/base/225586 A> A> Log: A> Add IPv6 support to the ng

Re: svn commit: r225868 - head/bin/ps

2011-10-08 Thread Alexander Best
On Sat Oct 1 11, Ed Schouten wrote: > * Alexander Best , 20110929 14:14: > > i think i wrote you about the two spaces between the "TT" and "TIME" field > > some time ago. you said that the "TT" entry can contain a postfix "-". can > > you describe a situation, where this can occur? i've never seen

svn commit: r226165 - head/sys/conf

2011-10-08 Thread Marius Strobl
Author: marius Date: Sat Oct 8 21:15:39 2011 New Revision: 226165 URL: http://svn.freebsd.org/changeset/base/226165 Log: Remove inphy(4), exphy(4) as well as ruephy(4) that no longer existed as a separate config option, which was missed in r226154. Modified: head/sys/conf/NOTES Modified:

Re: svn commit: r226162 - head/share/mk

2011-10-08 Thread Ed Schouten
* Chris Rees , 20111008 20:25: > Revert unapproved commit to bsd.port.mk. You're very nice... :-) -- Ed Schouten WWW: http://80386.nl/ pgpnv11pU3IHJ.pgp Description: PGP signature

svn commit: r226164 - head/usr.bin/kdump

2011-10-08 Thread Dag-Erling Smorgrav
Author: des Date: Sat Oct 8 21:02:08 2011 New Revision: 226164 URL: http://svn.freebsd.org/changeset/base/226164 Log: Address some of bde@'s concerns with the new code. Modified: head/usr.bin/kdump/kdump.c Modified: head/usr.bin/kdump/kdump.c

Re: svn commit: r226162 - head/share/mk

2011-10-08 Thread Chris Rees
On 8 Oct 2011 20:35, "Doug Barton" wrote: > > On 10/08/2011 11:25, Chris Rees wrote: > > Author: crees (ports committer) > > Date: Sat Oct 8 18:25:01 2011 > > New Revision: 226162 > > URL: http://svn.freebsd.org/changeset/base/226162 > > > > Log: > > Revert unapproved commit to bsd.port.mk. > >

Re: svn commit: r226162 - head/share/mk

2011-10-08 Thread Steve Kargl
On Sat, Oct 08, 2011 at 12:34:44PM -0700, Doug Barton wrote: > On 10/08/2011 11:25, Chris Rees wrote: > > Author: crees (ports committer) > > Date: Sat Oct 8 18:25:01 2011 > > New Revision: 226162 > > URL: http://svn.freebsd.org/changeset/base/226162 > > > > Log: > > Revert unapproved commit to

Re: svn commit: r226162 - head/share/mk

2011-10-08 Thread Doug Barton
On 10/08/2011 11:25, Chris Rees wrote: > Author: crees (ports committer) > Date: Sat Oct 8 18:25:01 2011 > New Revision: 226162 > URL: http://svn.freebsd.org/changeset/base/226162 > > Log: > Revert unapproved commit to bsd.port.mk. This is completely unacceptable. We don't do "commit wars" in

Re: svn commit: r226122 - head/share/mk

2011-10-08 Thread Kostik Belousov
On Sat, Oct 08, 2011 at 12:25:16PM -0700, Doug Barton wrote: > On 10/08/2011 08:23, Kostik Belousov wrote: > > On Sat, Oct 08, 2011 at 08:02:43AM -0700, Stanislav Sedov wrote: > >> On Oct 8, 2011, at 12:33 AM, Ed Schouten wrote: > >> > >>> Hi, > &g

Re: svn commit: r226122 - head/share/mk

2011-10-08 Thread Doug Barton
On 10/08/2011 08:23, Kostik Belousov wrote: > On Sat, Oct 08, 2011 at 08:02:43AM -0700, Stanislav Sedov wrote: >> On Oct 8, 2011, at 12:33 AM, Ed Schouten wrote: >> >>> Hi, >>> >>> * Stanislav Sedov , 20111008 01:43: >>>> - ${WRKSRC

svn commit: r226163 - head/sys/kern

2011-10-08 Thread Alan Cox
Author: alc Date: Sat Oct 8 18:29:30 2011 New Revision: 226163 URL: http://svn.freebsd.org/changeset/base/226163 Log: Fix the handling of an empty kmem map by sysctl_kmem_map_free(). In the unlikely event that sysctl_kmem_map_free() was performed on an empty kmem map, it would incorrectly

svn commit: r226162 - head/share/mk

2011-10-08 Thread Chris Rees
Author: crees (ports committer) Date: Sat Oct 8 18:25:01 2011 New Revision: 226162 URL: http://svn.freebsd.org/changeset/base/226162 Log: Revert unapproved commit to bsd.port.mk. This would have had more discussion, but it was explicitly rejected at submission by portmgr: http://list

svn commit: r226161 - head/usr.sbin/bsdinstall/partedit

2011-10-08 Thread Nathan Whitehorn
Author: nwhitehorn Date: Sat Oct 8 16:45:03 2011 New Revision: 226161 URL: http://svn.freebsd.org/changeset/base/226161 Log: Avoid magicking into existence sub-partitions due to leftover blocks when creating new ones by destroying any geom that may have come into existence immediately after

Re: svn commit: r226153 - head/usr.bin/kdump

2011-10-08 Thread Bruce Evans
On Sat, 8 Oct 2011, Dag-Erling Smorgrav wrote: Log: I appreciate the logic behind using a (void) cast to indicate that the return value is intentionally ignored, but frankly, all it does is get in the way of the code. It would be more useful if used precisely when ignoring the return value

Re: svn commit: r226122 - head/share/mk

2011-10-08 Thread Kostik Belousov
On Sat, Oct 08, 2011 at 08:02:43AM -0700, Stanislav Sedov wrote: > On Oct 8, 2011, at 12:33 AM, Ed Schouten wrote: > > > Hi, > > > > * Stanislav Sedov , 20111008 01:43: > >> - ${WRKSRC} might be missing when the autotools fixup is running. > >>

Re: svn commit: r226122 - head/share/mk

2011-10-08 Thread Stanislav Sedov
On Oct 8, 2011, at 12:33 AM, Ed Schouten wrote: > Hi, > > * Stanislav Sedov , 20111008 01:43: >> - ${WRKSRC} might be missing when the autotools fixup is running. >>Account for this. > > Maybe we should simplify this a bit? Sounds good! I didn't think abou

Re: svn commit: r226090 - head/sys/sys

2011-10-08 Thread David O'Brien
On Fri, Oct 07, 2011 at 02:14:46PM +0300, Andriy Gapon wrote: > > Increase MSGBUF_SIZE. > > The previous size lead to truncated /var/run/dmesg.boot when booted with > > "-v". ... > Heck, I am going to five! :-) Do you know of a machine that needs "* 5"? I went with the miniumum increase that

svn commit: r226160 - head/usr.sbin/bsdinstall/partedit

2011-10-08 Thread Nathan Whitehorn
Author: nwhitehorn Date: Sat Oct 8 14:59:12 2011 New Revision: 226160 URL: http://svn.freebsd.org/changeset/base/226160 Log: Usability enhancements: do not allow setting a mountpoint on bsdlabel container partitions, which didn't do anything anyway, and check for an existing freebsd-boot pa

Re: svn commit: r226089 - in head: share/man/man7 sys/kern

2011-10-08 Thread David O'Brien
On Fri, Oct 07, 2011 at 02:14:08PM +0300, Andriy Gapon wrote: > > Log: > > Disallow various debug.kdb sysctl's when securelevel is raised. .. > Maybe the same would also be a good idea for sysctls defined at the top of > sys/kern/kern_shutdown.c? Thanks for pointing those out. I'll take a look.

svn commit: r226159 - head/usr.bin/kdump

2011-10-08 Thread Dag-Erling Smorgrav
Author: des Date: Sat Oct 8 13:01:38 2011 New Revision: 226159 URL: http://svn.freebsd.org/changeset/base/226159 Log: Teach kdump(1) to decode capability bitmasks. MFC after:3 weeks Modified: head/usr.bin/kdump/kdump.c head/usr.bin/kdump/mksubr Modified: head/usr.bin/kdump/kdump.

svn commit: r226158 - head/usr.bin/kdump

2011-10-08 Thread Dag-Erling Smorgrav
Author: des Date: Sat Oct 8 12:59:41 2011 New Revision: 226158 URL: http://svn.freebsd.org/changeset/base/226158 Log: Fix the dependency issue properly by a) moving kdump_subr.c to the front of the SRCS list and b) listing kdump_subr.h in DPSRCS. Modified: head/usr.bin/kdump/Makefile Modi

svn commit: r226157 - head/usr.bin/kdump

2011-10-08 Thread Dag-Erling Smorgrav
Author: des Date: Sat Oct 8 12:47:00 2011 New Revision: 226157 URL: http://svn.freebsd.org/changeset/base/226157 Log: Bring ioctlname() in line with all the other *name() functions, which actually print the name (or the numeric value, if they can't figure out the correct name) instead of ju

svn commit: r226156 - head/libexec/rtld-elf/i386

2011-10-08 Thread Konstantin Belousov
Author: kib Date: Sat Oct 8 12:42:19 2011 New Revision: 226156 URL: http://svn.freebsd.org/changeset/base/226156 Log: Handle the R_386_TLS_TPOFF32 relocation, which is similar to R_386_TLS_TPOFF, but with negative relocation value. Found by: mpfr test suite, pointed to by ale Revie

svn commit: r226155 - head/libexec/rtld-elf

2011-10-08 Thread Konstantin Belousov
Author: kib Date: Sat Oct 8 12:39:47 2011 New Revision: 226155 URL: http://svn.freebsd.org/changeset/base/226155 Log: Setting up TLS block for the main thread must be done after the relocations are processed, since tls initialization section might be itself subject for relocations. Only set

svn commit: r226154 - in head/sys: conf dev/fxp dev/mii dev/usb/net dev/xl modules/fxp modules/mii modules/usb/rue modules/xl

2011-10-08 Thread Marius Strobl
Author: marius Date: Sat Oct 8 12:33:10 2011 New Revision: 226154 URL: http://svn.freebsd.org/changeset/base/226154 Log: - Follow the lead of dcphy(4) and pnphy(4) and move the reminder of the PHY drivers that only ever attach to a particular MAC driver, i.e. inphy(4), ruephy(4) and xlp

svn commit: r226153 - head/usr.bin/kdump

2011-10-08 Thread Dag-Erling Smorgrav
Author: des Date: Sat Oct 8 12:28:06 2011 New Revision: 226153 URL: http://svn.freebsd.org/changeset/base/226153 Log: I appreciate the logic behind using a (void) cast to indicate that the return value is intentionally ignored, but frankly, all it does is get in the way of the code. Al

svn commit: r226151 - head/usr.bin/kdump

2011-10-08 Thread Dag-Erling Smorgrav
Author: des Date: Sat Oct 8 12:21:51 2011 New Revision: 226151 URL: http://svn.freebsd.org/changeset/base/226151 Log: Fix casting. Modified: head/usr.bin/kdump/kdump.c head/usr.bin/kdump/mksubr Modified: head/usr.bin/kdump/kdump.c ==

svn commit: r226150 - head/usr.bin/kdump

2011-10-08 Thread Dag-Erling Smorgrav
Author: des Date: Sat Oct 8 12:10:16 2011 New Revision: 226150 URL: http://svn.freebsd.org/changeset/base/226150 Log: Whitespace. Modified: head/usr.bin/kdump/kdump.c Modified: head/usr.bin/kdump/kdump.c == --- head

svn commit: r226149 - head/sys/dev/siba

2011-10-08 Thread Christian Brueffer
Author: brueffer Date: Sat Oct 8 12:09:57 2011 New Revision: 226149 URL: http://svn.freebsd.org/changeset/base/226149 Log: Fix an infinite loop in siba_bwn_suspend(). CID: 3536 Found with: Coverity Prevent(tm) MFC after:1 week Modified: head/sys/dev/siba/siba_bwn.c M

svn commit: r226148 - head/usr.bin/kdump

2011-10-08 Thread Dag-Erling Smorgrav
Author: des Date: Sat Oct 8 11:51:48 2011 New Revision: 226148 URL: http://svn.freebsd.org/changeset/base/226148 Log: C has had swicth statements for 40 years or so. It's about time we started using them. Modified: head/usr.bin/kdump/kdump.c Modified: head/usr.bin/kdump/kdump.c =

svn commit: r226147 - head/usr.bin/kdump

2011-10-08 Thread Dag-Erling Smorgrav
Author: des Date: Sat Oct 8 11:39:00 2011 New Revision: 226147 URL: http://svn.freebsd.org/changeset/base/226147 Log: Sort and line up. Modified: head/usr.bin/kdump/mksubr Modified: head/usr.bin/kdump/mksubr == ---

svn commit: r226146 - head/sys/dev/bwn

2011-10-08 Thread Christian Brueffer
Author: brueffer Date: Sat Oct 8 10:50:48 2011 New Revision: 226146 URL: http://svn.freebsd.org/changeset/base/226146 Log: Remove dead code, "error" doesn't change between this check and the previous one. CID: 3254 Found with: Coverity Prevent(tm) MFC after:1 week Mod

svn commit: r226145 - head/usr.bin/kdump

2011-10-08 Thread Dag-Erling Smorgrav
Author: des Date: Sat Oct 8 09:57:29 2011 New Revision: 226145 URL: http://svn.freebsd.org/changeset/base/226145 Log: 1) Some of the #defines or enums for which we auto-generate naming functions may be wider than int, so use intmax_t throughout. Also add missing casts in printf() cal

svn commit: r226143 - head/sys/security/mac_mls

2011-10-08 Thread Christian Brueffer
Author: brueffer Date: Sat Oct 8 09:14:18 2011 New Revision: 226143 URL: http://svn.freebsd.org/changeset/base/226143 Log: Remove two dublicated assignments. CID: 9870 Found with: Coverity Prevent(tm) Confirmed by: rwatson MFC after:1 week Modified: head/sys/securit

Re: svn commit: r226141 - head/usr.bin/calendar

2011-10-08 Thread Dag-Erling Smørgrav
Ed Schouten writes: > Log: > Remove extraneous WARNS=7. > > WARNS above 6 has no use. Also, all of usr.bin is also built with > WARNS=6 by default. This one goes to seven. DES -- Dag-Erling Smørgrav - d...@des.no ___ svn-src-all@freebsd.org m

Re: svn commit: r226122 - head/share/mk

2011-10-08 Thread Ed Schouten
Hi, * Stanislav Sedov , 20111008 01:43: > - ${WRKSRC} might be missing when the autotools fixup is running. > Account for this. Maybe we should simplify this a bit? Index: bsd.port.mk === --- bsd.port.mk (revision

svn commit: r226141 - head/usr.bin/calendar

2011-10-08 Thread Ed Schouten
Author: ed Date: Sat Oct 8 07:20:12 2011 New Revision: 226141 URL: http://svn.freebsd.org/changeset/base/226141 Log: Remove extraneous WARNS=7. WARNS above 6 has no use. Also, all of usr.bin is also built with WARNS=6 by default. Discussed with: edwin Modified: head/usr.bin