Re: pf: increment rule counters after successful state insertion

2015-06-15 Thread Mike Belopuhov
On Tue, Jun 09, 2015 at 18:11 +0200, Mike Belopuhov wrote: Hi, I was surprised to see 'State Creations' rule counter go up when no real state creation happens. This is because we increment all counters too early, but then don't decrement 'states_tot' which is a total number of states

Re: [patch]rcs: usage functions above the main ones

2015-06-15 Thread Nicholas Marriott
this seems fine to me On Sun, Jun 14, 2015 at 10:38:40PM +0200, Fritjof Bornebusch wrote: Hi tech@, most of the tools implements the *usage* function above the *main* function. This patch makes it more consistent to these tools and where the different *usage* functions are implemented

Re: [patch]rcs: xstrdup just wrappes strdup

2015-06-15 Thread Nicholas Marriott
What about diff and ssh and file? They all use the a copy of the same xmalloc.c. On Mon, Jun 15, 2015 at 10:00:01AM +0200, Fritjof Bornebusch wrote: Hi, thanks for the hint. This one should do the trick. Index: xmalloc.c

Re: [patch]rcs: xstrdup just wrappes strdup

2015-06-15 Thread Nicholas Marriott
On Mon, Jun 15, 2015 at 10:09:39AM +0200, Fritjof Bornebusch wrote: On Sun, Jun 14, 2015 at 05:02:05PM -0600, Theo de Raadt wrote: But I am not sure about this change. xmalloc.c came from ssh (and is also used by file and diff). Would it be better to keep it in sync? How portable is

Re: Waiting for 'high priority' events with kqueue

2015-06-15 Thread Mark Kettenis
Date: Mon, 15 Jun 2015 12:59:50 +1000 From: Daurnimator q...@daurnimator.com On 14 June 2015 at 14:51, Philip Guenther guent...@gmail.com wrote: On Wed, Jun 10, 2015 at 10:21 PM, Daurnimator q...@daurnimator.com wrote: I'm working on adding support for high priority events to a user space

[patch] faq pf/carp

2015-06-15 Thread Julien Dhaille
Hi tech, few links on the bottom of http://www.openbsd.org/faq/pf/carp.html looks broken (Bad request, You specified an invalid manpath). cheers Index: www/faq/pf/carp.html === RCS file: /cvs/www/faq/pf/carp.html,v retrieving

fix stuck iwm(4) task thread

2015-06-15 Thread Stefan Sperling
After suspend/resume I've found an iwm task sleeping forever. ps showed: 0 7029 0 0 -22 0 0 0 iwmau2 DK??0:00.00 (iwmns) In this state, the driver awaits a notification from the firmware which it won't ever get after the system has resumed from sleep. The task running

uslcom(4): add ID for Netgear M7100

2015-06-15 Thread Andrew Daugherity
This particular 10Gb switch from Netgear has both a standard DB9 serial port for its console, and a mini-USB port, selectable via switch. The mini-USB is apparently connected to an internal CP2102 serial/USB adapter (0846:1100). Comes with Windows drivers on CD; worked out-of-the-box (with

Re: Waiting for 'high priority' events with kqueue

2015-06-15 Thread Daurnimator
On 15 June 2015 at 22:40, Mark Kettenis mark.kette...@xs4all.nl wrote: I'm using this inside an event handling library/framework. It uses a kqueue rather than poll() or select() under the hood. The users of our API expect that they'd get the same behaviour as if they used poll directly

[patch] cleaner checksum modification for pf

2015-06-15 Thread Richard Procter
Hi, These patches against HEAD re-instate the pf algorithm of OpenBSD 5.4 for preserving payload checksums end-to-end but rewritten without the ugly and error-prone (but speedy!) nested pf_cksum_fixup calls. I have been running this code on a small Alix (i386) IPv4 gateway for a month with no

FAQ 15.2, invalid manpath

2015-06-15 Thread vulcasian
In FAQ 15.2, clicking on any of the pkg_add, pkg_delete, pkg_info or pkg_create links results in a Bad Request from man.cgi. None of the other manpage links raise the error. E.g., http://www.openbsd.org/cgi-bin/man.cgi?query=pkg_addsektion=1manpath=OpenBSD+5.7 …from…

[patch]rcs: mark unlink as (void)

2015-06-15 Thread Fritjof Bornebusch
Hi tech@, mark this unlink(2) call as *(void)*, as there is no need to check the return value. This makes it more consistent to all other unlink(2) calls, since they are marked as *(void)* as well. Regards, --F. Index: co.c ===

Re: fix stuck iwm(4) task thread

2015-06-15 Thread Stefan Sperling
On Mon, Jun 15, 2015 at 05:04:38PM +0200, Stefan Sperling wrote: After suspend/resume I've found an iwm task sleeping forever. ps showed: 0 7029 0 0 -22 0 0 0 iwmau2 DK??0:00.00 (iwmns) In this state, the driver awaits a notification from the firmware which it

Re: [patch]rcs: xstrdup just wrappes strdup

2015-06-15 Thread Fritjof Bornebusch
Hi, thanks for the hint. This one should do the trick. Index: xmalloc.c === RCS file: /cvs/src/usr.bin/rcs/xmalloc.c,v retrieving revision 1.9 diff -u -p -r1.9 xmalloc.c --- xmalloc.c 13 Jun 2015 20:15:21 - 1.9 +++

Re: [patch]rcs: xstrdup just wrappes strdup

2015-06-15 Thread Fritjof Bornebusch
On Sun, Jun 14, 2015 at 05:02:05PM -0600, Theo de Raadt wrote: But I am not sure about this change. xmalloc.c came from ssh (and is also used by file and diff). Would it be better to keep it in sync? How portable is strdup? strdup is extremely portable. The last mainstream operating

patch: dump(8)'s Default Volume Behavior

2015-06-15 Thread Michael McConville
On Sun, Jun 14, 2015 at 06:14:09PM -0600, Theo de Raadt wrote: If so, let me know and I can write a patch. Please do, then it is easier for archaic people to play with it. This is my first patch, so I may have made some mistakes. I tried to correct and expand the man page while leaving its