Re: Add "-c command" option to script(1)

2017-12-13 Thread Paul de Weerd
Hi Jason, Thank you for your quick feedback! I've incorporated yours and off-list feedback from ian@ into a new diff included below. The diff now also includes the removal of /* ARGSUSED */ lint comments. On Wed, Dec 13, 2017 at 09:34:03PM +, Jason McIntyre wrote: | > +To run a | | i'd

Re: Add reset option to boot command of ddb(4)

2017-12-13 Thread Florian Riehm
I will prepare a new diff including the other architecures and try to find people who can test it. I have had such a diff already but then I decided to remove the untested parts because I didn't want to submit untested code. friehm On 12/13/17 21:59, Theo de Raadt wrote: As it is, this diff

Re: ntpd: mark packet used for settime as bad

2017-12-13 Thread Patrik Dahlström
Sorry, I seem to have messed up the whitespace. Updated patch: diff --git usr.sbin/ntpd/client.c usr.sbin/ntpd/client.c index 3de5268..5ee7ea8 100644 --- usr.sbin/ntpd/client.c +++ usr.sbin/ntpd/client.c @@ -402,8 +402,10 @@ client_dispatch(struct ntp_peer *p, u_int8_t settime) (long

ntpd: mark packet used for settime as bad

2017-12-13 Thread Patrik Dahlström
Hello, We recently discovered a bug in ntpd: When the '-s' flag is used for ntpd, the first reply received will be used to set the current time. If this reply also has the shortest round trip, it will be used to adjust the time later. Since openntpd uses a circular buffer of 8 replies, this leads

Symbols.map for libkvm

2017-12-13 Thread Philip Guenther
Inspired by kettenis@'s diff for libutil, here's a diff to do the (simpler) work for libkvm, The symbols this hides are just the linker defined ones; we've already declared the internal C symbols static or at least hidden. However, while here I've added the dance to have the three internally

Re: background scan for iwn(4)

2017-12-13 Thread Mike Larkin
On Wed, Dec 13, 2017 at 05:17:41PM +0100, Stefan Sperling wrote: > Since nobody is reporting problems with iwm(4), I took some time to write the > corresponding diff for iwn(4) as well. I hope this increases test coverage :) > > Works for me on: > iwn0 at pci3 dev 0 function 0 "Intel Centrino

Re: Symbols.map for libutil

2017-12-13 Thread Philip Guenther
On Thu, 14 Dec 2017, Mark Kettenis wrote: > Diff below makes sure libutil doesn't export any unwanted symbols. The > immediate motivation here is to prevent exportong certain libgcc symbols > on armv7 that cause issues when switching from gcc to clang. > > With this diff, only symbols present

simplify sched_barrier with a wait condition on the stack

2017-12-13 Thread David Gwynne
this started as just using struct cond to handle the sleep and wakeups for sched_barrier, but it got a bit bigger. the biggest semantic change is that it gets rid of the sbar taskq and uses systqmp instead. a whole thread (which is what is underneath a taskq) is a very big hammer for solving this

Re: background scan for iwn(4)

2017-12-13 Thread Mike Burns
On 2017-12-13 17.17.41 +0100, Stefan Sperling wrote: > Since nobody is reporting problems with iwm(4), I took some time to write the > corresponding diff for iwn(4) as well. I hope this increases test coverage :) > > Works for me on: > iwn0 at pci3 dev 0 function 0 "Intel Centrino Advanced-N

Re: Symbols.map for libutil

2017-12-13 Thread Theo de Raadt
Lovely. > Diff below makes sure libutil doesn't export any unwanted symbols. > The immediate motivation here is to prevent exportong certain libgcc > symbols on armv7 that cause issues when switching from gcc to clang. > > With this diff, only symbols present in man pages and public header >

Re: TL-WN722N usb ids and attach as urtwn(4)

2017-12-13 Thread Mark Kettenis
> Date: Thu, 14 Dec 2017 00:06:35 +0100 > From: Sebastian Benoit > > Dec 13 23:49:08 blap /bsd: urtwn0 at uhub0 port 1 configuration 1 interface 0 > "Realtek 802.11n NIC" rev 2.00/0.00 addr 4 > Dec 13 23:49:08 blap /bsd: urtwn0: MAC/BB RTL8188EU, RF 6052 1T1R, address > >

Re: Change bcopy to memcpy (i386 VIA PadLock driver)

2017-12-13 Thread David Gwynne
> On 14 Dec 2017, at 07:23, Frederic Cambus wrote: > > Hi tech@, > > This diff changes bcopy to memcpy in the i386 version of the VIA PadLock > driver. It was done a while ago in the amd64 version of the driver. > > Comments? OK? ok > > Index: sys/arch/i386/i386/via.c >

Symbols.map for libutil

2017-12-13 Thread Mark Kettenis
Diff below makes sure libutil doesn't export any unwanted symbols. The immediate motivation here is to prevent exportong certain libgcc symbols on armv7 that cause issues when switching from gcc to clang. With this diff, only symbols present in man pages and public header files are exported. A

TL-WN722N usb ids and attach as urtwn(4)

2017-12-13 Thread Sebastian Benoit
Dec 13 23:49:08 blap /bsd: urtwn0 at uhub0 port 1 configuration 1 interface 0 "Realtek 802.11n NIC" rev 2.00/0.00 addr 4 Dec 13 23:49:08 blap /bsd: urtwn0: MAC/BB RTL8188EU, RF 6052 1T1R, address and i am writing this mail using this device. ok? (benno_urtwn_tl-wn722n.diff) diff --git

Re: mess with regression tests

2017-12-13 Thread Alexander Bluhm
On Wed, Dec 06, 2017 at 11:19:18PM +0300, Sergey Bronnikov wrote: > Patch below adds similar scripts for ed, perl, gdb, libkeynote, ctags, > m4 and sed. Patch looks huge but it is actually simple. Anyway if you > want splitted patch I will resend. It is not the patch that is too big but the task.

Re: Add "-c command" option to script(1)

2017-12-13 Thread Jason McIntyre
On Wed, Dec 13, 2017 at 05:45:25PM +0100, Paul de Weerd wrote: > > Index: script.1 > === > RCS file: /cvs/src/usr.bin/script/script.1,v > retrieving revision 1.14 > diff -u -p -r1.14 script.1 > --- script.1 15 Jan 2012 20:06:40

Re: background scan for iwn(4)

2017-12-13 Thread Sebastian Benoit
Stefan Sperling(s...@stsp.name) on 2017.12.13 17:17:41 +0100: > Since nobody is reporting problems with iwm(4), I took some time to write the > corresponding diff for iwn(4) as well. I hope this increases test coverage :) > > Works for me on: > iwn0 at pci3 dev 0 function 0 "Intel Centrino

Re: Change bcopy to memcpy (i386 VIA PadLock driver)

2017-12-13 Thread Mike Larkin
On Wed, Dec 13, 2017 at 10:23:46PM +0100, Frederic Cambus wrote: > Hi tech@, > > This diff changes bcopy to memcpy in the i386 version of the VIA PadLock > driver. It was done a while ago in the amd64 version of the driver. > > Comments? OK? > ok mlarkin > Index: sys/arch/i386/i386/via.c >

Change bcopy to memcpy (i386 VIA PadLock driver)

2017-12-13 Thread Frederic Cambus
Hi tech@, This diff changes bcopy to memcpy in the i386 version of the VIA PadLock driver. It was done a while ago in the amd64 version of the driver. Comments? OK? Index: sys/arch/i386/i386/via.c === RCS file:

Re: update Mesa to 17.2.6

2017-12-13 Thread Bryan Steele
On Wed, Dec 13, 2017 at 04:19:03PM -0500, Bryan Steele wrote: > On Wed, Dec 13, 2017 at 09:56:32PM +0100, Matthieu Herrb wrote: > > On Mon, Nov 27, 2017 at 08:22:30PM +1100, Jonathan Gray wrote: > > > cd /usr/xenocara/lib > > > ftp https://mesa.freedesktop.org/archive/mesa-17.2.6.tar.gz > > > ftp

Re: update Mesa to 17.2.6

2017-12-13 Thread Bryan Steele
On Wed, Dec 13, 2017 at 09:56:32PM +0100, Matthieu Herrb wrote: > On Mon, Nov 27, 2017 at 08:22:30PM +1100, Jonathan Gray wrote: > > cd /usr/xenocara/lib > > ftp https://mesa.freedesktop.org/archive/mesa-17.2.6.tar.gz > > ftp http://jsg.id.au/mesa-update/mesa.diff.gz > > tar zxf mesa-17.2.6.tar.gz

Re: Add reset option to boot command of ddb(4)

2017-12-13 Thread Raf Czlonka
On Wed, Dec 13, 2017 at 06:09:14PM GMT, Florian Riehm wrote: > Hi, > > This patch follows bluhm's attempt for a ddb command 'boot reset'. > My first attempt was not architecture aware. > > Tested on i386 by bluhm@ and on amd64 by me. > > ok? > > friehm > > Index: share/man/man4/ddb.4 >

Re: Add reset option to boot command of ddb(4)

2017-12-13 Thread Theo de Raadt
As it is, this diff will not go in. Your 2nd attempt is not architecture aware either. There are more than 2 architectures. If you add a MI feature, you must attempt to add support for it to all the MD versions. And the process of mailing it out to the community gives people an opportunity to

Re: update Mesa to 17.2.6

2017-12-13 Thread Matthieu Herrb
On Mon, Nov 27, 2017 at 08:22:30PM +1100, Jonathan Gray wrote: > cd /usr/xenocara/lib > ftp https://mesa.freedesktop.org/archive/mesa-17.2.6.tar.gz > ftp http://jsg.id.au/mesa-update/mesa.diff.gz > tar zxf mesa-17.2.6.tar.gz > gunzip mesa.diff.gz > patch -p0 < mesa.diff > > sed -i

Add reset option to boot command of ddb(4)

2017-12-13 Thread Florian Riehm
Hi, This patch follows bluhm's attempt for a ddb command 'boot reset'. My first attempt was not architecture aware. Tested on i386 by bluhm@ and on amd64 by me. ok? friehm Index: share/man/man4/ddb.4 === RCS file:

Re: rpc: use monotime for timeouts

2017-12-13 Thread Jeremie Courreges-Anglas
On Tue, Dec 12 2017, Jeremie Courreges-Anglas wrote: > On Sun, Dec 10 2017, Scott Cheloha wrote: >> Hi, >> >> These timeouts in sunrpc need to be based on the monotonic >> clock to avoid a race with adjtime(2), settimeofday(2), etc. >> >> There are

Add "-c command" option to script(1)

2017-12-13 Thread Paul de Weerd
Hi all, As seen in NetBSD, FreeBSD and Linux, script(1) could do with a way to tell it what command to run instead of your $SHELL. This implements the "-c command" option. I've taken the logic to start the command (and its arguments, if any) from crontab(1), execv'ing `sh -c `. I wrote this

background scan for iwn(4)

2017-12-13 Thread Stefan Sperling
Since nobody is reporting problems with iwm(4), I took some time to write the corresponding diff for iwn(4) as well. I hope this increases test coverage :) Works for me on: iwn0 at pci3 dev 0 function 0 "Intel Centrino Advanced-N 6200" rev 0x35: msi, MIMO 2T2R, MoW Index: if_iwn.c

Re: libressl: crash in DES_fcrypt

2017-12-13 Thread Bob Beck
why AA? why not just choose two random ascii salt chars at that point? or since this is effectively a failure case encrypt a random ascii salt and random string? using AA will produce a usable result based on the original string. encrypting a random string with a random salt means the failure

Re: libressl: crash in DES_fcrypt

2017-12-13 Thread Theo de Raadt
I still don't understand. This feels like fail-open. > I would like to commit this fix. I tried to avoid the crash in > libcrypto with least possible impact for the DES_fcrypt() API. > > ok? > > bluhm > > On Tue, Dec 05, 2017 at 05:20:49PM +0100, Alexander Bluhm wrote: > > On Fri, Oct 27,

Re: libressl: crash in DES_fcrypt

2017-12-13 Thread Alexander Bluhm
Hi, I would like to commit this fix. I tried to avoid the crash in libcrypto with least possible impact for the DES_fcrypt() API. ok? bluhm On Tue, Dec 05, 2017 at 05:20:49PM +0100, Alexander Bluhm wrote: > On Fri, Oct 27, 2017 at 01:50:26AM +0200, Jan Engelhardt wrote: > > #include > > int

Re: OpenBSD OpenCL 2.x support perspective

2017-12-13 Thread Paul Irofti
There is no OpenCL support in OpenBSD.

Re: disable hw vlan tagging support in ix(4)

2017-12-13 Thread Kapetanakis Giannis
On 13/12/17 10:29, Martin Pieuchot wrote: > On 13/12/17(Wed) 09:54, David Gwynne wrote: >> im still looking at vlan performance problems, as discussed by mpi@ >> at http://www.grenadille.net/post/2017/02/13/What-happened-to-my-vlan. >> >> recently it occurred to me that we're making an implicit

Re: net/rtsock.c: size to free(9)

2017-12-13 Thread Martin Pieuchot
On 11/12/17(Mon) 15:46, kshe wrote: > On Sun, 10 Dec 2017 11:25:50 +, Martin Pieuchot wrote: > > On 08/12/17(Fri) 12:58, kshe wrote: > > > I noticed one instance where the size given to free(9) can easily be > > > determined. > > > > What about the other free(9)s in the same function? > >

Re: disable hw vlan tagging support in ix(4)

2017-12-13 Thread Martin Pieuchot
On 13/12/17(Wed) 09:54, David Gwynne wrote: > im still looking at vlan performance problems, as discussed by mpi@ > at http://www.grenadille.net/post/2017/02/13/What-happened-to-my-vlan. > > recently it occurred to me that we're making an implicit assumption > that having the chip handle the

Re: relayd and PUT

2017-12-13 Thread Rivo Nurges
Hi! Thanks for enlightening me, I’ll try to fix the real problem. Rivo > On 13 Dec 2017, at 08:42, Claudio Jeker wrote: > >> On Wed, Dec 13, 2017 at 12:25:39AM +, Rivo Nurges wrote: >> Hi! >> >> If you http PUT a "big" file through relayd, server<>relay read