Re: ld.so: -fno-builtin?

2016-12-22 Thread Philip Guenther
On Thu, Dec 22, 2016 at 12:33 PM, Joerg Sonnenberger wrote: > On Thu, Dec 22, 2016 at 05:47:05PM +0100, Christian Weisgerber wrote: >> Building ld.so with clang on amd64 fails with undefined references >> to memset and memcpy. That is odd, since neither function appears >> in the

Re: ospf6d: handle interface MTU changes

2016-12-22 Thread Remi Locherer
On Wed, Dec 21, 2016 at 12:08:23PM +0100, Jeremie Courreges-Anglas wrote: > > Hi, > > After ospfd here's a diff to make ospf6d refresh his view of an > interface's MTU at runtime. This needs a fresh kernel. > > The parent should pass the IFINFO message to its children first, and > then decide

Interrupt race in NET_LOCK/NET_UNLOCK

2016-12-22 Thread Visa Hankala
NET_LOCK() should raise IPL before acquiring the lock, and NET_UNLOCK() should restore the level after releasing the lock. Otherwise, lock recursion can occur, most likely right after the splx(). An example: nd6_slowtimo <- NET_LOCK() timeout_run softclock softintr_dispatch dosoftint interrupt

Re: if attach/detach netlocks

2016-12-22 Thread Martin Pieuchot
On 22/12/16(Thu) 20:45, Mike Belopuhov wrote: > On Wed, Dec 21, 2016 at 13:06 +0100, Alexander Bluhm wrote: > > On Wed, Dec 21, 2016 at 12:45:37PM +0100, Mike Belopuhov wrote: > > > Anyways, OK for the diff below? > > > > OK bluhm@ > > > > > diff --git sys/net/if_pflow.c sys/net/if_pflow.c > > >

Re: splassert: ip_output: want 1 have 0

2016-12-22 Thread Mark Kettenis
> Date: Thu, 22 Dec 2016 14:56:43 +0100 > From: Martin Pieuchot > > On 22/12/16(Thu) 10:45, Martin Pieuchot wrote: > > On 22/12/16(Thu) 00:32, Mark Kettenis wrote: > > > splassert: ip_output: want 1 have 0 > > > Starting stack trace... > > > ip_output() at ip_output+0x7d > > >

Re: ld.so: -fno-builtin?

2016-12-22 Thread Joerg Sonnenberger
On Thu, Dec 22, 2016 at 05:47:05PM +0100, Christian Weisgerber wrote: > Building ld.so with clang on amd64 fails with undefined references > to memset and memcpy. That is odd, since neither function appears > in the source. Apparently clang optimizes the _dl_memset and > _dl_bcopy functions into

Re: if attach/detach netlocks

2016-12-22 Thread Mike Belopuhov
On Wed, Dec 21, 2016 at 13:06 +0100, Alexander Bluhm wrote: > On Wed, Dec 21, 2016 at 12:45:37PM +0100, Mike Belopuhov wrote: > > Anyways, OK for the diff below? > > OK bluhm@ > > > diff --git sys/net/if_pflow.c sys/net/if_pflow.c > > index 0df0b69fd8a..8592d98a743 100644 > > ---

Re: snmpd improvements

2016-12-22 Thread Claudio Jeker
On Wed, Dec 21, 2016 at 10:40:48AM +0100, Franco Fichtner wrote: > Hi, > > Switching from net-snmp to OpenBSD's snmpd raised two > issues and I'd like to know if they make sense to address: > > A pid file is missing. Would a patch for this be accepted? Most probably not. We don't see the

Re: ospf6d: handle interface MTU changes

2016-12-22 Thread Claudio Jeker
On Wed, Dec 21, 2016 at 12:08:23PM +0100, Jeremie Courreges-Anglas wrote: > > Hi, > > After ospfd here's a diff to make ospf6d refresh his view of an > interface's MTU at runtime. This needs a fresh kernel. > > The parent should pass the IFINFO message to its children first, and > then decide

ld.so: -fno-builtin?

2016-12-22 Thread Christian Weisgerber
Building ld.so with clang on amd64 fails with undefined references to memset and memcpy. That is odd, since neither function appears in the source. Apparently clang optimizes the _dl_memset and _dl_bcopy functions into calls to memset and memcpy, respectively. I tentatively propose to add

Re: splassert: ip_output: want 1 have 0

2016-12-22 Thread Martin Pieuchot
On 22/12/16(Thu) 10:45, Martin Pieuchot wrote: > On 22/12/16(Thu) 00:32, Mark Kettenis wrote: > > splassert: ip_output: want 1 have 0 > > Starting stack trace... > > ip_output() at ip_output+0x7d > > ipsp_process_done() at ipsp_process_done+0x2ad > > esp_output_cb() at esp_output_cb+0x135 > >

Re: splassert: sowakeup/sorwakeup: want 1 have 0

2016-12-22 Thread Martin Pieuchot
On 22/12/16(Thu) 11:59, Stefan Sperling wrote: > Some additional stack traces which run through nd6: > > splassert: sowakeup: want 1 have 0 > Starting stack trace... > sowakeup(1,0,d09d8d63,d09c8abf,d03cfbb0) at sowakeup+0x43 > sowakeup(d95478ec,d954793c,d09d8be1,d378a4a0,d41150e0) at

Re: splassert: sowakeup/sorwakeup/sonewconn: want 1 have 0

2016-12-22 Thread Martin Pieuchot
On 22/12/16(Thu) 12:00, Theo Buehler wrote: > Not sure if these have been reported. With a kernel freshly built from > sources this morning, my dmesg is full of these: > > splassert: sorwakeup: want 1 have 0 > Starting stack trace... > sorwakeup() at sorwakeup+0x3c > soisconnected() at

Re: splassert: sowakeup/sorwakeup/sowwakeup: want 1 have 0

2016-12-22 Thread Martin Pieuchot
On 22/12/16(Thu) 11:56, Stefan Sperling wrote: > Getting a lot of these with 'dhclient iwm0'. > > splassert: sowakeup: want 1 have 0 > Starting stack trace... > sowakeup(1,0,d09d8d63,d09c8abf,d03cfbb0) at sowakeup+0x43 >

Re: ND6 and splsoftnet()

2016-12-22 Thread Alexander Bluhm
On Wed, Dec 21, 2016 at 07:51:03PM +0100, Martin Pieuchot wrote: > > This diff replaces the nd6_timer timeout and task with a timeout > > proc. > > Can we wait until the experiment is complete? Until then I'd like > to keep timeout_set_proc() only for timeout converted because they > need the

splassert: sowakeup/sorwakeup/sonewconn: want 1 have 0

2016-12-22 Thread Theo Buehler
Not sure if these have been reported. With a kernel freshly built from sources this morning, my dmesg is full of these: splassert: sorwakeup: want 1 have 0 Starting stack trace... sorwakeup() at sorwakeup+0x3c soisconnected() at soisconnected+0x6f unp_connect2() at unp_connect2+0x62

splassert: sowakeup/sorwakeup: want 1 have 0

2016-12-22 Thread Stefan Sperling
Some additional stack traces which run through nd6: splassert: sowakeup: want 1 have 0 Starting stack trace... sowakeup(1,0,d09d8d63,d09c8abf,d03cfbb0) at sowakeup+0x43 sowakeup(d95478ec,d954793c,d09d8be1,d378a4a0,d41150e0) at sowakeup+0x43 sorwakeup(d95478ec,d0b57b50,d94bd000,0,2) at

splassert: sowakeup/sorwakeup/sowwakeup: want 1 have 0

2016-12-22 Thread Stefan Sperling
Getting a lot of these with 'dhclient iwm0'. splassert: sowakeup: want 1 have 0 Starting stack trace... sowakeup(1,0,d09d8d63,d09c8abf,d03cfbb0) at sowakeup+0x43 sowakeup(d90431fc,d9043288,d09d8bd7,d3be1c00,d978ad00,f61cbd78,d978ad00,d03e5d6f,d90431fc) at sowakeup+0x43

Re: [patch]: Make ip4 mcast membership data structure similar to ip6

2016-12-22 Thread Dimitris Papastamos
On Tue, Dec 13, 2016 at 12:35:07PM +0100, Martin Pieuchot wrote: > Hello, > > On 10/12/16(Sat) 11:08, Dimitris Papastamos wrote: > > While I was playing around with the ip4 multicast code, I thought > > I would attempt to make the membership data structure similar to that > > of ip6. This means

Re: [patch] turn igmpstat into a set of percpu counters

2016-12-22 Thread Dimitris Papastamos
On Sun, Dec 11, 2016 at 07:24:40PM +, Dimitris Papastamos wrote: > Hi, > > I converted the igmp stats to use percpu counters. This work is > basically the same as what dlg@ did for other parts of the stack. > I looked at the diff and adjusted it for igmp. ping

Re: splassert: ip_output: want 1 have 0

2016-12-22 Thread Martin Pieuchot
On 22/12/16(Thu) 00:32, Mark Kettenis wrote: > splassert: ip_output: want 1 have 0 > Starting stack trace... > ip_output() at ip_output+0x7d > ipsp_process_done() at ipsp_process_done+0x2ad > esp_output_cb() at esp_output_cb+0x135 > taskq_thread() at taskq_thread+0x6c > end trace frame: 0x0,