Re: Call for Testing: Switching back to our BSD licensed dtc(1)

2016-07-21 Thread Warner Losh 
On Wed, Jul 20, 2016 at 9:51 AM, David Chisnall  wrote:
> On 20 Jul 2016, at 16:46, Warner Losh  wrote:
>>
>> I've been trying to get the final spec for it. Right now it's a
>> disorganized series
>> of patches, some of which have been merge some that haven't. I'll send you a
>> copy when I can find something better than "here's the code."
>
> Thanks.  From the information I can find, it looks as if most of the 
> machinery required to implement it is already in dtc, so it should 
> (hopefully) just be a matter of adding a new keyword to detect plugins, a 
> scan to find the cross-references (or possibly reusing the existing one) and 
> then a little bit of extra logic.

https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/overlay-notes.txt
has the specs.

Warner
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: kernel no longer builds without nvme

2016-07-21 Thread Michael Butler 
On 07/21/16 21:46, Warner Losh wrote:

>> On Jul 21, 2016, at 11:53 AM, Ngie Cooper (yaneurabeya) 
>>  wrote:
>>> On Jul 21, 2016, at 06:45, Michael Butler  
>>> wrote:
>>>
>>> For those of us who build "minimal" kernels:
>>>
>>> Building /usr/obj/usr/src/sys/VM02/vers.c
>>> Building /usr/obj/usr/src/sys/VM02/vers.o
>>> Building /usr/obj/usr/src/sys/VM02/kernel
>>> --- kernel ---
>>> linking kernel
>>> cam_xpt.o: In function `xpt_announce_periph':
>>> /usr/src/sys/cam/cam_xpt.c:(.text+0xcd2): undefined reference to
>>> `nvme_print_ident'

 [ .. snip .. ]

>> CCing Scott and Warner. This is one of many reports I’ve seen related to 
>> this issue.
> 
> Fixed in https://svnweb.freebsd.org/changeset/base/303165 . Sorry about the 
> mixup.
> I didn’t even think to test this case.

Thanks - it wasn't a big deal; I knew what I could do to work around it,

imb


___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: kernel no longer builds without nvme

2016-07-21 Thread Warner Losh 

> On Jul 21, 2016, at 11:53 AM, Ngie Cooper (yaneurabeya) 
>  wrote:
> 
>> 
>> On Jul 21, 2016, at 06:45, Michael Butler  
>> wrote:
>> 
>> For those of us who build "minimal" kernels:
>> 
>> Building /usr/obj/usr/src/sys/VM02/vers.c
>> Building /usr/obj/usr/src/sys/VM02/vers.o
>> Building /usr/obj/usr/src/sys/VM02/kernel
>> --- kernel ---
>> linking kernel
>> cam_xpt.o: In function `xpt_announce_periph':
>> /usr/src/sys/cam/cam_xpt.c:(.text+0xcd2): undefined reference to
>> `nvme_print_ident'
>> cam_xpt.o: In function `xpt_denounce_periph':
>> /usr/src/sys/cam/cam_xpt.c:(.text+0xe76): undefined reference to
>> `nvme_print_ident'
>> cam_xpt.o: In function `xpt_run_devq':
>> /usr/src/sys/cam/cam_xpt.c:(.text+0x2bc0): undefined reference to
>> `nvme_op_string'
>> /usr/src/sys/cam/cam_xpt.c:(.text+0x2bd8): undefined reference to
>> `nvme_cmd_string'
>> cam_xpt.o: In function `xpt_bus_register':
>> /usr/src/sys/cam/cam_xpt.c:(.text+0x620b): undefined reference to
>> `nvme_get_xport'
>> *** [kernel] Error code 1
> 
> CCing Scott and Warner. This is one of many reports I’ve seen related to this 
> issue.

Fixed in https://svnweb.freebsd.org/changeset/base/303165 . Sorry about the 
mixup.
I didn’t even think to test this case.

Warner


___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Jenkins build is still unstable: FreeBSD_HEAD #486

2016-07-21 Thread jenkins-admin 
See 

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Jenkins build is still unstable: FreeBSD_HEAD #485

2016-07-21 Thread jenkins-admin 
See 

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: Who broke world and how to fix?

2016-07-21 Thread Steven G. Kargl 
On Thu, Jul 21, 2016 at 03:50:48PM -0400, Michael Butler wrote:
> On 07/21/16 14:32, Steve Kargl wrote:
> > Updating uname -a
> > FreeBSD 11.0-ALPHA5 FreeBSD 11.0-ALPHA5
> > #3 r302248: Tue Jun 28 10:11:31 PDT 2016
> > data/obj/usr/src/sys/SPEW  amd64
> > 
> > to top-of-tree
> > 
> > ===> lib/libsbuf (obj,all,install)
> > cc   -O2 -pipe   -MD  -MF.depend.subr_prf.o -MTsubr_prf.o -std=gnu99 \
> > -fstack-protector-strong -Wsystem-headers -Werror -Wall -Wno-format-y2k \
> > -W -Wno-unused-parameter -Wstrict-prototypes -Wmissing-prototypes \
> > -Wpointer-arith -Wreturn-type -Wcast-qual -Wwrite-strings -Wswitch \
> >  -Wshadow -Wunused-parameter -Wcast-align -Wchar-subscripts -Winline \
> > -Wnested-externs -Wredundant-decls -Wold-style-definition \
> > -Wno-pointer-sign -Wmissing-variable-declarations -Wthread-safety \
> > -Wno-empty-body -Wno-string-plus-int -Wno-unused-const-variable  \
> > -Qunused-arguments  -c /usr/src/lib/libsbuf/../../sys/kern/subr_prf.c \
> > -o subr_prf.o
> > /usr/src/lib/libsbuf/../../sys/kern/subr_prf.c:1209:7: error: implicit
> >  declaration of function 'atomic_cmpset_int' is invalid in C99
> >  [-Werror,-Wimplicit-function-declaration]
> > if (atomic_cmpset_int(counter, c, c - 1)) {
> > ^
> > 
> 
> It's fixed at and after SVN r303155,
> 

Thanks.  I found r303155 about 5 minutes ago after scurry to
the svn-src-head mailing list.

Now, if only the guilty party would fix 'make buildkernel'.

--- kernel.full ---
linking kernel.full
cam_xpt.o: In function `xpt_announce_periph':
/usr/src/sys/cam/cam_xpt.c:1037: undefined reference to `nvme_print_ident'
cam_xpt.o: In function `xpt_denounce_periph':
/usr/src/sys/cam/cam_xpt.c:1092: undefined reference to `nvme_print_ident'
cam_xpt.o: In function `xpt_run_devq':
/usr/src/sys/cam/cam_xpt.c:3331: undefined reference to `nvme_op_string'
/usr/src/sys/cam/cam_xpt.c:3331: undefined reference to `nvme_cmd_string'
cam_xpt.o: In function `xpt_bus_register':
/usr/src/sys/cam/cam_xpt.c:3927: undefined reference to `nvme_get_xport'
*** [kernel.full] Error code 1

Two-for-two today.

-- 
Steve
http://troutmask.apl.washington.edu/~kargl/
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

FreeBSD_HEAD_i386 - Build #3653 - Fixed

2016-07-21 Thread jenkins-admin 
FreeBSD_HEAD_i386 - Build #3653 - Fixed:

Build information: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/3653/
Full change log: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/3653/changes
Full build log: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/3653/console

Change summaries:

303158 by emaste:
libmd: strip local absolute symbols (to link with lld)

Old versions of gas produce an invalid section index. That is ignored by
old versions of ld, but prevents a link with lld.

Submitted by:   Rafael Ávila de Espíndola (earlier version)
Reviewed by:allanjude
Sponsored by:   The FreeBSD Foundation
Differential Revision:  https://reviews.freebsd.org/D6789

303157 by emaste:
libcxxrt: add padding in __cxa_allocate_* to fix alignment

The addition of the referenceCount to __cxa_allocate_exception put the
unwindHeader at offset 0x58 in __cxa_exception, but it requires 16-byte
alignment. In order to avoid changing the current __cxa_exception ABI
(and thus breaking its consumers), add explicit padding in the
allocation routines (and account for it when freeing).

This is intended as a lower-risk change for FreeBSD 11. A "more correct"
fix should be prepared for upstream and -CURRENT.

Reviewed by:dim
MFC after:  3 days
Sponsored by:   The FreeBSD Foundation
Differential Revision:  https://reviews.freebsd.org/D7271

303156 by emaste:
Remove duplicate symbols from libroken version-script.map

Upstream commit r24759 (efed563) prefixed some symbols with rk_, but
introduced 6 duplicate symbols in the version script (because the
rk_-prefixed versions of the symbols were already present).

303155 by kib:
Hide counted_warning(9) under #ifdef _KERNEL braces, to allow building
subr_prf.c in userspace for libsbuf.

Sponsored by:   The FreeBSD Foundation
MFC after:  1 week

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: Who broke world and how to fix?

2016-07-21 Thread Michael Butler 
On 07/21/16 15:54, Steven G. Kargl wrote:
> On Thu, Jul 21, 2016 at 03:50:48PM -0400, Michael Butler wrote:
>> On 07/21/16 14:32, Steve Kargl wrote:
>>> Updating uname -a
>>> FreeBSD 11.0-ALPHA5 FreeBSD 11.0-ALPHA5
>>> #3 r302248: Tue Jun 28 10:11:31 PDT 2016
>>> data/obj/usr/src/sys/SPEW  amd64
>>>
>>> to top-of-tree
>>>
>>> ===> lib/libsbuf (obj,all,install)
>>> cc   -O2 -pipe   -MD  -MF.depend.subr_prf.o -MTsubr_prf.o -std=gnu99 \
>>> -fstack-protector-strong -Wsystem-headers -Werror -Wall -Wno-format-y2k \
>>> -W -Wno-unused-parameter -Wstrict-prototypes -Wmissing-prototypes \
>>> -Wpointer-arith -Wreturn-type -Wcast-qual -Wwrite-strings -Wswitch \
>>>  -Wshadow -Wunused-parameter -Wcast-align -Wchar-subscripts -Winline \
>>> -Wnested-externs -Wredundant-decls -Wold-style-definition \
>>> -Wno-pointer-sign -Wmissing-variable-declarations -Wthread-safety \
>>> -Wno-empty-body -Wno-string-plus-int -Wno-unused-const-variable  \
>>> -Qunused-arguments  -c /usr/src/lib/libsbuf/../../sys/kern/subr_prf.c \
>>> -o subr_prf.o
>>> /usr/src/lib/libsbuf/../../sys/kern/subr_prf.c:1209:7: error: implicit
>>>  declaration of function 'atomic_cmpset_int' is invalid in C99
>>>  [-Werror,-Wimplicit-function-declaration]
>>> if (atomic_cmpset_int(counter, c, c - 1)) {
>>> ^
>>>
>>
>> It's fixed at and after SVN r303155,
>>
> 
> Thanks.  I found r303155 about 5 minutes ago after scurry to
> the svn-src-head mailing list.
> 
> Now, if only the guilty party would fix 'make buildkernel'.
> 
> --- kernel.full ---
> linking kernel.full
> cam_xpt.o: In function `xpt_announce_periph':
> /usr/src/sys/cam/cam_xpt.c:1037: undefined reference to `nvme_print_ident'
> cam_xpt.o: In function `xpt_denounce_periph':
> /usr/src/sys/cam/cam_xpt.c:1092: undefined reference to `nvme_print_ident'
> cam_xpt.o: In function `xpt_run_devq':
> /usr/src/sys/cam/cam_xpt.c:3331: undefined reference to `nvme_op_string'
> /usr/src/sys/cam/cam_xpt.c:3331: undefined reference to `nvme_cmd_string'
> cam_xpt.o: In function `xpt_bus_register':
> /usr/src/sys/cam/cam_xpt.c:3927: undefined reference to `nvme_get_xport'
> *** [kernel.full] Error code 1
> 
> Two-for-two today.
> 

I just added "device nvme" to my kernel config pending the real fix,

imb

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: Who broke world and how to fix?

2016-07-21 Thread Michael Butler 
On 07/21/16 14:32, Steve Kargl wrote:
> Updating uname -a
> FreeBSD 11.0-ALPHA5 FreeBSD 11.0-ALPHA5
> #3 r302248: Tue Jun 28 10:11:31 PDT 2016
> data/obj/usr/src/sys/SPEW  amd64
> 
> to top-of-tree
> 
> ===> lib/libsbuf (obj,all,install)
> cc   -O2 -pipe   -MD  -MF.depend.subr_prf.o -MTsubr_prf.o -std=gnu99 \
> -fstack-protector-strong -Wsystem-headers -Werror -Wall -Wno-format-y2k \
> -W -Wno-unused-parameter -Wstrict-prototypes -Wmissing-prototypes \
> -Wpointer-arith -Wreturn-type -Wcast-qual -Wwrite-strings -Wswitch \
>  -Wshadow -Wunused-parameter -Wcast-align -Wchar-subscripts -Winline \
> -Wnested-externs -Wredundant-decls -Wold-style-definition \
> -Wno-pointer-sign -Wmissing-variable-declarations -Wthread-safety \
> -Wno-empty-body -Wno-string-plus-int -Wno-unused-const-variable  \
> -Qunused-arguments  -c /usr/src/lib/libsbuf/../../sys/kern/subr_prf.c \
> -o subr_prf.o
> /usr/src/lib/libsbuf/../../sys/kern/subr_prf.c:1209:7: error: implicit
>  declaration of function 'atomic_cmpset_int' is invalid in C99
>  [-Werror,-Wimplicit-function-declaration]
> if (atomic_cmpset_int(counter, c, c - 1)) {
> ^
> 

It's fixed at and after SVN r303155,

imb

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: svn: Skipped '.' Summary of conflicts: Skipped paths: 1

2016-07-21 Thread Benjamin Kaduk 
On Thu, 21 Jul 2016, Julian H. Stacey wrote:

> Hi, Reference:
> > From:   Benjamin Kaduk 
> > Date:   Thu, 21 Jul 2016 12:19:43 -0400 (EDT)
>
> Benjamin Kaduk wrote:
>
> OK, it wasn't an svn problem then, I was maybe missing meta data,
> & have learnt status command available, & I should have tried harder ;-)
> Thanks Ben ! Sorry for noise to others.

FWIW, you can also pass another argument to svn co that tells it what to
name the directory with the checkout, so you could (e.g.):

cd /usr
svn co -q file:///usr/svn/base/head src

-Ben
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Who broke world and how to fix?

2016-07-21 Thread Steve Kargl 
Updating uname -a
FreeBSD 11.0-ALPHA5 FreeBSD 11.0-ALPHA5
#3 r302248: Tue Jun 28 10:11:31 PDT 2016
data/obj/usr/src/sys/SPEW  amd64

to top-of-tree

===> lib/libsbuf (obj,all,install)
cc   -O2 -pipe   -MD  -MF.depend.subr_prf.o -MTsubr_prf.o -std=gnu99 \
-fstack-protector-strong -Wsystem-headers -Werror -Wall -Wno-format-y2k \
-W -Wno-unused-parameter -Wstrict-prototypes -Wmissing-prototypes \
-Wpointer-arith -Wreturn-type -Wcast-qual -Wwrite-strings -Wswitch \
 -Wshadow -Wunused-parameter -Wcast-align -Wchar-subscripts -Winline \
-Wnested-externs -Wredundant-decls -Wold-style-definition \
-Wno-pointer-sign -Wmissing-variable-declarations -Wthread-safety \
-Wno-empty-body -Wno-string-plus-int -Wno-unused-const-variable  \
-Qunused-arguments  -c /usr/src/lib/libsbuf/../../sys/kern/subr_prf.c \
-o subr_prf.o
/usr/src/lib/libsbuf/../../sys/kern/subr_prf.c:1209:7: error: implicit
 declaration of function 'atomic_cmpset_int' is invalid in C99
 [-Werror,-Wimplicit-function-declaration]
if (atomic_cmpset_int(counter, c, c - 1)) {
^

-- 
Steve
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Jenkins build is unstable: FreeBSD_HEAD #484

2016-07-21 Thread jenkins-admin 
See 

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: kernel no longer builds without nvme

2016-07-21 Thread Ngie Cooper (yaneurabeya) 

> On Jul 21, 2016, at 06:45, Michael Butler  wrote:
> 
> For those of us who build "minimal" kernels:
> 
> Building /usr/obj/usr/src/sys/VM02/vers.c
> Building /usr/obj/usr/src/sys/VM02/vers.o
> Building /usr/obj/usr/src/sys/VM02/kernel
> --- kernel ---
> linking kernel
> cam_xpt.o: In function `xpt_announce_periph':
> /usr/src/sys/cam/cam_xpt.c:(.text+0xcd2): undefined reference to
> `nvme_print_ident'
> cam_xpt.o: In function `xpt_denounce_periph':
> /usr/src/sys/cam/cam_xpt.c:(.text+0xe76): undefined reference to
> `nvme_print_ident'
> cam_xpt.o: In function `xpt_run_devq':
> /usr/src/sys/cam/cam_xpt.c:(.text+0x2bc0): undefined reference to
> `nvme_op_string'
> /usr/src/sys/cam/cam_xpt.c:(.text+0x2bd8): undefined reference to
> `nvme_cmd_string'
> cam_xpt.o: In function `xpt_bus_register':
> /usr/src/sys/cam/cam_xpt.c:(.text+0x620b): undefined reference to
> `nvme_get_xport'
> *** [kernel] Error code 1

CCing Scott and Warner. This is one of many reports I’ve seen related to this 
issue.
Thanks,
-Ngie


signature.asc
Description: Message signed with OpenPGP using GPGMail

FreeBSD_HEAD_i386 - Build #3652 - Failure

2016-07-21 Thread jenkins-admin 
FreeBSD_HEAD_i386 - Build #3652 - Failure:

Build information: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/3652/
Full change log: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/3652/changes
Full build log: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/3652/console

Change summaries:

303154 by kib:
Declare aio requests on files from local filesystems safe.
Two notes:
- I allow AIO on reclaimed vnodes, since it is deterministically terminated
  fast.
- devfs mounts are marked as MNT_LOCAL, but device vnodes have type
  VCHR, so the slow device io is not allowed.

Reviewed by:jhb
Sponsored by:   The FreeBSD Foundation
MFC after:  1 week
Differential revision:  https://reviews.freebsd.org/D7273

303153 by kib:
Remove spurious but harmless double-quote.

Noted by:   jhb

303151 by kib:
Provide counter_warning(9) KPI which allows to issue limited number of
warnings for some kernel events, mostly intended for the use of
obsoleted or otherwise undersired interfaces.

This is an abstracted and race-expelled code from compat pty driver.

Requested and reviewed by:  jhb
Sponsored by:   The FreeBSD Foundation
MFC after:  1 week
Differential revision:  https://reviews.freebsd.org/D7270

303150 by adrian:
[mx25l] add new device id.

Tested:

* (submitter) TP-Link TL-WR1043nd v3

Submitted by:   t...@enoti.me

303147 by pfg:
binutils: fix "Bad value" error in bfd for MIPS when using -Bsymbolic.

>From OpenBSD's log:

Inspired by https://sourceware.org/ml/binutils/2010-08/msg00333.html,
but expressed differently so there are no GPLv3 issues.

Obtained from:  OpenBSD (CVS rev. 1.7)
MFC after:  1 month



The end of the build log:

[...truncated 39186 lines...]
--- skein256hl.So ---
cc  -fpic -DPIC -g -O2 -pipe -I/usr/src/lib/libmd 
-I/usr/src/lib/libmd/../../sys/crypto/sha2 
-I/usr/src/lib/libmd/../../sys/crypto/skein -DWEAK_REFS -DSHA1_ASM -DRMD160_ASM 
  -MD  -MF.depend.skein256hl.So -MTskein256hl.So -std=gnu99 
-fstack-protector-strong -Wno-pointer-sign -Wno-empty-body -Wno-string-plus-int 
-Wno-unused-const-variable -Wno-tautological-compare -Wno-unused-value 
-Wno-parentheses-equality -Wno-unused-function -Wno-enum-conversion 
-Wno-unused-local-typedef -Wno-switch -Wno-switch-enum 
-Wno-knr-promoted-parameter -Wno-parentheses  -Qunused-arguments  -c 
skein256hl.c -o skein256hl.So
--- lib/ncurses/ncursesw__L ---
--- lib_slk.o ---
cc   -O2 -pipe   -D_XOPEN_SOURCE_EXTENDED -DENABLE_WIDEC -I. 
-I/usr/obj/usr/src/lib/ncurses/ncursesw/../ncursesw 
-I/usr/src/lib/ncurses/ncursesw/../ncursesw 
-I/usr/src/lib/ncurses/ncursesw/../ncurses 
-I/usr/src/lib/ncurses/ncursesw/../../../contrib/ncurses/include 
-I/usr/src/lib/ncurses/ncursesw/../../../contrib/ncurses/ncurses -Wall -DNDEBUG 
-DHAVE_CONFIG_H -DFREEBSD_NATIVE -DTERMIOS -MD  -MF.depend.lib_slk.o 
-MTlib_slk.o -std=gnu99 -fstack-protector-strong -Wsystem-headers -Werror -Wall 
-Wno-format-y2k -W -Wno-unused-parameter -Wstrict-prototypes 
-Wmissing-prototypes -Wpointer-arith -Wno-uninitialized -Wno-pointer-sign 
-Wno-empty-body -Wno-string-plus-int -Wno-unused-const-variable 
-Wno-tautological-compare -Wno-unused-value -Wno-parentheses-equality 
-Wno-unused-function -Wno-enum-conversion -Wno-unused-local-typedef  
-Qunused-arguments  -c 
/usr/src/lib/ncurses/ncursesw/../../../contrib/ncurses/ncurses/base/lib_slk.c 
-o lib_slk.o
--- lib/ncurses/ncurses__L ---
--- lib_wattroff.o ---
cc   -O2 -pipe   -I. -I/usr/obj/usr/src/lib/ncurses/ncurses/../ncurses 
-I/usr/src/lib/ncurses/ncurses/../ncurses 
-I/usr/src/lib/ncurses/ncurses/../ncurses 
-I/usr/src/lib/ncurses/ncurses/../../../contrib/ncurses/include 
-I/usr/src/lib/ncurses/ncurses/../../../contrib/ncurses/ncurses -Wall -DNDEBUG 
-DHAVE_CONFIG_H -DFREEBSD_NATIVE -DTERMIOS -MD  -MF.depend.lib_wattroff.o 
-MTlib_wattroff.o -std=gnu99 -fstack-protector-strong -Wsystem-headers -Werror 
-Wall -Wno-format-y2k -W -Wno-unused-parameter -Wstrict-prototypes 
-Wmissing-prototypes -Wpointer-arith -Wno-uninitialized -Wno-pointer-sign 
-Wno-empty-body -Wno-string-plus-int -Wno-unused-const-variable 
-Wno-tautological-compare -Wno-unused-value -Wno-parentheses-equality 
-Wno-unused-function -Wno-enum-conversion -Wno-unused-local-typedef  
-Qunused-arguments  -c 
/usr/src/lib/ncurses/ncurses/../../../contrib/ncurses/ncurses/base/lib_wattroff.c
 -o lib_wattroff.o
--- lib/ncurses/ncursesw__L ---
--- lib_slkatr_set.o ---
cc   -O2 -pipe   -D_XOPEN_SOURCE_EXTENDED -DENABLE_WIDEC -I. 
-I/usr/obj/usr/src/lib/ncurses/ncursesw/../ncursesw 
-I/usr/src/lib/ncurses/ncursesw/../ncursesw 
-I/usr/src/lib/ncurses/ncursesw/../ncurses 
-I/usr/src/lib/ncurses/ncursesw/../../../contrib/ncurses/include 
-I/usr/src/lib/ncurses/ncursesw/../../../contrib/ncurses/ncurses -Wall -DNDEBUG 
-DHAVE_CONFIG_H -DFREEBSD_NATIVE -DTERMIOS -MD  -MF.depend.lib_slkatr_set.o 
-MTlib_slkatr_set.o -std=gnu99 -fstack-protector-strong -Wsystem-headers 
-Werror -Wall -Wno-format-y2k -W -Wno-unused-parameter -Wstrict-prototypes

Re: svn: Skipped '.' Summary of conflicts: Skipped paths: 1

2016-07-21 Thread Julian H. Stacey 
Hi, Reference:
> From: Benjamin Kaduk 
> Date: Thu, 21 Jul 2016 12:19:43 -0400 (EDT)

Benjamin Kaduk wrote:
> On Thu, 21 Jul 2016, Julian H. Stacey wrote:
> 
> > Hi current@
> > I've seen this ### about 3 times lately, anyone else ?
> > uname -a
> > FreeBSD lapr.js.berklix.net 12.0-CURRENT FreeBSD
> > 12.0-CURRENT #1 r302560M: Wed Jul 13 01:28:27 CEST
> > 2016
> > j...@lapr.js.berklix.net:/usr/src/sys/amd64/compile/LAPR.small
> > amd64
> >
> > svn co -q file:///usr/svn/base/head
> > svn up
> > ### Skipped '.'
> > ### Summary of conflicts:
> > ###   Skipped paths: 1
> 
> Does 'svn st' have anything interesting to say?
> -Ben

cd /usr/src
svn st
svn: warning: W155007: '/data/release/12.0-CURRENT/usr/src' is not a 
working copy
I had removed content ready for a rebuild
ls -la
total 12
drwxr-xr-x   3 jhs   staff   512 Jul 21 14:43 ./
drwxr-xr-x  18 root  wheel   512 Jul 20 18:46 ../
drwxr-xr-x  24 jhs   staff  1024 Jul 21 15:32 head/
cd head
ls
# COPYRIGHT etc
svn st
# silent completion
svn up
# silent completion
pwd
/data/release/12.0-CURRENT/usr/src/head
cd ../.. ; mv src/head s ; rmdir src ; mv s src ; cd src
pwd
/data/release/12.0-CURRENT/usr/src 
svn up
Updating '.':
At revision 303135.

OK, it wasn't an svn problem then, I was maybe missing meta data,
& have learnt status command available, & I should have tried harder ;-)
Thanks Ben ! Sorry for noise to others.

Cheers,
Julian
--
Julian Stacey, BSD Linux Unix Sys Eng Consultant Munich http://berklix.eu/jhs/
 Mail plain text,  No quoted-printable, HTML, base64, MS.doc.
 Prefix old lines '> '  Reply below old, like play script.  Break lines by 80.
 http://berklix.eu/brexit/#stolen_votes
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: svn: Skipped '.' Summary of conflicts: Skipped paths: 1

2016-07-21 Thread Benjamin Kaduk 
On Thu, 21 Jul 2016, Julian H. Stacey wrote:

> Hi current@
> I've seen this ### about 3 times lately, anyone else ?
>   uname -a
>   FreeBSD lapr.js.berklix.net 12.0-CURRENT FreeBSD
>   12.0-CURRENT #1 r302560M: Wed Jul 13 01:28:27 CEST
>   2016
>   j...@lapr.js.berklix.net:/usr/src/sys/amd64/compile/LAPR.small
>   amd64
>
>   svn co -q file:///usr/svn/base/head
>   svn up
>   ### Skipped '.'
>   ### Summary of conflicts:
>   ###   Skipped paths: 1

Does 'svn st' have anything interesting to say?

-Ben
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

svn: Skipped '.' Summary of conflicts: Skipped paths: 1

2016-07-21 Thread Julian H. Stacey 
Hi current@
I've seen this ### about 3 times lately, anyone else ?
uname -a
FreeBSD lapr.js.berklix.net 12.0-CURRENT FreeBSD
12.0-CURRENT #1 r302560M: Wed Jul 13 01:28:27 CEST
2016
j...@lapr.js.berklix.net:/usr/src/sys/amd64/compile/LAPR.small
amd64

svn co -q file:///usr/svn/base/head
svn up
### Skipped '.'
### Summary of conflicts:
###   Skipped paths: 1

ls -l `which svn`
-rwxr-xr-x  1 root  wheel  259568 May 24  2015 
/usr/local/bin/svn*

Meantime to trace it, I'm running:
cd /usr/ports/devel/subversion ; make clean ; make install
cd /data/tmp/new_svn_bin/src
script
svn co file:///usr/svn/base/head

Cheers,
Julian
-- 
Julian Stacey, BSD Linux Unix Sys Eng Consultant Munich http://berklix.eu/jhs/
 Mail plain text,  No quoted-printable, HTML, base64, MS.doc.
 Prefix old lines '> '  Reply below old, like play script.  Break lines by 80.
 http://berklix.eu/brexit/#stolen_votes
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Build failed in Jenkins: FreeBSD_HEAD #483

2016-07-21 Thread jenkins-admin 
See 

--
[...truncated 275395 lines...]
--- vnode_if_newproto.h ---
awk -f /builds/workspace/FreeBSD_HEAD/src/sys/tools/vnode_if.awk 
/builds/workspace/FreeBSD_HEAD/src/sys/kern/vnode_if.src -p
--- vnode_if_typedef.h ---
awk -f /builds/workspace/FreeBSD_HEAD/src/sys/tools/vnode_if.awk 
/builds/workspace/FreeBSD_HEAD/src/sys/kern/vnode_if.src -q
--- vnode_if.h ---
awk -f /builds/workspace/FreeBSD_HEAD/src/sys/tools/vnode_if.awk 
/builds/workspace/FreeBSD_HEAD/src/sys/kern/vnode_if.src -h
--- udf_vnops.o ---
cc  -O2 -pipe  -fno-strict-aliasing -Werror -D_KERNEL -DKLD_MODULE -nostdinc   
-DHAVE_KERNEL_OPTION_HEADERS -include 
/builds/workspace/FreeBSD_HEAD/obj/builds/workspace/FreeBSD_HEAD/src/sys/GENERIC/opt_global.h
 -I. -I/builds/workspace/FreeBSD_HEAD/src/sys -fno-common -g 
-fno-omit-frame-pointer -mno-omit-leaf-frame-pointer 
-I/builds/workspace/FreeBSD_HEAD/obj/builds/workspace/FreeBSD_HEAD/src/sys/GENERIC
  -MD  -MF.depend.udf_vnops.o -MTudf_vnops.o -mcmodel=kernel -mno-red-zone 
-mno-mmx -mno-sse -msoft-float  -fno-asynchronous-unwind-tables -ffreestanding 
-fwrapv -fstack-protector -gdwarf-2 -Wall -Wredundant-decls -Wnested-externs 
-Wstrict-prototypes  -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual  
-Wundef -Wno-pointer-sign -D__printf__=__freebsd_kprintf__  
-Wmissing-include-dirs -fdiagnostics-show-option  -Wno-unknown-pragmas  
-Wno-error-tautological-compare -Wno-error-empty-body  
-Wno-error-parentheses-equality -Wno-error-unused-function  
-Wno-error-pointer-sign -Wno-error
 -shift-negative-value  -mno-aes -mno-avx  -std=iso9899:1999 -c 
/builds/workspace/FreeBSD_HEAD/src/sys/modules/udf/../../fs/udf/udf_vnops.c -o 
udf_vnops.o
--- all_subdir_uart ---
ctfconvert -L VERSION -g uart_dbg.o
--- uart_dev_ns8250.o ---
cc  -O2 -pipe  -fno-strict-aliasing -Werror -D_KERNEL -DKLD_MODULE -nostdinc   
-DHAVE_KERNEL_OPTION_HEADERS -include 
/builds/workspace/FreeBSD_HEAD/obj/builds/workspace/FreeBSD_HEAD/src/sys/GENERIC/opt_global.h
 -I. -I/builds/workspace/FreeBSD_HEAD/src/sys -fno-common -g 
-fno-omit-frame-pointer -mno-omit-leaf-frame-pointer 
-I/builds/workspace/FreeBSD_HEAD/obj/builds/workspace/FreeBSD_HEAD/src/sys/GENERIC
  -MD  -MF.depend.uart_dev_ns8250.o -MTuart_dev_ns8250.o -mcmodel=kernel 
-mno-red-zone -mno-mmx -mno-sse -msoft-float  -fno-asynchronous-unwind-tables 
-ffreestanding -fwrapv -fstack-protector -gdwarf-2 -Wall -Wredundant-decls 
-Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes -Wpointer-arith 
-Winline -Wcast-qual  -Wundef -Wno-pointer-sign 
-D__printf__=__freebsd_kprintf__  -Wmissing-include-dirs 
-fdiagnostics-show-option  -Wno-unknown-pragmas  
-Wno-error-tautological-compare -Wno-error-empty-body  
-Wno-error-parentheses-equality -Wno-error-unused-function  
-Wno-error-pointer-sig
 n -Wno-error-shift-negative-value  -mno-aes -mno-avx  -std=iso9899:1999 -c 
/builds/workspace/FreeBSD_HEAD/src/sys/modules/uart/../../dev/uart/uart_dev_ns8250.c
 -o uart_dev_ns8250.o
--- all_subdir_sound ---
--- all_subdir_sound/driver/cmi ---
ctfconvert -L VERSION -g cmi.o
--- snd_cmi.ko.full ---
ld -d -warn-common -r -d -o snd_cmi.ko.full cmi.o
ctfmerge -L VERSION -g -o snd_cmi.ko.full cmi.o
:> export_syms
awk -f /builds/workspace/FreeBSD_HEAD/src/sys/conf/kmod_syms.awk 
snd_cmi.ko.full  export_syms | xargs -J% objcopy % snd_cmi.ko.full
--- snd_cmi.ko.debug ---
objcopy --only-keep-debug snd_cmi.ko.full snd_cmi.ko.debug
--- snd_cmi.ko ---
objcopy --strip-debug --add-gnu-debuglink=snd_cmi.ko.debug  snd_cmi.ko.full 
snd_cmi.ko
--- all_subdir_uart ---
--- uart_dev_quicc.o ---
cc  -O2 -pipe  -fno-strict-aliasing -Werror -D_KERNEL -DKLD_MODULE -nostdinc   
-DHAVE_KERNEL_OPTION_HEADERS -include 
/builds/workspace/FreeBSD_HEAD/obj/builds/workspace/FreeBSD_HEAD/src/sys/GENERIC/opt_global.h
 -I. -I/builds/workspace/FreeBSD_HEAD/src/sys -fno-common -g 
-fno-omit-frame-pointer -mno-omit-leaf-frame-pointer 
-I/builds/workspace/FreeBSD_HEAD/obj/builds/workspace/FreeBSD_HEAD/src/sys/GENERIC
  -MD  -MF.depend.uart_dev_quicc.o -MTuart_dev_quicc.o -mcmodel=kernel 
-mno-red-zone -mno-mmx -mno-sse -msoft-float  -fno-asynchronous-unwind-tables 
-ffreestanding -fwrapv -fstack-protector -gdwarf-2 -Wall -Wredundant-decls 
-Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes -Wpointer-arith 
-Winline -Wcast-qual  -Wundef -Wno-pointer-sign 
-D__printf__=__freebsd_kprintf__  -Wmissing-include-dirs 
-fdiagnostics-show-option  -Wno-unknown-pragmas  
-Wno-error-tautological-compare -Wno-error-empty-body  
-Wno-error-parentheses-equality -Wno-error-unused-function  
-Wno-error-pointer-sign 
 -Wno-error-shift-negative-value  -mno-aes -mno-avx  -std=iso9899:1999 -c 
/builds/workspace/FreeBSD_HEAD/src/sys/modules/uart/../../dev/uart/uart_dev_quicc.c
 -o uart_dev_quicc.o
--- all_subdir_sound ---
--- all_subdir_sound/driver/mss ---
ctfconvert -L VERSION -g gusc.o
--- mss.o ---
cc  -O2 -pipe  -fno-strict-aliasing

FreeBSD_HEAD_amd64_gcc - Build #1447 - Failure

2016-07-21 Thread jenkins-admin 
FreeBSD_HEAD_amd64_gcc - Build #1447 - Failure:

Build information: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_amd64_gcc/1447/
Full change log: 
https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_amd64_gcc/1447/changes
Full build log: 
https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_amd64_gcc/1447/console

Change summaries:

303134 by kib:
Improve typesetting.

Reviewed by:bapt
No objections from: emaste
Sponsored by:   The FreeBSD Foundation
Differential revision:  https://reviews.freebsd.org/D7261

303133 by andrew:
Add support for arm64 to uart_dev_acpi by using the _HID property to find
the uart class to use in a similar way as the fdt driver.

Obtained from:  ABT Systems Ltd
MFC after:  1 month
Sponsored by:   The FreeBSD Foundation
Differential Revision:  https://reviews.freebsd.org/D7248

303132 by trasz:
Add documentation for "ctld -u", the UCL configuration file format.

Submitted by:   jceel@
Reviewed by:wblock@
MFC after:  3 days
Differential Revision:  https://reviews.freebsd.org/D7222

303131 by sephe:
hyperv: hv_vmbus_channel -> vmbus_channel

MFC after:  1 week
Sponsored by:   Microsoft OSTC
Differential Revision:  https://reviews.freebsd.org/D7260

303130 by bdrewery:
Add LOCAL_XTOOL_DIRS to add into cross-tools.

Sponsored by:   EMC / Isilon Storage Division

303129 by sephe:
hyperv/vmbus: Shuffle function declaration and macro definition.

MFC after:  1 week
Sponsored by:   Microsoft OSTC
Differential Revision:  https://reviews.freebsd.org/D7259

303128 by sephe:
hyperv/vmbus: Reorder channel fields.

MFC after:  1 week
Sponsored by:   Microsoft OSTC
Differential Revision:  https://reviews.freebsd.org/D7258

303127 by sephe:
hyperv/vmbus: Save event flag location and evet flag mask.

This avoids unnecessary access to the vmbus_softc struct on sending path.

MFC after:  1 week
Sponsored by:   Microsoft OSTC
Differential Revision:  https://reviews.freebsd.org/D7257

303126 by imp:
Actually import nvme_sim so the CAM attachment for NVME (nda) actually
works.

MFC after: 1 week

303125 by imp:
Remove some bogus comments and printfs. Also, we can't
cam_periph_releaes_locked() at the end of nvme_probe_start because we
hit an assertion which may be bogus. Instead, leak a periph until we
sort it out. Since these devices don't arrive and depart often, so
this is the lessor of two evils.

MFC after: 1 week

303124 by imp:
Fix mismerge and add NVME xport support to devstat (or rather, exclude
it from SCSI).

MFC after: 1 week

303123 by imp:
Fix mismerge and include the nvme support.
Also, print out the name of any CCB's functions that's not supported.

MFC after: 1 week

303122 by cem:
Mirror r303120 to gcore

Sponsored by:   EMC / Isilon Storage Division

303120 by cem:
imgact_elf: Rename the segment iterator to match reality

The each_writable_segment routine evaluates segments on a slightly little more
nuanced metric than simply "writable" or not.  Rename the function to more
closely match its behavior (each_dumpable_segment).

Suggested by:   jhb
Sponsored by:   EMC / Isilon Storage Division

303119 by gjb:
Create a /usr/home -> /home symlink for the arm images to
avoid /usr/home confusingly being created as a directory.

Reported by:Russell Haley
MFC after:  3 days
Sponsored by:   The FreeBSD Foundation

303118 by cem:
ANSI-fy imgact_elf.c

Sponsored by:   EMC / Isilon Storage Division

303115 by avos:
urtwn: fix A-MSDU tx aggregation when encryption is used.

Clear unused (undocumented) CAM bytes while setting a key;
without that, hardware does weird things when A-MSDU bit in QoS header
is set.

Tested with RTL8188CUS (AP) -> RTL8188EU (STA) (A-MSDU transmit).

Reported by:many
Obtained from:  https://github.com/s3erios/urtwm
MFC after:  5 days

303114 by cem:
Add extended segment support to gcore

A follow-up to r303099, D7255.  Basically, apply the exact same change, with
the exact same rationale, to gcore.  gcore's elfcore.c is largely a clone of
the kernel imgact_elf coredump facility.

Reviewed by:emaste (earlier version, not substantially different)
Requested by:   jhb
Sponsored by:   EMC / Isilon Storage Division
Differential Revision:  https://reviews.freebsd.org/D7265

303113 by mav:
Cross-link some SMBus controller drivers man pages.

Some compatibility cases are not obvious.

303111 by mav:
Document list of supported chipsets.

303110 by sbruno:
Remove uneeded parens.

MFC after:  1 day

303109 by jhb:
Update crashinfo to work with newer gdb from ports.

If gdb from ports is installed, use it instead of the base system gdb
to extract variables from a kernel.  Note that base gdb and ports gdb
do not support the same options for invoking a single command in batch
mode, so a wrapper shell function is used.  In addition, prefer kgdb
from ports when generating a backtrace if present.

PR: 193335
Reviewed by:emaste
MFC after:  1 week
Differential Revision:  https://reviews.freebsd.org/D7218

303106 by zeising:

kernel no longer builds without nvme

2016-07-21 Thread Michael Butler 
For those of us who build "minimal" kernels:

Building /usr/obj/usr/src/sys/VM02/vers.c
Building /usr/obj/usr/src/sys/VM02/vers.o
Building /usr/obj/usr/src/sys/VM02/kernel
--- kernel ---
linking kernel
cam_xpt.o: In function `xpt_announce_periph':
/usr/src/sys/cam/cam_xpt.c:(.text+0xcd2): undefined reference to
`nvme_print_ident'
cam_xpt.o: In function `xpt_denounce_periph':
/usr/src/sys/cam/cam_xpt.c:(.text+0xe76): undefined reference to
`nvme_print_ident'
cam_xpt.o: In function `xpt_run_devq':
/usr/src/sys/cam/cam_xpt.c:(.text+0x2bc0): undefined reference to
`nvme_op_string'
/usr/src/sys/cam/cam_xpt.c:(.text+0x2bd8): undefined reference to
`nvme_cmd_string'
cam_xpt.o: In function `xpt_bus_register':
/usr/src/sys/cam/cam_xpt.c:(.text+0x620b): undefined reference to
`nvme_get_xport'
*** [kernel] Error code 1

imb
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Jenkins build is still unstable: FreeBSD_HEAD #482

2016-07-21 Thread jenkins-admin 
See 

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Jenkins build is still unstable: FreeBSD_HEAD #481

2016-07-21 Thread jenkins-admin 
See 

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: panic with tcp timers

2016-07-21 Thread Hans Petter Selasky 

On 07/21/16 09:54, Julien Charbon wrote:


 Hi,

On 7/14/16 11:02 PM, Larry Rosenman wrote:

On 2016-07-14 12:01, Julien Charbon wrote:

On 6/20/16 11:55 AM, Julien Charbon wrote:

On 6/20/16 9:39 AM, Gleb Smirnoff wrote:

On Fri, Jun 17, 2016 at 11:27:39AM +0200, Julien Charbon wrote:
J> > Comparing stable/10 and head, I see two changes that could
J> > affect that:
J> >
J> > - callout_async_drain
J> > - switch to READ lock for inp info in tcp timers
J> >
J> > That's why you are in To, Julien and Hans :)
J> >
J> > We continue investigating, and I will keep you updated.
J> > However, any help is welcome. I can share cores.

Now, spending some time with cores and adding a bunch of
extra CTRs, I have a sequence of events that lead to the
panic. In short, the bug is in the callout system. It seems
to be not relevant to the callout_async_drain, at least for
now. The transition to READ lock unmasked the problem, that's
why NetflixBSD 10 doesn't panic.

The panic requires heavy contention on the TCP info lock.

[CPU 1] the callout fires, tcp_timer_keep entered
[CPU 1] blocks on INP_INFO_RLOCK(_tcbinfo);
[CPU 2] schedules the callout
[CPU 2] tcp_discardcb called
[CPU 2] callout successfully canceled
[CPU 2] tcpcb freed
[CPU 1] unblocks... panic

When the lock was WLOCK, all contenders were resumed in a
sequence they came to the lock. Now, that they are readers,
once the lock is released, readers are resumed in a "random"
order, and this allows tcp_discardcb to go before the old
running callout, and this unmasks the panic.


 Highly interesting.  I should be able to reproduce that (will be useful
for testing the corresponding fix).


 Finally, I was able to reproduce it (without glebius fix).   The trick
was to really lower TCP keep timer expiration:

$ sysctl -a | grep tcp.keep
net.inet.tcp.keepidle: 720
net.inet.tcp.keepintvl: 75000
net.inet.tcp.keepinit: 75000
net.inet.tcp.keepcnt: 8
$ sudo bash -c "sysctl net.inet.tcp.keepidle=10 && sysctl
net.inet.tcp.keepintvl=50 && sysctl net.inet.tcp.keepinit=10"
Password:
net.inet.tcp.keepidle: 720 -> 10
net.inet.tcp.keepintvl: 75000 -> 50
net.inet.tcp.keepinit: 75000 -> 10

 Note: It will certainly close all your ssh connections to the tested
server.

 Now I will test in order:

#1. glebius fix
https://svnweb.freebsd.org/base?view=revision=302350

#2. rss extra fix
https://reviews.freebsd.org/D7135

#3. rrs TCP Timer cleanup
https://reviews.freebsd.org/D7136


please see also https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=210884


 My tests result so far:

#1. r302350:  First glebius TCP timer fix:  No more TCP timer kernel
panic during 48h under 200k TCP query per second load.

 Sadly I was unable to reproduce the issue described here:

panic: bogus refcnt 0 on lle 0xf80004608c00
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=210884

#2. r303098:  Got all kernel callout changes since r302350, (updates on
callout code are indeed always full of surprises):
https://svnweb.freebsd.org/base/head/sys/kern/kern_timeout.c?view=log=303098

 No kernel panic either.

 Still to test:

#3. rss extra fix (if still relevant now)
https://reviews.freebsd.org/D7135

#4. rrs TCP Timer cleanup:
https://reviews.freebsd.org/D7136

 My 2 cents.



Hi,

You should also check for memory leaks using "vmstat -m" .

--HPS

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: panic with tcp timers

2016-07-21 Thread Julien Charbon 

 Hi,

On 7/14/16 11:02 PM, Larry Rosenman wrote:
> On 2016-07-14 12:01, Julien Charbon wrote:
>> On 6/20/16 11:55 AM, Julien Charbon wrote:
>>> On 6/20/16 9:39 AM, Gleb Smirnoff wrote:
 On Fri, Jun 17, 2016 at 11:27:39AM +0200, Julien Charbon wrote:
 J> > Comparing stable/10 and head, I see two changes that could
 J> > affect that:
 J> >
 J> > - callout_async_drain
 J> > - switch to READ lock for inp info in tcp timers
 J> >
 J> > That's why you are in To, Julien and Hans :)
 J> >
 J> > We continue investigating, and I will keep you updated.
 J> > However, any help is welcome. I can share cores.

 Now, spending some time with cores and adding a bunch of
 extra CTRs, I have a sequence of events that lead to the
 panic. In short, the bug is in the callout system. It seems
 to be not relevant to the callout_async_drain, at least for
 now. The transition to READ lock unmasked the problem, that's
 why NetflixBSD 10 doesn't panic.

 The panic requires heavy contention on the TCP info lock.

 [CPU 1] the callout fires, tcp_timer_keep entered
 [CPU 1] blocks on INP_INFO_RLOCK(_tcbinfo);
 [CPU 2] schedules the callout
 [CPU 2] tcp_discardcb called
 [CPU 2] callout successfully canceled
 [CPU 2] tcpcb freed
 [CPU 1] unblocks... panic

 When the lock was WLOCK, all contenders were resumed in a
 sequence they came to the lock. Now, that they are readers,
 once the lock is released, readers are resumed in a "random"
 order, and this allows tcp_discardcb to go before the old
 running callout, and this unmasks the panic.
>>>
>>>  Highly interesting.  I should be able to reproduce that (will be useful
>>> for testing the corresponding fix).
>>
>>  Finally, I was able to reproduce it (without glebius fix).   The trick
>> was to really lower TCP keep timer expiration:
>>
>> $ sysctl -a | grep tcp.keep
>> net.inet.tcp.keepidle: 720
>> net.inet.tcp.keepintvl: 75000
>> net.inet.tcp.keepinit: 75000
>> net.inet.tcp.keepcnt: 8
>> $ sudo bash -c "sysctl net.inet.tcp.keepidle=10 && sysctl
>> net.inet.tcp.keepintvl=50 && sysctl net.inet.tcp.keepinit=10"
>> Password:
>> net.inet.tcp.keepidle: 720 -> 10
>> net.inet.tcp.keepintvl: 75000 -> 50
>> net.inet.tcp.keepinit: 75000 -> 10
>>
>>  Note: It will certainly close all your ssh connections to the tested
>> server.
>>
>>  Now I will test in order:
>>
>> #1. glebius fix
>> https://svnweb.freebsd.org/base?view=revision=302350
>>
>> #2. rss extra fix
>> https://reviews.freebsd.org/D7135
>>
>> #3. rrs TCP Timer cleanup
>> https://reviews.freebsd.org/D7136
> 
> please see also https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=210884

 My tests result so far:

#1. r302350:  First glebius TCP timer fix:  No more TCP timer kernel
panic during 48h under 200k TCP query per second load.

 Sadly I was unable to reproduce the issue described here:

panic: bogus refcnt 0 on lle 0xf80004608c00
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=210884

#2. r303098:  Got all kernel callout changes since r302350, (updates on
callout code are indeed always full of surprises):
https://svnweb.freebsd.org/base/head/sys/kern/kern_timeout.c?view=log=303098

 No kernel panic either.

 Still to test:

#3. rss extra fix (if still relevant now)
https://reviews.freebsd.org/D7135

#4. rrs TCP Timer cleanup:
https://reviews.freebsd.org/D7136

 My 2 cents.

--
Julien



signature.asc
Description: OpenPGP digital signature

CURRENT r303131: builkernel failure xpt: undefined reference to `nvme_print_ident'

2016-07-21 Thread O. Hartmann 
Recent CURRENT (r303131) can not build a kernel and fails due to the error shown
below:


[...]
cc -target x86_64-unknown-freebsd12.0
--sysroot=/empty/nanoobj/FAFNER_amd64/empty/CURRENT/src/tmp
-B/empty/nanoobj/FAFNER_amd64/empty/CURRENT/src/tmp/usr/bin -c -O2 -pipe
-fno-strict-aliasing   -nostdinc  -I. -I/empty/CURRENT/src/sys
-I/empty/CURRENT/src/sys/contrib/libfdt -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS
-include opt_global.h   -mcmodel=kernel -mno-red-zone -mno-mmx -mno-sse
-msoft-float  -fno-asynchronous-unwind-tables -ffreestanding -fwrapv
-fstack-protector -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes
-Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual  -Wundef
-Wno-pointer-sign -D__printf__=__freebsd_kprintf__  -Wmissing-include-dirs
-fdiagnostics-show-option  -Wno-unknown-pragmas
-Wno-error-tautological-compare -Wno-error-empty-body
-Wno-error-parentheses-equality -Wno-error-unused-function
-Wno-error-pointer-sign -Wno-error-shift-negative-value  -mno-aes -mno-avx
-std=iso9899:1999 -Werror  vers.c --- kernel --- linking kernel cam_xpt.o: In
function
`xpt_announce_periph': /empty/CURRENT/src/sys/cam/cam_xpt.c:(.text+0xac2):
undefined reference to `nvme_print_ident' cam_xpt.o: In function
`xpt_denounce_periph': /empty/CURRENT/src/sys/cam/cam_xpt.c:(.text+0xc66):
undefined reference to `nvme_print_ident' cam_xpt.o: In function
`xpt_run_devq': /empty/CURRENT/src/sys/cam/cam_xpt.c:(.text+0x270f): undefined
reference to
`nvme_op_string' /empty/CURRENT/src/sys/cam/cam_xpt.c:(.text+0x2727): undefined
reference to `nvme_cmd_string' cam_xpt.o: In function
`xpt_bus_register': /empty/CURRENT/src/sys/cam/cam_xpt.c:(.text+0x5211):
undefined reference to `nvme_get_xport' *** [kernel] Error code 1
[...]


Kind regards,

oh
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Jenkins build is still unstable: FreeBSD_HEAD #480

2016-07-21 Thread jenkins-admin 
See 

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"