Re: svn commit: r250435 - head/sys/boot/fdt/dts

2013-05-10 Thread Oleksandr Tymoshenko
On 2013-05-09, at 10:34 PM, Tim Kientzle kient...@freebsd.org wrote: Author: kientzle Date: Fri May 10 05:34:08 2013 New Revision: 250435 URL: http://svnweb.freebsd.org/changeset/base/250435 Log: Move 'compatible' line out of the common am335x.dtsi and into the beaglebone-specific

svn commit: r250436 - head/sys/contrib/octeon-sdk

2013-05-10 Thread Warner Losh
Author: imp Date: Fri May 10 06:28:01 2013 New Revision: 250436 URL: http://svnweb.freebsd.org/changeset/base/250436 Log: Make the read/write routines default to the newer DS1337 part, but keep the supported bits board specific. Modified: head/sys/contrib/octeon-sdk/cvmx-rtc.h Modified:

Re: svn commit: r250430 - head/sbin/dmesg

2013-05-10 Thread Alexey Dokuchaev
On Fri, May 10, 2013 at 03:42:48AM +, Eitan Adler wrote: New Revision: 250430 URL: http://svnweb.freebsd.org/changeset/base/250430 @@ -79,15 +80,20 @@ main(int argc, char *argv[]) kvm_t *kd; size_t buflen, bufpos; long pri; - int all, ch; + int ch, clear; +

Re: svn commit: r250431 - head/usr.bin/xargs

2013-05-10 Thread Juli Mallett
Eitan, Are other changes coming to xargs for which these style changes make sense? If not, I'd say that style changes which only adjust vertical whitespace, especially those which mostly add vertical whitespace at the start of functions with no local variables (which is one of the most archaic

svn commit: r250441 - head/sys/dev/netmap

2013-05-10 Thread Luigi Rizzo
Author: luigi Date: Fri May 10 08:46:10 2013 New Revision: 250441 URL: http://svnweb.freebsd.org/changeset/base/250441 Log: another minor bugfix in the memory allocator, this time in the free routine. Modified: head/sys/dev/netmap/netmap_mem2.c Modified: head/sys/dev/netmap/netmap_mem2.c

svn commit: r250442 - head/sys/net80211

2013-05-10 Thread Adrian Chadd
Author: adrian Date: Fri May 10 09:37:58 2013 New Revision: 250442 URL: http://svnweb.freebsd.org/changeset/base/250442 Log: Fix a VAP BSS node reference in the HT code to actually take a reference before using said node. The blessed way here is to take a node reference before

svn commit: r250443 - head/usr.sbin/ctladm

2013-05-10 Thread Jaakko Heinonen
Author: jh Date: Fri May 10 09:58:32 2013 New Revision: 250443 URL: http://svnweb.freebsd.org/changeset/base/250443 Log: Check the return value of sbuf_finish(). MFC after:1 week Modified: head/usr.sbin/ctladm/ctladm.c Modified: head/usr.sbin/ctladm/ctladm.c

svn commit: r250444 - head/sys/dev/ath

2013-05-10 Thread Adrian Chadd
Author: adrian Date: Fri May 10 10:06:45 2013 New Revision: 250444 URL: http://svnweb.freebsd.org/changeset/base/250444 Log: Make sure the holding descriptor and link pointer are both freed during a non-loss reset. When the drain functions are called, the holding descriptor and link

Re: svn commit: r250431 - head/usr.bin/xargs

2013-05-10 Thread smckay
On Fri 10/05/13 17:50 , Juli Mallett jmall...@freebsd.org sent: Eitan, Are other changes coming to xargs for which these style changes make sense? If not, I'd say that style changes which only adjust vertical whitespace, especially those which mostly add vertical whitespace at

Re: svn commit: r250430 - head/sbin/dmesg

2013-05-10 Thread Eitan Adler
On 10 May 2013 03:42, Alexey Dokuchaev da...@freebsd.org wrote: On Fri, May 10, 2013 at 03:42:48AM +, Eitan Adler wrote: New Revision: 250430 URL: http://svnweb.freebsd.org/changeset/base/250430 @@ -79,15 +80,20 @@ main(int argc, char *argv[]) kvm_t *kd; size_t buflen,

svn commit: r250451 - in head: sbin/dmesg usr.bin/split

2013-05-10 Thread Eitan Adler
Author: eadler Date: Fri May 10 12:49:16 2013 New Revision: 250451 URL: http://svnweb.freebsd.org/changeset/base/250451 Log: Bump .Dd for recent content change. Reported by: delphij Modified: head/sbin/dmesg/dmesg.8 head/usr.bin/split/split.1 Modified: head/sbin/dmesg/dmesg.8

Re: svn commit: r250430 - head/sbin/dmesg

2013-05-10 Thread Alexey Dokuchaev
On Fri, May 10, 2013 at 08:43:37AM -0400, Eitan Adler wrote: On 10 May 2013 03:42, Alexey Dokuchaev da...@freebsd.org wrote: On Fri, May 10, 2013 at 03:42:48AM +, Eitan Adler wrote: New Revision: 250430 URL: http://svnweb.freebsd.org/changeset/base/250430 (void)

svn commit: r250453 - head/etc

2013-05-10 Thread Eitan Adler
Author: eadler Date: Fri May 10 13:57:44 2013 New Revision: 250453 URL: http://svnweb.freebsd.org/changeset/base/250453 Log: Bring /etc/protocols up to date. PR: conf/175397 Submitted by: ak Modified: head/etc/protocols Modified: head/etc/protocols

Re: svn commit: r250411 - in head/sys: conf kern sys

2013-05-10 Thread John Baldwin
On Thursday, May 09, 2013 4:56:33 pm Marcel Moolenaar wrote: On May 9, 2013, at 9:46 AM, Attilio Rao atti...@freebsd.org wrote: On Thu, May 9, 2013 at 6:28 PM, Marcel Moolenaar mar...@freebsd.org wrote: Author: marcel Date: Thu May 9 16:28:18 2013 New Revision: 250411 URL:

Re: svn commit: r250430 - head/sbin/dmesg

2013-05-10 Thread Eitan Adler
On 10 May 2013 09:36, Alexey Dokuchaev da...@freebsd.org wrote: Of course. But my question was about dropping the cast, not adding new option. Adding a new option should have resulted in the following diff: - (void)fprintf(stderr, usage: dmesg [-a] [-M core [-N system]]\n); +

Re: svn commit: r250430 - head/sbin/dmesg

2013-05-10 Thread David Chisnall
On 10 May 2013, at 10:01, Eitan Adler ead...@freebsd.org wrote: I thought style(9) disliked the leading (void) but I could be wrong. The cast to (void) tells the compiler that you are ignoring the return result. Without it, you will introduce new warnings and break the build on certain

Re: svn commit: r250430 - head/sbin/dmesg

2013-05-10 Thread Eitan Adler
On 10 May 2013 10:08, David Chisnall thera...@freebsd.org wrote: On 10 May 2013, at 10:01, Eitan Adler ead...@freebsd.org wrote: I thought style(9) disliked the leading (void) but I could be wrong. The cast to (void) tells the compiler that you are ignoring the return result. Without it,

Re: svn commit: r250430 - head/sbin/dmesg

2013-05-10 Thread David Chisnall
On 10 May 2013, at 10:12, Eitan Adler ead...@freebsd.org wrote: I don't remember any recent tool which uses that cast to remove errors. clang -Wunused-result gcc -Wunused-result David ___ svn-src-head@freebsd.org mailing list

Re: svn commit: r250411 - in head/sys: conf kern sys

2013-05-10 Thread Marcel Moolenaar
On May 10, 2013, at 6:52 AM, John Baldwin j...@freebsd.org wrote: The way to fix this is to implement LK_NOWITNESS on a per-lock basis into lockmgr, propagate the same concept to the vn_lock() (which should be basically done automatically) and finally identify the false-positive case and

Re: svn commit: r250220 - head/sys/kern

2013-05-10 Thread John Baldwin
On Thursday, May 09, 2013 11:05:46 pm Bruce Evans wrote: On Mon, 6 May 2013, John Baldwin wrote: On Saturday, May 04, 2013 4:47:43 am Bruce Evans wrote: Log: Fix FIONREAD on regular files. The computed result was being ignored and it was being passed down to VOP_IOCTL() where it

Re: svn commit: r250411 - in head/sys: conf kern sys

2013-05-10 Thread John Baldwin
On Friday, May 10, 2013 11:46:54 am Marcel Moolenaar wrote: 2) vnode locks from a local filesystem that report a LOR with a devfs vnode. Typical reports are either ufs - devfs or in some cases ufs - devfs - ufs. As with 1), I would much rather tag the offending location than to

svn commit: r250459 - head/share/syscons/fonts

2013-05-10 Thread Eitan Adler
Author: eadler Date: Fri May 10 16:30:39 2013 New Revision: 250459 URL: http://svnweb.freebsd.org/changeset/base/250459 Log: Add the letter 'q' to the font iso05-8x16.fnt PR: 72585 PR: 169797 Submitted by: Takefu tak...@airport.fm Submitted by: Satoshi KImura

svn commit: r250460 - in head: share/examples/ppi share/misc sys/cam/ctl sys/cam/scsi sys/dev/aic7xxx sys/dev/cp sys/dev/ctau sys/dev/cx sys/dev/hptmv sys/dev/isci/scil sys/dev/nsp sys/dev/sfxge/co...

2013-05-10 Thread Eitan Adler
Author: eadler Date: Fri May 10 16:41:26 2013 New Revision: 250460 URL: http://svnweb.freebsd.org/changeset/base/250460 Log: Fxi a bunch of typos. PR: misc/174625 Submitted by: Jeremy Chadwick j...@koitsu.org Modified: head/share/examples/ppi/ppilcd.c head/share/misc/pci_vendors

Re: svn commit: r250411 - in head/sys: conf kern sys

2013-05-10 Thread Alfred Perlstein
On 5/10/13 8:46 AM, Marcel Moolenaar wrote: And all I did is to allow someone (= Juniper) to not print the LOR for this well-known and mostly ignored case that is impacting our ability to keep witness enabled. And the reason I had to do that is that this is a long-standing LOR that isn't being

Re: svn commit: r250432 - head/usr.bin/split

2013-05-10 Thread Benjamin Kaduk
On Fri, May 10, 2013 at 12:23 AM, Eitan Adler ead...@freebsd.org wrote: Modified: head/usr.bin/split/split.c == --- head/usr.bin/split/split.c Fri May 10 03:49:05 2013(r250431) +++ head/usr.bin/split/split.c

svn commit: r250462 - head/usr.bin/lockf

2013-05-10 Thread Eitan Adler
Author: eadler Date: Fri May 10 17:30:29 2013 New Revision: 250462 URL: http://svnweb.freebsd.org/changeset/base/250462 Log: Add option to lockf to avoid creating a file if it does not exist. PR: bin/170775 Submitted by: Matthew Story matthewst...@gmail.com Reviewed by: scottl MFC

svn commit: r250463 - head/usr.sbin/mergemaster

2013-05-10 Thread Eitan Adler
Author: eadler Date: Fri May 10 17:42:13 2013 New Revision: 250463 URL: http://svnweb.freebsd.org/changeset/base/250463 Log: Permit relative paths with mergemaster -m Reviewed by: dim, brooks MFC after:3 days Modified: head/usr.sbin/mergemaster/mergemaster.sh Modified:

Re: svn commit: r250462 - head/usr.bin/lockf

2013-05-10 Thread Mateusz Guzik
On Fri, May 10, 2013 at 05:30:30PM +, Eitan Adler wrote: Author: eadler Date: Fri May 10 17:30:29 2013 New Revision: 250462 URL: http://svnweb.freebsd.org/changeset/base/250462 Log: Add option to lockf to avoid creating a file if it does not exist. Patch seems to be incomplete.

svn commit: r250466 - in head/sys: netinet netinet6

2013-05-10 Thread Michael Tuexen
Author: tuexen Date: Fri May 10 18:09:38 2013 New Revision: 250466 URL: http://svnweb.freebsd.org/changeset/base/250466 Log: Honor the net.inet6.ip6.v6only sysctl variable and the IPV6_V6ONLY socket option for SCTP sockets in the same way as for UDP or TCP sockets. MFC after: 2 weeks

svn commit: r250469 - head/bin/dd

2013-05-10 Thread Eitan Adler
Author: eadler Date: Fri May 10 18:43:36 2013 New Revision: 250469 URL: http://svnweb.freebsd.org/changeset/base/250469 Log: Make dd's signal handler async safe. PR: bin/75258 Submitted by: Oleg V. Nauman o...@reis.zp.ua Arrival Date: Sun Dec 19 14:50:21 GMT 2004 Reviewed

Re: svn commit: r250411 - in head/sys: conf kern sys

2013-05-10 Thread Marcel Moolenaar
On May 10, 2013, at 9:11 AM, John Baldwin j...@freebsd.org wrote: On Friday, May 10, 2013 11:46:54 am Marcel Moolenaar wrote: 2) vnode locks from a local filesystem that report a LOR with a devfs vnode. Typical reports are either ufs - devfs or in some cases ufs - devfs - ufs. As with

Re: svn commit: r250379 - in head/usr.sbin/bsnmpd/modules: . snmp_hast

2013-05-10 Thread Pawel Jakub Dawidek
On Thu, May 09, 2013 at 01:05:35PM +0300, Mikolaj Golub wrote: On Thu, May 09, 2013 at 11:22:43AM +0300, Konstantin Belousov wrote: Why do you need libl.a ? If only for yywrap(), then re-implementing it for the module seems to be trivial. Yes, it is like so. Thanks! I am now considering

Re: svn commit: r250463 - head/usr.sbin/mergemaster

2013-05-10 Thread Garrett Cooper
On Fri, May 10, 2013 at 10:42 AM, Eitan Adler ead...@freebsd.org wrote: Author: eadler Date: Fri May 10 17:42:13 2013 New Revision: 250463 URL: http://svnweb.freebsd.org/changeset/base/250463 Log: Permit relative paths with mergemaster -m Reviewed by: dim, brooks MFC after:3

Re: svn commit: r250463 - head/usr.sbin/mergemaster

2013-05-10 Thread Eitan Adler
On 10 May 2013 15:09, Garrett Cooper yaneurab...@gmail.com wrote: On Fri, May 10, 2013 at 10:42 AM, Eitan Adler ead...@freebsd.org wrote: Author: eadler Date: Fri May 10 17:42:13 2013 New Revision: 250463 URL: http://svnweb.freebsd.org/changeset/base/250463 Log: Permit relative paths

Re: svn commit: r250463 - head/usr.sbin/mergemaster

2013-05-10 Thread Garrett Cooper
On Fri, May 10, 2013 at 12:32 PM, Eitan Adler ead...@freebsd.org wrote: ... Yes, sorry for not including the PR number. No worries :). Thanks for the fix! -Garrett ___ svn-src-head@freebsd.org mailing list

svn commit: r250477 - head/usr.bin/calendar/calendars

2013-05-10 Thread Xin LI
Author: delphij Date: Fri May 10 20:02:15 2013 New Revision: 250477 URL: http://svnweb.freebsd.org/changeset/base/250477 Log: REST IN PEACE Alexander Botero-Lowry Born June 2, 1986 in Austin, Texas Died August 24, 2012 in San Francisco, California Thank you for your

Re: svn commit: r250411 - in head/sys: conf kern sys

2013-05-10 Thread John Baldwin
On Friday, May 10, 2013 2:51:20 pm Marcel Moolenaar wrote: On May 10, 2013, at 9:11 AM, John Baldwin j...@freebsd.org wrote: On Friday, May 10, 2013 11:46:54 am Marcel Moolenaar wrote: 2) vnode locks from a local filesystem that report a LOR with a devfs vnode. Typical reports are

svn commit: r250487 - head/sys/x86/cpufreq

2013-05-10 Thread Hiren Panchasara
Author: hiren Date: Fri May 10 22:43:27 2013 New Revision: 250487 URL: http://svnweb.freebsd.org/changeset/base/250487 Log: Adding a detach method to p4tcc driver. PR: 118739 Submitted by: Dan Lukes d...@obluda.cz (earlier version) Reviewed by: jhb Approved by: sbruno (mentor)

svn commit: r250495 - head/sys/amd64/amd64

2013-05-10 Thread Rui Paulo
Author: rpaulo Date: Sat May 11 01:31:51 2013 New Revision: 250495 URL: http://svnweb.freebsd.org/changeset/base/250495 Log: Fix several standard extended feature bits. Submitted by: Oliver Pinter oliver.pntr at gmail.com Modified: head/sys/amd64/amd64/identcpu.c Modified:

Re: svn commit: r250432 - head/usr.bin/split

2013-05-10 Thread Eitan Adler
On 10 May 2013 13:05, Benjamin Kaduk bjkf...@gmail.com wrote: This check relies on signed integer overflow, which is undefined behavior. Furthermore, even if one assumes a non-evil compiler and two's complement representation, this check fails for pattlen == 10. 10**9 is representable as both