Re: relayd(8) remove snmp keyword

2020-10-30 Thread Denis Fondras
On Thu, Oct 29, 2020 at 03:51:24PM +0100, Martijn van Duren wrote: > 6.8 is out in the wild. I guess this is as good a time as any to remove > the old snmp keyword. > > OK? > OK denis@ And while it is fresh, is this the right time to update plus.html and current.html ?

Minor tweak relayd agentx manpage

2020-10-30 Thread Martijn van Duren
I think metrics is a better word than statistics and it might help people if they knew where to query for these metrics. OK? martijn@ Index: relayd.conf.5 === RCS file: /cvs/src/usr.sbin/relayd/relayd.conf.5,v retrieving revision

Re: Minor tweak relayd agentx manpage

2020-10-30 Thread Denis Fondras
On Fri, Oct 30, 2020 at 09:53:08AM +0100, Martijn van Duren wrote: > I think metrics is a better word than statistics and it might help > people if they knew where to query for these metrics. > > OK? I also find it more accurate. OK denis@ > martijn@ > > Index: relayd.conf.5 >

Re: accton(8) requires a reboot after being enabled

2020-10-30 Thread Jason McIntyre
On Fri, Oct 30, 2020 at 04:24:43PM +0100, Solene Rapenne wrote: > reading accton(8) it's not clear that if you > enable it you need to restart the system for > accounting to be effective. > > Here is a change to add the explanation, but > I'm not sure if the wording is correct. > hi. i think

Re: accton(8) requires a reboot after being enabled

2020-10-30 Thread Theo de Raadt
Jason McIntyre wrote: > On Fri, Oct 30, 2020 at 04:24:43PM +0100, Solene Rapenne wrote: > > reading accton(8) it's not clear that if you > > enable it you need to restart the system for > > accounting to be effective. > > > > Here is a change to add the explanation, but > > I'm not sure if the

dig(1): Extended DNS Error (RFC 8914)

2020-10-30 Thread Florian Obser
$ obj/dig @1.1.1.1 dnssec-failed.org ; <<>> dig 9.10.8-P1 <<>> @1.1.1.1 dnssec-failed.org ; (1 server found) ;; global options: +cmd ;; Got answer: ;; ->>HEADER<<- opcode: QUERY, status: SERVFAIL, id: 26772 ;; flags: qr rd ra; QUERY: 1, ANSWER: 0, AUTHORITY: 0, ADDITIONAL: 1 ;; OPT

Re: Minor tweak relayd agentx manpage

2020-10-30 Thread Sebastian Benoit
sure Martijn van Duren(openbsd+t...@list.imperialat.at) on 2020.10.30 09:53:08 +0100: > I think metrics is a better word than statistics and it might help > people if they knew where to query for these metrics. > > OK? > martijn@ > > Index: relayd.conf.5 >

Re: relayd(8) remove snmp keyword

2020-10-30 Thread Sebastian Benoit
ok benno@ and yes, add a line to current.html. Denis Fondras(open...@ledeuns.net) on 2020.10.30 10:13:56 +0100: > On Thu, Oct 29, 2020 at 03:51:24PM +0100, Martijn van Duren wrote: > > 6.8 is out in the wild. I guess this is as good a time as any to remove > > the old snmp keyword. > > > > OK?

Re: amap: introduce amap_adjref_anons()

2020-10-30 Thread Martin Pieuchot
On 23/10/20(Fri) 10:31, Martin Pieuchot wrote: > More refactoring. This time let's introduce a helper to manipulate > references. The goal is to reduce the upcoming diff adding locking. > > This is extracted from a bigger diff from guenther@ as well as some > bits from NetBSD. Now with the

Re: dig(1): Extended DNS Error (RFC 8914)

2020-10-30 Thread Otto Moerbeek
On Fri, Oct 30, 2020 at 03:04:03PM +0100, Florian Obser wrote: Love it, -Otto > $ obj/dig @1.1.1.1 dnssec-failed.org > > ; <<>> dig 9.10.8-P1 <<>> @1.1.1.1 dnssec-failed.org > ; (1 server found) > ;; global options: +cmd > ;; Got answer: > ;; ->>HEADER<<- opcode: QUERY, status:

Re: accton(8) requires a reboot after being enabled

2020-10-30 Thread Theo de Raadt
Yes, that diff is a whole bunch of TOCTUO. If this was going to be changed, it should be in the kernel. But I don't know if it should be changed yet, which is why I asked a bunch of questions. Stepping back to the man page change, we could decide that accton should continue to behave how it

Re: Kernel hang under concurrent vfs ops on the same hierarchy.

2020-10-30 Thread Mathieu -
I'm moving this to tech as I think the issue is best discussed there but I might be wrong. So, I did some slow progress on that hang. A recent snapshots now panics in ufs code with "ufs_rename: lost dir entry". That's the second one at ufs_vnops.c:1084. This means that the source of the rename

accton(8) requires a reboot after being enabled

2020-10-30 Thread Solene Rapenne
reading accton(8) it's not clear that if you enable it you need to restart the system for accounting to be effective. Here is a change to add the explanation, but I'm not sure if the wording is correct. Index: accton.8 === RCS file:

Re: accton(8) requires a reboot after being enabled

2020-10-30 Thread Jason McIntyre
On Fri, Oct 30, 2020 at 09:59:09AM -0600, Theo de Raadt wrote: > Jason McIntyre wrote: > > > On Fri, Oct 30, 2020 at 04:24:43PM +0100, Solene Rapenne wrote: > > > reading accton(8) it's not clear that if you > > > enable it you need to restart the system for > > > accounting to be effective. > >

Re: accton(8) requires a reboot after being enabled

2020-10-30 Thread Ingo Schwarze
Hi Theo, Theo de Raadt wrote on Fri, Oct 30, 2020 at 09:59:09AM -0600: > With a careful reading of the current manual page, everything is there > and it is accurate. > > With an argument naming an existing file > > > Ok so let's create it with touch.

Re: accton(8) requires a reboot after being enabled

2020-10-30 Thread Ingo Schwarze
Hi Solene, Solene Rapenne wrote on Fri, Oct 30, 2020 at 06:34:09PM +0100: > Following diff changes accton(8) behavior. > > If the file given as parameter doesn't exists, accton will create it. > Then it will check the ownership and will make it root:wheel if > it's different. > > I added a

Re: Please test: switch select(2) to kqfilters

2020-10-30 Thread Martin Pieuchot
On 26/10/20(Mon) 11:57, Scott Cheloha wrote: > On Mon, Oct 12, 2020 at 11:11:36AM +0200, Martin Pieuchot wrote: > [...] > > +/* > > + * Scan the kqueue, blocking if necessary until the target time is reached. > > + * If tsp is NULL we block indefinitely. If tsp->ts_secs/nsecs are both > > + * 0

Re: accton(8) requires a reboot after being enabled

2020-10-30 Thread Solene Rapenne
Following diff changes accton(8) behavior. If the file given as parameter doesn't exists, accton will create it. Then it will check the ownership and will make it root:wheel if it's different. I added a check to be sure it's run as root because it's has no use if not run as root. I don't often

Re: [PATCH] ifconfig: keep track of allowed ips pointer correctly

2020-10-30 Thread wictory
Hi! On Tue, Oct 27, 2020 at 06:16:08PM +0100, Jason A. Donenfeld wrote: > Somebody on IRC mentioned that using ifconfig to set wgallowedips wasn't > working on macppc. It isn't working on armv7 either. I posted on bugs@ earlier, but it seems the problem I found might be the same as is being