Re: svn commit: r278634 - head/lib/libc/gen

2015-02-13 Thread Andrey Chernov
We even don't need to check arg excepting for < 0, because what is needed is rlimt_t and not arg. So this version will be better: rlimt_t targ; if (arg < 0) { errno = EINVAL; return (-1); } targ = arg; if (targ > RLIM_INFINITY / 512) targ = RLIM_INFINITY / 512; limit.rlim_max = limit.

svn commit: r278669 - head/usr.bin/passwd

2015-02-13 Thread Baptiste Daroussin
Author: bapt Date: Fri Feb 13 09:02:15 2015 New Revision: 278669 URL: https://svnweb.freebsd.org/changeset/base/278669 Log: Revert r278642 install(1) does not handle chflags on hardlinks Modified: head/usr.bin/passwd/Makefile Modified: head/usr.bin/passwd/Makefile ==

svn commit: r278670 - head/sys/dev/dwc

2015-02-13 Thread Ruslan Bukin
Author: br Date: Fri Feb 13 11:00:13 2015 New Revision: 278670 URL: https://svnweb.freebsd.org/changeset/base/278670 Log: o Correct the condition in reset procedure o Setup interrupt hander after reset, not before This fixes operation on ODROID-C1 (Amlogic S805) Submitted by: John We

svn commit: r278671 - head/sys/dev/smc

2015-02-13 Thread Ruslan Bukin
Author: br Date: Fri Feb 13 11:13:08 2015 New Revision: 278671 URL: https://svnweb.freebsd.org/changeset/base/278671 Log: o Correct the calculation how many pages we need o Ensure we use correct bank for MSK register o Save and restore current bank in interrupt handler o Stop TX watchdog o

svn commit: r278672 - in head/sys: cam/ctl sys

2015-02-13 Thread Alexander Motin
Author: mav Date: Fri Feb 13 13:26:23 2015 New Revision: 278672 URL: https://svnweb.freebsd.org/changeset/base/278672 Log: Teach CTL to ask GEOM devices about BIO_DELETE support. MFC after:1 week Modified: head/sys/cam/ctl/ctl_backend_block.c head/sys/cam/ctl/ctl_backend_ramdisk.c

Re: svn commit: r278634 - head/lib/libc/gen

2015-02-13 Thread Bruce Evans
On Fri, 13 Feb 2015, Andrey Chernov wrote: On 13.02.2015 10:18, Bruce Evans wrote: if (arg > RLIM_INFINITY) err(...); Checking for RLIM_INFINITY is wrong here, since it is ulong long max, No, it is correct. rlim_t is neither unsigned not specificially ulong long. It is int64_t

svn commit: r278673 - head/sys/dev/vt/font

2015-02-13 Thread Alexey Dokuchaev
Author: danfe (ports committer) Date: Fri Feb 13 13:55:38 2015 New Revision: 278673 URL: https://svnweb.freebsd.org/changeset/base/278673 Log: Improve vt(4) default mouse pointer image. Previous one looked rather ugly, especially on native monitor resolutions. This one essentially matches X1

Re: svn commit: r278634 - head/lib/libc/gen

2015-02-13 Thread Bruce Evans
On Fri, 13 Feb 2015, Andrey Chernov wrote: We even don't need to check arg excepting for < 0, because what is needed is rlimt_t and not arg. So this version will be better: rlimt_t targ; if (arg < 0) { errno = EINVAL; return (-1); } This is reasonable, but not encouraged by the API or

Re: svn commit: r278634 - head/lib/libc/gen

2015-02-13 Thread Bruce Evans
On Sat, 14 Feb 2015, Bruce Evans wrote: ... However, I don't like using rlim_t for the scaled value that is not an rlimit. An incomplete fix with handling of negative values restored is something like: intmax_t targ; targ = arg; if (targ > RLIM_INFINITY / 512)

Re: svn commit: r278634 - head/lib/libc/gen

2015-02-13 Thread Pedro Giffuni
On 02/13/15 09:29, Bruce Evans wrote: On Fri, 13 Feb 2015, Andrey Chernov wrote: We even don't need to check arg excepting for < 0, because what is needed is rlimt_t and not arg. So this version will be better: rlimt_t targ; if (arg < 0) { errno = EINVAL; return (-1); } This is reas

svn commit: r278681 - head/sys/ofed/include/linux

2015-02-13 Thread Hans Petter Selasky
Author: hselasky Date: Fri Feb 13 16:35:12 2015 New Revision: 278681 URL: https://svnweb.freebsd.org/changeset/base/278681 Log: Add more functions to the Linux kernel compatibility layer. Add some missing includes which are needed when the header files are not included in a particular order.

Re: svn commit: r278634 - head/lib/libc/gen

2015-02-13 Thread Bruce Evans
On Fri, 13 Feb 2015, Pedro Giffuni wrote: On 02/13/15 09:29, Bruce Evans wrote: On Fri, 13 Feb 2015, Andrey Chernov wrote: We even don't need to check arg excepting for < 0, because what is needed is rlimt_t and not arg. So this version will be better: rlimt_t targ; if (arg < 0) { errno

Re: svn commit: r278634 - head/lib/libc/gen

2015-02-13 Thread Pedro Giffuni
On 02/13/15 11:46, Bruce Evans wrote: On Fri, 13 Feb 2015, Pedro Giffuni wrote: On 02/13/15 09:29, Bruce Evans wrote: On Fri, 13 Feb 2015, Andrey Chernov wrote: We even don't need to check arg excepting for < 0, because what is needed is rlimt_t and not arg. So this version will be better:

svn commit: r278682 - head/share/mk

2015-02-13 Thread John Baldwin
Author: jhb Date: Fri Feb 13 17:33:27 2015 New Revision: 278682 URL: https://svnweb.freebsd.org/changeset/base/278682 Log: Make the extra dependencies in DPADD be dependencies of PROG_FULL and SHLIB_NAME_FULL so that the full binary is relinked when a dependency changes. Right now the exist

Re: svn commit: r278634 - head/lib/libc/gen

2015-02-13 Thread Andrey Chernov
On 13.02.2015 17:29, Bruce Evans wrote: >> if (arg < 0) { >>errno = EINVAL; >>return (-1); >> } > > > This is reasonable, but not encouraged by the API or compatible with > what setrlimit() does with negative args. (setrlimit() still uses > my hack from 1994, of converting negative args

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

2015-02-13 Thread hiren panchasara
On 02/12/15 at 01:31P, Randall Stewart wrote: > Author: rrs > Date: Thu Feb 12 13:31:08 2015 > New Revision: 278623 > URL: https://svnweb.freebsd.org/changeset/base/278623 > > Log: > This fixes a bug I in-advertantly inserted when I updated the callout > code in my last commit. The cc_exec_nex

svn commit: r278691 - head/usr.sbin/gssd

2015-02-13 Thread Mark Johnston
Author: markj Date: Fri Feb 13 18:34:04 2015 New Revision: 278691 URL: https://svnweb.freebsd.org/changeset/base/278691 Log: Add an auto-generated file to CLEANFILES. MFC after:3 days Sponsored by: EMC / Isilon Storage Division Modified: head/usr.sbin/gssd/Makefile Modified: head/

svn commit: r278690 - head/usr.sbin/gssd

2015-02-13 Thread Mark Johnston
Author: markj Date: Fri Feb 13 18:32:55 2015 New Revision: 278690 URL: https://svnweb.freebsd.org/changeset/base/278690 Log: Check for an error from daemon(3), and correct the check for an error from socket(2). MFC after:3 days Sponsored by: EMC / Isilon Storage Division Modified:

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

2015-02-13 Thread Peter Holm
On Fri, Feb 13, 2015 at 10:29:01AM -0800, hiren panchasara wrote: > On 02/12/15 at 01:31P, Randall Stewart wrote: > > Author: rrs > > Date: Thu Feb 12 13:31:08 2015 > > New Revision: 278623 > > URL: https://svnweb.freebsd.org/changeset/base/278623 > > > > Log: > > This fixes a bug I in-advertant

svn commit: r278695 - head/sys/dev/oce

2015-02-13 Thread John-Mark Gurney
Author: jmg Date: Fri Feb 13 19:44:04 2015 New Revision: 278695 URL: https://svnweb.freebsd.org/changeset/base/278695 Log: srandom has no influence on read_random, at least not this late... Modified: head/sys/dev/oce/oce_mbox.c Modified: head/sys/dev/oce/oce_mbox.c ==

svn commit: r278697 - head/sys/kern

2015-02-13 Thread Alan Cox
Author: alc Date: Fri Feb 13 19:58:53 2015 New Revision: 278697 URL: https://svnweb.freebsd.org/changeset/base/278697 Log: Preset the object's color, or alignment, to maximize superpage usage. MFC after:5 days Modified: head/sys/kern/sysv_shm.c Modified: head/sys/kern/sysv_shm.c ===

svn commit: r278704 - head/etc/rc.d

2015-02-13 Thread Garrett Cooper
Author: ngie Date: Fri Feb 13 20:49:43 2015 New Revision: 278704 URL: https://svnweb.freebsd.org/changeset/base/278704 Log: Unbreak rcorder when MK_UNBOUND == no by moving local_unbound from REQUIRE: in NETWORKING to BEFORE: in the script MFC after: 2 weeks Modified: head/etc/rc.d/NETW

svn commit: r278706 - head/etc/rc.d

2015-02-13 Thread Garrett Cooper
Author: ngie Date: Fri Feb 13 20:52:23 2015 New Revision: 278706 URL: https://svnweb.freebsd.org/changeset/base/278706 Log: Unbreak rcorder when MK_UTX == no by moving utx from REQUIRE: in LOGIN to BEFORE: in utx MFC after: 1 week Modified: head/etc/rc.d/LOGIN head/etc/rc.d/utx Modi

Re: svn commit: r278472 - in head/sys: netinet netinet6

2015-02-13 Thread Gleb Smirnoff
On Mon, Feb 09, 2015 at 03:11:21PM -0500, John Baldwin wrote: J> On Monday, February 09, 2015 07:28:12 PM Randall Stewart wrote: J> > Author: rrs J> > Date: Mon Feb 9 19:28:11 2015 J> > New Revision: 278472 J> > URL: https://svnweb.freebsd.org/changeset/base/278472 J> > J> > Log: J> > This fixe

Re: svn commit: r278472 - in head/sys: netinet netinet6

2015-02-13 Thread Randall Stewart via svn-src-head
On Feb 13, 2015, at 4:21 PM, Gleb Smirnoff wrote: > On Mon, Feb 09, 2015 at 03:11:21PM -0500, John Baldwin wrote: > J> On Monday, February 09, 2015 07:28:12 PM Randall Stewart wrote: > J> > Author: rrs > J> > Date: Mon Feb 9 19:28:11 2015 > J> > New Revision: 278472 > J> > URL: https://svnweb.f

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

2015-02-13 Thread hiren panchasara
On 02/13/15 at 08:11P, Peter Holm wrote: > On Fri, Feb 13, 2015 at 10:29:01AM -0800, hiren panchasara wrote: > > On 02/12/15 at 01:31P, Randall Stewart wrote: > > > Author: rrs > > > Date: Thu Feb 12 13:31:08 2015 > > > New Revision: 278623 > > > URL: https://svnweb.freebsd.org/changeset/base/27862

Re: svn commit: r278472 - in head/sys: netinet netinet6

2015-02-13 Thread Randall Stewart via svn-src-head
Gleb: Ok here is the diff of the arp timer function that this changed made (238990): arptimer(void *arg) { + struct llentry *lle = (struct llentry *)arg; struct ifnet *ifp; - struct llentry *lle; - int pkts_dropped; + size_t pkts_dro

svn commit: r278728 - head/usr.sbin/freebsd-update

2015-02-13 Thread Thomas Quinot
Author: thomas Date: Fri Feb 13 22:55:25 2015 New Revision: 278728 URL: https://svnweb.freebsd.org/changeset/base/278728 Log: (backup_kernel_finddir, backup_kernel, install_files): Add missing references to $BASEDIR, in order to allow correct operation when updating a system mounted at anoth

svn commit: r278729 - head/sys/sys

2015-02-13 Thread Simon J. Gerraty
Author: sjg Date: Fri Feb 13 23:19:35 2015 New Revision: 278729 URL: https://svnweb.freebsd.org/changeset/base/278729 Log: sbspace: size of bleft, mleft must match sockbuf fields to avoid overflow on amd64 Submitted by: anshu...@juniper.net Obtained from:Juniper Networks Modifi

svn commit: r278733 - head/lib/libiconv

2015-02-13 Thread Bryan Drewery
Author: bdrewery Date: Fri Feb 13 23:35:58 2015 New Revision: 278733 URL: https://svnweb.freebsd.org/changeset/base/278733 Log: Remove disconnected (and broken) directory libiconv. It was added in the initial citrus import in r219019 but never used as iconv is just built with libc. A libic

Re: svn commit: r278472 - in head/sys: netinet netinet6

2015-02-13 Thread Gleb Smirnoff
Randall, thanks a lot for investigating that. On Fri, Feb 13, 2015 at 05:27:40PM -0500, Randall Stewart wrote: R> Gleb: R> R> Ok here is the diff of the arp timer function that this changed made (238990): R> R> arptimer(void *arg) R> { R> + struct llentr

Phabricator + 'Reviewed by' [was Re: svn commit: r278472 - in head/sys: netinet netinet6]

2015-02-13 Thread Bryan Drewery
On 2/9/2015 3:45 PM, Bjoern A. Zeeb wrote: >> Commented upon by hiren and sbruno >> See Phabricator D1777 for more details. >> >> Commented upon by hiren and sbruno >> Reviewed by:adrian, jhb and bz > I have not reviewed this; as a matter of fact you are aware that I still > wanted to

svn commit: r278737 - head/usr.sbin/flowctl

2015-02-13 Thread Gleb Smirnoff
Author: glebius Date: Fri Feb 13 23:57:20 2015 New Revision: 278737 URL: https://svnweb.freebsd.org/changeset/base/278737 Log: Use less ugly code to allocate buffer of SORCVBUF_SIZE. Modified: head/usr.sbin/flowctl/flowctl.c Modified: head/usr.sbin/flowctl/flowctl.c =

svn commit: r278738 - head/cddl/contrib/opensolaris/cmd/dtrace/test/tst/common/io

2015-02-13 Thread Mark Johnston
Author: markj Date: Sat Feb 14 00:03:43 2015 New Revision: 278738 URL: https://svnweb.freebsd.org/changeset/base/278738 Log: Tweak the fds test program so that it actually compiles. Also use 0 instead of -1 for the bogus ioctl command so that dmesg doesn't get spammed with sign extension war

Re: svn commit: r278640 - head/sys/netgraph

2015-02-13 Thread Gleb Smirnoff
Hi! On Thu, Feb 12, 2015 at 10:20:34PM +, Gleb Smirnoff wrote: T> Author: glebius T> Date: Thu Feb 12 22:20:34 2015 T> New Revision: 278640 T> URL: https://svnweb.freebsd.org/changeset/base/278640 T> T> Log: T> Revise default limit for maximum of netgraph data items. T> With modern inte

svn commit: r278739 - head/lib/libc/regex

2015-02-13 Thread Xin LI
Author: delphij Date: Sat Feb 14 00:23:53 2015 New Revision: 278739 URL: https://svnweb.freebsd.org/changeset/base/278739 Log: Disallow pattern spaces which would cause intermediate calculations to overflow size_t. Obtained from:DragonFly (2841837793bd095a82f477e9c370cfe6cfb3862c

Re: svn commit: r278739 - head/lib/libc/regex

2015-02-13 Thread Bryan Drewery
On 2/13/2015 6:23 PM, Xin LI wrote: > Author: delphij > Date: Sat Feb 14 00:23:53 2015 > New Revision: 278739 > URL: https://svnweb.freebsd.org/changeset/base/278739 > > Log: > Disallow pattern spaces which would cause intermediate calculations to > overflow size_t. > > Obtained from:

Re: svn commit: r278739 - head/lib/libc/regex

2015-02-13 Thread Bryan Drewery
On 2/13/2015 6:37 PM, Bryan Drewery wrote: > On 2/13/2015 6:23 PM, Xin LI wrote: >> Author: delphij >> Date: Sat Feb 14 00:23:53 2015 >> New Revision: 278739 >> URL: https://svnweb.freebsd.org/changeset/base/278739 >> >> Log: >> Disallow pattern spaces which would cause intermediate calculations

Re: svn commit: r278739 - head/lib/libc/regex

2015-02-13 Thread Xin Li
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 02/13/15 16:37, Bryan Drewery wrote: > On 2/13/2015 6:23 PM, Xin LI wrote: >> Author: delphij Date: Sat Feb 14 00:23:53 2015 New Revision: >> 278739 URL: https://svnweb.freebsd.org/changeset/base/278739 >> >> Log: Disallow pattern spaces which wo

svn commit: r278741 - head/sys/contrib/dev/ath/ath_hal/ar9300

2015-02-13 Thread Adrian Chadd
Author: adrian Date: Sat Feb 14 04:28:51 2015 New Revision: 278741 URL: https://svnweb.freebsd.org/changeset/base/278741 Log: Update the AR9300 HAL to the latest public available HAL from QCA. I've been sitting on this for a year or so now; I've finally tested it on enough devices to be r

Re: svn commit: r278321 - head/sys/dev/ipmi

2015-02-13 Thread Will Andrews
On Fri, Feb 06, 2015 at 12:00:02PM -0500, John Baldwin wrote: > I think we should further change this to only allow the watchdog to be used > with KCS and SMIC so that watchdog requests never sleep. I think that we > could then revert r272366 as to my knowledge this was the only watchdog > driv

svn commit: r278742 - head/bin/pkill/tests

2015-02-13 Thread Garrett Cooper
Author: ngie Date: Sat Feb 14 06:19:24 2015 New Revision: 278742 URL: https://svnweb.freebsd.org/changeset/base/278742 Log: Simplify jail_name_to_jid and try to be more fault tolerant when scanning for the jail ID (poll up to 10 times for the jail IDs to become available) If the scan fail