Re: mg(1): dired-jump

2021-03-18 Thread George Koehler
On Thu, 18 Mar 2021 20:22:44 + (UTC) Mark Lumsden wrote: > This diff was first seen on the tech list just under a year ago. It is > from Philip K. and was tested by George Koehler. I have modified it > slightly. From Philips original post: You sent, or I received, your diff with wrong

mg(1): dired-jump

2021-03-18 Thread Mark Lumsden
This diff was first seen on the tech list just under a year ago. It is from Philip K. and was tested by George Koehler. I have modified it slightly. From Philips original post: this patch implements "dired-jump" for mg. For those not familiar with what dired-jump does in GNU Emacs, here's it's

Re: vmctl status does not reflect the stopping state of a VM

2021-03-18 Thread Dave Voutila
Preben Guldberg writes: > In "vmctl status", VMs that are being stopped but are still running > will simply show up as "running". > > The diff below gives preference to showing the stopping state akin to > how a paused VM is handled. I've tested this and it works as advertised. I guess I had

installer: remove libLLVM.so.2.0 during upgrade

2021-03-18 Thread Klemens Nanni
It is over a year old: $ ls -lh /usr/lib/libLLVM.so.* -r--r--r-- 1 root bin 47.3M Jul 28 2020 /usr/lib/libLLVM.so.2.0 -r--r--r-- 1 root bin 56.3M Aug 6 2020 /usr/lib/libLLVM.so.3.0 -r--r--r-- 1 root bin 56.4M Mar 9 19:17 /usr/lib/libLLVM.so.4.0 Do

mg(1): regress tests and pty

2021-03-18 Thread Mark Lumsden
I am currently writing regress tests for mg which use an mg startup file's capabilities to process mg commands to test mg's internal functions. In order to facilitate mg fitting into the OpenBSD regress test framework and be able to run via a cron job I have amended mg to be able to run with a

syslogd kernel timestamp

2021-03-18 Thread Alexander Bluhm
Hi, Since we stash log messages in the kernel, the timestamps added by syslogd are delayed. The kernel could add the timestamp when it receives the message by sendsyslog(2). This is more precise and can be expressed by more digits in the ISO timestamp. I have to copyin(9) at the beginning of

Re: rpki-client: do not include ':' in port

2021-03-18 Thread Claudio Jeker
On Thu, Mar 18, 2021 at 04:46:08PM +0100, Theo Buehler wrote: > The port number starts after the ':'. Agreed. OK claudio@ > Index: http.c > === > RCS file: /cvs/src/usr.sbin/rpki-client/http.c,v > retrieving revision 1.7 > diff -u

Re: uvm: sync some comments with NetBSD

2021-03-18 Thread Mark Kettenis
> Date: Thu, 18 Mar 2021 09:26:14 +0100 > From: Martin Pieuchot > > Diff below only touches comments in sys/uvm. It reverts the commit from > 2014 that turned three line comments into one line comments and sync > some more block with NetBSD -current. This helps reducing the diff with > NetBSD.

rpki-client: do not include ':' in port

2021-03-18 Thread Theo Buehler
The port number starts after the ':'. Index: http.c === RCS file: /cvs/src/usr.sbin/rpki-client/http.c,v retrieving revision 1.7 diff -u -p -r1.7 http.c --- http.c 18 Mar 2021 15:40:45 - 1.7 +++ http.c 18 Mar 2021

Re: rpki-client: avoid NULL access in http_parse_uri()

2021-03-18 Thread Claudio Jeker
On Thu, Mar 18, 2021 at 03:54:48PM +0100, Theo Buehler wrote: > A malformed URI such as "https://[::1/index.html; causes a NULL access > in the hosttail[1] == ":" check. Good catch. I think your diff makes this code a bit easier to understand. OK claudio@ > Index: http.c >

rpki-client: avoid NULL access in http_parse_uri()

2021-03-18 Thread Theo Buehler
A malformed URI such as "https://[::1/index.html; causes a NULL access in the hosttail[1] == ":" check. Index: http.c === RCS file: /cvs/src/usr.sbin/rpki-client/http.c,v retrieving revision 1.6 diff -u -p -r1.6 http.c --- http.c

Re: ifconfig and MPLS: document commands and ioctls, SIOCDELLABEL inconsistency

2021-03-18 Thread Claudio Jeker
On Tue, Mar 16, 2021 at 09:46:27AM +0100, Klemens Nanni wrote: > On Tue, Mar 16, 2021 at 08:55:43AM +0100, Claudio Jeker wrote: > > > .Sh HISTORY > > > -The > > > +The , > > > > Why did you add this ',' that looks strange to me. > > > > > .Nm > > > device first appeared in > > > .Ox 4.4 . >

Re: rpki-client cleanup

2021-03-18 Thread Theo Buehler
On Thu, Mar 18, 2021 at 11:25:25AM +0100, Claudio Jeker wrote: > This diff aims at removing some warnings seen in -portable. > > - gcc has a hard time to realize when a variable like noop ensures that > an other variable is not used. > > - Similar the switch () statements in http.c include all

Re: Patch for crypt(3) man page.

2021-03-18 Thread Rubén Llorente
I suspect this patch has been forgotten, so I am bumping this thread as a reminder. I'd like this man page cleaned up :-) On Wed, Jan 27, 2021 at 10:45:46PM +0100, Ingo Schwarze wrote: > Hi, > > this page is a mess. It is full of unclear wordings, in some cases > verging incorrect statements.

iwn(4): clear frames before firmware's BA window only

2021-03-18 Thread Stefan Sperling
Our iwn(4) driver tries to be a bit smart and clears already acknowledged frames from Tx aggregation queues, even if those frames are still in the firmware's current block ack window. Instead, the driver can simply clear frames before the firmware's BA window. This matches what the Linux driver

ntpd offset loop refactoring

2021-03-18 Thread Alexander Bluhm
Hi, While hunting a bug in ntpd offset handling, I found some things that could be improved. Call the index of the offset loops "shift" consistently. Merge the two offset loops in client_update() into one. Assign the best value instead of memcpy. Use the same mechanism everywhere to avoid an

rpki-client cleanup

2021-03-18 Thread Claudio Jeker
This diff aims at removing some warnings seen in -portable. - gcc has a hard time to realize when a variable like noop ensures that an other variable is not used. - Similar the switch () statements in http.c include all possible enums but gcc warns about control reaches end of non-void function.

Re: Fix iwx wrong ac indexs.

2021-03-18 Thread Stefan Sperling
On Thu, Mar 18, 2021 at 11:41:33AM +0800, zxystd wrote: > The indexes into the ac array in the iwx_mac_ctxt_cmd_common are from the > iwx_ac enum and not the txfs. The current code therefore puts the edca params > in the wrong indexes of the array, causing wrong priority for data-streams of >

uvm: sync some comments with NetBSD

2021-03-18 Thread Martin Pieuchot
Diff below only touches comments in sys/uvm. It reverts the commit from 2014 that turned three line comments into one line comments and sync some more block with NetBSD -current. This helps reducing the diff with NetBSD. ok? Index: uvm/uvm_addr.c