Re: KASSERT() @ pf_test() is back

2016-03-03 Thread Stuart Henderson
On 2016/02/28 13:01, Martin Pieuchot wrote: > On 08/02/16(Mon) 01:55, Alexandr Nedvedicky wrote: > > Hello, > > > > I don't expect to see O.K. to patch below. > > > > The patch is the part of the change, which has been backed out last weekend. > > Too many things were wrong so I'm trying to untan

/sys/dev/pci/if_oce.c - Missing parenthesis

2016-03-03 Thread Tiago Silva
Hi to all, On /sys/dev/pci/if_oce.c there's a missing parenthesis that pops when the OCE_RSS macro is defined. Thanks, Tiago Silva --- dev/pci/if_oce.cMon Jun 29 18:58:04 2015 +++ dev/pci/if_oce.cThu Mar 03 22:02:22 2016 @@ -3077,7 +3077,7 @@ memset(&cmd, 0, sizeof(cmd));

[ksh] [patch] Make "$@" POSIX-compliant with empty IFS

2016-03-03 Thread Martijn Dekker
Hi all, I'm new here and posting at Theo de Raadt's request. I'm developing a general-purpose cross-platform library for the POSIX shell language and in the process I encounter lots of bugs in various shells. I will be posting here a few times with some patches and bug reports against OpenBSD ksh.

Re: Remove WSDISPLAY_COMPAT_SYSCONS

2016-03-03 Thread Miod Vallat
> jmc@ stumbled over this. The WSDISPLAY_COMPAT_SYSCONS option was > only ever in i386 GENERIC and it has been disabled there for 15 > years, so I think we can safely delete it, whatever it was used for > (running FreeBSD X11 server binaries??). Not really. The native X server used to issue thes

Re: Remove WSDISPLAY_COMPAT_SYSCONS

2016-03-03 Thread Mike Larkin
On Thu, Mar 03, 2016 at 05:30:05PM +0100, Christian Weisgerber wrote: > jmc@ stumbled over this. The WSDISPLAY_COMPAT_SYSCONS option was > only ever in i386 GENERIC and it has been disabled there for 15 > years, so I think we can safely delete it, whatever it was used for > (running FreeBSD X11 se

Re: Remove WSDISPLAY_COMPAT_SYSCONS

2016-03-03 Thread Mark Kettenis
> Date: Thu, 3 Mar 2016 17:30:05 +0100 > From: Christian Weisgerber > > jmc@ stumbled over this. The WSDISPLAY_COMPAT_SYSCONS option was > only ever in i386 GENERIC and it has been disabled there for 15 > years, so I think we can safely delete it, whatever it was used for > (running FreeBSD X11

kern_exit.c: move declarations from proc.h

2016-03-03 Thread Michal Mazurek
These functions appear to only be used in kern_exit. Index: sys/kern/kern_exit.c === RCS file: /cvs/src/sys/kern/kern_exit.c,v retrieving revision 1.154 diff -u -p -r1.154 kern_exit.c --- sys/kern/kern_exit.c9 Oct 2015 01:10:

ping(8) -n flag

2016-03-03 Thread Florian Obser
So, who knows what the -n flag is doing in ping(8)? The man page has this to say: -n Numeric output only. No attempt will be made to look up symbolic names for host addresses. $ ping -c1 amd64.openbsd.adns.de PING amd64.openbsd.adns.de (217.31.84.226): 56 data bytes 6

ping(8): implement -H

2016-03-03 Thread Florian Obser
on top of previous mail ("ping(8) -n flag") OK? diff --git sbin/ping/ping.8 sbin/ping/ping.8 index e3d3d0b..07318e4 100644 --- sbin/ping/ping.8 +++ sbin/ping/ping.8 @@ -39,7 +39,7 @@ .Sh SYNOPSIS .Nm ping .Bk -words -.Op Fl DdEefLqRv +.Op Fl DdEefHLnqRv .Op Fl c Ar count .Op Fl I Ar ifaddr

Re: /32 routes don't stick

2016-03-03 Thread Matthieu Herrb
On Thu, Mar 03, 2016 at 02:38:16PM +0100, Martin Pieuchot wrote: > > So you found a regression due to a change in arptfree(). Previously the > function was looking as its reference counter to decide if it should > delete the route or not. I changed that do stop look at the reference > because th

Re: ksh: variable substitution in double quoted string

2016-03-03 Thread Todd C. Miller
On Thu, 03 Mar 2016 07:14:43 +0100, "Dmitrij D. Czarkoff" wrote: > Bob Beck said: > > This now starts to sounds sane, although the regress test needs to be > > named like paren-expansion-quotes not xxx-what-do-you-call this... > > The test my previous diff failed was "xxx-what-do-you-call-this-1"

Remove WSDISPLAY_COMPAT_SYSCONS

2016-03-03 Thread Christian Weisgerber
jmc@ stumbled over this. The WSDISPLAY_COMPAT_SYSCONS option was only ever in i386 GENERIC and it has been disabled there for 15 years, so I think we can safely delete it, whatever it was used for (running FreeBSD X11 server binaries??). Index: sys/dev/wscons/wsdisplay_compat_usl.c ==

Re: ksh: variable substitution in double quoted string

2016-03-03 Thread Otto Moerbeek
On Thu, Mar 03, 2016 at 07:14:43AM +0100, Dmitrij D. Czarkoff wrote: > Bob Beck said: > > This now starts to sounds sane, although the regress test needs to be > > named like paren-expansion-quotes not xxx-what-do-you-call this... > > The test my previous diff failed was "xxx-what-do-you-call-thi

Re: /32 routes don't stick

2016-03-03 Thread Martin Pieuchot
On 03/03/16(Thu) 13:39, Matthieu Herrb wrote: > On Thu, Mar 03, 2016 at 01:30:10PM +0100, Martin Pieuchot wrote: > > On 03/03/16(Thu) 09:32, Matthieu Herrb wrote: > > > One more data point: I've checked with 5.8, the patch is not needed > > > there. (the "route add .../32 -link -iface if" stays and

Re: /32 routes don't stick

2016-03-03 Thread Matthieu Herrb
On Thu, Mar 03, 2016 at 01:30:10PM +0100, Martin Pieuchot wrote: > On 03/03/16(Thu) 09:32, Matthieu Herrb wrote: > > One more data point: I've checked with 5.8, the patch is not needed > > there. (the "route add .../32 -link -iface if" stays and renews the > > arp entry when needed). > > Which rou

Re: /32 routes don't stick

2016-03-03 Thread Martin Pieuchot
On 03/03/16(Thu) 09:32, Matthieu Herrb wrote: > On Wed, Mar 02, 2016 at 06:03:47PM +0100, Matthieu Herrb wrote: > > On Sat, Feb 27, 2016 at 04:45:09PM +0100, Matthieu Herrb wrote: > > > On Sat, Feb 27, 2016 at 03:20:49PM +0100, Martin Pieuchot wrote: > > > > > > > > Instead of adding a "link" entr

Re: fdt: move code from socppc to sys/dev/fdt/

2016-03-03 Thread Mark Kettenis
> Date: Thu, 3 Mar 2016 00:56:34 +0100 > From: Patrick Wildt > > On Thu, Mar 03, 2016 at 09:38:25AM +1000, David Gwynne wrote: > > > > > On 2 Mar 2016, at 20:49, Patrick Wildt wrote: > > > > > > Hi, > > > > > > to allow FDT to be used on ARM I think we should move the FDT code from > > > socp

Re: /32 routes don't stick

2016-03-03 Thread Matthieu Herrb
On Wed, Mar 02, 2016 at 06:03:47PM +0100, Matthieu Herrb wrote: > On Sat, Feb 27, 2016 at 04:45:09PM +0100, Matthieu Herrb wrote: > > On Sat, Feb 27, 2016 at 03:20:49PM +0100, Martin Pieuchot wrote: > > > > > > Instead of adding a "link" entry I would add a cloning route that will > > > generate i