look(1): drop "rpath" promise after open(2)/fstat(2)

The look(1) program needs to open(2) and fstat(2) exactly one file during its runtime. Using unveil(2) seems like overkill here. This seems closer to what we want: - pledge(2) initially with "stdio rpath" at the top of main(). We know we need to read a file at this point but don't yet know

Re: ps STAT sorted

On Tue, Feb 08, 2022 at 08:39:35PM +0100, Alexander Bluhm wrote: > Hi, > > Sort the ps(1) STAT characters alphabetically like in the man page. > Note that the 'else' I have removed is redundant. > > ok? > > bluhm > > Index: bin/ps/print.c >

ps STAT sorted

Hi, Sort the ps(1) STAT characters alphabetically like in the man page. Note that the 'else' I have removed is redundant. ok? bluhm Index: bin/ps/print.c === RCS file: /data/mirror/openbsd/cvs/src/bin/ps/print.c,v retrieving

Re: mandoc.1: update example to reflect current options

Hello Anders, Anders Damsgaard wrote on Tue, Feb 08, 2022 at 08:36:11AM +0100: > The mandoc(1) option alias -l for -a was removed from the documentation > in revision 1.107: > > https://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin/mandoc/mandoc.1.diff?r1=1.106=1.107 > > The -l option still

Re: wskbd_set_mixervolume

On Tue, Feb 08, 2022 at 07:32:38AM +0100, Alexandre Ratchov wrote: > On Mon, Feb 07, 2022 at 06:55:21PM +0100, Anton Lindqvist wrote: > > On Mon, Feb 07, 2022 at 11:21:43AM +0100, Alexandre Ratchov wrote: > > > On Sun, Feb 06, 2022 at 08:40:35AM +0100, Anton Lindqvist wrote: > > > > > > > >

Unify f_modify and f_process event filter callbacks for pipes

The current code pattern used with MP-safe event filters is verbose. I would like to find a more concise form before making more event filters MP-safe. Of struct filterops callbacks, f_modify and f_process essentially use f_event. By utilizing the f_event function pointer, a subsystem can have

Re: Increase armv7 ramdisk size

Fine. Visa Hankala wrote: > On Mon, Feb 07, 2022 at 09:35:21AM -0700, Theo de Raadt wrote: > > Please don't do such a small increase, because it is likely we'll need to > > do it again the near future. > > > > I suggest taking ns#600 to ns#650 or ns#700, and adjusting all the numbers. > >

Re: tr(1): improve table names

On Sun, Jan 30, 2022 at 10:23:43AM -0600, Scott Cheloha wrote: > In tr(1), we have these two global arrays, "string1" and "string2". > > I have a few complaints: > > 1. They are not strings. They are lookup tables. The names are >misleading. > > 2. The arguments given to tr(1) in argv[]

Re: rev(1): drop "rpath" promise in no-file branch

On Tue, 08 Feb 2022 10:40:00 -0600, Scott Cheloha wrote: > We don't need "rpath" if we're only processing the standard input. Sure. OK millert@ - todd

rev(1): drop "rpath" promise in no-file branch

We don't need "rpath" if we're only processing the standard input. ok? Index: rev.c === RCS file: /cvs/src/usr.bin/rev/rev.c,v retrieving revision 1.15 diff -u -p -r1.15 rev.c --- rev.c 29 Jan 2022 00:11:54 - 1.15 +++

Re: Increase armv7 ramdisk size

On Mon, Feb 07, 2022 at 09:35:21AM -0700, Theo de Raadt wrote: > Please don't do such a small increase, because it is likely we'll need to > do it again the near future. > > I suggest taking ns#600 to ns#650 or ns#700, and adjusting all the numbers. Here is a diff that uses ns#700. OK? Index:

Re: perl clang -Wcompound-token-split-by-macro

On Tue, Feb 08, 2022 at 01:59:51PM +, Stuart Henderson wrote: > > > I think we should start updating Devel::PPPort in base. Then we > > > can regenerate other ppport.h in base and ports. Note that some ports > > > need newer Devel::PPPort than we have now. So updating seems > > >

Re: perl clang -Wcompound-token-split-by-macro

> > I think we should start updating Devel::PPPort in base. Then we > > can regenerate other ppport.h in base and ports. Note that some ports > > need newer Devel::PPPort than we have now. So updating seems > > unavoidable. I do worry a bit that if we regenerate in ports, that some ports might

Re: fix active scan on iwm and iwx

> Date: Tue, 8 Feb 2022 14:21:52 +0100 > From: Stefan Sperling > > On Tue, Jan 25, 2022 at 11:22:45AM +0100, Mark Kettenis wrote: > > > The KASSERT triggers but for the wrong reason: We don't have outstanding > > > tasks, we have a bad reference counter. Only setting the ref counter to 1 > > >

iked restart clean entry tcpmd5 of bgpd

Hi, When iked restart, the daemon cleans tcpmd5 entries (SAD entries) of bgpd. Then de daemon bgpd stop working properly. Thanks in advance. Best Regards, Breno.

Re: fix active scan on iwm and iwx

On Tue, Jan 25, 2022 at 11:22:45AM +0100, Mark Kettenis wrote: > > The KASSERT triggers but for the wrong reason: We don't have outstanding > > tasks, we have a bad reference counter. Only setting the ref counter to 1 if > > we are about to launch a task during resume should fix it, and this

Make pipes and sockets use KNOTE() instead of selwakeup()

Now that poll(2) is based on kqueue, the old, non-MP-safe poll backend is not used any longer. Event sources can call KNOTE() directly instead of selwakeup(). This diff does the KNOTE() conversion for pipes and sockets, removing a kernel-locked section from a frequently used code path. The

mandoc.1: update example to reflect current options

Hello, The mandoc(1) option alias -l for -a was removed from the documentation in revision 1.107: https://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin/mandoc/mandoc.1.diff?r1=1.106=1.107 The -l option still works, but this diff makes the example consistent with the current option description.