Re: [patch] cleaner checksum modification for pf

2015-09-24 Thread Richard Procter
On 14/09/2015, at 11:51 PM, Henning Brauer wrote: > * Martin Pieuchot [2015-09-11 13:54]: >> On 11/09/15(Fri) 13:28, Henning Brauer wrote: >>> Ryan pointed me to this diff and we briefly discussed it; we remain >>> convinced that the in-tree approach is better than this. >> Could you elaborate wh

Re: mpsafe ip_carp

2015-09-24 Thread mxb
With setup like: node1: carpnodes 1:0,10:0 carpdev vmx0 balancing arp node2: carpnodes 1:100,10:100 carped vmx0 balancing arp (eg, I forced one machine to be a master for a while in my configs) node2 (-current) with your diff started to pollute dmesg with: duplicate IP address 192.168.78.123 sen

Re: PF SMP: mutex for fragcache

2015-09-24 Thread mxb
Applied. > On 12 sep. 2015, at 19:20, Alexandr Nedvedicky > wrote: > > Hello, > > very small first step towards MP(i) friendly PF. Patch adds mutex around > fragment cache. > > Patch adds a lock around fragment cache. Unlike other parts of PF the fragment > cache is self-contained subsystem.

Re: mpsafe vmx(4)

2015-09-24 Thread mxb
This one in the tree, so it’s live on my side. > On 14 sep. 2015, at 13:09, David Gwynne wrote: > > this is an attempt to make the interrupt path in vmx mpsafe. > > seems to hold up under load here, but more testing would be > appreciated. > > Index: if_vmx.c > ===

mg(1) transpose-paragraph

2015-09-24 Thread Mark Lumsden
While I am 'paragraph' mode, here is the useful 'transpose-paragraph' command. Comments/ok? mark Index: def.h === RCS file: /cvs/src/usr.bin/mg/def.h,v retrieving revision 1.148 diff -u -p -u -p -r1.148 def.h --- def.h 24 Sep 2

Re: delete reference to inetd from afterboot.8

2015-09-24 Thread Jason McIntyre
On Thu, Sep 24, 2015 at 10:46:12AM -0400, Rob Pierce wrote: > Since inetd is no longer started by default and inetd.conf has been relegated > to /etc/examples, this sentence can probably be deleted from afterboot.8. > > Regards, > fair enough. i knocked out the following Ss to avoid being left w

Re: arch(1): small cleanup

2015-09-24 Thread Ted Unangst
Michael Reed wrote: > Hi all, > > I test it out as both `arch' and `machine' and didn't observe > any differences. Also, I figured I might as well convert exit(3) > to return while touching this code, as was done in [1]. this is cleaner i think. Index: arch.c ==

Re: Purge route entries when an address is removed

2015-09-24 Thread Stuart Henderson
On 2015/09/13 11:15, Martin Pieuchot wrote: > Currently we leave RTF_STATIC route entries in the table when the > address they are attached to is removed from a system. > > That's why ifas need to be refcounted and that's why we have *a lot* > of checks in the stack to not use cached routes attach

Re: Plug a mem-leak in dhcpd(8)

2015-09-24 Thread Ted Unangst
Michael McConville wrote: > Am I interpreting this correctly? > > This is the least invasive fix, but it's unfortunate that this function > allows the supplied buffer to be NULL. If we made it unconditionally > allocate a new buffer, we would have to change some program logic > because uses pass s

delete reference to inetd from afterboot.8

2015-09-24 Thread Rob Pierce
Since inetd is no longer started by default and inetd.conf has been relegated to /etc/examples, this sentence can probably be deleted from afterboot.8. Regards, Index: afterboot.8 === RCS file: /cvs/src/share/man/man8/afterboot.8,v r

Keep RTF_LOCAL routes UP

2015-09-24 Thread Martin Pieuchot
This is now needed to make sure local addresses can be used even if the interface their are configured on is down. ok? Index: net/route.c === RCS file: /cvs/src/sys/net/route.c,v retrieving revision 1.242 diff -u -p -r1.242 route.c -