Re: diff: Fix send(2) EACCES mistake

2018-11-09 Thread Jason McIntyre
On Fri, Nov 09, 2018 at 03:49:32PM -0700, Alexander Bluhm wrote: > On Fri, Nov 09, 2018 at 09:03:20PM +0100, Jan Klemkow wrote: > > On Fri, Nov 09, 2018 at 12:36:20PM -0700, Alexander Bluhm wrote: > > > On Fri, Nov 09, 2018 at 08:24:47PM +0100, Jan Klemkow wrote: > > > > Perfect, I also think its m

Re: [PATCH] Fix broken bus voltage setting in sdhc

2018-11-09 Thread Ben Pye
On Wed, Nov 07, 2018 at 05:33:13PM +0100, Mark Kettenis wrote: > > From: Ben Pye > > Content-Type: text/plain; charset="utf-8" > > > > I have been attempting to run OpenBSD on my HP Chromebook 13, it's a > > Skylake device with eMMC storage. Previously sdhc attempted to set the > > same bus volta

add -D (don't fragment) to ping6

2018-11-09 Thread David Gwynne
this brings -D over to ping6. ok? Index: ping.8 === RCS file: /cvs/src/sbin/ping/ping.8,v retrieving revision 1.60 diff -u -p -r1.60 ping.8 --- ping.8 10 Nov 2018 05:03:23 - 1.60 +++ ping.8 10 Nov 2018 05:25:30 -00

Re: 6.4 - RX not working on new supported BCM574xx (bnxt)

2018-11-09 Thread David Gwynne
> On 10 Nov 2018, at 12:22 pm, Jonathan Matthew wrote: > > On Fri, Nov 09, 2018 at 04:35:38AM -0700, Luthing wrote: >> Hello there, >> >> I am facing an issue with a Broadcom NIC (specs here >> https://www.broadcom.com/products/ethernet-connectivity/controllers/bcm57416/#specifications). >>

Re: [PATCH] parse ! on hostname.if for autoinstall

2018-11-09 Thread Florian Obser
+ _cmd=$(print -- "${_c[@]}" | sed 's/\$if/'$_if'/g') > + _cmds[${#_cmds[*]}]="route -q add${_cmd#route}" > + ;; > '!'*) _cmd=$(print -- "${_c[@]}" | sed 's/\$if/'$_if'/

Re: 6.4 - RX not working on new supported BCM574xx (bnxt)

2018-11-09 Thread Jonathan Matthew
On Fri, Nov 09, 2018 at 04:35:38AM -0700, Luthing wrote: > Hello there, > > I am facing an issue with a Broadcom NIC (specs here > https://www.broadcom.com/products/ethernet-connectivity/controllers/bcm57416/#specifications). > > After some troubleshooting, I am not able to resolve listen ARP req

Re: [PATCH] parse ! on hostname.if for autoinstall

2018-11-09 Thread Julien Dhaille
e -q add${_cmd#route}" > + ;; > '!'*) _cmd=$(print -- "${_c[@]}" | sed 's/\$if/'$_if'/g') > _cmds[${#_cmds[*]}]="${_cmd#!}" > ;; > > > > If this is worth pursuing, I'll

Re: [PATCH] parse ! on hostname.if for autoinstall

2018-11-09 Thread Todd T. Fries
-- "${_c[@]}" | sed 's/\$if/'$_if'/g') _cmds[${#_cmds[*]}]="${_cmd#!}" ;; If this is worth pursuing, I'll create and test an installer equivalent version. Thanks, Penned by Theo de Raadt on 20181109 10:13.33, we have: | This w

Re: diff: Fix send(2) EACCES mistake

2018-11-09 Thread Alexander Bluhm
On Fri, Nov 09, 2018 at 09:03:20PM +0100, Jan Klemkow wrote: > On Fri, Nov 09, 2018 at 12:36:20PM -0700, Alexander Bluhm wrote: > > On Fri, Nov 09, 2018 at 08:24:47PM +0100, Jan Klemkow wrote: > > > Perfect, I also think its more intuitive to get a "permission denied" > > > in case of a pf(4) block

Re: diff: Fix send(2) EACCES mistake

2018-11-09 Thread Jan Klemkow
On Fri, Nov 09, 2018 at 12:36:20PM -0700, Alexander Bluhm wrote: > On Fri, Nov 09, 2018 at 08:24:47PM +0100, Jan Klemkow wrote: > > Perfect, I also think its more intuitive to get a "permission denied" > > in case of a pf(4) block then a "Host is unreachable". The diff below > > corrects kernel an

Re: diff: Fix send(2) EACCES mistake

2018-11-09 Thread Alexander Bluhm
On Fri, Nov 09, 2018 at 08:24:47PM +0100, Jan Klemkow wrote: > Perfect, I also think its more intuitive to get a "permission denied" > in case of a pf(4) block then a "Host is unreachable". The diff below > corrects kernel and extents the manpage for pf(4) blocks. Don't forget divert_output() and

Re: diff: Fix send(2) EACCES mistake

2018-11-09 Thread Jan Klemkow
On Fri, Nov 09, 2018 at 06:57:16PM +0100, Claudio Jeker wrote: > On Fri, Nov 09, 2018 at 06:09:34PM +0100, Jan Klemkow wrote: > > I printed the code path below to make it easier to review the diff. > > While I was following the code path again, I found an inconsistency > > between udp_output() and

Re: diff: Fix send(2) EACCES mistake

2018-11-09 Thread Alexander Bluhm
On Fri, Nov 09, 2018 at 06:57:16PM +0100, Claudio Jeker wrote: > I would prefer to remove this errno translation magic for pf(4) in the > midlayer of the network code. As shown it is always prone for errors. Yes. I always confuses me when I see "no route to host" if pf blocks the packet. If a pr

bgpctl: show relevant error message

2018-11-09 Thread Denis Fondras
When querying a non-existent table, do not reply "no such peer" Index: bgpd/bgpd.h === RCS file: /cvs/src/usr.sbin/bgpd/bgpd.h,v retrieving revision 1.352 diff -u -p -r1.352 bgpd.h --- bgpd/bgpd.h 4 Nov 2018 14:34:00 - 1.352

Re: diff: Fix send(2) EACCES mistake

2018-11-09 Thread Claudio Jeker
On Fri, Nov 09, 2018 at 06:09:34PM +0100, Jan Klemkow wrote: > On Tue, Oct 30, 2018 at 07:13:24AM +, Jason McIntyre wrote: > > On Mon, Oct 29, 2018 at 11:55:52PM +0100, Jan Klemkow wrote: > > > On Sun, Oct 28, 2018 at 10:58:34PM +, Jason McIntyre wrote: > > > > On Sun, Oct 28, 2018 at 09:40

Re: diff: Fix send(2) EACCES mistake

2018-11-09 Thread Jan Klemkow
On Tue, Oct 30, 2018 at 07:13:24AM +, Jason McIntyre wrote: > On Mon, Oct 29, 2018 at 11:55:52PM +0100, Jan Klemkow wrote: > > On Sun, Oct 28, 2018 at 10:58:34PM +, Jason McIntyre wrote: > > > On Sun, Oct 28, 2018 at 09:40:33PM +0100, Jan Klemkow wrote: > > > > Unlike the manpage saids or o

Re: YP/NIS support in /etc/ethers, libc ether_ntohost/ether_hostton

2018-11-09 Thread Theo de Raadt
Sure but the issue is that binaries over-reach beyond in a way that "getpw" supports for name lookups cannot help here. So the ether library routines need another non-obvious pledge. I refer to remove such YP support since use of it would be completely fringe. > On Thu, Nov 08, 2018 at 08:05:13P

Re: [PATCH] parse ! on hostname.if for autoinstall

2018-11-09 Thread Julien Dhaille
diff --git distrib/miniroot/install.sub distrib/miniroot/install.sub index bce1fa50358..31dbafdc95d 100644 --- distrib/miniroot/install.sub +++ distrib/miniroot/install.sub @@ -2319,8 +2319,11 @@ parse_hn_line() { _cmds[${#_cmds[*]}]="ifconfig $_if ${_c[@]} up;dhclient $_if"

Re: [PATCH] Add elan(4) touchpad driver

2018-11-09 Thread Ben Pye
On Thu, Nov 08, 2018 at 11:25:37PM -0600, joshua stein wrote: > On Fri, 09 Nov 2018 at 03:30:07 +, b...@curlybracket.co.uk wrote: > > Hi all, > > > > This patch adds a new touchpad driver, elan(4), which supports older non > > PTP I2C Elantech touchpads. I have tested this on my HP Chromebook

Re: [PATCH] parse ! on hostname.if for autoinstall

2018-11-09 Thread Gilles Chehade
On Fri, Nov 09, 2018 at 05:10:00PM +0100, Julien Dhaille wrote: > Hi. > > During auto upgrade via the auto_upgrade.conf file (no DHCP server), > shell commands are skipped. > This small diff is coming from parse_hn_line() in /etc/netstart, thus, > it keeps the behavior similar. > I think it's hand

Re: [PATCH] parse ! on hostname.if for autoinstall

2018-11-09 Thread Theo de Raadt
This was intentional because a user could put crazy script commands in the files which don't work in the bsd.rd context. bsd.rd is quite gutted. That means some circumstances could fail quite badly Could be reconsidered... > During auto upgrade via the auto_upgrade.conf file (no DHCP server), >

[PATCH] parse ! on hostname.if for autoinstall

2018-11-09 Thread Julien Dhaille
Hi. During auto upgrade via the auto_upgrade.conf file (no DHCP server), shell commands are skipped. This small diff is coming from parse_hn_line() in /etc/netstart, thus, it keeps the behavior similar. I think it's handy, especially if you need a static route in order to reach a mirror for exampl

Re: 6.4 - RX not working on new supported BCM574xx (bnxt)

2018-11-09 Thread Carlos Cardenas
On Fri, Nov 09, 2018 at 04:35:38AM -0700, Luthing wrote: > Hello there, > > I am facing an issue with a Broadcom NIC (specs here > https://www.broadcom.com/products/ethernet-connectivity/controllers/bcm57416/#specifications). > > After some troubleshooting, I am not able to resolve listen ARP req

Re: let ping6 set the IPv6 traffic class like it does for IPv4 TOS

2018-11-09 Thread Denis Fondras
On Fri, Nov 09, 2018 at 06:43:06PM +1000, David Gwynne wrote: > On Fri, Nov 09, 2018 at 08:30:08AM +0100, Denis Fondras wrote: > > On Fri, Nov 09, 2018 at 05:07:10PM +1000, David Gwynne wrote: > > > On Fri, Nov 09, 2018 at 06:33:46AM +0100, Florian Obser wrote: > > > > please update usage(), then i

Re: YP/NIS support in /etc/ethers, libc ether_ntohost/ether_hostton

2018-11-09 Thread Ricardo Mestre
Paraphrasing an excerpt of my commit on getent(1) to add unveil(2): "After a discussion with millert@ regarding YP then deraadt@ chimed in referring that when he wrote this code even though we can have YP mappings with several of these dbs "it doesn't mean that things use it, or should, or will" s

6.4 - RX not working on new supported BCM574xx (bnxt)

2018-11-09 Thread Luthing
Hello there, I am facing an issue with a Broadcom NIC (specs here https://www.broadcom.com/products/ethernet-connectivity/controllers/bcm57416/#specifications). After some troubleshooting, I am not able to resolve listen ARP request from my connected switch. The NIC has negociated 10G with auto n

Re: YP/NIS support in /etc/ethers, libc ether_ntohost/ether_hostton

2018-11-09 Thread Jonathan Matthew
On Thu, Nov 08, 2018 at 08:05:13PM -0500, Bryan Steele wrote: > These libc functions are used to map hardware MAC addresses to hostnames > and vice versa. If it exists, /etc/ethers will typically contain a > number of lines like so: > > 34:00:8a:56:10:20 superman > > In addition to that, ther

Re: let ping6 set the IPv6 traffic class like it does for IPv4 TOS

2018-11-09 Thread Florian Obser
OK florian@ On Fri, Nov 09, 2018 at 06:43:06PM +1000, David Gwynne wrote: > On Fri, Nov 09, 2018 at 08:30:08AM +0100, Denis Fondras wrote: > > On Fri, Nov 09, 2018 at 05:07:10PM +1000, David Gwynne wrote: > > > On Fri, Nov 09, 2018 at 06:33:46AM +0100, Florian Obser wrote: > > > > please update us

Re: let ping6 set the IPv6 traffic class like it does for IPv4 TOS

2018-11-09 Thread David Gwynne
On Fri, Nov 09, 2018 at 08:30:08AM +0100, Denis Fondras wrote: > On Fri, Nov 09, 2018 at 05:07:10PM +1000, David Gwynne wrote: > > On Fri, Nov 09, 2018 at 06:33:46AM +0100, Florian Obser wrote: > > > please update usage(), then it's OK florian@ > > > > > > Could you do tracerute6, too? > > > > li