Re: Remove comcnspeed and comcnmode on armv7

2018-05-30 Thread Jonathan Gray
On Wed, May 30, 2018 at 03:27:22PM +0200, Mark Kettenis wrote: > Remove unused infrastructure. After this we arm64 and armv7 com_fdt.c > are identical and I can move it into dev/fdt. > > ok? ok, there is also dev/fdt/imxuart.c:extern int comcnspeed; dev/fdt/imxuart.c:extern int comcnmode; >

Re: [Patch] mg(1): Experimental UTF-8 support

2018-05-30 Thread Theo de Raadt
This approach seems misguided. Let me tell a story. More than two decades ago, I made a fork of mg which was 100% byte clean. Unfortunately I lost the source code of that change. mg's data buffers are a linked list of lines, with a \n implied by the end of each string. 0 bytes are

Enable bwfm(4) on Loongson for USB devices

2018-05-30 Thread Frederic Cambus
Hi tech@, Here is a diff to enable bwfm(4) on Loongson for USB devices. Tested on a Lemote Yeeloong 8101B. Comments? OK? Index: sys/arch/loongson/conf/GENERIC === RCS file: /cvs/src/sys/arch/loongson/conf/GENERIC,v retrieving

Re: bgpd ignore aspath with to large attributes

2018-05-30 Thread Sebastian Benoit
Claudio Jeker(cje...@diehard.n-r-g.com) on 2018.05.30 15:18:45 +0200: > Hi, > > This adds a protection to handle aspaths overly large attributes in bgpd. > The main idea is to protect other bgp routes downstream for hitting the > limit with is often not well catched. > The limit is currently a

Re: [Patch] mg(1): Experimental UTF-8 support

2018-05-30 Thread S. Gilles
On 2018-05-30T09:17:22-0600, Theo de Raadt wrote: > This approach seems misguided. Let me tell a story. > > More than two decades ago, I made a fork of mg which was 100% byte > clean. Unfortunately I lost the source code of that change. mg's data > buffers are a linked list of lines, with a \n

Re: [Patch] mg(1): Experimental UTF-8 support

2018-05-30 Thread S. Gilles
On 2018-05-30T11:50:37+0200, Stefan Sperling wrote: > Comments inline. I think this still needs a lot of work... Thanks for the review; replies inline (and omitted where the reply is the same as one above). By the time you read this, I'll have pushed the changes I mention to my branch in hboetes'

Re: Libressl question

2018-05-30 Thread Ted Unangst
Brent Cook wrote: > On Mon, Feb 12, 2018 at 09:27:16AM -0600, ed...@pettijohn-web.com wrote: > > Has there been any discussion of packaging libtls separately from libressl > > for portable use? With my limited skills I was able to write a program to > > talk to smtpd and starttls using nothing

Re: Libressl question

2018-05-30 Thread Theo de Raadt
> libtls is tied to libressl as a matter of convenience and happenstance, > but it doesn't need to be that way. If I recall, libtls requires the ability to pass certs as memory rather than directly loading it from a file. And OpenSSL native lacks any way of doing this, and their team rejected

Re: fdinsert(), take 2

2018-05-30 Thread Alexander Bluhm
On Mon, May 28, 2018 at 01:10:29PM +0200, Martin Pieuchot wrote: > This diff has been tested as part of the big "unlocking" diff I sent > last week, so I'm confident, but reviews & tests are more than welcome! > > Ok? Passes all regress tests. OK bluhm@ > Index: kern/kern_exec.c >

Use IPL_MPFLOOR for 'f_mtx'

2018-05-30 Thread Mathieu -
Hello, Now that IPL_MPFLOOR is available on all platforms we can use it. The first user is the f_mtx mutex in kern_descrip. Rev 1.160 initialized it using IPL_VM, we can now use the more appropriate IPL_MPFLOOR value as already done in the MI mutex implementation for the exact same reason.

Re: [Patch] mg(1): Experimental UTF-8 support

2018-05-30 Thread Hiltjo Posthuma
On Tue, May 29, 2018 at 05:22:43PM +0300, Leonid Bobrov wrote: > On Tue, May 29, 2018 at 03:33:08PM +0200, Henning Brauer wrote: > > Hi, > > > > very welcome! > > > > I have applied the diff and don't notice immediate breakage. Pls poke > > You didn't notice cursor movement bugs? o_O > Well,

Re: [Patch] mg(1): Experimental UTF-8 support

2018-05-30 Thread Leonid Bobrov
On Wed, May 30, 2018 at 09:05:12AM +0200, Hiltjo Posthuma wrote: > On Tue, May 29, 2018 at 05:22:43PM +0300, Leonid Bobrov wrote: > > On Tue, May 29, 2018 at 03:33:08PM +0200, Henning Brauer wrote: > > > Hi, > > > > > > very welcome! > > > > > > I have applied the diff and don't notice immediate

Re: [Patch] mg(1): Experimental UTF-8 support

2018-05-30 Thread Reyk Floeter
> Am 30.05.2018 um 10:10 schrieb Leonid Bobrov : > >> On Wed, May 30, 2018 at 09:05:12AM +0200, Hiltjo Posthuma wrote: >>> On Tue, May 29, 2018 at 05:22:43PM +0300, Leonid Bobrov wrote: On Tue, May 29, 2018 at 03:33:08PM +0200, Henning Brauer wrote: Hi, very welcome!

[patch] Add kvm_close in mib_hrsystemprocs function

2018-05-30 Thread Nan Xiao
Hi tech@, Maybe kvm_close is needed if kvm_getprocs returns NULL here? Sorry if I am wrong, thanks! Index: mib.c === RCS file: /cvs/src/usr.sbin/snmpd/mib.c,v retrieving revision 1.87 diff -u -p -r1.87 mib.c --- mib.c 25 May

const for X509_certificate_type()

2018-05-30 Thread Theo Buehler
Here's one that is not entirely trivial. This needs my previous diff because of EVP_PKEY_size(). Instead of using the refcounting in X509_get_pubkey() and EVP_PKEY_free(), use X509_get0_pubkey() and check its return value in the proper place. Zap an ugly comment that lived for 20 years and

Re: [Patch] mg(1): Experimental UTF-8 support

2018-05-30 Thread Stefan Sperling
Comments inline. I think this still needs a lot of work... I've done one pass over this to weed out some obvious problems. There might be many I've missed. Not going to do any run-time testing because emacs/mg break my fingers ;) On Mon, May 28, 2018 at 09:32:10PM +0300, Leonid Bobrov wrote: >

Re: [patch] Add kvm_close in mib_hrsystemprocs function

2018-05-30 Thread Gerhard Roth
On Wed, 30 May 2018 16:25:55 +0800 Nan Xiao wrote: > Hi tech@, > > Maybe kvm_close is needed if kvm_getprocs returns NULL here? Sorry if I > am wrong, thanks! > > Index: mib.c > === > RCS file: /cvs/src/usr.sbin/snmpd/mib.c,v >

Re: httpd request rewrite

2018-05-30 Thread Sebastian Benoit
Reyk Floeter(r...@openbsd.org) on 2018.05.29 18:48:31 +0200: > Hi, > > it's about time. > > server "default" { > listen on * port 80 > location match "/de/(.*)" { > request rewrite "/ch/%1" > } > } > > You can also you

Re: httpd request rewrite

2018-05-30 Thread Sebastian Benoit
Sebastian Benoit(be...@openbsd.org) on 2018.05.30 10:10:51 +0200: > Reyk Floeter(r...@openbsd.org) on 2018.05.29 18:48:31 +0200: > > Hi, > > > > it's about time. > > > > server "default" { > > listen on * port 80 > > location match "/de/(.*)" { > >

Re: httpd request rewrite

2018-05-30 Thread Reyk Floeter
> Am 30.05.2018 um 10:12 schrieb Sebastian Benoit : > > Sebastian Benoit(be...@openbsd.org) on 2018.05.30 10:10:51 +0200: >> Reyk Floeter(r...@openbsd.org) on 2018.05.29 18:48:31 +0200: >>> Hi, >>> >>> it's about time. >>> >>>server "default" { >>>listen on * port 80 >>>

Adding IPL_MPFLOOR to sh/armv7

2018-05-30 Thread Mathieu -
Hi, The following patch adds IPL_MPFLOOR to the two last archs missing it, namely landisk and armv7. The goal is to be able to use the define in MI code (note that it's already used but hidden behind a #ifdef MULTIPROCESSOR). I define it to IPL_NONE because those two archs are only

Re: [Patch] mg(1): Experimental UTF-8 support

2018-05-30 Thread Leonid Bobrov
Hi Stefan! I'm sorry, fileio.c was out of sync that day, that's my bad, I forgot to remove its diff. Sorry for not reading diffs before sending them (I read your comments), that's why I call it experimental: I blindly applied diffs and tested mg in runtime. I'll send your comments to S. Gilles,

bgpd ignore aspath with to large attributes

2018-05-30 Thread Claudio Jeker
Hi, This adds a protection to handle aspaths overly large attributes in bgpd. The main idea is to protect other bgp routes downstream for hitting the limit with is often not well catched. The limit is currently a bit arbitarily set to 4096 - 1024 which is afaik the same limit that bird uses.

Re: Adding IPL_MPFLOOR to sh/armv7

2018-05-30 Thread Martin Pieuchot
On 30/05/18(Wed) 10:54, Mathieu - wrote: > The following patch adds IPL_MPFLOOR to the two last archs missing it, > namely landisk and armv7. The goal is to be able to use the define in MI > code (note that it's already used but hidden behind a #ifdef > MULTIPROCESSOR). Committed thanks, please

Re: [Patch] mg(1): Experimental UTF-8 support

2018-05-30 Thread Leonid Bobrov
I'm updating a diff, but it is just a fix for cursor movement, it is still experimental and obvious problems are not fixed yet, this is just a diff taken from latest commit at github: Index: basic.c === RCS file:

Remove comcnspeed and comcnmode on armv7

2018-05-30 Thread Mark Kettenis
Remove unused infrastructure. After this we arm64 and armv7 com_fdt.c are identical and I can move it into dev/fdt. ok? Index: armv7/armv7_machdep.c === RCS file: /cvs/src/sys/arch/armv7/armv7/armv7_machdep.c,v retrieving revision