Re: svn commit: r325841 - in head/sys: conf dev/mlx4 dev/mlx4/mlx4_core dev/mlx4/mlx4_en dev/mlx4/mlx4_ib modules/mlx4

2017-12-18 Thread Hans Petter Selasky
On 12/18/17 05:29, Colin Percival wrote: Also, it breaks some work I have in progress for instrumenting SYSINITs. Would you mind moving the DEFINE_MUTEX line to occur immediately prior to the set_port_type function, rather than being placed inside it? I'll have a look at this later today. Your

svn commit: r326929 - head/sys/i386/i386

2017-12-18 Thread Bruce Evans
Author: bde Date: Mon Dec 18 09:32:56 2017 New Revision: 326929 URL: https://svnweb.freebsd.org/changeset/base/326929 Log: Fix the undersupported option KERNLOAD, part 1: fix crashes in locore when KERNLOAD is not a multiple of NBPDR (not the default) and PSE is enabled (the default if the C

svn commit: r326930 - head/usr.sbin/newsyslog

2017-12-18 Thread Baptiste Daroussin
Author: bapt Date: Mon Dec 18 09:35:04 2017 New Revision: 326930 URL: https://svnweb.freebsd.org/changeset/base/326930 Log: newsyslog: Fix issues after r326616 When building the command to execute for compression, newsyslog was modifying the generic arguments array instead of its own copy

Re: svn commit: r326617 - head/usr.sbin/newsyslog

2017-12-18 Thread Baptiste Daroussin
On Fri, Dec 15, 2017 at 05:40:15PM -0800, Cy Schubert wrote: > In message <201712060944.vb69izqe027...@repo.freebsd.org>, Baptiste > Daroussin w > rites: > > Author: bapt > > Date: Wed Dec 6 09:44:35 2017 > > New Revision: 326617 > > URL: https://svnweb.freebsd.org/changeset/base/326617 > > > > L

svn commit: r326931 - head/sys/i386/i386

2017-12-18 Thread Bruce Evans
Author: bde Date: Mon Dec 18 11:57:05 2017 New Revision: 326931 URL: https://svnweb.freebsd.org/changeset/base/326931 Log: Fix the undersupported option KERNLOAD, part 2: fix crashes in locore when KERNLOAD is smaller than NBPDR (not the default) and PG_G is enabled (the default if the CPU s

svn commit: r326932 - in head/sys: i386/i386 x86/acpica x86/x86

2017-12-18 Thread Bruce Evans
Author: bde Date: Mon Dec 18 13:53:22 2017 New Revision: 326932 URL: https://svnweb.freebsd.org/changeset/base/326932 Log: Remove the permanent double mapping of low physical memory and replace it by a transient double mapping for the one instruction in ACPI wakeup where it is needed (and fo

svn commit: r326933 - head/sys/i386/i386

2017-12-18 Thread Bruce Evans
Author: bde Date: Mon Dec 18 14:29:48 2017 New Revision: 326933 URL: https://svnweb.freebsd.org/changeset/base/326933 Log: Also forgotten in the previous that removed the permanent double mapping of low physical memory: Update the comment about leaving the permanent mapping in place. Thi

Re: svn commit: r326932 - in head/sys: i386/i386 x86/acpica x86/x86

2017-12-18 Thread Bruce Evans
On Mon, 18 Dec 2017, Bruce Evans wrote: Author: bde Date: Mon Dec 18 13:53:22 2017 New Revision: 326932 URL: https://svnweb.freebsd.org/changeset/base/326932 Log: Remove the permanent double mapping of low physical memory and replace it by a transient double mapping for the one instruction in

Re: svn commit: r326643 - head/sys/cam

2017-12-18 Thread Mark Johnston
On Wed, Dec 06, 2017 at 11:05:07PM +, Warner Losh wrote: > Author: imp > Date: Wed Dec 6 23:05:07 2017 > New Revision: 326643 > URL: https://svnweb.freebsd.org/changeset/base/326643 > > Log: > Make cam_periph_runccb be safe to call when we can only do polling. > > Sponsored by: Netflix

Re: svn commit: r326643 - head/sys/cam

2017-12-18 Thread Warner Losh
On Mon, Dec 18, 2017 at 7:59 AM, Mark Johnston wrote: > On Wed, Dec 06, 2017 at 11:05:07PM +, Warner Losh wrote: > > Author: imp > > Date: Wed Dec 6 23:05:07 2017 > > New Revision: 326643 > > URL: https://svnweb.freebsd.org/changeset/base/326643 > > > > Log: > > Make cam_periph_runccb be s

Re: svn commit: r326643 - head/sys/cam

2017-12-18 Thread Warner Losh
Sorry for the top post, but https://reviews.freebsd.org/D13531 should have a fix. Warner On Mon, Dec 18, 2017 at 8:03 AM, Warner Losh wrote: > > > On Mon, Dec 18, 2017 at 7:59 AM, Mark Johnston wrote: > >> On Wed, Dec 06, 2017 at 11:05:07PM +, Warner Losh wrote: >> > Author: imp >> > Date:

Re: svn commit: r326617 - head/usr.sbin/newsyslog

2017-12-18 Thread Cy Schubert
In message <20171218093635.ilumboo655ual...@ivaldir.net>, Baptiste Daroussin wr ites: > On Fri, Dec 15, 2017 at 05:40:15PM -0800, Cy Schubert wrote: > > In message <201712060944.vb69izqe027...@repo.freebsd.org>, Baptiste=20 > > Daroussin w > > rites: > > > Author: bapt > > > Date: Wed Dec 6 09:44

Re: svn commit: r326617 - head/usr.sbin/newsyslog

2017-12-18 Thread Cy Schubert
In message <20171218093635.ilumboo655ual...@ivaldir.net>, Baptiste Daroussin wr ites: > On Fri, Dec 15, 2017 at 05:40:15PM -0800, Cy Schubert wrote: > > In message <201712060944.vb69izqe027...@repo.freebsd.org>, Baptiste=20 > > Daroussin w > > rites: > > > Author: bapt > > > Date: Wed Dec 6 09:44

svn commit: r326935 - in head/sys/cddl/dev/dtrace: amd64 i386

2017-12-18 Thread Mark Johnston
Author: markj Date: Mon Dec 18 17:26:24 2017 New Revision: 326935 URL: https://svnweb.freebsd.org/changeset/base/326935 Log: Avoid CPU migration in dtrace_gethrtime() on x86. dtrace_gethrtime() may be called outside of probe context, and in particular, from the DTRACEIOC_BUFSNAP handler.

svn commit: r326937 - head/sys/dev/nvme

2017-12-18 Thread Warner Losh
Author: imp Date: Mon Dec 18 18:38:00 2017 New Revision: 326937 URL: https://svnweb.freebsd.org/changeset/base/326937 Log: When we're disabling the nvme device, some drives have a controller bug that requires 'hands off' for a period of time (2.3s) before we check the RDY bit. Sicne this is

svn commit: r326940 - head/sys/dev/nvme

2017-12-18 Thread Warner Losh
Author: imp Date: Mon Dec 18 20:11:21 2017 New Revision: 326940 URL: https://svnweb.freebsd.org/changeset/base/326940 Log: Although we only have one quirk at the moment, guard against the day we have more than one by checking the actual quirk bit before delaying the reset. Noticed by: r

Re: svn commit: r325841 - in head/sys: conf dev/mlx4 dev/mlx4/mlx4_core dev/mlx4/mlx4_en dev/mlx4/mlx4_ib modules/mlx4

2017-12-18 Thread Colin Percival
On 12/18/17 00:15, Hans Petter Selasky wrote: > On 12/18/17 05:29, Colin Percival wrote: >> Also, it breaks some work I have in progress for instrumenting SYSINITs. >> Would you mind moving the DEFINE_MUTEX line to occur immediately prior to >> the set_port_type function, rather than being placed i

svn commit: r326953 - head/tests/sys/kern

2017-12-18 Thread John Baldwin
Author: jhb Date: Mon Dec 18 23:35:14 2017 New Revision: 326953 URL: https://svnweb.freebsd.org/changeset/base/326953 Log: Catch up to r325719 which makes the kern.proc.pid sysctl "work" for zombies. Some of the ptrace tests need to wait for a child process to become a zombie before prece

svn commit: r326954 - head/tools/boot

2017-12-18 Thread Warner Losh
Author: imp Date: Tue Dec 19 00:18:17 2017 New Revision: 326954 URL: https://svnweb.freebsd.org/changeset/base/326954 Log: Support more images (but still no geli) Print a qemu line to a shell script to ease testing each image Start to support multiple architectures (still very green) Creat

svn commit: r326956 - head/sys/dev/acpica

2017-12-18 Thread Conrad Meyer
Author: cem Date: Tue Dec 19 02:49:11 2017 New Revision: 326956 URL: https://svnweb.freebsd.org/changeset/base/326956 Log: Implement ACPI CPU support when Processor object is not present By the ACPI standard (ACPI 5 chapter 8.4 Declaring Processors) Processors can be implemented in 2 dist

svn commit: r326957 - head/contrib/llvm/tools/lld/ELF

2017-12-18 Thread Ed Maste
Author: emaste Date: Tue Dec 19 03:15:20 2017 New Revision: 326957 URL: https://svnweb.freebsd.org/changeset/base/326957 Log: lld: Don't write preemptible symbol values to the .got. It is not necessary and matches what bfd and gold do. This was a regression from [LLVM] r315658. Ob

svn commit: r326958 - head

2017-12-18 Thread Eitan Adler
Author: eadler Date: Tue Dec 19 03:35:39 2017 New Revision: 326958 URL: https://svnweb.freebsd.org/changeset/base/326958 Log: arclint: revert in prep for recommitting Modified: head/.arclint Modified: head/.arclint =

svn commit: r326959 - head

2017-12-18 Thread Eitan Adler
Author: eadler Date: Tue Dec 19 03:38:06 2017 New Revision: 326959 URL: https://svnweb.freebsd.org/changeset/base/326959 Log: arc lint: ignore /tests/ in chmod shell scripts in scripts don't need to be chmod +x to work. In fact most are not. Of the tests I found from a simple search:

Re: svn commit: r326958 - head

2017-12-18 Thread Rodney W. Grimes
[ Charset UTF-8 unsupported, converting... ] > Author: eadler > Date: Tue Dec 19 03:35:39 2017 > New Revision: 326958 > URL: https://svnweb.freebsd.org/changeset/base/326958 > > Log: > arclint: revert in prep for recommitting When you revert you need to sight the Rx(s) being reverted. > >

Re: svn commit: r326958 - head

2017-12-18 Thread Eitan Adler
I give up. Not touching this anymore. If you feel like fixing it do it. On 18 December 2017 at 19:51, Rodney W. Grimes wrote: > [ Charset UTF-8 unsupported, converting... ] >> Author: eadler >> Date: Tue Dec 19 03:35:39 2017 >> New Revision: 326958 >> URL: https://svnweb.freebsd.org/changeset/bas

svn commit: r326960 - head/stand/common

2017-12-18 Thread Warner Losh
Author: imp Date: Tue Dec 19 04:05:43 2017 New Revision: 326960 URL: https://svnweb.freebsd.org/changeset/base/326960 Log: Simplify things a little. The RETURN macro isn't required. It's only used once, inside an #ifdef where it would be defined to be return. Sponsored by: Netflix Modifi

svn commit: r326962 - in head/stand/i386: . boot2 gptboot libfirewire libi386 loader zfsboot

2017-12-18 Thread Warner Losh
Author: imp Date: Tue Dec 19 04:06:02 2017 New Revision: 326962 URL: https://svnweb.freebsd.org/changeset/base/326962 Log: Hoist btx include stuff to i386/Makefile.inc Sponsored by: Netflix Modified: head/stand/i386/Makefile.inc head/stand/i386/boot2/Makefile head/stand/i386/gptboot/

svn commit: r326961 - in head/stand: common efi/loader i386/loader mips/beri/loader ofw/common powerpc/kboot powerpc/ps3 sparc64/loader uboot/common userboot/userboot

2017-12-18 Thread Warner Losh
Author: imp Date: Tue Dec 19 04:05:55 2017 New Revision: 326961 URL: https://svnweb.freebsd.org/changeset/base/326961 Log: Interact is always called with NULL. Simplify code a little by removing this argument, and expanding when rc is NULL. This effectively completes the back out of custom s

svn commit: r326963 - head/stand/i386/libi386

2017-12-18 Thread Warner Losh
Author: imp Date: Tue Dec 19 04:06:07 2017 New Revision: 326963 URL: https://svnweb.freebsd.org/changeset/base/326963 Log: No need to use relative paths like this here. Sponsored by: Netflix Modified: head/stand/i386/libi386/multiboot.c Modified: head/stand/i386/libi386/multiboot.c

svn commit: r326964 - head/sys/cam

2017-12-18 Thread Warner Losh
Author: imp Date: Tue Dec 19 04:13:22 2017 New Revision: 326964 URL: https://svnweb.freebsd.org/changeset/base/326964 Log: When doing a dump, the scheduler is normally not running, so this changed worked to capture dumps for me. However, the test for SCHEDULER_STOPPED() isn't right. We can a

Re: svn commit: r326959 - head

2017-12-18 Thread Conrad Meyer
Thanks for fixing it. Conrad On Mon, Dec 18, 2017 at 7:38 PM Eitan Adler wrote: > Author: eadler > Date: Tue Dec 19 03:38:06 2017 > New Revision: 326959 > URL: https://svnweb.freebsd.org/changeset/base/326959 > > Log: > arc lint: ignore /tests/ in chmod > > shell scripts in scripts don't ne