switchd.conf(5): mention default values used for global configurations

2018-11-08 Thread Ayaka Koshibe
Hi all, After a second thought, I've split off the diff for switchd.conf(5) from the diffs for changing switchd(8)'s default listen port. As suggested by kn@, it now notes the default listen port and address. Comments/OKs? Thanks, Ayaka Index: switchd.conf.5

Re: tcpdump: revisiting some old diffs, cleanup unused functions

2018-11-08 Thread Klemens Nanni
OK

Re: tcpdump: revisiting some old diffs, cleanup unused functions

2018-11-08 Thread Ricardo Mestre
Still works as advertised, ok mestre@ On 19:32 Wed 07 Nov , Bryan Steele wrote: > On Wed, Nov 07, 2018 at 07:06:09PM -0500, Bryan Steele wrote: > > I'm revisiting some old tcpdump diffs, now that mestre@ has added proper > > unveil(2) support! :-) > > > > Refresher:

Re: unveil error - src/usr.bin/passwd/local_passwd.c (-current)

2018-11-08 Thread Ricardo Mestre
Hi Mark, Thanks for the report, don't know how I missed it but it's now fixed. On 16:20 Thu 08 Nov , Mark Patruck wrote: > Hi Ricardo, > > when running > > $ passwd > > with src/usr.bin/passwd/local_passwd.c v1.54 > > i see the following error in /var/log/messages > > passwd: cannot

batch copyout(9) in bridge(4)

2018-11-08 Thread Martin Pieuchot
I'd like to protect bridge(4) data structures with a mutex. Using a mutex is necessary because bridge_output() is called from interrupt handlers in wireless drivers. This forces us to decouple data gathering from copying to userland when executing some ioctl(2)s. Diff below does that by moving

Re: tcpdump: revisiting some old diffs, remove unused pledges

2018-11-08 Thread Bryan Steele
On Wed, Nov 07, 2018 at 07:32:25PM -0500, Bryan Steele wrote: > On Wed, Nov 07, 2018 at 07:06:09PM -0500, Bryan Steele wrote: > > I'm revisiting some old tcpdump diffs, now that mestre@ has added proper > > unveil(2) support! :-) > > > > Refresher:

netcat report listen

2018-11-08 Thread Alexander Bluhm
Hi, It is easier to write race free tests if netcat reports when the listen system call has finished. This avoids ugly kernel lookups with fstat. Just report bind or listen like accepted connections in verbose mode to stderr. Then the other end can proceed. ok? Index: usr.bin/nc/netcat.c

Re: netcat report listen

2018-11-08 Thread Theo Buehler
On Thu, Nov 08, 2018 at 02:12:35PM -0700, Alexander Bluhm wrote: > Hi, > > It is easier to write race free tests if netcat reports when the > listen system call has finished. This avoids ugly kernel lookups > with fstat. Just report bind or listen like accepted connections > in verbose mode to

[PATCH] Add elan(4) touchpad driver

2018-11-08 Thread ben
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 13 and it appears to work well, multitouch is working as well as the three physical mouse buttons. I do not know how well this will work on other

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

2018-11-08 Thread Bryan Steele
This was suggested by deraadt@, sorry. 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

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

2018-11-08 Thread David Gwynne
This sets ping6 up to specify the traffic class field for the IPv6 traffic class field. The v6 traffic class is equivalent to the IPv6 TOS/DSCP field, so this uses the same getopt param and parsing to get the value. ok? Index: ping.8

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

2018-11-08 Thread Bryan Steele
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, there is support for using a YP (nee Yellow Pee) lookup service: "If a '+'

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

2018-11-08 Thread joshua stein
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 13 > and it appears to work well, multitouch is working as well as the

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

2018-11-08 Thread Denis Fondras
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? > > like this? > Can you also update traceroute.8 ? :) > Index:

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

2018-11-08 Thread David Gwynne
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? like this? Index: sbin/ping/ping.8 === RCS file: /cvs/src/sbin/ping/ping.8,v retrieving

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

2018-11-08 Thread Florian Obser
please update usage(), then it's OK florian@ Could you do tracerute6, too? On Fri, Nov 09, 2018 at 03:08:20PM +1000, David Gwynne wrote: > This sets ping6 up to specify the traffic class field for the IPv6 > traffic class field. The v6 traffic class is equivalent to the IPv6 > TOS/DSCP field, so

Unused defines in ed25519

2018-11-08 Thread Geoff Hill
Looking at the ed25519 code, these defines are unused. They appear to be unused upstream as well (in djt's supercop ed25519/ref implementation), so maybe this should be suggested there instead of deviating. Up to you. Index: usr.bin/signify/fe25519.c

free(9) size for wi(4) @ USB

2018-11-08 Thread Martin Pieuchot
The driver already keeps track of the size of the allocated buffer, so use it. Diff is untested as I don't have a wi(4) @ USB. ok? Index: if_wi_usb.c === RCS file: /cvs/src/sys/dev/usb/if_wi_usb.c,v retrieving revision 1.68 diff -u

free(9) sizes for uhub(4)

2018-11-08 Thread Martin Pieuchot
The driver already keeps track of the number of ports, so use this piece of information to free allocated structures. Ok? Index: uhub.c === RCS file: /cvs/src/sys/dev/usb/uhub.c,v retrieving revision 1.90 diff -u -p -r1.90 uhub.c

free(9) size for USB endpoints

2018-11-08 Thread Martin Pieuchot
Keep track of the number of allocated endpoints and use it when freeing the array. ok? Index: usb_subr.c === RCS file: /cvs/src/sys/dev/usb/usb_subr.c,v retrieving revision 1.138 diff -u -p -r1.138 usb_subr.c --- usb_subr.c 19 Jul

free(9) sizes for USB cdesc

2018-11-08 Thread Martin Pieuchot
The length of the configuration descriptor is already used in usbd_parse_idesc(). The diff below reuses the same pattern to add the size argument to free(9), ok? Index: usb_subr.c === RCS file: /cvs/src/sys/dev/usb/usb_subr.c,v