Re: Kill uvm/uvm_stat.c

2019-12-04 Thread Mike Larkin
On Wed, Dec 04, 2019 at 03:19:41PM +0100, Martin Pieuchot wrote: > Less is more. Fewer files to look at, simpler it becomes to understand > UVM. uvm/uvm_stat.c contains just a ddb(4) function. Let's move it to > uvm/uvm_meter.c which also deals with counters. ok? > Also reads ok to me. -ml >

Re: un-boolean_t amd64's pmap

2019-12-04 Thread Mike Larkin
On Wed, Dec 04, 2019 at 03:31:07PM +0100, Martin Pieuchot wrote: > Similar to recent ddb(4) changes, replace boolean_t/TRUE/FALSE by > int/1/0. > > ok? > No objection here, unsure if anyone else has commented either way. -ml > Index: arch/amd64/amd64/pmap.c > ==

Does rpki-client need to unveil(NULL, NULL)?

2019-12-04 Thread George Brown
After pledge is immediately called without the unveil promise so this seems redundant. diff --git a/usr.sbin/rpki-client/main.c b/usr.sbin/rpki-client/main.c index f05ec1c5837..53ee4223371 100644 --- a/usr.sbin/rpki-client/main.c +++ b/usr.sbin/rpki-client/main.c @@ -1498,8 +1498,6 @@ main(int arg

ftp(1): https:// handling with NOSSL

2019-12-04 Thread Jeremie Courreges-Anglas
ftp(1) built with no TLS support is confused (confusing?) when handled an https url. I have noticed this during tests with /usr/src/distrib/special/ftp. Now: --8<-- ritchie /usr/src/distrib/special/ftp$ obj/ftp -o/dev/null https://www.openbsd.org/ ftp: https: no address associated with name ft

Fix SHA256Init.3

2019-12-04 Thread Martin
Hey! Somehow the SHA256Init(3) man page ended up being the Makefile. This patch restores the last one and changes the references in the SEE ALSO section accordingly (like SHA1Init(3)). Kind regards, Martin Index: SHA256Init.3 === R

Re: ure(4): add preliminary support for RTL8156

2019-12-04 Thread Shawn Chiou
On Tue, Dec 3, 2019 at 5:24 PM Kevin Lo wrote: > Hi, > > This diff adds preliminary support for RTL8156 to ure(4). > Tested with the Planex USB-LAN2500R. > > Index: share/man/man4/ure.4 > === > RCS file: /cvs/src/share/man/man4/ure.4

un-boolean_t amd64's pmap

2019-12-04 Thread Martin Pieuchot
Similar to recent ddb(4) changes, replace boolean_t/TRUE/FALSE by int/1/0. ok? Index: arch/amd64/amd64/pmap.c === RCS file: /cvs/src/sys/arch/amd64/amd64/pmap.c,v retrieving revision 1.136 diff -u -p -r1.136 pmap.c --- arch/amd64/amd

Kill uvm/uvm_stat.c

2019-12-04 Thread Martin Pieuchot
Less is more. Fewer files to look at, simpler it becomes to understand UVM. uvm/uvm_stat.c contains just a ddb(4) function. Let's move it to uvm/uvm_meter.c which also deals with counters. ok? Index: conf/files === RCS file: /cvs/s

thrsleep_unlock: check pointer against NULL

2019-12-04 Thread Masato Asou
ok? Index: sys/kern/kern_synch.c === RCS file: /cvs/src/sys/kern/kern_synch.c,v retrieving revision 1.155 diff -u -p -r1.155 kern_synch.c --- sys/kern/kern_synch.c 30 Nov 2019 11:19:17 - 1.155 +++ sys/kern/kern_synch.c