Re: Use proper TAILQ functions for slpque

2018-10-15 Thread Mark Kettenis
> From: Christian Ludwig > Date: Tue, 16 Oct 2018 08:31:14 +0200 > > Do not roll a custom for-loop, use the appropriate TAILQ function. > > Also use unsleep() instead of coding the same functionality here again. > There is only one place in the system that resets p_wchan now. And the > unsleep-p

Re: Use proper TAILQ functions for slpque

2018-10-15 Thread Claudio Jeker
On Tue, Oct 16, 2018 at 08:31:14AM +0200, Christian Ludwig wrote: > Do not roll a custom for-loop, use the appropriate TAILQ function. > > Also use unsleep() instead of coding the same functionality here again. > There is only one place in the system that resets p_wchan now. And the > unsleep-part

Use proper TAILQ functions for slpque

2018-10-15 Thread Christian Ludwig
Do not roll a custom for-loop, use the appropriate TAILQ function. Also use unsleep() instead of coding the same functionality here again. There is only one place in the system that resets p_wchan now. And the unsleep-part has the same pattern like in endtsleep(). --- sys/kern/kern_synch.c | 12 +

Re: switchctl(8): Add ability to reference control device for 'dump' option

2018-10-15 Thread Claudio Jeker
On Mon, Oct 15, 2018 at 03:17:29PM -0700, Ayaka Koshibe wrote: > Hi all, > > Currently, switchctl(8) is able to query a switch for information by > specifying > the switch's network address. The following adds support for querying a > switch(4) instance via its control device: > > > $ doas swit

Re: pf: honor quick on anchor rules

2018-10-15 Thread Fabian Mueller-Knapp
On 18-10-15 23:45:58, Alexandr Nedvedicky wrote: > Hello, > > > That is still different from the original (2006) behaviour which > > would terminate ruleset-evaluation IFF any rules inside the anchor > > actually match. > > Perhaps I still misunderstand the whole anchor thing. > > let m

AEAD_XChaCha20_Poly1305 support for libressl libcrypto

2018-10-15 Thread David Gwynne
XChaCha20 is a variant of ChaCha20 that provides an extended nonce, which in turn makes it more usable when using randomly generated nonces for long running connections. The longer nonce makes it less likely that a randomly generated one will collide. I want this cos a particular protocol uses AEA

Re: spf walk: lookup aaaa records with "a" mechanism

2018-10-15 Thread Todd T. Fries
In principal I like this. In practice, I note something is missing. I get a different output: $ echo netsend.nl | smtpctl spf walk 32.1.9.129 32.1.9.132 80.127.98.234 80.127.135.115 I also note the dns is being requested, as per below, just not printed for some reason. 09:42:12.175535 d0:7e:3

switchctl(8): Add ability to reference control device for 'dump' option

2018-10-15 Thread Ayaka Koshibe
Hi all, Currently, switchctl(8) is able to query a switch for information by specifying the switch's network address. The following adds support for querying a switch(4) instance via its control device: $ doas switchctl switch /dev/switch0 dump features any > /dev/switch0: version 1_3 type FEATU

Re: pf: honor quick on anchor rules

2018-10-15 Thread Alexandr Nedvedicky
Hello, > That is still different from the original (2006) behaviour which > would terminate ruleset-evaluation IFF any rules inside the anchor > actually match. Perhaps I still misunderstand the whole anchor thing. let me clarify the proposed change by two examples. Ruleset below allows

Re: pf: honor quick on anchor rules

2018-10-15 Thread Fabian Mueller-Knapp
On 18-10-15 13:38:32, Alexandr Nedvedicky wrote: > Hello, > > I just got back on-line after a week. Thank you all for detailed analysis. > This particular bug, which Klemens tries to fix is caused by my commit 1.1024: > > - percpu anchor stacks > we actually don't need to pre-allocate p

Re: spf walk: lookup aaaa records with "a" mechanism

2018-10-15 Thread Gilles Chehade
On Mon, Oct 15, 2018 at 01:08:06AM +0200, Tim Kuijsten wrote: > Hi, > Hi, > When the "a" designated sender mechanism is used in an spf txt record, both > v4 and v6 addresses are matched according to [1], so let `smtpctl spf walk` > resolve both A and records. > > [...] > > -Tim > > [1] h

Re: doas(1): Exit Status

2018-10-15 Thread Theo de Raadt
Stefan Filipek wrote: > According to this discussion, there is an issue with nice(1) > documentation, which the patch was trying to emulate: > > > EXIT STATUS > > ... > > Otherwise, the exit status of nice shall be that of utility. > > Which is incorrect, since 'nice' doesn't exit in this c

Re: user mod -u lead to segfault

2018-10-15 Thread Alexander Bluhm
On Sun, Oct 14, 2018 at 09:58:08PM -0600, Todd C. Miller wrote: > Updated diff that adds wrapper functions for improved readability. > It's arguable that user(8) should check the local master.passwd and > group files directly but that's a discussion for another time. OK bluhm@ > Index: usr.sbin/u

Re: doas(1): Exit Status

2018-10-15 Thread Stefan Filipek
According to this discussion, there is an issue with nice(1) documentation, which the patch was trying to emulate: > EXIT STATUS > ... > Otherwise, the exit status of nice shall be that of utility. Which is incorrect, since 'nice' doesn't exit in this case at all, just like 'doas'. Perhaps t

Re: top cpu stats are wrong with hyper threading

2018-10-15 Thread Scott Cheloha
On Fri, Oct 05, 2018 at 04:27:17PM +0200, Moritz Buhl wrote: > > Due to the SMT stuff the output of top showed the first few cpus instead > of the ones that are actually active. > > To reproduce the bad output: > Use a machine with hyper therading, top should show half the cpus, of > which every

Re: pf/pfctl: use PFR_FB_NONE consistently

2018-10-15 Thread Alexandr Nedvedicky
Hello, On Sat, Oct 13, 2018 at 10:56:21PM +0200, Klemens Nanni wrote: > Replace hardcoded 0 and implicit checks with enum as done in all other > use cases of `pfra_fback'. either way is fine with me, so I'm not going to object your change. OK sashan

Re: pf: honor quick on anchor rules

2018-10-15 Thread Alexandr Nedvedicky
Hello, I just got back on-line after a week. Thank you all for detailed analysis. This particular bug, which Klemens tries to fix is caused by my commit 1.1024: - percpu anchor stacks we actually don't need to pre-allocate per_anchor_stack[], if we use a 'natural' recursion, when

Re: pkg_add -Dsnap not fetching any updates and packages on -current

2018-10-15 Thread Neeraj Pal
It worked. yeah, I configured the timezone and forgot to check the date and time. So, now after syncing the date and time with ntp servers, everything works fine. Thank you Benno for the help. openbsd$ nc -cvvv fastly.cdn.openbsd.org 443 Connection to fastly.cdn.openbsd.org 443 port [tcp/https] s