Re: armv7/arm64: enable fifo in com_fdt for "snps,dw-apb-uart"

2017-11-14 Thread Artturi Alm
On Mon, Nov 13, 2017 at 10:02:35PM +0100, Mark Kettenis wrote: > > Date: Fri, 10 Nov 2017 00:16:26 +0200 > > From: Artturi Alm > > > > Hi, > > > > does also remove ti,omapX-uart compatibles from arm64 com_fdt, > > as they're never to be found there. > > iirc. the lenght

if_rxr_livelocked

2017-11-14 Thread David Gwynne
if a driver determines that an rx ring is generating too much load, it can notify the rxr ring accounting stuff of this. the effect is that it may lower the current watermark on the ring, therefore restricting the number of packets that will enter the system in the future. this also adds

Re: multiple interface input queues

2017-11-14 Thread David Gwynne
> On 14 Nov 2017, at 23:17, Martin Pieuchot wrote: > > On 14/11/17(Tue) 14:42, David Gwynne wrote: >> this replaces the single mbuf_queue and task in struct ifnet with >> a new ifiqueue structure modelled on ifqueues. > > The name is confusing, should we rename ifqueues

csh(1): minor tweaks

2017-11-14 Thread Anton Lindqvist
Hi, In revision 1.35 of file.c tenex() was modified to respect the inputline_size argument. I did miss one conditional which depends on the size of the input buffer. This change does not cause any functional change since tenex() only has one call site where `inputline_size == sizeof(buf)` but for

add an iqdrops view to systat to show interface queue drops

2017-11-14 Thread David Gwynne
im adding numbers to input and output qdrops in the kernel, so im aware that they exist now. however, i don't really see these values in userland. it seems netstat and systat think errors are more important. i tried adding qdrops to the ifstat view, but it got too cluttered. so i made a new view

Re: iked: support MOBIKE (RFC 4555)

2017-11-14 Thread Patrick Wildt
On Thu, Nov 09, 2017 at 11:40:30AM +0100, Patrick Wildt wrote: > Hi, > > this diff implements MOBIKE (RFC 4555) support in iked(8), with us > acting as responder. In practice this support means that clients like > iPhones can roam in different networks (LTE, WiFi) and change their > external

Re: multiple interface input queues

2017-11-14 Thread Martin Pieuchot
On 14/11/17(Tue) 14:42, David Gwynne wrote: > this replaces the single mbuf_queue and task in struct ifnet with > a new ifiqueue structure modelled on ifqueues. The name is confusing, should we rename ifqueues 'ifoqueue' then? > the main motivation behind this was to show mpsafe input counters.

frag6 splassert net lock

2017-11-14 Thread Alexander Bluhm
Hi, On my regress test machine I see this trace: splassert: rt_match: want 2 have 0 Starting stack trace... rt_match(2,0,d0bb032e,604a56c3) at rt_match+0x45 rt_match(f5475540,0,0,0) at rt_match+0x45 rtalloc(f5475540,0,0) at rtalloc+0x13 icmp6_reflect(d818de00,28) at icmp6_reflect+0x1f7

Re: NET_RLOCK in ifioctl()

2017-11-14 Thread Alexandr Nedvedicky
Hello, I like this change and I think it should go in. ok sashan On Mon, Nov 13, 2017 at 04:54:11PM +0100, Theo Buehler wrote: > The diff below pushes the NET_LOCK into ifioctl() and reduces it to > NET_RLOCK where possible. In particular, this will allow SIOCG* requests > to run in parallel.

Re: frag6 splassert net lock

2017-11-14 Thread Martin Pieuchot
On 14/11/17(Tue) 14:20, Alexandr Nedvedicky wrote: > Hello, > > I would go for NET_RLOCK() instead of NET_LOCK(), which is currently alias > to NET_WLOCK(). My point is the icmp6_reflect(), which is the workhorse for > icmp6_error(), is a typical 'READER-user' of network stack. It does not >

Re: frag6 splassert net lock

2017-11-14 Thread Alexandr Nedvedicky
Hello, I would go for NET_RLOCK() instead of NET_LOCK(), which is currently alias to NET_WLOCK(). My point is the icmp6_reflect(), which is the workhorse for icmp6_error(), is a typical 'READER-user' of network stack. It does not change any network configuration. So it should be fine to let it

faster printf

2017-11-14 Thread Theo Buehler
There is a simplification and optimization for __vfprintf() from android pointed out by tedu: https://github.com/aosp-mirror/platform_bionic/commit/5305a4d4a723b06494b93f2df81733b83a0c46d3 If we only support UTF-8 and ASCII, we do not need complicated multibyte decoding to recognize a '%' in the

Re: xf86-video-intel patch to test

2017-11-14 Thread Robert
q Am 13. November 2017 17:56:19 MEZ schrieb Stefan Sperling : >On Sat, Nov 11, 2017 at 07:57:16PM +0100, Matthieu Herrb wrote: >> Hi, >> >> the patch below should not affect the intel(4) X.Org driver >> functionality. It's sole purpose is to make it compatible with the >> future

error checking for __find_arguments() in vfwprintf(3)

2017-11-14 Thread Theo Buehler
As promised in my previous mail, here's the part of the android commit https://github.com/aosp-mirror/platform_bionic/commit/5305a4d4a723b06494b93f2df81733b83a0c46d3 that adds some missing error checking analogous to some of the checking added by schwarze in vfprintf.c r1.71. ok? Index:

Re: Remove commented out pledge in tsort

2017-11-14 Thread Marc Espie
On Mon, Nov 13, 2017 at 10:38:17PM -0800, Ori Bernstein wrote: > On Sun, 12 Nov 2017 13:31:10 +0100, Marc Espie wrote: > > > I did this in an eager way a while back, was told in no uncertain terms > > "not yet, we're still looking at things". > > > > Well, see pledge(2).

Re: un-KERNEL_LOCK() TCP/UDP input & co

2017-11-14 Thread Martin Pieuchot
On 13/11/17(Mon) 17:29, Visa Hankala wrote: > On Mon, Nov 13, 2017 at 01:25:42PM +0100, Martin Pieuchot wrote: > > I'd love is somebody could do the changes in the rwlock API to let us > > check if we're holding a lock. Maybe this is already present in > > witness(4), visa? > > witness(4) does

Re: Question about tables in nested anchor on pf since 6.1

2017-11-14 Thread Alexandr Nedvedicky
Hello Leo, this looks like my bad, which goes back to commit [1], which tried to fix 'mix up of anchor names and anchor paths'. I've completely forgot to take care of pfctl/parse.y back then. Please let me know if patch below solves your problem. thank you for great troubleshooting and excellent

armv7 ld.so tweak

2017-11-14 Thread Mark Kettenis
The arm ld.so code has #define _BM(x) (x == 32? ~0 : ~(-(1UL << (x and uses _BM(32) in several places. For some reason clang doesn't realize that the ~(-(1UL << (x))) branch is dead and warns that we shift a 32-bit integer by 32 bits. Arguably a bug in clang, but I noticed that the i386

Re: multiple interface input queues

2017-11-14 Thread David Gwynne
On Tue, Nov 14, 2017 at 02:42:30PM +1000, David Gwynne wrote: > this replaces the single mbuf_queue and task in struct ifnet with > a new ifiqueue structure modelled on ifqueues. > > the main motivation behind this was to show mpsafe input counters. > > ifiqueues, like ifqueues, allow a driver

Fwd [misc@]: Suppessing logging of arp movement messages

2017-11-14 Thread Stuart Henderson
In gmane.os.openbsd.misc, open...@me.com wrote: > hello all, > > I have finally build an internet gateway with OpenBSD 6.2 (AMD64), including > pf and IPSec. Great stuff. > Now I am seeing a lot of arp movement, that I know are caused by Apple's > Bonjour Sleep Proxy. > > Nov 8 00:00:27

Re: faster printf

2017-11-14 Thread Todd C. Miller
On Tue, 14 Nov 2017 09:26:47 +0100, Theo Buehler wrote: > If we only support UTF-8 and ASCII, we do not need complicated multibyte > decoding to recognize a '%' in the format string. > > In his commit message, enh claims that there is a 10x speedup. In my own > benchmarking on amd64, a speedup

Re: error checking for __find_arguments() in vfwprintf(3)

2017-11-14 Thread Todd C. Miller
On Tue, 14 Nov 2017 09:28:40 +0100, Theo Buehler wrote: > As promised in my previous mail, here's the part of the android commit > https://github.com/aosp-mirror/platform_bionic/commit/5305a4d4a723b06494b93f2 > df81733b83a0c46d3 > that adds some missing error checking analogous to some of the

Re: Fwd [misc@]: Suppessing logging of arp movement messages

2017-11-14 Thread Theo de Raadt
Might this be better per-interface, using ifconfig, then people can disable logging on a particular subnet in hostname.* files? > On Tue, Nov 14, 2017 at 02:04:27PM +, Stuart Henderson wrote: > > Any thoughts (and suggestions for mib for an af-independent one if > > that's the way to go)? >

Re: Question about tables in nested anchor on pf since 6.1

2017-11-14 Thread Leonardo Guardati
Hi, I've applied the patch on 6.2 and it fixes the problem. # pfctl -ef /etc/pf.conf pf enabled # pfctl -a /uno/due -t foo -T show 10.0.0.1 # pfctl

Re: Fwd [misc@]: Suppessing logging of arp movement messages

2017-11-14 Thread Stuart Henderson
On 2017/11/14 09:13, Theo de Raadt wrote: > Might this be better per-interface, using ifconfig, then people can > disable logging on a particular subnet in hostname.* files? Hmm yes, that sounds a much better idea actually. And in that case keeping v4 and v6 in one setting isn't as awkward as

Re: Fwd [misc@]: Suppessing logging of arp movement messages

2017-11-14 Thread Alexander Bluhm
On Tue, Nov 14, 2017 at 02:04:27PM +, Stuart Henderson wrote: > Any thoughts (and suggestions for mib for an af-independent one if > that's the way to go)? I have the requirement to log all arp overwrites in the local network. So I would like to keep the log message, it does no harm in a

Re: un-KERNEL_LOCK() TCP/UDP input & co

2017-11-14 Thread Visa Hankala
On Tue, Nov 14, 2017 at 10:37:16AM +0100, Martin Pieuchot wrote: > Note that the diff below still includes a false positive. If the > current thread doesn't have the read-lock but another one has it, > then RW_READ will still be returned. That's true if witness(4) has not been enabled.

Re: maximum size of http headers in relayd

2017-11-14 Thread Sebastian Benoit
rediff, i fixed the two lines that were too long. Sebastian Benoit(be...@openbsd.org) on 2017.11.14 18:01:26 +0100: > Currently relayd limits the size of all http headers to > RELAY_MAXHEADERLENGTH = 8192. > > Current webbrowsers support for example as default > > httpd: 8k > Apache: 8190 byte

maximum size of http headers in relayd

2017-11-14 Thread Sebastian Benoit
Currently relayd limits the size of all http headers to RELAY_MAXHEADERLENGTH = 8192. Current webbrowsers support for example as default httpd: 8k Apache: 8190 byte per header line, 100 header nginx: 32k (max 8k per line) iis: 16k per line Tomcat: 8k All except httpd are configurable and