Re: poll magic for pflogd

2015-09-22 Thread Ted Unangst
Tobias Ulmer wrote: > On Tue, Sep 22, 2015 at 11:46:08AM -0400, Ted Unangst wrote: > > Todd C. Miller wrote: > > > On Mon, 21 Sep 2015 20:13:05 -0400, "Ted Unangst" wrote: > > > > > > > We can put a "long" poll() in front of pcap to wait until there are > > > > packets > > > > (maybe never if you

Re: poll magic for pflogd

2015-09-22 Thread Tobias Ulmer
On Tue, Sep 22, 2015 at 11:46:08AM -0400, Ted Unangst wrote: > Todd C. Miller wrote: > > On Mon, 21 Sep 2015 20:13:05 -0400, "Ted Unangst" wrote: > > > > > We can put a "long" poll() in front of pcap to wait until there are > > > packets > > > (maybe never if you aren't using pf logging), and the

Re: bsd.port.mk.5: Two small tweaks

2015-09-22 Thread Ingo Schwarze
Hi Michael, Michael Reed wrote on Tue, Sep 22, 2015 at 04:11:26PM -0400: > I'm pretty sure the use of Nm was previously wrong, Indeed. The .Nm macro is only intended for command names that occur in the NAME section of the respective manual. > but let me know if I'm off the mark. > > Index: sr

Re: bsd.port.mk.5: Remove unneeded args to Nm

2015-09-22 Thread Ingo Schwarze
Hi Michael, Michael Reed wrote on Tue, Sep 22, 2015 at 04:24:37PM -0400: > In bsd.port.mk.5, the use of arguments to Nm is inconsistent, > and when they are used it's seemingly redundant; this patch > fixes that. > > I verified there's no change in the formatted manual with this: > $ diff <{mand

Re: Purge route entries when an address is removed

2015-09-22 Thread Stuart Henderson
On 2015/09/22 23:07, Sebastian Benoit wrote: > Martin Pieuchot(m...@openbsd.org) on 2015.09.13 16:08:50 +0200: > > On 13/09/15(Sun) 15:51, Alexander Bluhm wrote: > > > On Sun, Sep 13, 2015 at 11:15:50AM +0200, Martin Pieuchot wrote: > > > > This makes the kernel simpler as it no longer try to find

Re: Purge route entries when an address is removed

2015-09-22 Thread Sebastian Benoit
Martin Pieuchot(m...@openbsd.org) on 2015.09.13 16:08:50 +0200: > On 13/09/15(Sun) 15:51, Alexander Bluhm wrote: > > On Sun, Sep 13, 2015 at 11:15:50AM +0200, Martin Pieuchot wrote: > > > This makes the kernel simpler as it no longer try to find a new ifa > > > when a route with a stale address is

Re: Merge rt_use counters

2015-09-22 Thread Sebastian Benoit
Claudio Jeker(cje...@diehard.n-r-g.com) on 2015.09.22 16:01:34 +0200: > On Tue, Sep 22, 2015 at 03:14:18PM +0200, Martin Pieuchot wrote: > > Instead of incrementing the rt_use counter when a rtalloc(9) call > > succeeds, let's do it inside ralloc(9). > > > > The route(8) regress tests will need to

bsd.port.mk.5: Remove unneeded args to Nm

2015-09-22 Thread Michael Reed
In bsd.port.mk.5, the use of arguments to Nm is inconsistent, and when they are used it's seemingly redundant; this patch fixes that. I verified there's no change in the formatted manual with this: $ diff <{mandoc bsd.port.mk.5.orig} <{mandoc bsd.port.mk.5} Finally, if anyone is wondering, line 1

bsd.port.mk.5: Two small tweaks

2015-09-22 Thread Michael Reed
I'm pretty sure the use of Nm was previously wrong, but let me know if I'm off the mark. Index: src/share/man/man5/bsd.port.mk.5 === RCS file: /cvs/src/share/man/man5/bsd.port.mk.5,v retrieving revision 1.422 diff -u -p -r1.422 bsd.po

Re: iwm(4) rx reset fix

2015-09-22 Thread Stefan Sperling
On Tue, Sep 22, 2015 at 01:23:17PM -0400, Ted Unangst wrote: > Stefan Sperling wrote: > > Properly reset the RX ring by clearing RX buffer status exposed to hardware. > > Found by Matthew Dillon while porting FreeBSD's iwm(4) to Dragonfly. > > > > ok? > > > > Index: if_iwm.c > > =

iwm mac ctxt

2015-09-22 Thread Stefan Sperling
Adrian Chadd noted that Linux iwlwifi adds a mac context to the firmware when the driver initialises the interface, and changes it (rather than adding it) whenever the interface is brought up. This does the same for iwm(4). ok? Index: if_iwm.c

Re: iwm(4) rx reset fix

2015-09-22 Thread Ted Unangst
Stefan Sperling wrote: > Properly reset the RX ring by clearing RX buffer status exposed to hardware. > Found by Matthew Dillon while porting FreeBSD's iwm(4) to Dragonfly. > > ok? > > Index: if_iwm.c > === > RCS file: /cvs/src/sys/d

iwm(4) bssid addr fix

2015-09-22 Thread Stefan Sperling
Adrian Chadd noted that Linux iwlwifi sends the broadcast address to the firmware when not associated, rather than zeros. ok? Index: if_iwm.c === RCS file: /cvs/src/sys/dev/pci/if_iwm.c,v retrieving revision 1.46 diff -u -p -r1.46 if

iwm(4) rx reset fix

2015-09-22 Thread Stefan Sperling
Properly reset the RX ring by clearing RX buffer status exposed to hardware. Found by Matthew Dillon while porting FreeBSD's iwm(4) to Dragonfly. ok? Index: if_iwm.c === RCS file: /cvs/src/sys/dev/pci/if_iwm.c,v retrieving revision 1

Re: poll magic for pflogd

2015-09-22 Thread Ted Unangst
Todd C. Miller wrote: > On Mon, 21 Sep 2015 20:13:05 -0400, "Ted Unangst" wrote: > > > We can put a "long" poll() in front of pcap to wait until there are packets > > (maybe never if you aren't using pf logging), and then let the timeout work > > it's magic. > > This: > > > + if (poll(

Re: poll magic for pflogd

2015-09-22 Thread Todd C. Miller
On Mon, 21 Sep 2015 20:13:05 -0400, "Ted Unangst" wrote: > We can put a "long" poll() in front of pcap to wait until there are packets > (maybe never if you aren't using pf logging), and then let the timeout work > it's magic. This: > + if (poll(&pfd, 1, INFTIM) == 1) { will also ma

Re: lighter sleep

2015-09-22 Thread Brian Conway
I recall this discussion on UTF-8 and locales: http://undeadly.org/cgi?action=article&sid=20150722182236 I imagine there was more elsewhere that I didn't see. Brian On Sep 22, 2015 7:07 AM, "Benny Lofgren" wrote: > On 2015-09-21 16:45, Mark Kettenis wrote: > >> From: Christian Weisgerber > >

Re: Merge rt_use counters

2015-09-22 Thread Claudio Jeker
On Tue, Sep 22, 2015 at 03:14:18PM +0200, Martin Pieuchot wrote: > Instead of incrementing the rt_use counter when a rtalloc(9) call > succeeds, let's do it inside ralloc(9). > > The route(8) regress tests will need to be updated because all the > paths calling rtalloc(9) do not increment rt_use.

Merge rt_use counters

2015-09-22 Thread Martin Pieuchot
Instead of incrementing the rt_use counter when a rtalloc(9) call succeeds, let's do it inside ralloc(9). The route(8) regress tests will need to be updated because all the paths calling rtalloc(9) do not increment rt_use. This change gives us a better understanding of which routes are queried an

Re: lighter sleep

2015-09-22 Thread Benny Lofgren
On 2015-09-21 16:45, Mark Kettenis wrote: >> From: Christian Weisgerber >> Date: Mon, 21 Sep 2015 14:29:03 + (UTC) >> >> On 2015-09-21, Stefan Sperling wrote: >> > You could argue that the thousands separator should be supported though: > > $ sleep 1.000.000 > > if your locale is somethin

Re: panic with latest ugen.c and pcsc-lite

2015-09-22 Thread Martin Pieuchot
On 17/09/15(Thu) 02:34, Grant Czajkowski wrote: > On Tue, Sep 15, 2015 at 12:04:35PM +0200, Martin Pieuchot wrote: > > On 15/09/15(Tue) 04:50, Grant Czajkowski wrote: > > > On Fri, Sep 11, 2015 at 02:41:04AM -0600, David Coppa wrote: > > > > > > > > Hi! > > > > > > > > Repeatedly hit the panic be