at(1) job files have to belong to group crontab

Hi, since 2017/06/07, spooled at(1) files have to be owned by group crontab, and at(1) makes sure that this is the case for newly created at(1) jobs. However, if you create an at(1) job while still running OpenBSD-6.1 and then update to OpenBSD-6.2 *before* that at(1) job had been run, it will

Re: sshd(8) logging of client disconnect from ClientAliveInterval

On 17 October 2017 at 06:04, Lars Noodén wrote: > > + logit("Timeout, client not responding from %s on port %d.", > + ssh_remote_ipaddr(ssh), ssh_remote_port(ssh) ); > probably better to use fmt_connection_id() instead of hand-rolling the

Re: man pages for OpenBSD 6.2

Hi, Raf Czlonka wrote on Mon, Oct 16, 2017 at 08:04:00AM +0100: > Manual pages for OpenBSD 6.2 are not yet available on > https://man.openbsd.org/ > > I thought I'd mention it in case this has slipped your attention :^) Oh my... I ought to know by now that i have a job to do twice each year...

sshd(8) logging of client disconnect from ClientAliveInterval

sshd(8) used to log connection information when ClientAliveInterval disconnected an idle client. It used to look something like this a while back: Oct 16 14:42:08 eee sshd[83709]: packet_write_poll: Connection from 192.0.2.97 port 57608: Host is down Now it looks like this:

Re: apmd: manual tweak

On Mon, Oct 16, 2017 at 07:51:47PM +0200, Anton Lindqvist wrote: > On Mon, Oct 16, 2017 at 06:10:02PM +0100, Jason McIntyre wrote: > > On Mon, Oct 16, 2017 at 06:38:31PM +0200, Anton Lindqvist wrote: > > > > > > Good call. Without knowing too much about the apmd internals, here's a > > > first

Re: apmd: manual tweak

On Mon, Oct 16, 2017 at 06:10:02PM +0100, Jason McIntyre wrote: > On Mon, Oct 16, 2017 at 06:38:31PM +0200, Anton Lindqvist wrote: > > > > Good call. Without knowing too much about the apmd internals, here's a > > first stab. As I read the code, the period polling is also used by > >

Re: apmd: manual tweak

On Mon, Oct 16, 2017 at 06:38:31PM +0200, Anton Lindqvist wrote: > > Good call. Without knowing too much about the apmd internals, here's a > first stab. As I read the code, the period polling is also used by > auto{suspend,hibernate}. Is this worth mentioning? Since a more frequent > interval

Re: apmd: manual tweak

On Mon, Oct 16, 2017 at 03:57:22PM +0100, Jason McIntyre wrote: > On Mon, Oct 16, 2017 at 04:06:59PM +0200, Anton Lindqvist wrote: > > Hi, > > A proposal on improving the `-t` option documentation. Most importantly, > > remove the recursive reference to the option being documented. > > > >

Re: apmd: manual tweak

On Mon, Oct 16, 2017 at 04:06:59PM +0200, Anton Lindqvist wrote: > Hi, > A proposal on improving the `-t` option documentation. Most importantly, > remove the recursive reference to the option being documented. > > Comments? OK? > > Index: apmd.8 >

apmd: manual tweak

Hi, A proposal on improving the `-t` option documentation. Most importantly, remove the recursive reference to the option being documented. Comments? OK? Index: apmd.8 === RCS file: /cvs/src/usr.sbin/apmd/apmd.8,v retrieving

Re: libfuse: improved command line parsing

On 14/10/17(Sat) 01:23, Helg Bredow wrote: > The attached patch vastly improves fuse argument and option parsing. For > example, all sshfs options will now be parsed successfully. It's a > substantial patch and can't be easily broken down further. > > In addition, this also adds support for the

Re: route warning

On Mon, Oct 16, 2017 at 01:18:27PM +0200, Martin Pieuchot wrote: > Removing the if () block should do it. It introduces a small change in > behavior, route(8) now exists with an error code if an entry doesn't > exist in the table. That's a good thing, because one can now test if a > route is

Re: Kill SIOCSIFPHYADDR & friends

On Mon, Oct 16, 2017 at 12:35:35PM +0200, Martin Pieuchot wrote: > The following ioctl(2)s are deprecated since 2001. I'd like to > remove our kernel support for them: > - SIOCSIFPHYADDR > - SIOCSIFPHYADDR_IN6 > - SIOCGIFPSRCADDR > - SIOCGIFPSRCADDR_IN6 > -

Re: ifioctl() tweak #3

On Mon, Oct 16, 2017 at 12:13:09PM +0200, Martin Pieuchot wrote: > All interfaces define an `if_ioctl' function pointer. So do not check > for it. > > Change the remaining 'return' into 'break'. > > In SIOCSIFLLADDR change the address *after* querying the driver. This > make the logic in

Re: route warning

On 12/10/17(Thu) 00:22, Julien Dhaille wrote: > Hi, > > when a default gateway is not set : > > # route get 4.4.4.4 > route: writing to routing socket: No such process > > this small patch uses oerrno translation : > > # route get 4.4.4.4 > get host 4.4.4.4: not in table Nice, but there's no

Re: dirty1, for those w/vanishing memory to keep up w/multiple cu's

On Sun, Oct 15, 2017 at 10:45:53AM +0300, Artturi Alm wrote: > Hi, > > i've been saving time+frustration for having had this, but i'm not proposing > anything, i'm aware how this what most likely exists only because of how badly > scripts stick w/me across reinstalls/need of backups, compared to

Please test: IPsec w/o KERNEL_LOCK()

On 11/10/17(Wed) 17:01, Martin Pieuchot wrote: > OpenBSD 6.2 includes nice performance and latency improvements due to > the work done in the Network Stack in the previous years. However as > soon as IPsec is enabled, all network related processing are affected. > In other words you cannot profit

Kill SIOCSIFPHYADDR & friends

The following ioctl(2)s are deprecated since 2001. I'd like to remove our kernel support for them: - SIOCSIFPHYADDR - SIOCSIFPHYADDR_IN6 - SIOCGIFPSRCADDR - SIOCGIFPSRCADDR_IN6 - SIOCGIFPDSTADDR - SIOCGIFPDSTADDR_IN6 ok? Index: net/if.c

ifioctl() tweak #3

All interfaces define an `if_ioctl' function pointer. So do not check for it. Change the remaining 'return' into 'break'. In SIOCSIFLLADDR change the address *after* querying the driver. This make the logic in vlan(4) work as intended. ok? Index: net/if.c

Re: rwlock tweak

> Date: Mon, 16 Oct 2017 11:53:30 +0200 > From: Martin Pieuchot > > On 16/10/17(Mon) 11:36, Mark Kettenis wrote: > > > Date: Mon, 16 Oct 2017 10:52:09 +0200 > > > From: Martin Pieuchot > > > > > > As pointed by Mateusz Guzik [0], on x86 the cmpxchg{q,l} used

Re: rwlock tweak

On Mon, Oct 16, 2017 at 09:36 +, Mark Kettenis wrote: > > Date: Mon, 16 Oct 2017 10:52:09 +0200 > > From: Martin Pieuchot > > > > As pointed by Mateusz Guzik [0], on x86 the cmpxchg{q,l} used by > > rw_enter(9) and rw_exit(9) already include an implicit memory > > barrier,

Re: rwlock tweak

On 16/10/17(Mon) 11:36, Mark Kettenis wrote: > > Date: Mon, 16 Oct 2017 10:52:09 +0200 > > From: Martin Pieuchot > > > > As pointed by Mateusz Guzik [0], on x86 the cmpxchg{q,l} used by > > rw_enter(9) and rw_exit(9) already include an implicit memory > > barrier, so we can

Re: rwlock tweak

> Date: Mon, 16 Oct 2017 10:52:09 +0200 > From: Martin Pieuchot > > As pointed by Mateusz Guzik [0], on x86 the cmpxchg{q,l} used by > rw_enter(9) and rw_exit(9) already include an implicit memory > barrier, so we can avoids using an explicit expensive one by > using the

Re: carp(4) fix

On Mon, Oct 16, 2017 at 10:15:28AM +0200, Martin Pieuchot wrote: > Hrvoje Popovski reported the following panic: > > panic: kernel diagnostic assertion "sc->sc_carpdev != NULL" failed > panic() at panic+0x128 > __assert() at __assert+0x24 > carp_output() at carp_output+0xde >

rwlock tweak

As pointed by Mateusz Guzik [0], on x86 the cmpxchg{q,l} used by rw_enter(9) and rw_exit(9) already include an implicit memory barrier, so we can avoids using an explicit expensive one by using the following variants. [0] https://marc.info/?l=openbsd-tech=150765959923113=2 ok? Index:

carp(4) fix

Hrvoje Popovski reported the following panic: panic: kernel diagnostic assertion "sc->sc_carpdev != NULL" failed panic() at panic+0x128 __assert() at __assert+0x24 carp_output() at carp_output+0xde ip_output() at ip_output+0x948 ipsp_process_done() at ipsp_process_done+0x254

man pages for OpenBSD 6.2

Hi all, Manual pages for OpenBSD 6.2 are not yet available on https://man.openbsd.org/ I thought I'd mention it in case this has slipped your attention :^) Regards, Raf